-
Notifications
You must be signed in to change notification settings - Fork 130
[SYCL] updating device selectors #1160
[SYCL] updating device selectors #1160
Conversation
…SYCL 2020 ones Signed-off-by: Chris Perkins <[email protected]>
/verify with intel/llvm#6599 |
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.
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.
will have a failure in |
Signed-off-by: Chris Perkins <[email protected]>
Signed-off-by: Chris Perkins <[email protected]>
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 |
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
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 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 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]