-
Notifications
You must be signed in to change notification settings - Fork 788
[SYCL][NFC] Update sub_group test for native subgroups mode #1973
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great to have some meaningful PR description on why this test needs update. Also, an [NFC]
tag probably fits the purpose of the patch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. @wuw2 please update PR description with some rationale. What changes make the updated version of test possible?
ef0f5ea
to
d1a30e9
Compare
Updated |
I'd like @Pennycook to sign off on this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you address John's comments?
Previously, the execution mode of CPU subgroup is emulation mode, which means a subgroup == a workgroup. Since OCL CPU RT has enabled native subgroups by default, we need to update these sycl subgroup tests.
d1a30e9
to
8a39ba3
Compare
This instruction is generated for DWARF `DISPFlagMainSubprogram` flag of function as well as for `spir_kernel` functions. Spec: https://github.com/KhronosGroup/SPIRV-Registry/blob/main/nonsemantic/NonSemantic.Shader.DebugInfo.100.asciidoc#DebugEntryPoint Original commit: KhronosGroup/SPIRV-LLVM-Translator@8565381
These sycl subgroup tests assumed cpu subgroup executes in emulation mode, which means a subgroup == a workgroup. Since OCL CPU RT has enabled native subgroups by default, we need to update these tests