[PATCH] D18487: [ThinLTO] Add optional import message and statistics

[Teresa Johnson via llvm-commits]

tejohnson updated this revision to Diff 51726.
tejohnson added a comment.

Rebase on http://reviews.llvm.org/D18343/r264503.

Also, with r264503, aliases are now being added to the GlobalsToImport set even
when their aliasees can't be imported due to their linkage type. While the
importing worked correctly (the aliases imported as declarations) due to the
logic in FunctionImportGlobalProcessing::doImportAsDefinition, there is no
point to adding them to the GlobalsToImport set and it was resulting in
incorrectly printing a message indicating that the alias was imported.
To avoid this, delay adding aliases to the GlobalsToImport set until after
the linkage type of the aliasee is checked.


http://reviews.llvm.org/D18487

Files:
  lib/Transforms/IPO/FunctionImport.cpp
  test/Transforms/FunctionImport/funcimport.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18487.51726.patch
Type: text/x-patch
Size: 5713 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160327/577ec184/attachment.bin>