[llvm] [ThinLTO] Reduce the number of renaming due to promotions in distribu… (PR #188074)

via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 18 09:53:49 PDT 2026


================
@@ -1607,6 +1607,23 @@ void llvm::gatherImportedSummariesForModule(
 
     SummariesForIndex[GUID] = DS->second;
   }
+
+  // For each source module we import from, also include summaries for local
----------------
yonghong-song wrote:

Yes, use declaration is much better than definition. I fixed it in the new revision and also added an assert as you suggested.

Thanks for review!

https://github.com/llvm/llvm-project/pull/188074


More information about the llvm-commits mailing list