[clang-scan-deps] Remove const from ModuleDeps loop to enable move. (#161109)

naveen-seth · web-flow · commit d28c07b7550a · 2025-09-30T00:12:55.000+02:00
This changes the iteration from const to non-const so that std::move
results in a true move rather than a copy.
diff --git a/clang/tools/clang-scan-deps/ClangScanDeps.cpp b/clang/tools/clang-scan-deps/ClangScanDeps.cpp
@@ -420,7 +420,7 @@ class FullDeps {
     std::vector<ModuleDeps *> NewMDs;
     {
       std::unique_lock<std::mutex> ul(Lock);
-      for (const ModuleDeps &MD : Graph) {
+      for (ModuleDeps &MD : Graph) {
         auto I = Modules.find({MD.ID, 0});
         if (I != Modules.end()) {
           I->first.InputIndex = std::min(I->first.InputIndex, InputIndex);

Original file line number	Diff line number	Diff line change
`@@ -420,7 +420,7 @@ class FullDeps {`
`420`	`420`	`std::vector<ModuleDeps *> NewMDs;`
`421`	`421`	`{`
`422`	`422`	`std::unique_lock<std::mutex> ul(Lock);`
`423`		`- for (const ModuleDeps &MD : Graph) {`
	`423`	`+ for (ModuleDeps &MD : Graph) {`
`424`	`424`	`auto I = Modules.find({MD.ID, 0});`
`425`	`425`	`if (I != Modules.end()) {`
`426`	`426`	`I->first.InputIndex = std::min(I->first.InputIndex, InputIndex);`