commit 3dabfc6b0d62433d9beedfd2b3dc8fd067870a9b
author Rafael Espindola <rafael.espindola@gmail.com> Mon Oct 31 13:14:53 2016 +0000
committer Rafael Espindola <rafael.espindola@gmail.com> Mon Oct 31 13:14:53 2016 +0000
tree f02a05e3b9425dbe9493660fb67e2b81a2649850
parent 6788ddeac96193f2cd5670d3061ef816a4a4737a

Correctly merge visibility of linkerscript created symbols.

llvm-svn: 285567

lld/ELF/LinkerScript.cpp

diff --git a/lld/ELF/LinkerScript.cpp b/lld/ELF/LinkerScript.cpp
index f15d170..96862cc 100644
--- a/lld/ELF/LinkerScript.cpp
+++ b/lld/ELF/LinkerScript.cpp
@@ -46,9 +46,9 @@
 ScriptConfiguration *elf::ScriptConfig;
 
 template <class ELFT> static void addRegular(SymbolAssignment *Cmd) {
-  Symbol *Sym = Symtab<ELFT>::X->addRegular(Cmd->Name, STV_DEFAULT, nullptr,
+  uint8_t Visibility = Cmd->Hidden ? STV_HIDDEN : STV_DEFAULT;
+  Symbol *Sym = Symtab<ELFT>::X->addRegular(Cmd->Name, Visibility, nullptr,
                                             STB_GLOBAL, STT_NOTYPE, 0);
-  Sym->Visibility = Cmd->Hidden ? STV_HIDDEN : STV_DEFAULT;
   Cmd->Sym = Sym->body();
 
   // If we have no SECTIONS then we don't have '.' and don't call