llvm
llvm copied to clipboard
[SYCL][DOC] Add spec and design for "if_device"
Add a proposed extension specification and design for if_device
, an alternative to __SYCL_DEVICE_ONLY__
, which is implementable in a 1-pass compiler.
This commit also renames the proposed extension "sycl_ext_oneapi_device_if" to "sycl_ext_oneapi_if_device_has" in order to avoid confusion with this new extension.
Ping @intel/dpcpp-doc-reviewers @tfzhu. We'd like to merge this if possible. We already have #10169 which implements this proposed extension for the current 2-pass compiler.
Ping to reviewers (@rolandschulz @steffenlarsen). If there are no more comments, I'd like to merge this PR. Merging is not a commitment to implement this feature. We can decide that separately.
This feature looks really useful, in particular we need in it in ESIMD component/development. Unfortunately, it seems like this proposal is stuck. Is it blocked by some other ongoing SYCL changes/re-design activities?
This feature looks really useful, in particular we need in it in ESIMD component/development. Unfortunately, it seems like this proposal is stuck. Is it blocked by some other ongoing SYCL changes/re-design activities?
No, it's not blocked by anything in particular; our focus just turned to other things. I think we just need approvals from @intel/dpcpp-doc-reviewers and @intel/llvm-reviewers-runtime, and I need to fix the merge conflicts.