[PATCH] D19752: ELF: New symbol table design.

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 30 20:31:35 PDT 2016


davide accepted this revision.
davide added a comment.
This revision is now accepted and ready to land.

Thank for doing this, I found your code to be up to 15% faster while linking some FreeBSD system binaries.
As a general comment, I find the new logic slightly easier to understand, so my hope is that you convert the COFF linker once you get a chance. I don't find any obvious problems with the patch, and others commented at length on the style, so I have very little to add. lgtm.


http://reviews.llvm.org/D19752





More information about the llvm-commits mailing list