-
Notifications
You must be signed in to change notification settings - Fork 131
[SYCL][CUDA] Re-enable tests that were marked as UNSUPPORTED for the CUDA backend #1317
[SYCL][CUDA] Re-enable tests that were marked as UNSUPPORTED for the CUDA backend #1317
Conversation
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.
Sub-group changes LGTM.
I wanted to check in on the status of this PR. It has passed all tests, and I believe it is ready to be merged. Is there any blocker preventing this from being reviewed or merged? |
Code review from @intel/llvm-reviewers-runtime team is missing. |
We can merge this PR now; I'm not authorized to do so, could you @bader, please? |
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.
Great!
…_cmd-topic [CMPLRTST-18383] valgrind command needs to be optimized by adding "--track-origins=yes"
…CUDA backend (intel#1317) This PR re-enables the test marked as UNSUPPORTED for the CUDA backend. As a result of these tests ran to update this issue: intel#249; Specifically, the part which mentions the tests that passed but are marked as unsupported; I'm proposing this PR to re-enable them.
…CUDA backend (intel/llvm-test-suite#1317) This PR re-enables the test marked as UNSUPPORTED for the CUDA backend. As a result of these tests ran to update this issue: intel/llvm-test-suite#249; Specifically, the part which mentions the tests that passed but are marked as unsupported; I'm proposing this PR to re-enable them.
This PR re-enables the test marked as UNSUPPORTED for the CUDA backend.
As a result of these tests ran to update this issue: #249; Specifically, the part which mentions the tests that passed but are marked as unsupported;
I'm proposing this PR to re-enable them.