[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 13:43:04 PDT 2016


tejohnson created this revision.
tejohnson added a reviewer: joker.eph.
tejohnson added a subscriber: llvm-commits.
Herald added a subscriber: joker.eph.

The function import pass was computing all the imports for all the
modules in the index, and only using the imports for the current module.
Change this to instead compute only for the given module. This means
that the exports list can't be populated, but they weren't being used
anyway.

Note that if we want to use the export lists to control linkage changes,
it would be better to compute them a single time in the thin link step
and mark any required changes in the index.

http://reviews.llvm.org/D18945

Files:
  include/llvm/Transforms/IPO/FunctionImport.h
  lib/Transforms/IPO/FunctionImport.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18945.53183.patch
Type: text/x-patch
Size: 6481 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160410/0e978b19/attachment.bin>


More information about the llvm-commits mailing list