[PATCH] D65242: [ELF] More dynamic relocation packing
Vic Yang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 20 09:35:07 PDT 2019
victoryang added inline comments.
================
Comment at: lld/ELF/SyntheticSections.cpp:1711
+ // For Rela, the addend for most non-relative relocations is zero, and thus
+ // we can usually get a smaller relocation section if we group by addend
+ // as well.
----------------
MaskRay wrote:
> > if we group by addend as well
>
> I think you meant relocations with 0 addend are grouped.
Yes. Thanks!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65242/new/
https://reviews.llvm.org/D65242
More information about the llvm-commits
mailing list