Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix cooperative matrix prefetch test (scope parameter) #2234

Conversation

VyacheslavLevytskyy
Copy link
Contributor

Change Scope argument to one of two available options: ScopeWorkgroup/ScopeWorkgroup.

@VyacheslavLevytskyy VyacheslavLevytskyy marked this pull request as ready for review November 27, 2023 11:38
@VyacheslavLevytskyy
Copy link
Contributor Author

@MrSidims Can you please have a look

@MrSidims MrSidims merged commit 4e1d3e0 into KhronosGroup:main Nov 27, 2023
7 checks passed
MrSidims pushed a commit to MrSidims/SPIRV-LLVM-Translator that referenced this pull request Feb 13, 2024
…2234)

- change Scope argument to one of two available options: ScopeWorkgroup/ScopeWorkgroup
- fix arguments order in calls to OpCooperativeMatrixLoadKHR()
YuriPlyakhin pushed a commit to YuriPlyakhin/SPIRV-LLVM-Translator that referenced this pull request Jul 3, 2024
…2234)

- change Scope argument to one of two available options: ScopeWorkgroup/ScopeWorkgroup
- fix arguments order in calls to OpCooperativeMatrixLoadKHR()
MrSidims pushed a commit that referenced this pull request Jul 4, 2024
This PR aims to introduce CooperativeMatrixPrefetchINTEL capability and operation, and make initial introduction of entities in llvm-spirv translator.

* Fix cooperative matrix prefetch test (scope parameter) (#2234)

- change Scope argument to one of two available options: ScopeWorkgroup/ScopeWorkgroup
- fix arguments order in calls to OpCooperativeMatrixLoadKHR()

* Remove CoordX and CoordY arguments of OpCooperativeMatrixPrefetchINTEL

According to a new specification, CoordX and CoordY parameters are not needed for prefetch in CooperativeMatrixPrefetchINTEL(), only offset pointer is enough. This PR is to fix support for joint_matrix_prefetch in SPIRV according to the new specification.

Co-authored-by: Vyacheslav Levytskyy <[email protected]>
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