[clang-tools-extra] [modularize] Use a range-based for loop (NFC) (PR #139398)

Kazu Hirata via cfe-commits cfe-commits at lists.llvm.org
Sat May 10 10:26:07 PDT 2025


https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/139398

None

>From 1b2af11d771f4e6903e48ac356fa8e6eb3cfc28f Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Sat, 10 May 2025 08:17:21 -0700
Subject: [PATCH] [modularize] Use a range-based for loop (NFC)

---
 clang-tools-extra/modularize/Modularize.cpp | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/clang-tools-extra/modularize/Modularize.cpp b/clang-tools-extra/modularize/Modularize.cpp
index bd180681c9154..8609692945344 100644
--- a/clang-tools-extra/modularize/Modularize.cpp
+++ b/clang-tools-extra/modularize/Modularize.cpp
@@ -357,11 +357,9 @@ getModularizeArgumentsAdjuster(DependencyMap &Dependencies) {
     std::string InputFile = findInputFile(Args);
     DependentsVector &FileDependents = Dependencies[InputFile];
     CommandLineArguments NewArgs(Args);
-    if (int Count = FileDependents.size()) {
-      for (int Index = 0; Index < Count; ++Index) {
-        NewArgs.push_back("-include");
-        NewArgs.push_back(FileDependents[Index]);
-      }
+    for (const std::string &Dep : FileDependents) {
+      NewArgs.push_back("-include");
+      NewArgs.push_back(Dep);
     }
     // Ignore warnings.  (Insert after "clang_tool" at beginning.)
     NewArgs.insert(NewArgs.begin() + 1, "-w");



More information about the cfe-commits mailing list