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

Eugene Leviant via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 20 09:25:59 PST 2017

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.


More information about the llvm-commits mailing list