diff options
| author | Richard Trieu <rtrieu@google.com> | 2014-08-23 00:30:57 +0000 |
|---|---|---|
| committer | Richard Trieu <rtrieu@google.com> | 2014-08-23 00:30:57 +0000 |
| commit | 7ec1a318ff6b86f3c06a24dc933cb1d33b187ba1 (patch) | |
| tree | 765588ac3e81bbedfdd3a4ba2852f8e71b4178a3 /clang/lib/Sema/SemaExpr.cpp | |
| parent | e3eface17ac15ff3818e76e4fcadef5afad92f2a (diff) | |
| download | bcm5719-llvm-7ec1a318ff6b86f3c06a24dc933cb1d33b187ba1.tar.gz bcm5719-llvm-7ec1a318ff6b86f3c06a24dc933cb1d33b187ba1.zip | |
Fix a bad location in -Wparentheses fix-it hint
The code used getLocStart() instead of getLocEnd(). This works for single
token expressions, but breaks if the expression is longer.
llvm-svn: 216306
Diffstat (limited to 'clang/lib/Sema/SemaExpr.cpp')
| -rw-r--r-- | clang/lib/Sema/SemaExpr.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp index b74429ab557..5bb042ce3da 100644 --- a/clang/lib/Sema/SemaExpr.cpp +++ b/clang/lib/Sema/SemaExpr.cpp @@ -9515,7 +9515,7 @@ static void DiagnoseBitwisePrecedence(Sema &Self, BinaryOperatorKind Opc, StringRef OpStr = isLeftComp ? LHSBO->getOpcodeStr() : RHSBO->getOpcodeStr(); SourceRange ParensRange = isLeftComp ? SourceRange(LHSBO->getRHS()->getLocStart(), RHSExpr->getLocEnd()) - : SourceRange(LHSExpr->getLocStart(), RHSBO->getLHS()->getLocStart()); + : SourceRange(LHSExpr->getLocStart(), RHSBO->getLHS()->getLocEnd()); Self.Diag(OpLoc, diag::warn_precedence_bitwise_rel) << DiagRange << BinaryOperator::getOpcodeStr(Opc) << OpStr; |
