-
Notifications
You must be signed in to change notification settings - Fork 131
Test the case where range is over allowed limit. #1500
Conversation
/verify with intel/llvm#7968 |
verify /intel/llvm#7968 @mmoadeli, the command format is /verify with <link to the PR> |
/verify with intel/llvm#7968 |
@mmoadeli Since this test works for HIP backend as well. May be: |
This will disable the test. There are no machines in CI where all three features are true at the same time. |
FYI: /verify with validates the change with OpenCL and Level Zero only. Tests requiring CUDA are not covered. |
Thanks @bader. Would |
This looks like we expect the test to run with any back end, so it's better just remove |
Thanks for the pointer. |
thanks @bader, @abagusetty, REQUIRES is removed. |
/verify with intel/llvm#7968 |
@intel/llvm-reviewers-runtime, @bso-intel, ping. |
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.
LGTM
intel#1402) (intel#1500) [SYCL] Reflect support of AMD, NVIDIA GPU architectures as argument of the -fsycl-targets. Co-authored-by: mmoadeli <mmoadeli@hotmail.co.uk>
Provides a test for intel/llvm#7968
…te#1500) Provides a test for intel#7968
Provides a test for
intel/llvm#7968