[PATCH] D47975: [ELF] Fix copy relocation when two symbols share the same Symbol instance.

[Benjamin Kramer via Phabricator via llvm-commits]

bkramer added inline comments.


================
Comment at: ELF/Relocations.cpp:455
+  // Use a set to deduplicate symbols.
+  SmallSet<SharedSymbol *, 4> Ret;
   for (const Elf_Sym &S : File.getGlobalELFSyms()) {
----------------
SmallSet of pointers doesn't have a stable iteration order. Does that matter here?


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D47975