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

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 26 21:25:43 PDT 2016

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.



-------------- 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>

More information about the llvm-commits mailing list