-
Notifications
You must be signed in to change notification settings - Fork 82
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
Update checks for sycl::info::device::preferred_interop_user_sync
and sycl::info::device::profile
in device_info
test
#992
base: main
Are you sure you want to change the base?
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 looks like sycl::make_error_code
isn't supported by hipSYCL
, so we probably should disable checks or the test case for that implementation.
Co-authored-by: Alexey Bader <alexey.bader@intel.com>
I updated code. Could you look at it if it's okay? |
@KhronosGroup/sycl-cts-reviewers, we need one approve from a person not affiliated with Intel. Could you take a look, please? |
Based on specification https://registry.khronos.org/SYCL/specs/sycl-2020/html/sycl-2020.html#_device_information_descriptors
sycl::info::device::preferred_interop_user_sync
andsycl::info::device::profile
info descriptors are supposed to throw an exception when the backend is not OpenCL. This is why checks indevice_info.cpp
were modified for these specific info device descriptors.