Skip to content

Conversation

@kpet
Copy link
Contributor

@kpet kpet commented Sep 22, 2025

Fixes #279

Comment on lines +231 to +232
greater than 0 when passed to
*OpCooperativeMatrixStoreKHR*.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We did discuss potentially requiring the stride be such that no write collisions occur. Do we want to do this too?

@bashbaug
Copy link
Collaborator

Discussed in the October 8th teleconference:

  • Why does constraint only apply to stores? Suggest updating the constraint so it applies more broadly such that no write collisions occur, which should also help to describe why the constraint does not apply to loads. Figuring out the right spec text to describe this is challenging - suggestions are welcome.
  • What does it mean to say "interpreted as unsigned"? In short, this means that an integer stride with no signed-ness information is interpreted as an unsigned value, so if the upper bit is set it is interpreted as a very large positive value and not a negative value.
  • We're aiming to update the constraint before merging.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unclear requirements on Stride operand to OpCooperativeMatrix{Load,Store}KHR

2 participants