[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