diff options
| author | Vlad Tsyrklevich <vlad@tsyrklevich.net> | 2018-03-13 05:08:48 +0000 |
|---|---|---|
| committer | Vlad Tsyrklevich <vlad@tsyrklevich.net> | 2018-03-13 05:08:48 +0000 |
| commit | aab60006842d6172ca7963e4d96540bad33ebe3d (patch) | |
| tree | b1b3d0e5c94b8cc3f635b13c3e3a28e66974f363 /llvm/lib/ObjectYAML | |
| parent | ade40dd3c8f25b243a55cb9d1960efd7ca9e25a3 (diff) | |
| download | bcm5719-llvm-aab60006842d6172ca7963e4d96540bad33ebe3d.tar.gz bcm5719-llvm-aab60006842d6172ca7963e4d96540bad33ebe3d.zip | |
Reland r327041: [ThinLTO] Keep available_externally symbols live
Summary:
This change fixes PR36483. The bug was originally introduced by a change
that marked non-prevailing symbols dead. This broke LowerTypeTests
handling of available_externally functions, which are non-prevailing.
LowerTypeTests uses liveness information to avoid emitting thunks for
unused functions.
Marking available_externally functions dead is incorrect, the functions
are used though the function definitions are not. This change keeps them
live, and lets the EliminateAvailableExternally/GlobalDCE passes remove
them later instead.
(Reland with a suspected fix for a unit test failure I haven't been able
to reproduce locally)
Reviewers: pcc, tejohnson
Reviewed By: tejohnson
Subscribers: grimar, mehdi_amini, inglorion, eraman, llvm-commits
Differential Revision: https://reviews.llvm.org/D43690
llvm-svn: 327360
Diffstat (limited to 'llvm/lib/ObjectYAML')
0 files changed, 0 insertions, 0 deletions
