-
Notifications
You must be signed in to change notification settings - Fork 123
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
Clarify spec around isNativeHandleOwned. #2193
Clarify spec around isNativeHandleOwned. #2193
Conversation
fc0039a
to
e456967
Compare
1724bae
to
a955bec
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cuda/hip 👍
6b90beb
to
ad1d903
Compare
ad1d903
to
6664a86
Compare
ping @oneapi-src/unified-runtime-opencl-write |
6664a86
to
0ba9457
Compare
f5d6491
to
48c91d1
Compare
…switch statement in one test. Remove tests for owned/unowned native queue, as it is being handled in oneapi-src#2193.
…switch statement in one test. Remove tests for owned/unowned native queue, as it is being handled in oneapi-src#2193.
…switch statement in one test. Remove tests for owned/unowned native queue, as it is being handled in oneapi-src#2193.
…switch statement in one test. Remove tests for owned/unowned native queue, as it is being handled in oneapi-src#2193.
…switch statement in one test. Remove tests for owned/unowned native queue, as it is being handled in oneapi-src#2193.
48c91d1
to
1bf1809
Compare
1bf1809
to
b4e54fd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to fix the typos but otherwise LGTM.
I've decided this isn't quite NFC enough to dodge a tag bump intel/llvm#16730 |
) oneapi-src/unified-runtime#2193 --------- Co-authored-by: Kenneth Benzie (Benie) <[email protected]>
No description provided.