Skip to content

Conversation

npmiller
Copy link
Contributor

There's no need for a wrapper here we can directly throw ur_result_t.

There's no need for a wrapper here we can directly throw `ur_result_t`.
@npmiller npmiller requested a review from a team as a code owner April 14, 2025 16:56
@npmiller npmiller requested a review from Seanst98 April 14, 2025 16:56
Copy link
Contributor

@ldrumm ldrumm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice cleanup!

@npmiller
Copy link
Contributor Author

@intel/llvm-gatekeepers this is ready to merge

@kbenzie kbenzie merged commit df87c03 into intel:sycl Apr 15, 2025
34 checks passed
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