[llvm-branch-commits] [llvm] [ThinLTO] Generate import status in per-module combined summary (PR #88024)

Jan Voung via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Thu May 9 11:23:57 PDT 2024


================
@@ -207,11 +211,16 @@ bool convertToDeclaration(GlobalValue &GV);
 /// \p ModuleToSummariesForIndex will be populated with the needed summaries
 /// from each required module path. Use a std::map instead of StringMap to get
 /// stable order for bitcode emission.
+///
+/// \p ModuleToDecSummaries will be populated with the set of declarations \p
+/// ModulePath need from other modules. They key is module path, and the value
----------------
jvoung wrote:

nit: ModuleToDecSummaries -> DecSummaries now
and update the "The key is ..." part

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


More information about the llvm-branch-commits mailing list