Index: ELF/Relocations.cpp =================================================================== --- ELF/Relocations.cpp +++ ELF/Relocations.cpp @@ -52,6 +52,7 @@ #include "Thunks.h" #include "lld/Common/Memory.h" #include "lld/Common/Strings.h" +#include "llvm/ADT/SmallSet.h" #include "llvm/Support/Endian.h" #include "llvm/Support/raw_ostream.h" #include @@ -443,14 +444,15 @@ // // If two or more symbols are at the same offset, and at least one of // them are copied by a copy relocation, all of them need to be copied. -// Otherwise, they would refer different places at runtime. +// Otherwise, they would refer to different places at runtime. template -static std::vector getSymbolsAt(SharedSymbol &SS) { +static SmallSet getSymbolsAt(SharedSymbol &SS) { typedef typename ELFT::Sym Elf_Sym; SharedFile &File = SS.getFile(); - std::vector Ret; + // Use a set to deduplicate symbols. + SmallSet Ret; for (const Elf_Sym &S : File.getGlobalELFSyms()) { if (S.st_shndx == SHN_UNDEF || S.st_shndx == SHN_ABS || S.st_value != SS.Value) @@ -458,7 +460,7 @@ StringRef Name = check(S.getName(File.getStringTable())); Symbol *Sym = Symtab->find(Name); if (auto *Alias = dyn_cast_or_null(Sym)) - Ret.push_back(Alias); + Ret.insert(Alias); } return Ret; }