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

[ur] Allow native handles to be unsupported #724

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

kbenzie
Copy link
Contributor

@kbenzie kbenzie commented Jul 17, 2023

Some adapters do not map directly to Unified Runtime's set of API objects and as such are unable to support getting the underlying native handle or creating objects from native handles. This patch adds the UR_RESULT_ERROR_UNSUPPORTED_FEATURE to all native handle entry-points and updates the conformance tests to match.

@kbenzie kbenzie changed the title benie/unsupported native handles [ur] Allow native handles to be unsupported Jul 17, 2023
@kbenzie kbenzie force-pushed the benie/unsupported-native-handles branch 2 times, most recently from c2fc29f to c502d99 Compare July 18, 2023 09:56
@kbenzie kbenzie marked this pull request as ready for review July 18, 2023 09:56
@kbenzie kbenzie requested a review from aarongreig July 18, 2023 09:57
Some adapters do not map directly to Unified Runtime's set of API
objects and as such are unable to support getting the underlying native
handle or creating objects from native handles. This patch adds the
`UR_RESULT_ERROR_UNSUPPORTED_FEATURE` to all native handle entry-points
and updates the conformance tests to match.
@kbenzie kbenzie force-pushed the benie/unsupported-native-handles branch from c502d99 to c1ee9f3 Compare July 19, 2023 12:23
@kbenzie kbenzie merged commit a00e9ba into oneapi-src:main Jul 19, 2023
@kbenzie kbenzie deleted the benie/unsupported-native-handles branch January 29, 2024 13:49
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