simoll added inline comments.
================ Comment at: clang/lib/Sema/SemaExpr.cpp:10244-10245 + // This operation may not be performed on boolean vectors. + if (!AllowBoolOperation && (isBoolOrExtVectorBoolType(LHSType) && + isBoolOrExtVectorBoolType(RHSType))) + return ReportInvalid ? InvalidOperands(Loc, LHS, RHS) : QualType(); ---------------- aaron.ballman wrote: > No need for the parens, but should that `&&` have been `||`? The `&&` is intentional. We cannot step into this if `AllowBoolOperation` is true. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88905/new/ https://reviews.llvm.org/D88905 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
