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

[OpenCL] Add more mappings from CL error codes to UR error codes. #975

Closed

Conversation

aarongreig
Copy link
Contributor

Also merge urQueueCreate InvalidValueProperties test into InvalidQueueProperties test.

@aarongreig aarongreig requested review from a team as code owners October 20, 2023 08:59
Also merge urQueueCreate InvalidValueProperties test into
InvalidQueueProperties test.
@kbenzie
Copy link
Contributor

kbenzie commented Oct 20, 2023

Looks like this is now passing in the CUDA adapter. Need to update the match file.

urQueueCreateTest.InvalidValueProperties/NVIDIA_CUDA_BACKEND___{{.*}}_

Copy link
Contributor

@fabiomestre fabiomestre left a comment

Choose a reason for hiding this comment

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

LGTM

@kbenzie kbenzie added the ready to merge Added to PR's which are ready to merge label Oct 20, 2023
@kbenzie
Copy link
Contributor

kbenzie commented Nov 2, 2023

@aarongreig there does not appear to be an https://github.com/intel/llvm PR for this as per the Adapter Change Process

@kbenzie kbenzie removed the ready to merge Added to PR's which are ready to merge label Nov 2, 2023
@aarongreig
Copy link
Contributor Author

forgot to do that before I went on holiday, is it worth doing now? this is technically a cts fix so it could wait to go in with the rest of them

@kbenzie
Copy link
Contributor

kbenzie commented Nov 2, 2023

forgot to do that before I went on holiday, is it worth doing now? this is technically a cts fix so it could wait to go in with the rest of them

Okay, we can test it in the combined branch.

@aarongreig aarongreig added the conformance Conformance test suite issues. label Nov 2, 2023
@aarongreig aarongreig closed this Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conformance Conformance test suite issues.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants