Skip to content

Replace GetElementCoord to cooperative matrix version#20855

Draft
vmaksimo wants to merge 3 commits intosyclfrom
sycl-devops-pr/vmaksimo/joint-matrix-getelemcoord
Draft

Replace GetElementCoord to cooperative matrix version#20855
vmaksimo wants to merge 3 commits intosyclfrom
sycl-devops-pr/vmaksimo/joint-matrix-getelemcoord

Conversation

@vmaksimo
Copy link
Copy Markdown
Contributor

@vmaksimo vmaksimo commented Dec 8, 2025

No description provided.

@vmaksimo vmaksimo marked this pull request as ready for review December 9, 2025 13:50
@vmaksimo vmaksimo requested review from a team as code owners December 9, 2025 13:50
@vmaksimo vmaksimo requested a review from vinser52 December 9, 2025 13:50
@dkhaldi
Copy link
Copy Markdown
Contributor

dkhaldi commented Dec 9, 2025

@MrSidims, @vmaksimo, We need to check that this works with current version of IGC used by CI before we merge it.

Copy link
Copy Markdown
Contributor

@dkhaldi dkhaldi left a comment

Choose a reason for hiding this comment

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

LGTM as opcode is same.

@vmaksimo
Copy link
Copy Markdown
Contributor Author

This one should be merged together/as a part of the next pulldown from Khronos which will include merged KhronosGroup/SPIRV-LLVM-Translator#3468

Copy link
Copy Markdown
Collaborator

@iclsrc iclsrc left a comment

Choose a reason for hiding this comment

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

Clean rename completing the pre-existing TODO. The SPIRV backend (SPIRVBuiltins.td) already maps __spirv_CooperativeMatrixGetElementCoordINTEL to OpCooperativeMatrixGetElementCoordINTEL (opcode 6440), so the declaration and call-site updates in these headers are correct and sufficient. E2e coverage exists in sycl/test-e2e/Matrix/get_coordinate_ops.cpp.

@jsji jsji removed the claude-review label Apr 8, 2026
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.

6 participants