Add support for comdats to the gold plugin.
authorRafael Espindola <rafael.espindola@gmail.com>
Fri, 22 Aug 2014 23:26:10 +0000 (23:26 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Fri, 22 Aug 2014 23:26:10 +0000 (23:26 +0000)
commit03d2823e0289477355d76c9e8f8dacb393895439
tree7a3a45c69cefdf5f4dca0f2f07bc642e95705071
parent6c7a6a1ba2ba93a7d7e96d88c39c3cd48ff082e5
Add support for comdats to the gold plugin.

There are two parts to this. First, the plugin needs to tell gold the comdat by
setting comdat_key.

What gets things a bit more complicated is that gold only seems
symbols. In particular, if A is an alias to B, it only sees the symbols
A and B. It can then ask us to keep symbol A but drop symbol B. What
we have to do instead is to create an internal version of B and make A
an alias to that.

At some point some of this logic should be moved to lib/Linker so that
we don't map a Constant to an internal version just to have lib/Linker
map that again to the destination module.

The reason for implementing this in tools/gold for now is simplicity.
With it in place it should be possible to update clang to use comdats
for constructors and destructors on ELF without breaking the LTO
bootstrap. Once that is done I intend to come back and improve the
interface lib/Linker exposes.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@216302 91177308-0d34-0410-b5e6-96231b3b80d8
test/tools/gold/Inputs/comdat.ll [new file with mode: 0644]
test/tools/gold/bad-alias.ll [new file with mode: 0644]
test/tools/gold/comdat.ll [new file with mode: 0644]
tools/gold/gold-plugin.cpp