[NFC] Strenghten preconditions for warning
authorDavid Bolvansky <david.bolvansky@gmail.com>
Tue, 24 Sep 2019 20:10:57 +0000 (20:10 +0000)
committerDavid Bolvansky <david.bolvansky@gmail.com>
Tue, 24 Sep 2019 20:10:57 +0000 (20:10 +0000)
llvm-svn: 372775

clang/lib/Sema/SemaChecking.cpp

index 02c0d48..28f4e06 100644 (file)
@@ -11321,7 +11321,8 @@ static void DiagnoseIntInBoolContext(Sema &S, Expr *E) {
       const auto *RHS = getIntegerLiteral(BO->getRHS());
       if (LHS && LHS->getValue() == 0)
         S.Diag(ExprLoc, diag::warn_left_shift_always) << 0;
-      else if (RHS && RHS->getValue().isNonNegative() &&
+      else if (!E->isValueDependent() && LHS && RHS &&
+               RHS->getValue().isNonNegative() &&
                E->EvaluateAsInt(Result, S.Context, Expr::SE_AllowSideEffects))
         S.Diag(ExprLoc, diag::warn_left_shift_always)
             << (Result.Val.getInt() != 0);