[llvm] a44620d - [libc++abi][NFC] Enable modernize-loop-convert clang-tidy check (#213250)

via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 4 01:48:04 PDT 2026


Author: Nikolas Klauser
Date: 2026-08-04T10:47:58+02:00
New Revision: a44620dad09537ba10d22e241c6f2ba011c65bea

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

LOG: [libc++abi][NFC] Enable modernize-loop-convert clang-tidy check (#213250)

Added: 
    

Modified: 
    libcxxabi/.clang-tidy
    libcxxabi/src/demangle/ItaniumDemangle.h
    llvm/include/llvm/Demangle/ItaniumDemangle.h

Removed: 
    


################################################################################
diff  --git a/libcxxabi/.clang-tidy b/libcxxabi/.clang-tidy
index 365548b0bfab2..cc06a4c8f6f75 100644
--- a/libcxxabi/.clang-tidy
+++ b/libcxxabi/.clang-tidy
@@ -10,6 +10,7 @@ Checks: >
   misc-non-copyable-objects,
   misc-uniqueptr-reset-release,
 
+  modernize-loop-convert,
   modernize-redundant-void-arg,
   modernize-use-override,
 
@@ -48,7 +49,6 @@ CheckOptions:
 # TODO: Enable these checks
 # llvm-include-order,
 # llvm-namespace-comment,
-# modernize-loop-convert,
 # modernize-use-equals-delete,
 # modernize-use-nullptr,
 # readability-identifier-naming,

diff  --git a/libcxxabi/src/demangle/ItaniumDemangle.h b/libcxxabi/src/demangle/ItaniumDemangle.h
index 94d780e229631..2ae943e1998f1 100644
--- a/libcxxabi/src/demangle/ItaniumDemangle.h
+++ b/libcxxabi/src/demangle/ItaniumDemangle.h
@@ -2858,8 +2858,8 @@ template <typename Derived, typename Alloc> struct AbstractManglingParser {
     ParsingLambdaParamsAtLevel = (size_t)-1;
     TryToParseTemplateArgs = true;
     PermitForwardTemplateReferences = false;
-    for (int I = 0; I != 3; ++I)
-      NumSyntheticTemplateParameters[I] = 0;
+    for (unsigned int & NumSyntheticTemplateParameter : NumSyntheticTemplateParameters)
+      NumSyntheticTemplateParameter = 0;
     ASTAllocator.reset();
   }
 

diff  --git a/llvm/include/llvm/Demangle/ItaniumDemangle.h b/llvm/include/llvm/Demangle/ItaniumDemangle.h
index 04338e1d7cb51..0f1deb3d710e2 100644
--- a/llvm/include/llvm/Demangle/ItaniumDemangle.h
+++ b/llvm/include/llvm/Demangle/ItaniumDemangle.h
@@ -2858,8 +2858,8 @@ template <typename Derived, typename Alloc> struct AbstractManglingParser {
     ParsingLambdaParamsAtLevel = (size_t)-1;
     TryToParseTemplateArgs = true;
     PermitForwardTemplateReferences = false;
-    for (int I = 0; I != 3; ++I)
-      NumSyntheticTemplateParameters[I] = 0;
+    for (unsigned int & NumSyntheticTemplateParameter : NumSyntheticTemplateParameters)
+      NumSyntheticTemplateParameter = 0;
     ASTAllocator.reset();
   }
 


        


More information about the llvm-commits mailing list