Skip to content

[CoopVec] Check for minimum support set#7457

Merged
alsepkow merged 2 commits intomicrosoft:staging-sm6.9from
jholewinski-nv:dev/jholewinski/coopvec-exectests-minset
May 16, 2025
Merged

[CoopVec] Check for minimum support set#7457
alsepkow merged 2 commits intomicrosoft:staging-sm6.9from
jholewinski-nv:dev/jholewinski/coopvec-exectests-minset

Conversation

@jholewinski-nv
Copy link
Copy Markdown

This change refactors the CoopVec test config selection to enforce running the minimum support set. Drivers that do not report all required data formats and interpretations will fail the test.

The handling of D3D12EnableExperimentalFeatures is also refactored to apply only to the CoopVec tests, without the use of -p:CooperativeVectorExperimental=*.

Comment thread tools/clang/unittests/HLSLExec/ExecutionTest.cpp Outdated
Comment thread tools/clang/unittests/HLSLExec/ExecutionTest.cpp
@jholewinski-nv jholewinski-nv force-pushed the dev/jholewinski/coopvec-exectests-minset branch from 7fade28 to 89985d8 Compare May 16, 2025 15:43
D3D12_LINEAR_ALGEBRA_DATATYPE_FLOAT_E5M2, /* MatrixInterpretation */
D3D12_LINEAR_ALGEBRA_DATATYPE_FLOAT16, /* BiasInterpretation */
D3D12_LINEAR_ALGEBRA_DATATYPE_FLOAT16}, /* OutputType */
{D3D12_LINEAR_ALGEBRA_DATATYPE_UINT32, /* InputType */
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.

I don't think I see this particular set of configs in the linked spec?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This matches the version in microsoft/hlsl-specs#502.

Copy link
Copy Markdown
Contributor

@alsepkow alsepkow left a comment

Choose a reason for hiding this comment

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

lgtm

@alsepkow alsepkow merged commit 21d37a4 into microsoft:staging-sm6.9 May 16, 2025
12 checks passed
@github-project-automation github-project-automation Bot moved this from Active to Closed in HLSL Support May 16, 2025
@github-project-automation github-project-automation Bot moved this from New to Done in HLSL Roadmap May 16, 2025
@damyanp damyanp removed this from HLSL Support Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants