[PATCH] D22356: [ThinLTO] Perform conservative weak/linkonce resolution in distributed backend case

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 19:51:41 PDT 2016


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

Rebase and address review comments:

- Fold the isExported check into the isPrevailing callback from

gold-plugin, to avoid needing to pass that callback.

- Remove FunctionImport.cpp changes, since they will be obviated by

https://reviews.llvm.org/D21545.

- Rework test case to use llvm-lto for weak symbol resolution and

importing, to simulate the behavior of the ThinLTO backend invoked by
clang after https://reviews.llvm.org/D21545.


https://reviews.llvm.org/D22356

Files:
  include/llvm/LTO/LTO.h
  lib/LTO/LTO.cpp
  test/tools/gold/X86/Inputs/thinlto_preserve_nonprevailodr-1.ll
  test/tools/gold/X86/Inputs/thinlto_preserve_nonprevailodr-2.ll
  test/tools/gold/X86/Inputs/thinlto_preserve_nonprevailodr-3.ll
  test/tools/gold/X86/thinlto_preserve_nonprevailodr.ll
  tools/gold/gold-plugin.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22356.64081.patch
Type: text/x-patch
Size: 13113 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160715/9deefc30/attachment.bin>


More information about the llvm-commits mailing list