[llvm] ca48b01 - [LTO] Use a helper function to add a definition (NFC) (#105721)

via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 22 16:01:40 PDT 2024


Author: Kazu Hirata
Date: 2024-08-22T16:01:36-07:00
New Revision: ca48b015a1719ba7be2d357056f348473d495d3d

URL: https://github.com/llvm/llvm-project/commit/ca48b015a1719ba7be2d357056f348473d495d3d
DIFF: https://github.com/llvm/llvm-project/commit/ca48b015a1719ba7be2d357056f348473d495d3d.diff

LOG: [LTO] Use a helper function to add a definition (NFC) (#105721)

I missed this one when I introduced helper functions in:

  commit 3082a381f57ef2885c270f41f2955e08c79634c5
  Author: Kazu Hirata <kazu at google.com>
  Date:   Thu Aug 22 12:06:47 2024 -0700

Added: 
    

Modified: 
    llvm/lib/Transforms/IPO/FunctionImport.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/IPO/FunctionImport.cpp b/llvm/lib/Transforms/IPO/FunctionImport.cpp
index 354ad0fde092a7..55803670071d16 100644
--- a/llvm/lib/Transforms/IPO/FunctionImport.cpp
+++ b/llvm/lib/Transforms/IPO/FunctionImport.cpp
@@ -600,8 +600,8 @@ class WorkloadImportsManager : public ModuleImportsManager {
       LLVM_DEBUG(dbgs() << "[Workload][Including]" << VI.name() << " from "
                         << ExportingModule << " : "
                         << Function::getGUID(VI.name()) << "\n");
-      ImportList[ExportingModule][VI.getGUID()] =
-          GlobalValueSummary::Definition;
+      FunctionImporter::addDefinition(ImportList, ExportingModule,
+                                      VI.getGUID());
       GVI.onImportingSummary(*GVS);
       if (ExportLists)
         (*ExportLists)[ExportingModule].insert(VI);


        


More information about the llvm-commits mailing list