Skip to content

Conversation

@EwanC
Copy link
Contributor

@EwanC EwanC commented Sep 16, 2024

Update OpenCL adapter code to reflect the 2 API breaking changes to the command-buffer family of extensions that have been made upstream:

PR also contains fixes for CTS tests that have been added recently:

  • Test passing a null handle to urCommandBufferAppendKernelLaunchExp checks for nullptr
  • Test urCommandBufferAppendMemBufferWriteExp doesn't die (other unsupported commands in the adapter don't do this)

DPC++ PR intel/llvm#15410

@github-actions github-actions bot added the command-buffer Command Buffer feature addition/changes/specification label Sep 16, 2024
Update OpenCL adapter code to reflect the 2 API breaking changes
to the command-buffer family of extensions that have been
made upstream:

* [Add properties parameter to all command-buffer commands](KhronosGroup/OpenCL-Headers#260)
* [Use array for
  clUpdateMutableCommandsKHR](KhronosGroup/OpenCL-Headers#245)
@EwanC EwanC force-pushed the ewan/update_opencl_headers branch from 050919c to 762eb9f Compare September 16, 2024 13:09
@EwanC EwanC marked this pull request as ready for review September 17, 2024 08:22
@EwanC EwanC requested review from a team as code owners September 17, 2024 08:22
EwanC pushed a commit to reble/llvm that referenced this pull request Sep 17, 2024
Update UR commit for L0 adapter to use most recent Khronos
cl_khr_command_buffer APIs oneapi-src/unified-runtime#2095
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

command-buffer Command Buffer feature addition/changes/specification ready to merge Added to PR's which are ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants