From b12fcc6e5ac25c21e01429455fe44b042123b5e5 Mon Sep 17 00:00:00 2001 From: Eric Christopher Date: Jun 21 2017 23:26:30 +0000 Subject: 2017-06-21 Eric Christopher Apply from master 2017-06-20 Eric Christopher * aarch64.cc (scan_reloc_for_stub): Use plt_address_for_global to calculate the symbol value. (scan_reloc_section_for_stubs): Allow stubs to be created for section symbols. (maybe_apply_stub): Handle creating stubs for weak symbols to match the code in scan_reloc_for_stub. --- diff --git a/gold/ChangeLog b/gold/ChangeLog index 3a2b3f3..4c65d4d 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,6 +1,18 @@ 2017-06-21 Eric Christopher Apply from master + 2017-06-20 Eric Christopher + + * aarch64.cc (scan_reloc_for_stub): Use plt_address_for_global to + calculate the symbol value. + (scan_reloc_section_for_stubs): Allow stubs to be created for + section symbols. + (maybe_apply_stub): Handle creating stubs for weak symbols to + match the code in scan_reloc_for_stub. + +2017-06-21 Eric Christopher + + Apply from master 2017-06-15 Eric Christopher * aarch64.cc: Fix a few typos and grammar-os. diff --git a/gold/aarch64.cc b/gold/aarch64.cc index 9d4ea50..9d8815a 100644 --- a/gold/aarch64.cc +++ b/gold/aarch64.cc @@ -3734,8 +3734,7 @@ Target_aarch64::scan_reloc_for_stub( if (gsym->use_plt_offset(arp->reference_flags())) { // This uses a PLT, change the symbol value. - symval.set_output_value(this->plt_section()->address() - + gsym->plt_offset()); + symval.set_output_value(this->plt_address_for_global(gsym)); psymval = &symval; } else if (gsym->is_undefined()) @@ -3968,11 +3967,6 @@ Target_aarch64::scan_reloc_section_for_stubs( psymval = &symval2; } - // If symbol is a section symbol, we don't know the actual type of - // destination. Give up. - if (psymval->is_section_symbol()) - continue; - this->scan_reloc_for_stub(relinfo, r_type, sym, r_sym, psymval, addend, view_address + offset); } // End of iterating relocs in a section @@ -5404,8 +5398,14 @@ maybe_apply_stub(unsigned int r_type, const The_aarch64_relobj* aarch64_relobj = static_cast(object); - // We don't create stubs for undefined symbols so don't look for one. - if (gsym && gsym->is_undefined()) + const AArch64_reloc_property* arp = + aarch64_reloc_property_table->get_reloc_property(r_type); + gold_assert(arp != NULL); + + // We don't create stubs for undefined symbols, but do for weak. + if (gsym + && !gsym->use_plt_offset(arp->reference_flags()) + && gsym->is_undefined()) { gold_debug(DEBUG_TARGET, "stub: looking for a stub for undefined symbol %s in file %s", @@ -5424,9 +5424,6 @@ maybe_apply_stub(unsigned int r_type, Address new_branch_target = stub_table->address() + stub->offset(); typename elfcpp::Swap::Valtype branch_offset = new_branch_target - address; - const AArch64_reloc_property* arp = - aarch64_reloc_property_table->get_reloc_property(r_type); - gold_assert(arp != NULL); typename This::Status status = This::template rela_general<32>(view, branch_offset, 0, arp); if (status != This::STATUS_OKAY)