[llvm] [ThinLTO] Use a set rather than a map to track exported ValueInfos. (PR #97360)

[Teresa Johnson via llvm-commits]

================
@@ -1154,29 +1147,25 @@ void llvm::ComputeCrossModuleImport(
   for (auto &ModuleImports : ImportLists) {
     auto ModName = ModuleImports.first;
     auto &Exports = ExportLists[ModName];
-    unsigned DefinedGVS = 0, DefinedFS = 0;
-    unsigned NumGVS =
-        numGlobalVarSummaries(Index, Exports, DefinedGVS, DefinedFS);
+    unsigned DefinedFS = 0;
+    unsigned NumGVS = numGlobalVarSummaries(Index, Exports, DefinedFS);
     LLVM_DEBUG(dbgs() << "* Module " << ModName << " exports " << DefinedFS
                       << " function as definitions, "
                       << Exports.size() - NumGVS - DefinedFS
----------------
teresajohnson wrote:

I think this will result in 0, since I believe NumGVS + DefinedFS should be the same as Exports.size() (see my comment above).

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