Re: [GOLD][PATCH] Handle initial TLS relocations in static linking.

[Ian Lance Taylor <iant at google dot com>]

"Doug Kwan (éæå)" <dougkwan@google.com> writes:

>      This is an updated patch.  I simplified code in
> Arm_output_data_got::do_write.   The original patch was also wrong in
> handling undefined symbols.  This patch fixes that.  There is no
> change in ChangeLog.

>>>> 2010-02-23  Doug Kwan  <dougkwan@google.com>
>>>>
>>>>        * arm.cc (Arm_output_data_got): New class.
>>>>        (ARM_TCB_SIZE): New constant
>>>>        (Target_arm): Use Arm_output_data_got instead of Output_data_got.
>>>>        (Arm_output_section::fix_exidx_coverage): Add a parameter for layout.
>>>>        If user uses a script with a SECTIONS clause, issue only a warning
>>>>        for a misplaced EXIDX input section.  Otherwise, issue an error.
>>>>        (Arm_relobj::do_gc_process_relocs): Exit early if we are not doing
>>>>        garbage collection.
>>>>        (Target_arm::got_mode_index_entry): Handle static linking.
>>>>        (Target_arm::Scan::local): Ditto.
>>>>        (Target_arm::Scan::global): Ditto.
>>>>        (Target_arm::Relocate::relocate_tls): Handle static linking.  Fix
>>>>        all incorrectly implemented relocations.
>>>>        (Target_arm::fix_exidx_coverage): Pass layout to
>>>>        Arm_output_section::fix_exidx_coverage.
>>>>        * layout.cc (Layout::section_name_mapping): Remove trailing dots
>>>>        from ".ARM.exidx." and ".ARM.extab.".

This is OK.

Thanks.

Ian