[gold patch] Incremental 16/18: COMDAT groups

Cary Coutant ccoutant@google.com
Thu May 26 18:46:00 GMT 2011


>>       * incremental-dump.cc (dump_incremental_inputs): Print COMDAT groups.
>>       * incremental.cc (Incremental_inputs::report_input_section): Fix
>>       comment, indentation.
>>       (Incremental_inputs::report_comdat_group): New function.
>>       (Output_section_incremental_inputs::set_final_data_size): Adjust size
>>       of data for incremental input file entry.
>>       (Output_section_incremental_inputs::write_info_blocks): Write COMDAT
>>       group count, COMDAT group signatures.
>>       (Sized_incr_relobj::do_layout): Record kept COMDAT group info from
>>       an unchanged input file.
>>       * incremental.h (Incremental_object_entry::Incremental_object_entry):
>>       Initialize new data member.
>>       (Incremental_object_entry::add_comdat_group): New function.
>>       (Incremental_object_entry::get_comdat_group_count): New function.
>>       (Incremental_object_entry::get_comdat_signature_key): New function.
>>       (Incremental_object_entry::groups_): New data member.
>>       (Incremental_inputs::report_comdat_group): New function.
>>       (Incremental_input_entry_reader::get_symbol_offset): Adjust size of
>>       data for incremental input file entry.
>>       (Incremental_input_entry_reader::get_comdat_group_count): New function.
>>       (Incremental_input_entry_reader::get_input_section): Adjust size of
>>       data for incremental input file entry.
>>       (Incremental_input_entry_reader::get_global_symbol_reader): Likewise.
>>       (Incremental_input_entry_reader::get_comdat_group_signature): New
>>       function.
>>       * object.cc (Sized_relobj::include_section_group): Report kept
>>       COMDAT groups for incremental links.
>
> This is OK.

Thanks, committed.

> At some point some of these 28's and other offsets ought to turn into
> named consts.  Or perhaps even go the elfcpp route.

Agreed. Those magic constants started out few and manageable, but
started spreading, and I should have done something about that. Sorry.

-cary



More information about the Binutils mailing list