Skip to content

[SYCL] Throw right errc for work_group_size query #8645

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

Merged
merged 7 commits into from
Mar 16, 2023

Conversation

KornevNikita
Copy link
Contributor

clGetKernelWorkGroupInfo returns CL_INVALID_VALUE if param_name is CL_KERNEL_GLOBAL_WORK_SIZE and device is not a custom device and kernel is not a built-in kernel. According to SYCL2020 an exception with the errc::invalid error code should be thrown.

clGetKernelWorkGroupInfo returns CL_INVALID_VALUE if param_name is
CL_KERNEL_GLOBAL_WORK_SIZE and device is not a custom device and kernel
is not a built-in kernel. According to SYCL2020 an exception with the
errc::invalid error code should be thrown.
@KornevNikita KornevNikita requested a review from a team as a code owner March 14, 2023 15:50
@KornevNikita KornevNikita marked this pull request as draft March 14, 2023 15:53
@KornevNikita KornevNikita temporarily deployed to aws March 14, 2023 16:17 — with GitHub Actions Inactive
@KornevNikita
Copy link
Contributor Author

/verify with intel/llvm-test-suite#1660

@KornevNikita KornevNikita temporarily deployed to aws March 14, 2023 17:03 — with GitHub Actions Inactive
@KornevNikita KornevNikita temporarily deployed to aws March 14, 2023 23:36 — with GitHub Actions Inactive
@KornevNikita KornevNikita requested a review from a team March 15, 2023 10:06
@KornevNikita KornevNikita temporarily deployed to aws March 15, 2023 11:43 — with GitHub Actions Inactive
@KornevNikita KornevNikita temporarily deployed to aws March 15, 2023 12:10 — with GitHub Actions Inactive
@KornevNikita
Copy link
Contributor Author

/verify with intel/llvm-test-suite#1660

@KornevNikita KornevNikita temporarily deployed to aws March 15, 2023 19:32 — with GitHub Actions Inactive
@KornevNikita KornevNikita temporarily deployed to aws March 15, 2023 20:18 — with GitHub Actions Inactive
@KornevNikita
Copy link
Contributor Author

/verify with intel/llvm-test-suite#1660

@KornevNikita KornevNikita temporarily deployed to aws March 16, 2023 10:23 — with GitHub Actions Inactive
@KornevNikita
Copy link
Contributor Author

/verify with intel/llvm-test-suite#1660

@KornevNikita KornevNikita temporarily deployed to aws March 16, 2023 11:01 — with GitHub Actions Inactive
@KornevNikita KornevNikita temporarily deployed to aws March 16, 2023 11:30 — with GitHub Actions Inactive
Copy link
Contributor

@steffenlarsen steffenlarsen 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! 🐙

@steffenlarsen steffenlarsen merged commit 5b183b4 into intel:sycl Mar 16, 2023
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.

3 participants