llvm icon indicating copy to clipboard operation
llvm copied to clipboard

[SYCL][DOC] Add spec and design for "if_device"

Open gmlueck opened this issue 1 year ago • 4 comments

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.

gmlueck avatar Mar 31 '23 20:03 gmlueck

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.

gmlueck avatar Jul 06 '23 21:07 gmlueck

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.

gmlueck avatar Sep 21 '23 19:09 gmlueck

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?

v-klochkov avatar Feb 22 '24 04:02 v-klochkov

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.

gmlueck avatar Feb 22 '24 13:02 gmlueck