[llvm] [ThinLTO][BitcodeWriter] Write import type in per-module combined summary (PR #87600)

Mingming Liu via llvm-commits llvm-commits at lists.llvm.org
Tue May 7 14:07:49 PDT 2024


================
@@ -428,6 +428,10 @@ class IndexBitcodeWriter : public BitcodeWriterBase {
   /// The combined index to write to bitcode.
   const ModuleSummaryIndex &Index;
 
+  /// For each module, provides the set of global value summaries for which the
----------------
minglotus-6 wrote:

Updated the comment to "When writing combined summaries, provides the set of global value summaries...".

While we write a subset of the index only for combined summaries, 'combined summary' is slightly preferred as it hints this map doesn't apply for per-module summary or the index.

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


More information about the llvm-commits mailing list