[PATCH] D18945: [ThinLTO] Only compute imports for current module in FunctionImport pass

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 10 15:11:00 PDT 2016


tejohnson added a comment.

In http://reviews.llvm.org/D18945#396592, @joker.eph wrote:

> About your note: `ComputeCrossModuleImport()` *is* the API I am calling in the Thin-link step.


Right, I haven't changed that callsite and that module will also be called from the gold-plugin.

> The export list is gonna be needed to drive a "smart" promotion/renaming, so I expect it to be needed at the same stage you need the import list. 

>  What will be the need for this?


The callsite in the function importer (see the last change in this file) uses the new interface. It doesn't need to recompute all imports for all modules. Should reduce compile time in the back ends that use this pass.


http://reviews.llvm.org/D18945





More information about the llvm-commits mailing list