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

[Rafael Ávila de Espíndola via llvm-commits]

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?


http://reviews.llvm.org/D18122