-
Notifications
You must be signed in to change notification settings - Fork 752
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
[SYCL][NFC][Doc] Update sycl_ext_intel_device_info backend limitations #16792
Conversation
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
@intel/dpcpp-specification-reviewers |
It's better to take out the wording about which backend support each query. Users should check the aspect. It looks like you have mostly done this already, but there are some that you did not remove. Is there a reason these remain? |
Yes, I left them since they are still relevant and info is supported on L0 only. Should I left all backend mentions then? |
Yes, it would be better to remove all mention of the backends that support each query. Applications should really use the aspects instead. |
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
done in 0620cdb |
SYCL_ENABLE_PCI is deprecated here 2d12863. Remove it as requirement for device info query.
Remove statement that some device info is supported on L0 only if there is at least one another backend that could return it. (mostly it is L0 +hip/cuda).