[llvm] [ThinLTO] Use a set rather than a map to track exported ValueInfos. (PR #97360)
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 2 08:55:15 PDT 2024
================
@@ -818,10 +817,8 @@ static void computeImportForFunction(
// Since definition takes precedence over declaration for the same VI,
// try emplace <VI, declaration> pair without checking insert result.
// If insert doesn't happen, there must be an existing entry keyed by
- // VI.
- if (ExportLists)
- (*ExportLists)[DeclSourceModule].try_emplace(
- VI, GlobalValueSummary::Declaration);
+ // VI. Note `ExportLists` only keeps track of definitions so VI won't
----------------
teresajohnson wrote:
Maybe "only keeps track of exports due to imported definitions" ?
https://github.com/llvm/llvm-project/pull/97360
More information about the llvm-commits
mailing list