diff options
author | David Bolvansky <david.bolvansky@gmail.com> | 2019-10-05 13:28:15 +0000 |
---|---|---|
committer | David Bolvansky <david.bolvansky@gmail.com> | 2019-10-05 13:28:15 +0000 |
commit | 83b81c1f6ee878ffe458aee3162ce0ef004d3374 (patch) | |
tree | 92c19482736ef76038fbd2e857345f1d1d87d895 /clang/lib/Sema/SemaExpr.cpp | |
parent | f4cc3b3e10edd5c8d9b23f1d6806829d20b2c9df (diff) | |
download | bcm5719-llvm-83b81c1f6ee878ffe458aee3162ce0ef004d3374.tar.gz bcm5719-llvm-83b81c1f6ee878ffe458aee3162ce0ef004d3374.zip |
[Diagnostics] Highlight expr's source range for -Wbool-operation
Warning message looks better; and GCC adds it too.
llvm-svn: 373828
Diffstat (limited to 'clang/lib/Sema/SemaExpr.cpp')
-rw-r--r-- | clang/lib/Sema/SemaExpr.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp index c18f54cefde..3cb999dacc4 100644 --- a/clang/lib/Sema/SemaExpr.cpp +++ b/clang/lib/Sema/SemaExpr.cpp @@ -13481,6 +13481,7 @@ ExprResult Sema::CreateBuiltinUnaryOp(SourceLocation OpLoc, << resultType << Input.get()->getSourceRange(); else if (Input.get()->isKnownToHaveBooleanValue()) Diag(OpLoc, diag::warn_bitwise_negation_bool) + << Input.get()->getSourceRange() << FixItHint::CreateReplacement(OpLoc, "!"); else if (resultType->hasIntegerRepresentation()) break; |