[PATCH] D39594: [ELF] - Stop using SectionKey for creating output sections.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 4 02:01:02 PDT 2017


grimar added a comment.

In https://reviews.llvm.org/D39594#915573, @ruiu wrote:

> LGTM
>
> This is very nice improvement. The 7% improvement you observed feels a bit too good, and I'm not sure if it is applicable to other test cases, but this patch itself is pretty nice.


Yeah I have the same feelings. 7% is too good to be true for general case, I tried to link mozilla today and time goes 
from 3,210261947 seconds ( +-  0,06% ) to 3,153782940 seconds ( +-  0,06% ) for me, it is about 2%, still a nice time bonus for cleanup change.


https://reviews.llvm.org/D39594





More information about the llvm-commits mailing list