[llvm] [LTO] Introduce a helper function summarizeImports (NFC) (PR #106179)

Jan Voung via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 27 06:53:41 PDT 2024


================
@@ -1082,21 +1082,29 @@ numGlobalVarSummaries(const ModuleSummaryIndex &Index,
   return NumGVS;
 }
 
-// Given ImportMap, return the number of global variable summaries and record
-// the number of defined function summaries as output parameter.
-static unsigned
-numGlobalVarSummaries(const ModuleSummaryIndex &Index,
-                      const FunctionImporter::FunctionsToImportTy &ImportMap,
-                      unsigned &DefinedFS) {
+struct ImportSummary {
----------------
jvoung wrote:

nit: I don't feel too strongly, but Summary is also used in the sense of GlobalVarSummary, FunctionSummary, etc. and so I wonder if using a different term like "Statistics" might be better.

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


More information about the llvm-commits mailing list