-
Notifications
You must be signed in to change notification settings - Fork 790
[SYCLNATIVECPU] Re-enable host device #9178
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
Conversation
For us this was the quickest way to get kernels to run through the SYCL native CPU path, but I understand the concerns about the 2020 spec and in general maintainability of the host device, so I'm closing this PR while we work on a better solution, thank you |
The plan was to use the host device for parts and no longer expose it to the user - so for the current work we've just re-enabled it, and in that PR we then wanted to gradually remove the bits we no longer need. |
This PR re-enables the host device, making it again available in user code. This is needed by the SYCL Native CPU compiler flow to accelerate SYCL on CPUs. This PR only contains changes needed to re-enable the host device and no changes related to SYCL Native CPU. The changes relative to SYCL Native CPU are contained in a separate PR, which, for ease of review, currently lives in a separate fork, see this. Once this re-enablement PR is merged in, we will re-target the SYCL Native CPU PR to this repo.
This PR in particular:
sycl::stream::flush
(f9f3008) and device filtering (28a6353).