summaryrefslogtreecommitdiffstats
path: root/clang/lib/StaticAnalyzer/Core/BugReporter.cpp
diff options
context:
space:
mode:
authorTed Kremenek <kremenek@apple.com>2012-09-08 07:18:18 +0000
committerTed Kremenek <kremenek@apple.com>2012-09-08 07:18:18 +0000
commite7ec4ef48d5b1e07769a9285f60a3a97d04ef56d (patch)
treee23b97e35b059f406baa393d35dfb2aeb13642ca /clang/lib/StaticAnalyzer/Core/BugReporter.cpp
parent639b8d05dd2432f0609774f7963b5780dd5366e1 (diff)
downloadbcm5719-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.cpp4
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:
OpenPOWER on IntegriCloud