diff options
author | Rafael Espindola <rafael.espindola@gmail.com> | 2018-01-12 17:03:43 +0000 |
---|---|---|
committer | Rafael Espindola <rafael.espindola@gmail.com> | 2018-01-12 17:03:43 +0000 |
commit | 3b9843f0ffbbef83eeecffb6d97cf9b3be3b159f (patch) | |
tree | 14df8a4d75e1eb23f82f3936b92c374c99f38fc1 /llvm/lib/MC/MCCodeView.cpp | |
parent | 84652cc63cce8e0afd445de0d6803d19f59cfbee (diff) | |
download | llvm-3b9843f0ffbbef83eeecffb6d97cf9b3be3b159f.zip llvm-3b9843f0ffbbef83eeecffb6d97cf9b3be3b159f.tar.gz llvm-3b9843f0ffbbef83eeecffb6d97cf9b3be3b159f.tar.bz2 |
Allow dso_local on ifunc.
It was never fully disallowed. We were rejecting it in the asm parser,
but not in the verifier.
Currently TargetMachine::shouldAssumeDSOLocal returns true for hidden
ifuncs. I considered changing it and moving the check from the asm
parser to the verifier.
The reason for deciding to allow it instead is that all linkers handle
a direct reference just fine. They use the plt address as the address
of the function. In fact doing that means that clang doesn't have the
same bug as gcc: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83782.
This patch then removes the check from the asm parser and updates the
bitcode reader and writer.
llvm-svn: 322378
Diffstat (limited to 'llvm/lib/MC/MCCodeView.cpp')
0 files changed, 0 insertions, 0 deletions