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

[SYCL] updating device selectors #1160

Merged

Conversation

cperkinsintel
Copy link

@cperkinsintel cperkinsintel commented Aug 18, 2022

The SYCL 1.2.1 style device_selector subclasses are now deprecated. In this PR we are updating the tests to use the newer SYCL 2020 ones. This avoids encountering all the "deprecation" warnings when running verbose tests.

The deprecation warnings are being added in intel/llvm#6599 .

Signed-off-by: Chris Perkins [email protected]

@cperkinsintel cperkinsintel requested review from a team and Pennycook as code owners August 18, 2022 05:24
@cperkinsintel
Copy link
Author

/verify with intel/llvm#6599

AlexeySachkov
AlexeySachkov previously approved these changes Aug 18, 2022
Copy link

@v-klochkov v-klochkov 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 in general, but there is 1 or 2 test fails caused by this PR.
In particular program_link.cpp fail (syntax error) is definitely caused by this change.

@cperkinsintel
Copy link
Author

will have a failure in event_creation_oepncl.cpp until #1161 is merged.

v-klochkov
v-klochkov previously approved these changes Aug 19, 2022
AlexeySachkov
AlexeySachkov previously approved these changes Sep 6, 2022
v-klochkov
v-klochkov previously approved these changes Sep 8, 2022
@cperkinsintel
Copy link
Author

ping to reviewers. I had to set this aside to work on something else, but now it has been deconflicted again and is passing all tests. Would love to get this in and merged.

This PR mostly just replaces instances of gpu_selector{} (et al) with gpu_selector_v to avoid the deprecation warnings that the compiler is outputting now on those old style selectors.

Copy link

@smaslov-intel smaslov-intel left a comment

Choose a reason for hiding this comment

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

LGTM

@againull againull merged commit 907a49b into intel:intel Oct 3, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
The SYCL 1.2.1 style device_selector subclasses are now deprecated. In this PR we are updating the tests to use the newer SYCL 2020 ones. This avoids encountering all the "deprecation" warnings when running verbose tests.
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
The SYCL 1.2.1 style device_selector subclasses are now deprecated. In this PR we are updating the tests to use the newer SYCL 2020 ones. This avoids encountering all the "deprecation" warnings when running verbose tests.
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.

5 participants