aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/Object/MachOObjectFile.cpp
diff options
context:
space:
mode:
authorPeter Collingbourne <peter@pcc.me.uk>2016-07-07 18:31:51 +0000
committerPeter Collingbourne <peter@pcc.me.uk>2016-07-07 18:31:51 +0000
commit73589f321b88ff6f0896b6beb5d73c40d84ebe41 (patch)
treea754447d7802d3b8fab00d90a8847d737cc52f22 /llvm/lib/Object/MachOObjectFile.cpp
parentbed6779c7a839ed6dab85d0198e105021b63552f (diff)
downloadllvm-73589f321b88ff6f0896b6beb5d73c40d84ebe41.zip
llvm-73589f321b88ff6f0896b6beb5d73c40d84ebe41.tar.gz
llvm-73589f321b88ff6f0896b6beb5d73c40d84ebe41.tar.bz2
ThinLTO: Do not take into account whether a definition has multiple copies when promoting.
We currently do not touch a symbol's linkage in the case where a definition has a single copy. However, this code is effectively unnecessary: either the definition is not exported, in which case the internalize phase sets its linkage to internal, or it is exported, in which case we need to promote linkage to weak. Those two cases are already handled by existing code. I believe that the only real functional change here is in the case where we have a single definition which does not prevail (e.g. because the definition in a native object file prevails). In that case we now lower linkage to available_externally following the existing code path for that case. As a result we can remove the isExported function parameter from the thinLTOResolveWeakForLinkerInIndex function. Differential Revision: http://reviews.llvm.org/D21883 llvm-svn: 274784
Diffstat (limited to 'llvm/lib/Object/MachOObjectFile.cpp')
0 files changed, 0 insertions, 0 deletions