diff options
author | Pavel Samolysov <samolisov@gmail.com> | 2024-06-11 05:30:50 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-06-11 05:30:50 +0300 |
commit | 69e9e779b783bb34a3c1f73c93ca63ee6b89ab09 (patch) | |
tree | e7786893fa62faab929d98e0c4dfd853257abba5 /clang/lib/Sema/AnalysisBasedWarnings.cpp | |
parent | cb63abca27e1813ae58ded466cd81ba3952ab888 (diff) | |
download | llvm-69e9e779b783bb34a3c1f73c93ca63ee6b89ab09.zip llvm-69e9e779b783bb34a3c1f73c93ca63ee6b89ab09.tar.gz llvm-69e9e779b783bb34a3c1f73c93ca63ee6b89ab09.tar.bz2 |
[clang] Replace X && isa<Y>(X) with isa_and_nonnull<Y>(X). NFC (#94987)
This addresses a clang-tidy suggestion.
Diffstat (limited to 'clang/lib/Sema/AnalysisBasedWarnings.cpp')
-rw-r--r-- | clang/lib/Sema/AnalysisBasedWarnings.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/clang/lib/Sema/AnalysisBasedWarnings.cpp b/clang/lib/Sema/AnalysisBasedWarnings.cpp index b9d0b59..0f604c6 100644 --- a/clang/lib/Sema/AnalysisBasedWarnings.cpp +++ b/clang/lib/Sema/AnalysisBasedWarnings.cpp @@ -442,7 +442,7 @@ static ControlFlowKind CheckFallThrough(AnalysisDeclContext &AC) { if (!live[B->getBlockID()]) { if (B->pred_begin() == B->pred_end()) { const Stmt *Term = B->getTerminatorStmt(); - if (Term && isa<CXXTryStmt>(Term)) + if (isa_and_nonnull<CXXTryStmt>(Term)) // When not adding EH edges from calls, catch clauses // can otherwise seem dead. Avoid noting them as dead. count += reachable_code::ScanReachableFromBlock(B, live); @@ -1100,7 +1100,7 @@ namespace { // issue a warn_fallthrough_attr_unreachable for them. for (const auto *B : *Cfg) { const Stmt *L = B->getLabel(); - if (L && isa<SwitchCase>(L) && ReachableBlocks.insert(B).second) + if (isa_and_nonnull<SwitchCase>(L) && ReachableBlocks.insert(B).second) BlockQueue.push_back(B); } @@ -1128,7 +1128,7 @@ namespace { if (!P) continue; const Stmt *Term = P->getTerminatorStmt(); - if (Term && isa<SwitchStmt>(Term)) + if (isa_and_nonnull<SwitchStmt>(Term)) continue; // Switch statement, good. const SwitchCase *SW = dyn_cast_or_null<SwitchCase>(P->getLabel()); @@ -1327,7 +1327,7 @@ static void DiagnoseSwitchLabelsFallthrough(Sema &S, AnalysisDeclContext &AC, B = *B->succ_begin(); Term = B->getTerminatorStmt(); } - if (!(B->empty() && Term && isa<BreakStmt>(Term))) { + if (!(B->empty() && isa_and_nonnull<BreakStmt>(Term))) { Preprocessor &PP = S.getPreprocessor(); StringRef AnnotationSpelling = getFallthroughAttrSpelling(PP, L); SmallString<64> TextToInsert(AnnotationSpelling); |