diff options
| author | Anna Zaks <ganna@apple.com> | 2013-06-08 00:29:24 +0000 |
|---|---|---|
| committer | Anna Zaks <ganna@apple.com> | 2013-06-08 00:29:24 +0000 |
| commit | 22895473af4cd87cd00a44410a2d383d2784974e (patch) | |
| tree | 0216a17504b6b3b836d0767362a925f6cc4db0d5 /clang/lib/StaticAnalyzer/Core/BugReporter.cpp | |
| parent | abfeabb565cf24c09ba9d7ec8faf0f1329bb8015 (diff) | |
| download | bcm5719-llvm-22895473af4cd87cd00a44410a2d383d2784974e.tar.gz bcm5719-llvm-22895473af4cd87cd00a44410a2d383d2784974e.zip | |
[analyzer; alternate edges] Fix the edge locations in presence of macros.
We drew the diagnostic edges to wrong statements in cases the note was on a macro.
The fix is simple, but seems to work just fine for a whole bunch of test cases (plist-macros.cpp).
Also, removes an unnecessary edge in edges-new.mm, when function signature starts with a macro.
llvm-svn: 183599
Diffstat (limited to 'clang/lib/StaticAnalyzer/Core/BugReporter.cpp')
| -rw-r--r-- | clang/lib/StaticAnalyzer/Core/BugReporter.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/StaticAnalyzer/Core/BugReporter.cpp b/clang/lib/StaticAnalyzer/Core/BugReporter.cpp index e93bb3fce5e..dff7d8e59ee 100644 --- a/clang/lib/StaticAnalyzer/Core/BugReporter.cpp +++ b/clang/lib/StaticAnalyzer/Core/BugReporter.cpp @@ -1562,7 +1562,7 @@ static void addEdgeToPath(PathPieces &path, return; SourceLocation NewLocL = NewLoc.asLocation(); - if (NewLocL.isInvalid() || NewLocL.isMacroID()) + if (NewLocL.isInvalid()) return; if (!PrevLoc.isValid() || !PrevLoc.asLocation().isValid()) { |
