-
Notifications
You must be signed in to change notification settings - Fork 741
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] Extended vectorized_binary support to logical operators + Tests #15759
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.
Thanks for finding out why this didn't work @OuadiElfarouki!
@intel/llvm-gatekeepers This should be ready, thank you! |
@OuadiElfarouki got some merge conflicts to resolve first |
224531e
to
7a87e96
Compare
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/llvm-gatekeepers Hi again, it should be ready now. Thanks again. |
We add support for logical operators with
vectorized_binary
as well as the relevant unit-tests.