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

Rename urCommandBufferEnqueueExp to urEnqueueCommandBufferExp #2606

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

EwanC
Copy link
Contributor

@EwanC EwanC commented Jan 23, 2025

The API to enqueue a closed command-buffer to a queue is defined in the YAML as a part of the command-buffer class, but it should be part of the enqueue class like other enqueue API extensions.

This PR updates the YAML and regenerates UR code, making the associated changes to adapters and CTS.

DPC++ PR intel/llvm#16747
Closes #2600

@github-actions github-actions bot added loader Loader related feature/bug conformance Conformance test suite issues. specification Changes or additions to the specification experimental Experimental feature additions/changes/specification level-zero L0 adapter specific issues cuda CUDA adapter specific issues hip HIP adapter specific issues opencl OpenCL adapter specific issues native-cpu Native CPU adapter specific issues command-buffer Command Buffer feature addition/changes/specification labels Jan 23, 2025
@EwanC EwanC force-pushed the cmd-buf_enqueue_refactor branch from d96133f to aec66e8 Compare January 23, 2025 11:27
EwanC added a commit to EwanC/llvm that referenced this pull request Jan 23, 2025
Reflects change in name of UR entry-point from
`urCommandBufferEnqueueExp` to `urEnqueueCommandBufferExp`
in oneapi-src/unified-runtime#2606
The API to enqueue a closed command-buffer to a queue is defined
in the YAML as a part of the command-buffer class, but it should be
part of the enqueue class like other enqueue API extensions.

This PR updates the YAML and regenerates UR code, making the associated
changes to adapters and CTS.

Closes oneapi-src#2600
@EwanC EwanC force-pushed the cmd-buf_enqueue_refactor branch from aec66e8 to c62847a Compare January 23, 2025 11:56
@EwanC EwanC marked this pull request as ready for review January 23, 2025 13:37
@EwanC EwanC requested review from a team as code owners January 23, 2025 13:37
@EwanC EwanC requested review from reble and omarahmed1111 January 23, 2025 13:37
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 conformance Conformance test suite issues. cuda CUDA adapter specific issues experimental Experimental feature additions/changes/specification hip HIP adapter specific issues level-zero L0 adapter specific issues loader Loader related feature/bug native-cpu Native CPU adapter specific issues opencl OpenCL adapter specific issues specification Changes or additions to the specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename urCommandBufferEnqueueExp to urEnqueueCommandBufferExp
3 participants