Skip to content

Add asfloat, asuint and asint matrices test#1186

Open
joaosaffran wants to merge 5 commits into
llvm:mainfrom
joaosaffran:intrinsic/asfloat-matrices
Open

Add asfloat, asuint and asint matrices test#1186
joaosaffran wants to merge 5 commits into
llvm:mainfrom
joaosaffran:intrinsic/asfloat-matrices

Conversation

@joaosaffran
Copy link
Copy Markdown
Contributor

This patch adds matrices versions for asfloat, asuint and asint intrinsics tests.
Fix: llvm/llvm-project#184471, llvm/llvm-project#184472 and llvm/llvm-project#184474

Assisted by: Claude Opus 4.6

@joaosaffran joaosaffran added the test-all When applied to a PR this will opt-in to additional pre-merge test configurations.. label May 13, 2026
Copy link
Copy Markdown
Contributor

@Icohedron Icohedron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add tests involving constant literals in the shaders

@joaosaffran joaosaffran requested a review from Icohedron May 13, 2026 23:15
Copy link
Copy Markdown
Contributor

@kmpeng kmpeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI you linked the implementation issues instead of the test issues (#781, #782, #784) in the description. Also I think we wanted to name the files <intrinsic>.matrix.test

RWStructuredBuffer<float4> Out2 : register(u5);

[numthreads(1,1,1)]
void main() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we get 1 dimensional matrix tests? The requirements say to test columns > row, row > columns, 1 dimensional, and the max size

RWStructuredBuffer<int4> Out3 : register(u5);

[numthreads(1,1,1)]
void main() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above - 1 dimensional matrix tests

RWStructuredBuffer<uint4> Out2 : register(u5);

[numthreads(1,1,1)]
void main() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above - 1 dimensional matrix tests

RWStructuredBuffer<uint4> Highs : register(u2);

[numthreads(1,1,1)]
void main() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above - 1 dimensional matrix tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test-all When applied to a PR this will opt-in to additional pre-merge test configurations..

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Matrix] Implement matrix support for the asfloat HLSL Function

3 participants