-
Notifications
You must be signed in to change notification settings - Fork 734
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
[SYCL][COMPAT] Add math extend_vcompare[2/4] to SYCLCompat #14079
[SYCL][COMPAT] Add math extend_vcompare[2/4] to SYCLCompat #14079
Conversation
Co-authored-by: Alberto Cabrera Pérez <alberto.cabrera@intel.com>
Co-authored-by: Joe Todd <joe.todd@codeplay.com>
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.
Very minor stuff in the docs, otherwise LGTM, The review assumes that extend_vbinary2
and extend_vbinary4
are gonna be reviewed in their respective PRs.
Co-authored-by: Alberto Cabrera Pérez <alberto.cabrera@intel.com>
Co-authored-by: Yihan Wang <yihan.wang@intel.com>
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
Due to the timeout of the |
@intel/llvm-gatekeepers PR should be ready! Thanks. |
This PR adds math
extend_vcompare[2/4]
operators (4 in total) along with unit-tests for signed and unsigned int32 cases.Also, Unit-tests from previous
extend_v*4
#14078 andextend_v*2
#13953 are moved to two different files.