diff options
| author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2020-01-09 12:05:30 +0000 |
|---|---|---|
| committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2020-01-09 12:05:48 +0000 |
| commit | e315ce2bd3a5347ab58d50fab2d449e8a4e60e00 (patch) | |
| tree | bf292620b603bf68cf26b18b8e0027029742ab07 /clang/lib/Sema | |
| parent | e3e72a2619ccec9654ef9ad45e1e926841fbaa58 (diff) | |
| download | bcm5719-llvm-e315ce2bd3a5347ab58d50fab2d449e8a4e60e00.tar.gz bcm5719-llvm-e315ce2bd3a5347ab58d50fab2d449e8a4e60e00.zip | |
Fix "pointer is null" static analyzer warnings. NFCI.
Use castAs<> instead of getAs<> since the pointer is dereferenced immediately below and castAs will perform the null assertion for us.
Diffstat (limited to 'clang/lib/Sema')
| -rw-r--r-- | clang/lib/Sema/SemaCoroutine.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/clang/lib/Sema/SemaCoroutine.cpp b/clang/lib/Sema/SemaCoroutine.cpp index 271c4a10f3e..6dc9e342beb 100644 --- a/clang/lib/Sema/SemaCoroutine.cpp +++ b/clang/lib/Sema/SemaCoroutine.cpp @@ -1228,7 +1228,7 @@ bool CoroutineStmtBuilder::makeNewAndDeleteExpr() { return false; if (RequiresNoThrowAlloc) { - const auto *FT = OperatorNew->getType()->getAs<FunctionProtoType>(); + const auto *FT = OperatorNew->getType()->castAs<FunctionProtoType>(); if (!FT->isNothrow(/*ResultIfDependent*/ false)) { S.Diag(OperatorNew->getLocation(), diag::err_coroutine_promise_new_requires_nothrow) @@ -1281,7 +1281,7 @@ bool CoroutineStmtBuilder::makeNewAndDeleteExpr() { // Check if we need to pass the size. const auto *OpDeleteType = - OpDeleteQualType.getTypePtr()->getAs<FunctionProtoType>(); + OpDeleteQualType.getTypePtr()->castAs<FunctionProtoType>(); if (OpDeleteType->getNumParams() > 1) DeleteArgs.push_back(FrameSize); |
