[PATCH] D45325: [ELF] - Reorder local symbols.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 11 02:19:57 PDT 2018


grimar added a comment.

In https://reviews.llvm.org/D45325#1063671, @espindola wrote:

> LGTM
>
> I think this is fine. It is a bit funny that we count the section symbols as being part of the first file that had that section, but that is unlikely to be a problem.


We could just stop passing the file when we create them in `addSectionSymbols` probably.


https://reviews.llvm.org/D45325





More information about the llvm-commits mailing list