Skip to content

Conversation

gmlueck
Copy link
Contributor

@gmlueck gmlueck commented Sep 5, 2025

Specify a simplified way to enqueue free function kernels without first getting a kernel bundle. The style follows the enqueue functions in "sycl_ext_oneapi_enqueue_functions".

Once the KHR extension "sycl_khr_free_function_commands" is finalized, we may also add enqueue functions that follow that style. However, we will first need to extend the requirements in that KHR to accept kernel launch properties.

Specify a simplified way to enqueue free function kernels without first
getting a kernel bundle.  The style follows the enqueue functions in
"sycl_ext_oneapi_enqueue_functions".

Once the KHR extension "sycl_khr_free_function_commands" is finalized,
we may also add enqueue functions that follow that style.  However, we
will first need to extend the `requirements` in that KHR to accept
kernel launch properties.
@gmlueck gmlueck requested a review from a team as a code owner September 5, 2025 21:30
Move this note earlier in the specification, so it appears before all
APIs that take a `Func` parameter.
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.

LGTM!

Copy link
Contributor

@intel/llvm-gatekeepers please consider merging

3 similar comments
Copy link
Contributor

github-actions bot commented Oct 3, 2025

@intel/llvm-gatekeepers please consider merging

Copy link
Contributor

github-actions bot commented Oct 6, 2025

@intel/llvm-gatekeepers please consider merging

Copy link
Contributor

github-actions bot commented Oct 9, 2025

@intel/llvm-gatekeepers please consider merging

@gmlueck
Copy link
Contributor Author

gmlueck commented Oct 10, 2025

@intel/llvm-gatekeepers I think this is ready to merge.

@sarnex sarnex merged commit dfd72fc into intel:sycl Oct 10, 2025
4 checks passed
@gmlueck gmlueck deleted the gmlueck/free-function-launch branch October 10, 2025 20:41
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