[PATCH] D28956: [LLD][ELF] Add support for local symbols in Synthetic Sections

[Eugene Leviant via Phabricator via llvm-commits]

evgeny777 added a comment.

Looks good, but wait for others, please.



================
Comment at: ELF/SyntheticSections.cpp:1143
          File->KeptLocalSyms) {
       const DefinedRegular<ELFT> &Body = *P.first;
       auto *ESym = reinterpret_cast<Elf_Sym *>(Buf);
----------------
I'd add lambda which takes 'DefinedRegular<ELFT> *' and call it in both places.


https://reviews.llvm.org/D28956