diff options
author | Ted Kremenek <kremenek@apple.com> | 2012-09-08 07:18:18 +0000 |
---|---|---|
committer | Ted Kremenek <kremenek@apple.com> | 2012-09-08 07:18:18 +0000 |
commit | e7ec4ef48d5b1e07769a9285f60a3a97d04ef56d (patch) | |
tree | e23b97e35b059f406baa393d35dfb2aeb13642ca /clang/lib/StaticAnalyzer/Core/BugReporter.cpp | |
parent | 639b8d05dd2432f0609774f7963b5780dd5366e1 (diff) | |
download | bcm5719-llvm-e7ec4ef48d5b1e07769a9285f60a3a97d04ef56d.tar.gz bcm5719-llvm-e7ec4ef48d5b1e07769a9285f60a3a97d04ef56d.zip |
Fix bug in BugReporter::RemoveUneededCalls() where "prunable"
PathDiagnosticEventPieces were *always* pruned. Instead, they
are suppose to only be pruned if the entire call gets pruned.
llvm-svn: 163460
Diffstat (limited to 'clang/lib/StaticAnalyzer/Core/BugReporter.cpp')
-rw-r--r-- | clang/lib/StaticAnalyzer/Core/BugReporter.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/clang/lib/StaticAnalyzer/Core/BugReporter.cpp b/clang/lib/StaticAnalyzer/Core/BugReporter.cpp index 68cc7d88b1a..80ece9a3b0a 100644 --- a/clang/lib/StaticAnalyzer/Core/BugReporter.cpp +++ b/clang/lib/StaticAnalyzer/Core/BugReporter.cpp @@ -158,9 +158,7 @@ bool BugReporter::RemoveUneededCalls(PathPieces &pieces, BugReport *R) { PathDiagnosticEventPiece *event = cast<PathDiagnosticEventPiece>(piece); // We never throw away an event, but we do throw it away wholesale // as part of a path if we throw the entire path away. - if (event->isPrunable()) - continue; - containsSomethingInteresting = true; + containsSomethingInteresting |= !event->isPrunable(); break; } case PathDiagnosticPiece::ControlFlow: |