diff options
| author | Fariborz Jahanian <fjahanian@apple.com> | 2014-12-18 23:14:51 +0000 |
|---|---|---|
| committer | Fariborz Jahanian <fjahanian@apple.com> | 2014-12-18 23:14:51 +0000 |
| commit | 0fc95ad7584eacbf5b82a4890709e54b44318351 (patch) | |
| tree | ed08146e587c6f11dd2a68a4bc3eea5d452cdc02 /clang/lib | |
| parent | a2bd8d98a18e4927b247dc2443b2a63d0c0feb24 (diff) | |
| download | bcm5719-llvm-0fc95ad7584eacbf5b82a4890709e54b44318351.tar.gz bcm5719-llvm-0fc95ad7584eacbf5b82a4890709e54b44318351.zip | |
[c Sema]. Patch fixes pointer-bool-conversion warning on C code
when source range is incorrect causing the warning to be
issued when it should not because expression is in a macro.
rdar://19256338
llvm-svn: 224549
Diffstat (limited to 'clang/lib')
| -rw-r--r-- | clang/lib/Sema/SemaChecking.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/clang/lib/Sema/SemaChecking.cpp b/clang/lib/Sema/SemaChecking.cpp index 550474107c0..55de70826bc 100644 --- a/clang/lib/Sema/SemaChecking.cpp +++ b/clang/lib/Sema/SemaChecking.cpp @@ -6685,11 +6685,11 @@ void AnalyzeImplicitConversions(Sema &S, Expr *OrigE, SourceLocation CC) { if (BO && BO->isLogicalOp()) { Expr *SubExpr = BO->getLHS()->IgnoreParenImpCasts(); if (!IsLogicalAndOperator || !isa<StringLiteral>(SubExpr)) - ::CheckBoolLikeConversion(S, SubExpr, SubExpr->getExprLoc()); + ::CheckBoolLikeConversion(S, SubExpr, BO->getExprLoc()); SubExpr = BO->getRHS()->IgnoreParenImpCasts(); if (!IsLogicalAndOperator || !isa<StringLiteral>(SubExpr)) - ::CheckBoolLikeConversion(S, SubExpr, SubExpr->getExprLoc()); + ::CheckBoolLikeConversion(S, SubExpr, BO->getExprLoc()); } if (const UnaryOperator *U = dyn_cast<UnaryOperator>(E)) |
