Index: lld/trunk/ELF/LinkerScript.cpp =================================================================== --- lld/trunk/ELF/LinkerScript.cpp +++ lld/trunk/ELF/LinkerScript.cpp @@ -57,12 +57,12 @@ ScriptConfiguration *elf::ScriptConfig; template static SymbolBody *addRegular(SymbolAssignment *Cmd) { + Symbol *Sym; uint8_t Visibility = Cmd->Hidden ? STV_HIDDEN : STV_DEFAULT; - Symbol *Sym = Symtab::X->addUndefined( - Cmd->Name, /*IsLocal=*/false, STB_GLOBAL, Visibility, - /*Type*/ 0, - /*CanOmitFromDynSym*/ false, /*File*/ nullptr); - + std::tie(Sym, std::ignore) = Symtab::X->insert( + Cmd->Name, /*Type*/ 0, Visibility, /*CanOmitFromDynSym*/ false, + /*File*/ nullptr); + Sym->Binding = STB_GLOBAL; replaceBody>(Sym, Cmd->Name, /*IsLocal=*/false, Visibility, STT_NOTYPE, 0, 0, nullptr, nullptr); @@ -70,14 +70,14 @@ } template static SymbolBody *addSynthetic(SymbolAssignment *Cmd) { + Symbol *Sym; uint8_t Visibility = Cmd->Hidden ? STV_HIDDEN : STV_DEFAULT; const OutputSectionBase *Sec = ScriptConfig->HasSections ? nullptr : Cmd->Expression.Section(); - Symbol *Sym = Symtab::X->addUndefined( - Cmd->Name, /*IsLocal=*/false, STB_GLOBAL, Visibility, - /*Type*/ 0, - /*CanOmitFromDynSym*/ false, /*File*/ nullptr); - + std::tie(Sym, std::ignore) = Symtab::X->insert( + Cmd->Name, /*Type*/ 0, Visibility, /*CanOmitFromDynSym*/ false, + /*File*/ nullptr); + Sym->Binding = STB_GLOBAL; replaceBody(Sym, Cmd->Name, 0, Sec); return Sym->body(); } Index: lld/trunk/ELF/SymbolTable.h =================================================================== --- lld/trunk/ELF/SymbolTable.h +++ lld/trunk/ELF/SymbolTable.h @@ -77,6 +77,11 @@ uint8_t Binding, uint8_t StOther, uint8_t Type, InputFile *File); + std::pair insert(StringRef Name); + std::pair insert(StringRef Name, uint8_t Type, + uint8_t Visibility, bool CanOmitFromDynSym, + InputFile *File); + void scanUndefinedFlags(); void scanShlibUndefined(); void scanVersionScript(); @@ -90,11 +95,6 @@ std::vector *> Sections; private: - std::pair insert(StringRef Name); - std::pair insert(StringRef Name, uint8_t Type, - uint8_t Visibility, bool CanOmitFromDynSym, - InputFile *File); - std::vector findByVersion(SymbolVersion Ver); std::vector findAllByVersion(SymbolVersion Ver); Index: lld/trunk/test/ELF/linkerscript/Inputs/lazy-symbols.s =================================================================== --- lld/trunk/test/ELF/linkerscript/Inputs/lazy-symbols.s +++ lld/trunk/test/ELF/linkerscript/Inputs/lazy-symbols.s @@ -0,0 +1,2 @@ +.globl foo +foo: Index: lld/trunk/test/ELF/linkerscript/lazy-symbols.s =================================================================== --- lld/trunk/test/ELF/linkerscript/lazy-symbols.s +++ lld/trunk/test/ELF/linkerscript/lazy-symbols.s @@ -0,0 +1,13 @@ +# REQUIRES: x86 +# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %S/Inputs/lazy-symbols.s -o %t1 +# RUN: llvm-ar rcs %tar %t1 +# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %s -o %t2 +# RUN: echo "foo = 1;" > %t.script +# RUN: ld.lld %t2 %tar --script %t.script -o %tout +# RUN: llvm-readobj -symbols %tout | FileCheck %s + +# This test is to ensure a linker script can define a symbol which have the same +# name as a lazy symbol. + +# CHECK: Name: foo +# CHECK-NEXT: Value: 0x1