[llvm] [ThinLTO] Populate declaration import status except for distributed ThinLTO (PR #88024)
Mingming Liu via llvm-commits
llvm-commits at lists.llvm.org
Fri May 10 17:10:12 PDT 2024
================
@@ -1634,17 +1752,27 @@ Expected<bool> FunctionImporter::importFunctions(
return std::move(Err);
auto &ImportGUIDs = FunctionsToImportPerModule->second;
+
// Find the globals to import
SetVector<GlobalValue *> GlobalsToImport;
for (Function &F : *SrcModule) {
if (!F.hasName())
continue;
auto GUID = F.getGUID();
- auto Import = ImportGUIDs.count(GUID);
- LLVM_DEBUG(dbgs() << (Import ? "Is" : "Not") << " importing function "
- << GUID << " " << F.getName() << " from "
- << SrcModule->getSourceFileName() << "\n");
- if (Import) {
+ auto ImportType = maybeGetImportType(ImportGUIDs, GUID);
+
+ if (!ImportType) {
----------------
minglotus-6 wrote:
Added 'declarations' into debug message and kept the similar flow as before (no new `continue` path).
Added `IMPORTDUMP:` lines to regression test.
https://github.com/llvm/llvm-project/pull/88024
More information about the llvm-commits
mailing list