[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.


https://reviews.llvm.org/D28956





More information about the llvm-commits mailing list