[PATCH] D18122: Rework linkInModule(), making it oblivious to ThinLTO

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 16 09:19:31 PDT 2016

rafael added a subscriber: rafael.
rafael added a comment.

I really like this change, but Teresa should be the one to give the final LGTM.

Comment at: include/llvm/Linker/Linker.h:50
@@ -49,4 +49,3 @@
   bool linkInModule(std::unique_ptr<Module> Src, unsigned Flags = Flags::None,
-                    const FunctionInfoIndex *Index = nullptr,
-                    DenseSet<const GlobalValue *> *FunctionsToImport = nullptr,
+                    DenseSet<const GlobalValue *> *GlobalsToImport = nullptr,
                     DenseMap<unsigned, MDNode *> *ValIDToTempMDMap = nullptr);
Can you remove the llvm/IR/ModuleSummaryIndex.h include?


More information about the llvm-commits mailing list