diff options
| author | Erik Pilkington <erik.pilkington@gmail.com> | 2019-11-20 15:39:22 -0800 |
|---|---|---|
| committer | Erik Pilkington <erik.pilkington@gmail.com> | 2019-11-20 16:29:31 -0800 |
| commit | d9957c7405bc726422dbc2736ad62f704916fbe8 (patch) | |
| tree | 986901d3ddc31d5daf7b31d983963a417d3ceaf4 /clang/lib/Sema/SemaChecking.cpp | |
| parent | a0da875a3ace9f4d4ade54f7a4764e327169a629 (diff) | |
| download | bcm5719-llvm-d9957c7405bc726422dbc2736ad62f704916fbe8.tar.gz bcm5719-llvm-d9957c7405bc726422dbc2736ad62f704916fbe8.zip | |
[Sema] Add a 'Semantic' parameter to Expr::isKnownToHaveBooleanValue
Some clients of this function want to know about any expression that is known
to produce a 0/1 value, and others care about expressions that are semantically
boolean.
This fixes a -Wswitch-bool regression I introduced in 8bfb353bb33c, pointed out
by Chris Hamilton!
Diffstat (limited to 'clang/lib/Sema/SemaChecking.cpp')
| -rw-r--r-- | clang/lib/Sema/SemaChecking.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaChecking.cpp b/clang/lib/Sema/SemaChecking.cpp index 65e4112d5e5..a4218add4e7 100644 --- a/clang/lib/Sema/SemaChecking.cpp +++ b/clang/lib/Sema/SemaChecking.cpp @@ -11845,7 +11845,7 @@ static void CheckImplicitConversion(Sema &S, Expr *E, QualType T, return; if (isObjCSignedCharBool(S, T) && !Source->isCharType() && - !E->isKnownToHaveBooleanValue()) { + !E->isKnownToHaveBooleanValue(/*Semantic=*/false)) { return adornObjCBoolConversionDiagWithTernaryFixit( S, E, S.Diag(CC, diag::warn_impcast_int_to_objc_signed_char_bool) |
