Skip to content

[SM6.10] LinAlg Validation: MatrixAccumulate - #8627

Merged
V-FEXrt merged 3 commits into
microsoft:mainfrom
V-FEXrt:linalg-validation-matrixaccumulate
Jul 21, 2026
Merged

[SM6.10] LinAlg Validation: MatrixAccumulate#8627
V-FEXrt merged 3 commits into
microsoft:mainfrom
V-FEXrt:linalg-validation-matrixaccumulate

Conversation

@V-FEXrt

@V-FEXrt V-FEXrt commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Fixes #8501

@alsepkow alsepkow left a comment

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.

LGTM

Comment thread lib/DxilValidation/DxilValidation.cpp Outdated
Comment thread utils/hct/hctdb.py Outdated
Comment on lines +6 to +11
%dx.types.LinAlgMatrixC8M4N4U2S2 = type { i8* }
%dx.types.LinAlgMatrixC8M4N4U0S2 = type { i8* }
%dx.types.LinAlgMatrixC8M4N4U2S0 = type { i8* }
%dx.types.LinAlgMatrixC8M4N4U1S0 = type { i8* }
%dx.types.LinAlgMatrixC8M4N4U1S1 = type { i8* }
%dx.types.LinAlgMatrixC8M8N8U0S2 = type { i8* }

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Every matrix in this test uses the same component type (C8), so no component-type mismatch is exercised (rule 6: The element types are compatible).

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Good catch:) I actually talked to @llvm-beanz about this yesterday and he said not to worry about that rule for now!

@github-project-automation github-project-automation Bot moved this from New to In progress in HLSL Roadmap Jul 16, 2026
@V-FEXrt
V-FEXrt merged commit 5955df0 into microsoft:main Jul 21, 2026
14 checks passed
@github-project-automation github-project-automation Bot moved this from In progress to Done in HLSL Roadmap Jul 21, 2026
@V-FEXrt
V-FEXrt deleted the linalg-validation-matrixaccumulate branch July 21, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

LinAlg Validation: MatrixAccumulate

4 participants