intel / intel/llvm

Add SYCL extension specification for max_work_group_size

Open
#7,700 3 comments 0 reactions 1 assignee Claimed by @steffenlarsen View on GitHub
confirmed enhancement
Dominant language
LLVM
Stars
1.5k
Forks
854
Avg merge
3d 17h
Merged PRs (30d)
137

Description

DPC++ already supports `intel::max_work_group_size` but though the documentation for the attribute specifies the behavior at SYCL level, the attribute does not have a related SYCL extension detailing the behavior. Such an extensions should be added.

Additional options for this extension specification:

1. Allow 1-3 arguments in the attribute and add the requirement that the kernel must be launched with the same dimensionality as the attribute. This makes it in line with other SYCL 2020 core attributes for work groups (`sycl::reqd_work_group_size` and `sycl::work_group_size_hint`). See discussion https://github.com/intel/llvm/pull/7450#discussion_r1027925050.
2. ~~Add new `sycl` spelling, i.e. `sycl::max_work_group_size`.~~
3. Add a kernel compile-time property analogue. See https://github.com/intel/llvm/blob/sycl/sycl/doc/extensions/experimental/sycl_ext_oneapi_kernel_properties.asciidoc.
** Note: Since this would be experimental it may have to be in a separate extension.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.