diff options
author | Rafael Espindola <rafael.espindola@gmail.com> | 2016-04-08 15:43:43 +0000 |
---|---|---|
committer | Rafael Espindola <rafael.espindola@gmail.com> | 2016-04-08 15:43:43 +0000 |
commit | b729b2131dc8314b64abcef057dc75e0dfac7bab (patch) | |
tree | 945a31f49b5206be2e5d4a210c77baa2a87f73d3 /lld/ELF/OutputSections.cpp | |
parent | badd397aa7c2af53dd302cd7ebd6a96e68f9df1c (diff) | |
download | bcm5719-llvm-b729b2131dc8314b64abcef057dc75e0dfac7bab.tar.gz bcm5719-llvm-b729b2131dc8314b64abcef057dc75e0dfac7bab.zip |
Produce STV_DEFAULT for symbols in shared libraries.
The spec says:
If a symbol definition with STV_PROTECTED visibility from a shared
object is taken as resolving a reference from an executable or another
shared object, the SHN_UNDEF symbol table entry created has STV_DEFAULT
visibility.
llvm-svn: 265792
Diffstat (limited to 'lld/ELF/OutputSections.cpp')
-rw-r--r-- | lld/ELF/OutputSections.cpp | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/lld/ELF/OutputSections.cpp b/lld/ELF/OutputSections.cpp index 64949f2e23e..24258fe40cc 100644 --- a/lld/ELF/OutputSections.cpp +++ b/lld/ELF/OutputSections.cpp @@ -1458,6 +1458,12 @@ void SymbolTableSection<ELFT>::writeLocalSymbols(uint8_t *&Buf) { } } +static uint8_t getSymbolVisibility(SymbolBody *Body) { + if (Body->isShared()) + return STV_DEFAULT; + return Body->getVisibility(); +} + template <class ELFT> void SymbolTableSection<ELFT>::writeGlobalSymbols(uint8_t *Buf) { // Write the internal symbol table contents to the output symbol table @@ -1473,7 +1479,7 @@ void SymbolTableSection<ELFT>::writeGlobalSymbols(uint8_t *Buf) { ESym->setBindingAndType(getSymbolBinding(Body), Type); ESym->st_size = Size; ESym->st_name = StrOff; - ESym->setVisibility(Body->getVisibility()); + ESym->setVisibility(getSymbolVisibility(Body)); ESym->st_value = Body->getVA<ELFT>(); if (const OutputSectionBase<ELFT> *OutSec = getOutputSection(Body)) |