This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [patch] Remove is_merge_section_for
- From: Rafael EspÃndola <rafael dot espindola at gmail dot com>
- To: Binutils <binutils at sourceware dot org>
- Cc: Cary Coutant <ccoutant at google dot com>
- Date: Thu, 19 Mar 2015 11:40:07 -0400
- Subject: Re: [patch] Remove is_merge_section_for
- Authentication-results: sourceware.org; auth=none
- References: <CAG3jRe+oXa9JnUi3xAnHpmKNtpRaqdg3PBBOWZ19Jbst7_vwHQ at mail dot gmail dot com> <CAG3jReLND4e0+c67uOo+2g9=Auy3qvGzA61N1HSkzKkr39Sekw at mail dot gmail dot com>
Ping, it does make the link 1.08X faster :-)
On 16 March 2015 at 10:57, Rafael EspÃndola <rafael.espindola@gmail.com> wrote:
> ping.
>
> On 9 March 2015 at 12:01, Rafael EspÃndola <rafael.espindola@gmail.com> wrote:
>> Hi Cary,
>>
>> Now that Input_merge_map has an Output_section_data, we can use it in
>> implementing find_merge_section and replace the only use of
>> is_merge_section_for with it.
>>
>> This simplifies the code a bit and is faster. Linking chromium goes from
>>
>> 6173.193320 task-clock (msec) # 0.999 CPUs utilized
>> ( +- 0.62% )
>>
>> to
>>
>> 5693.481663 task-clock (msec) # 0.999 CPUs utilized
>> ( +- 0.49% )
>>
>> Cheers,
>> Rafael