================
@@ -5558,8 +5558,8 @@ VisitAbstractConditionalOperator(const
AbstractConditionalOperator *E) {
// OpenCL: If the condition is a vector, we can treat this condition like
// the select function.
- if ((CGF.getLangOpts().OpenCL && condExpr->getType()->isVectorType()) ||
- condExpr->getType()->isExtVectorType()) {
+ if (CGF.getLangOpts().OpenCL && (condExpr->getType()->isVectorType() ||
+ condExpr->getType()->isExtVectorType())) {
----------------
jhuber6 wrote:
That's true, it changes the codegen and behavior slightly, though I honestly
don't know in practice who would ever trip the difference between the MSB and
-1 as their mask. I'd prefer to have more generically useful vectors than be
shackled to OpenCL decisions outside of OpenCL mode, but I'll cede to others on
these issues. I can make this only apply to bitvectors if needed, but I would
prefer not to.
https://github.com/llvm/llvm-project/pull/154145
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
