[Linker] Support weak symbols in nodeduplicate COMDAT group
authorPetr Hosek <phosek@google.com>
Wed, 25 Aug 2021 08:12:19 +0000 (01:12 -0700)
committerPetr Hosek <phosek@google.com>
Wed, 1 Sep 2021 00:44:33 +0000 (17:44 -0700)
commit92f54e1c752253b5a17eb9ec7942f580d878f64d
tree914d039ebc13766cc6bbca1721b7a75eb0addf52
parentc2162e4d89d6f99bbbe96beaa9156dbb2dba8344
[Linker] Support weak symbols in nodeduplicate COMDAT group

When a nodeduplicate COMDAT group contains a weak symbol, choose
a non-weak symbol (or one of the weak ones) rather than reporting
an error. This should address issue PR51394.

With the current IR representation, a generic comdat nodeduplicate
semantics is not representable for LTO. In the linker, sections and
symbols are separate concepts. A dropped weak symbol does not force the
defining input section to be dropped as well (though it can be collected
by GC). In the IR, when a weak linkage symbol is dropped, its associate
section content is dropped as well.

For InstrProfiling, which is where ran into this issue in PR51394, the
deduplication semantic is a sufficient workaround.

Differential Revision: https://reviews.llvm.org/D108689
llvm/lib/Linker/LinkModules.cpp
llvm/test/Linker/comdat-nodeduplicate.ll