[PATCH] D19954: ELF: Undefine all symbols, not just those that we expect to be defined.

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Wed May 4 20:13:32 PDT 2016


pcc created this revision.
pcc added reviewers: rafael, ruiu.
pcc added a subscriber: llvm-commits.
Herald added a subscriber: joker.eph.

This allows the combined LTO object to provide a definition with the same
name as a symbol that was internalized without causing a duplicate symbol
error.  This normally happens during parallel codegen which externalizes
originally-internal symbols, for example.

In order to make this work, I needed to relax the undefined symbol error to
only report an error for symbols that are used in regular objects.

http://reviews.llvm.org/D19954

Files:
  ELF/LTO.cpp
  ELF/Writer.cpp
  test/ELF/lto/parallel-internalize.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19954.56234.patch
Type: text/x-patch
Size: 3845 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160505/3da47880/attachment.bin>


More information about the llvm-commits mailing list