[PATCH] D15025: [ThinLTO] Option to invoke ThinLTO backend passes and importing

Mehdi AMINI via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 4 08:15:06 PST 2015


joker.eph added inline comments.

================
Comment at: lib/CodeGen/CodeGenAction.cpp:822
@@ +821,3 @@
+      TheModule = std::move(Combined);
+    }
+
----------------
tejohnson wrote:
> joker.eph wrote:
> > Could we refactor this in a helper in llvm?
> I can create a new Linker interface that takes the index path, builds the index, does the link and returns the ErrorOr index.
I was thinking about a free function that would be something like: `std::unique_ptr<Module> llvm::renameModuleForThinLTO(std::unique_ptr<Module> &M, const FunctionInfoIndex &Index);`



http://reviews.llvm.org/D15025





More information about the cfe-commits mailing list