[PATCH] D39406: Merge SymbolBody and Symbol into one class, SymbolBody.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 30 14:02:41 PDT 2017


ruiu added inline comments.


================
Comment at: lld/ELF/SymbolTable.cpp:262
     Sym->VersionId = Config->DefaultSymbolVersion;
     SymVector.push_back(Sym);
   } else {
----------------
grimar wrote:
> (1)
Yes, that code is almost duplicate. I'll factor this out in a follow-up patch. 


https://reviews.llvm.org/D39406





More information about the llvm-commits mailing list