[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