Author: Simon Pilgrim Date: 2020-10-08T20:02:19+01:00 New Revision: 85681131019db5d6831fb3980aec683d517e1674
URL: https://github.com/llvm/llvm-project/commit/85681131019db5d6831fb3980aec683d517e1674 DIFF: https://github.com/llvm/llvm-project/commit/85681131019db5d6831fb3980aec683d517e1674.diff LOG: Fix Wparentheses warning. NFCI. Wrap the containErrors() calls together - assert we have any containErrors cases in the conditional operator. Added: Modified: clang/lib/Sema/SemaExpr.cpp Removed: ################################################################################ diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp index 17bb82af975f..dd2e8f8c0d25 100644 --- a/clang/lib/Sema/SemaExpr.cpp +++ b/clang/lib/Sema/SemaExpr.cpp @@ -8071,9 +8071,9 @@ QualType Sema::CheckConditionalOperands(ExprResult &Cond, ExprResult &LHS, (Cond.get()->isTypeDependent() || LHS.get()->isTypeDependent() || RHS.get()->isTypeDependent())) { assert(!getLangOpts().CPlusPlus); - assert(Cond.get()->containsErrors() || LHS.get()->containsErrors() || - RHS.get()->containsErrors() && - "should only occur in error-recovery path."); + assert((Cond.get()->containsErrors() || LHS.get()->containsErrors() || + RHS.get()->containsErrors()) && + "should only occur in error-recovery path."); return Context.DependentTy; } _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits