[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