Skip to content
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

Clarify max work-group size #548

Merged
merged 4 commits into from
Jun 27, 2024

Conversation

gmlueck
Copy link
Contributor

@gmlueck gmlueck commented Apr 4, 2024

Clarify the two information descriptors:

  • info::device::max_work_group_size
  • info::kernel_device_specific::work_group_size

These queries correspond to the OpenCL queries
CL_DEVICE_MAX_WORK_GROUP_SIZE and CL_KERNEL_WORK_GROUP_SIZE, and I shamelessly stole much of the description from the OpenCL specification.

Clarify the two information descriptors:

* `info::device::max_work_group_size`
* `info::kernel_device_specific::work_group_size`

These queries correspond to the OpenCL queries
`CL_DEVICE_MAX_WORK_GROUP_SIZE` and `CL_KERNEL_WORK_GROUP_SIZE`, and I
shamelessly stole much of the description from the OpenCL specification.
@gmlueck
Copy link
Contributor Author

gmlueck commented May 16, 2024

We need some more reviewers on this also.

Copy link
Collaborator

@AerialMantis AerialMantis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just a minor suggestion on the wording.

@TApplencourt
Copy link
Contributor

Nevin was faster than me!

Copy link
Contributor

@illuhad illuhad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tomdeakin tomdeakin merged commit 6636940 into KhronosGroup:SYCL-2020/master Jun 27, 2024
2 checks passed
@gmlueck gmlueck deleted the gmlueck/max-wg-size branch June 27, 2024 17:35
keryell pushed a commit that referenced this pull request Sep 10, 2024
gmlueck pushed a commit that referenced this pull request Nov 7, 2024
Clarify max work-group size

(cherry picked from commit 6636940)
gmlueck pushed a commit that referenced this pull request Nov 7, 2024
Clarify max work-group size

(cherry picked from commit 6636940)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants