Skip to content

Conversation

@aarongreig
Copy link
Contributor

The api now takes a ur_adapter_handle_t instead of a ur_platform_handle_t, which is easier for sycl RT to correctly provide and enables a fix for a few instances where we were illegally passing nullptr.

The api now takes a ur_adapter_handle_t instead of a
ur_platform_handle_t, which is easier for sycl RT to correctly provide
and enables a fix for a few instances where we were illegally passing
nullptr.
@omarahmed1111 omarahmed1111 force-pushed the aaron/updateURDeviceCreateWithNative branch from 55d261d to bbf2abe Compare August 15, 2024 16:53
@omarahmed1111 omarahmed1111 marked this pull request as ready for review August 15, 2024 16:54
@omarahmed1111 omarahmed1111 requested review from a team as code owners August 15, 2024 16:54
@omarahmed1111
Copy link
Contributor

@intel/llvm-reviewers-runtime Please review, Thanks!

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!

@omarahmed1111
Copy link
Contributor

@intel/llvm-gatekeepers Please merge, Thanks!

@steffenlarsen steffenlarsen merged commit 36347a3 into intel:sycl Aug 16, 2024
AlexeySachkov pushed a commit to AlexeySachkov/llvm that referenced this pull request Nov 26, 2024
intel#15023)

The api now takes a ur_adapter_handle_t instead of a
ur_platform_handle_t, which is easier for sycl RT to correctly provide
and enables a fix for a few instances where we were illegally passing
nullptr.
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