diff options
| author | George Karpenkov <ekarpenkov@apple.com> | 2018-01-02 23:05:47 +0000 |
|---|---|---|
| committer | George Karpenkov <ekarpenkov@apple.com> | 2018-01-02 23:05:47 +0000 |
| commit | 96625fdc6bff9f9bbf96239860a501778623b680 (patch) | |
| tree | fe1b39b9f57aa82cd3e10acfc15df6f1d5297277 /clang/lib/StaticAnalyzer/Core/BugReporter.cpp | |
| parent | 30d927a12805189a6e30efe8e917295fe3b92451 (diff) | |
| download | bcm5719-llvm-96625fdc6bff9f9bbf96239860a501778623b680.tar.gz bcm5719-llvm-96625fdc6bff9f9bbf96239860a501778623b680.zip | |
[analyzer] do not crash with assertion on processing locations of bodyfarmed functions
This addresses an issue introduced in r183451: since
`removePiecesWithInvalidLocations` is called *after* `adjustCallLocations`,
it is not necessary, and in fact harmful, to have this assertion in
adjustCallLocations.
Addresses rdar://36170689
Differential Revision: https://reviews.llvm.org/D41680
llvm-svn: 321682
Diffstat (limited to 'clang/lib/StaticAnalyzer/Core/BugReporter.cpp')
| -rw-r--r-- | clang/lib/StaticAnalyzer/Core/BugReporter.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/clang/lib/StaticAnalyzer/Core/BugReporter.cpp b/clang/lib/StaticAnalyzer/Core/BugReporter.cpp index dc284888eb0..de0e87bb824 100644 --- a/clang/lib/StaticAnalyzer/Core/BugReporter.cpp +++ b/clang/lib/StaticAnalyzer/Core/BugReporter.cpp @@ -229,7 +229,6 @@ adjustCallLocations(PathPieces &Pieces, PathDiagnosticCallPiece *Call = dyn_cast<PathDiagnosticCallPiece>(I->get()); if (!Call) { - assert((*I)->getLocation().asLocation().isValid()); continue; } |
