Skip to content

Conversation

@EwanC
Copy link
Contributor

@EwanC EwanC commented Apr 23, 2025

The merged PR #17658 added the requirement for OpenCL cl_khr_command_buffer implementations to support the simultaneous use capability in order to support UR command-buffers.

There was an oversight in this PR where the simultaneous use property wasn't set on command-buffer creation in the OpenCL adapter. This is rectified in this patch.

The merged PR intel#17658 added the
requirement for OpenCL cl_khr_command_buffer implementations to support
the simultaneous use capability in order to support UR command-buffers.

There was an oversight in this PR where the simultaneous use property
wasn't set on command-buffer creation in the OpenCL adapter. This is
rectified in this patch.
@EwanC EwanC requested a review from a team as a code owner April 23, 2025 10:26
@EwanC EwanC requested a review from fabiomestre April 23, 2025 10:26
@EwanC EwanC temporarily deployed to WindowsCILock April 23, 2025 10:27 — with GitHub Actions Inactive
@EwanC EwanC temporarily deployed to WindowsCILock April 23, 2025 10:47 — with GitHub Actions Inactive
@EwanC EwanC temporarily deployed to WindowsCILock April 23, 2025 10:57 — with GitHub Actions Inactive
@EwanC EwanC temporarily deployed to WindowsCILock April 23, 2025 10:57 — with GitHub Actions Inactive
@EwanC
Copy link
Contributor Author

EwanC commented Apr 24, 2025

@intel/llvm-gatekeepers This is ready to merge, thanks

@sommerlukas sommerlukas merged commit 99cd1ad into intel:sycl Apr 24, 2025
38 of 39 checks passed
KornevNikita pushed a commit that referenced this pull request May 27, 2025
The merged PR #17658 added the
requirement for OpenCL cl_khr_command_buffer implementations to support
the simultaneous use capability in order to support UR command-buffers.

There was an oversight in this PR where the simultaneous use property
wasn't set on command-buffer creation in the OpenCL adapter. This is
rectified in this patch.
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