Skip to content

[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

Merged
merged 1 commit into from
Jun 26, 2020

Conversation

wuw2
Copy link
Contributor

@wuw2 wuw2 commented Jun 24, 2020

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

@wuw2 wuw2 requested a review from a team as a code owner June 24, 2020 11:07
@wuw2 wuw2 requested a review from alexbatashev June 24, 2020 11:07
Copy link
Contributor

@alexbatashev alexbatashev left a 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.

@wuw2 wuw2 changed the title [SYCL] Update sub_group test for native subgroups mode [SYCL][NFC] Update sub_group test for native subgroups mode Jun 24, 2020
alexbatashev
alexbatashev previously approved these changes Jun 24, 2020
Copy link
Contributor

@alexbatashev alexbatashev left a 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?

@wuw2
Copy link
Contributor Author

wuw2 commented Jun 24, 2020

LGTM. @wuw2 please update PR description with some rationale. What changes make the updated version of test possible?

Updated

@jbrodman jbrodman requested a review from Pennycook June 24, 2020 17:08
@jbrodman
Copy link
Contributor

I'd like @Pennycook to sign off on this.

Copy link
Contributor

@vladimirlaz vladimirlaz left a 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.
@wuw2 wuw2 force-pushed the cpu_native_subgroup_test branch from d1a30e9 to 8a39ba3 Compare June 25, 2020 13:44
@wuw2 wuw2 requested review from Pennycook and vladimirlaz June 25, 2020 14:08
@bader bader merged commit 539972d into intel:sycl Jun 26, 2020
againull pushed a commit to againull/llvm that referenced this pull request May 4, 2023
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