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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
