Author: Pavel Samolysov
Date: 2024-06-11T12:24:46+03:00
New Revision: ec81c9b1dd856dbe77b042e7d293e437b587d914

URL: 
https://github.com/llvm/llvm-project/commit/ec81c9b1dd856dbe77b042e7d293e437b587d914
DIFF: 
https://github.com/llvm/llvm-project/commit/ec81c9b1dd856dbe77b042e7d293e437b587d914.diff

LOG: [clang] Remove a redundant check in Mangle. NFC (#95071)

This addresses a review comment for PR #94987 Because that PR is a big
automatic change, this change was moved in a separate one.

Added: 
    

Modified: 
    clang/lib/AST/Mangle.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/AST/Mangle.cpp b/clang/lib/AST/Mangle.cpp
index 4af4d7c00c5cb..4fbf0e3b42dbc 100644
--- a/clang/lib/AST/Mangle.cpp
+++ b/clang/lib/AST/Mangle.cpp
@@ -301,9 +301,8 @@ void MangleContext::mangleBlock(const DeclContext *DC, 
const BlockDecl *BD,
   } else {
     assert((isa<NamedDecl>(DC) || isa<BlockDecl>(DC)) &&
            "expected a NamedDecl or BlockDecl");
-    if (isa<BlockDecl>(DC))
-      for (; isa_and_nonnull<BlockDecl>(DC); DC = DC->getParent())
-        (void) getBlockId(cast<BlockDecl>(DC), true);
+    for (; isa_and_nonnull<BlockDecl>(DC); DC = DC->getParent())
+      (void)getBlockId(cast<BlockDecl>(DC), true);
     assert((isa<TranslationUnitDecl>(DC) || isa<NamedDecl>(DC)) &&
            "expected a TranslationUnitDecl or a NamedDecl");
     if (const auto *CD = dyn_cast<CXXConstructorDecl>(DC))


        
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to