[llvm] [ThinLTO] Populate declaration import status except for distributed ThinLTO (PR #88024)

Mingming Liu via llvm-commits llvm-commits at lists.llvm.org
Fri May 10 17:10:12 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
----------------
minglotus-6 wrote:

thanks for the catch. I'll update `gatherImportedSummariesForModule` in the next change (https://github.com/llvm/llvm-project/pull/87600) mostly because `DecSummaries` is used by bitcode writer.

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


More information about the llvm-commits mailing list