This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] Use a std::vector instead of a std::map to hold Input_merge_map


ping


On 20 May 2015 at 11:40, Rafael EspÃndola <rafael.espindola@gmail.com> wrote:
> A new patch with the style fixes is attached.
>
>
> On 20 May 2015 at 11:30, Sriraman Tallam <tmsriram@google.com> wrote:
>> Some minor changes:
>>
>> +  const Section_merge_maps &maps = this->section_merge_maps_;
>> +  for (Section_merge_maps::const_iterator i = maps.begin(), e =maps.end();
>> +       i != e; ++i) {
>>
>> space between '=' and maps.end()
>> '{' should come in a new line?
>>
>> Otherwise patch makes the code simpler and looks good to me.  I do not
>> have the authority to approve.
>>
>> Thanks
>> Sri
>>
>> On Wed, May 20, 2015 at 6:12 AM, Rafael EspÃndola
>> <rafael.espindola@gmail.com> wrote:
>>> ping
>>>
>>> On 23 April 2015 at 12:36, Rafael EspÃndola <rafael.espindola@gmail.com> wrote:
>>>> A std::map is hardly the best data structure for a small map from
>>>> small integers.
>>>>
>>>> The attached patch uses a std::vector<std::pair>> instead.
>>>>
>>>> This simplifies the code and speeds up linking of chromium (see
>>>> attached perf logs).
>>>>
>>>> Cheers,
>>>> Rafael
>>>>
>>>> 2015-04-23  Rafael Ãvila de EspÃndola <rafael.espindola@gmail.com>
>>>>
>>>> * merge.cc (get_input_merge_map): Update for data structure change.
>>>> (get_or_make_input_merge_map): Update for data structure change.
>>>> * merge.h (Object_merge_map): Use a std::vector<std::pair>> instead of
>>>> a std::map.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]