[llvm-branch-commits] [llvm] [ThinLTO] Generate import status in per-module combined summary (PR #88024)

Teresa Johnson via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Wed May 8 15:37:47 PDT 2024


================
@@ -1670,11 +1798,15 @@ Expected<bool> FunctionImporter::importFunctions(
       if (!GV.hasName())
         continue;
       auto GUID = GV.getGUID();
-      auto Import = ImportGUIDs.count(GUID);
-      LLVM_DEBUG(dbgs() << (Import ? "Is" : "Not") << " importing global "
-                        << GUID << " " << GV.getName() << " from "
-                        << SrcModule->getSourceFileName() << "\n");
-      if (Import) {
+      auto ImportType = maybeGetImportType(ImportGUIDs, GUID);
+      if (!ImportType)
----------------
teresajohnson wrote:

Do we need to emit a debug message in this case like you are doing for functions above? Ditto for aliases below

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


More information about the llvm-branch-commits mailing list