Skip to content

Comments

[SYCL][Joint Matrix] Add support for Offset joint_matrix_load and joi#15526

Draft
dkhaldi wants to merge 3 commits intointel:syclfrom
dkhaldi:offset-ld-st
Draft

[SYCL][Joint Matrix] Add support for Offset joint_matrix_load and joi#15526
dkhaldi wants to merge 3 commits intointel:syclfrom
dkhaldi:offset-ld-st

Conversation

@dkhaldi
Copy link
Contributor

@dkhaldi dkhaldi commented Sep 26, 2024

Spec added in #15499

__spv::MatrixLayout L = __spv::MatrixLayout::RowMajor,
__spv::Scope::Flag S = __spv::Scope::Flag::Subgroup>
extern __DPCPP_SYCL_EXTERNAL
__spv::__spirv_JointMatrixINTEL<Tp, R, C, L, S, U> *
Copy link
Contributor

@YixingZhang007 YixingZhang007 Dec 9, 2024

Choose a reason for hiding this comment

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

Hello Dounia! I found this line needs to be updated to __spv::__spirv_CooperativeMatrixKHR<Tp, S, R, C, U> * because a commit merged two weeks ago switched operations to use SPV_KHR_cooperative_matrix (#16045)

__spv::Scope::Flag S = __spv::Scope::Flag::Subgroup>
extern __DPCPP_SYCL_EXTERNAL void __spirv_CooperativeMatrixStoreOffsetINTEL(
T *Ptr, int32_t RowIndex, int32_t ColIndex,
__spv::__spirv_JointMatrixINTEL<Tp, R, C, L, S, U> *Object,
Copy link
Contributor

Choose a reason for hiding this comment

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

This line would also need to be updated to __spv::__spirv_CooperativeMatrixKHR<Tp, S, R, C, U> *Object for the same reason above.

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.

2 participants