[PATCH] D14914: Add a FunctionImporter helper to perform summary-based cross-module function importing

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 23 08:34:01 PST 2015


tejohnson added a comment.

In http://reviews.llvm.org/D14914#295006, @tejohnson wrote:

> [...] Another thing is that there potentially needs to be static promotion/renaming in the current module being compiled. I have a clang FE patch to handle that part, since it should be done early.


I guess you won't be doing this through clang since you are planning to invoke from ld64, but remember to do this part there for each module being compiled. Will need to do from 'opt' as well for testing. My clang patch also includes support for an option to trigger the pass and pass down the index file path via a CodeGenOption, with corresponding support in llvm to invoke the pass. I can add that part after this goes in.


http://reviews.llvm.org/D14914





More information about the llvm-commits mailing list