Skip to content

Conversation

@V-FEXrt
Copy link
Collaborator

@V-FEXrt V-FEXrt commented Dec 19, 2025

This PR reserves the DXIL opcodes needed for the SM6.10 LinAlg feature. dx.types.MatrixRef and $vec2 are intentionally not yet added to the PR so those have a placeholder i32 type. Three opcodes were reserved in case any new LinAlg operations come up later.

Only utils/hct/hctdb.py was manually edited, all other changes are generated code.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

✅ With the latest revision this PR passed the Python code formatter.

Copy link
Collaborator

@bob80905 bob80905 left a comment

Choose a reason for hiding this comment

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

Just 1 question, otherwise LGTM

],
)

op_table.reserve_dxil_op_range("LinAlgMatrixReserved", 3)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would you mind explaining where the 3 is coming from?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Arbitrarily picked! This is just putting some opcodes aside since folks have the impression that there will be more LinAlg ops coming later.

Since this is the experimental pool they'll get freed eventually if they aren't used.

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

Labels

None yet

Projects

Status: New

Development

Successfully merging this pull request may close these issues.

2 participants