[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