Skip to content

[SYCL][UR][L0 v2] Implement missing bindless images functionality #19808

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 1 commit into from
Aug 15, 2025

Conversation

igchor
Copy link
Member

@igchor igchor commented Aug 14, 2025

Implement urBindlessImagesSignalExternalSemaphoreExp and urBindlessImagesWaitExternalSemaphoreExp. The implementation is the same as in the legace adapter, except that we don't to a fallback in case old loader is used (v2 is supposed to be used with a new enough L0 loader version).

@igchor igchor requested a review from a team as a code owner August 14, 2025 16:13
Implement urBindlessImagesSignalExternalSemaphoreExp and
urBindlessImagesWaitExternalSemaphoreExp. The implementation
is the same as in the legace adapter, except that we don't to
a fallback in case old loader is used (v2 is supposed to be used
with a new enough L0 loader version).
@igchor igchor requested a review from nrspruit August 14, 2025 17:40
Copy link
Contributor

@nrspruit nrspruit 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 to me, thank you for the implementation!

@nrspruit
Copy link
Contributor

@intel/llvm-gatekeepers , please merge when available.

@againull againull merged commit cbea600 into intel:sycl Aug 15, 2025
40 of 41 checks passed
@igchor igchor deleted the bindless_v2 branch August 15, 2025 14:34
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