[clang-tools-extra] [modularize] Use a range-based for loop (NFC) (PR #139398)
via cfe-commits
cfe-commits at lists.llvm.org
Sat May 10 10:26:37 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-tools-extra
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/139398.diff
1 Files Affected:
- (modified) clang-tools-extra/modularize/Modularize.cpp (+3-5)
``````````diff
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");
``````````
</details>
https://github.com/llvm/llvm-project/pull/139398
More information about the cfe-commits
mailing list