diff options
| author | Aaron Puchert <aaronpuchert@alice-dsl.net> | 2020-07-26 01:53:32 +0200 |
|---|---|---|
| committer | Aaron Puchert <aaronpuchert@alice-dsl.net> | 2020-09-05 17:26:12 +0200 |
| commit | b2ce79ef66157dd752e3864ece57915e23a73f5d (patch) | |
| tree | 03cdb802ddd79e64c839710673b0062b07f4dd36 /clang/lib/Analysis/ThreadSafety.cpp | |
| parent | 9dcc82f34ea9b623d82d2577b93aaf67d36dabd2 (diff) | |
| download | llvm-b2ce79ef66157dd752e3864ece57915e23a73f5d.zip llvm-b2ce79ef66157dd752e3864ece57915e23a73f5d.tar.gz llvm-b2ce79ef66157dd752e3864ece57915e23a73f5d.tar.bz2 | |
Thread safety analysis: ValueDecl in Project is non-null
The constructor asserts that, use it in the ThreadSafetyAnalyzer.
Also note that the result of a cast<> cannot be null.
Diffstat (limited to 'clang/lib/Analysis/ThreadSafety.cpp')
| -rw-r--r-- | clang/lib/Analysis/ThreadSafety.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/clang/lib/Analysis/ThreadSafety.cpp b/clang/lib/Analysis/ThreadSafety.cpp index 1d4aaba..5b97265 100644 --- a/clang/lib/Analysis/ThreadSafety.cpp +++ b/clang/lib/Analysis/ThreadSafety.cpp @@ -1277,9 +1277,8 @@ bool ThreadSafetyAnalyzer::inCurrentScope(const CapabilityExpr &CapE) { if (const auto *P = dyn_cast<til::Project>(SExp)) { if (!CurrentMethod) return false; - const auto *VD = P->clangDecl(); - if (VD) - return VD->getDeclContext() == CurrentMethod->getDeclContext(); + const ValueDecl *VD = P->clangDecl(); + return VD->getDeclContext() == CurrentMethod->getDeclContext(); } return false; |