Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL][Matrix] Add missing explicit SG size statement #764

Merged
merged 3 commits into from
Jan 25, 2022

Conversation

dkhaldi
Copy link

@dkhaldi dkhaldi commented Jan 24, 2022

Since for DPAS, per current implementation limitation, only SG size of 8 is supported, this statement has to be there.

@dkhaldi dkhaldi requested a review from a team as a code owner January 24, 2022 16:25
@dkhaldi dkhaldi requested review from againull and mbelicki January 24, 2022 16:25
mbelicki
mbelicki previously approved these changes Jan 24, 2022
Copy link

@mbelicki mbelicki left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@againull
Copy link

@dkhaldi Could you please fix the formatting.
Changes look good to me.

againull
againull previously approved these changes Jan 24, 2022
@vladimirlaz vladimirlaz merged commit 7bb961a into intel:intel Jan 25, 2022
@dkhaldi dkhaldi deleted the add-sg-size branch January 25, 2022 18:59
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Apr 12, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Jun 17, 2022
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants