From d1a8cabd8fd4d4a74294a54045ef3d08e93969f4 Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Tue, 14 Aug 2012 01:07:01 +0000 Subject: [PATCH] * powerpc.cc (Powerpc_relobj::set_got2_shndx): Delete. (Powerpc_relobj::do_find_special_sections): Don't use set_got2_shndx. (Output_data_glink::add_entry,find_entry): Remove shndx param. (class Glink_sym_ent): Rename from struct Glink_sym_ent. Remove all references to shndx_. Handle special case for R_PPC_PLTREL24 here. (class Glink_sym_ent_hash): Rename from struct Glink_sym_ent_hash. (Output_data_glink::do_write): Retrieve got2_shdnx from object. (Target_powerpc::make_plt_entry): Don't special case R_PPC_PLTREL24 here. (Target_powerpc::Scan::global): Nor on make_plt_entry call. (Target_powerpc::Relocate::relocate): Nor on glink->find_entry call. --- gold/ChangeLog | 15 ++++++++ gold/powerpc.cc | 96 ++++++++++++++++++------------------------------- 2 files changed, 50 insertions(+), 61 deletions(-) diff --git a/gold/ChangeLog b/gold/ChangeLog index 6a19d864741..50838739611 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,3 +1,18 @@ +2012-08-14 Alan Modra + + * powerpc.cc (Powerpc_relobj::set_got2_shndx): Delete. + (Powerpc_relobj::do_find_special_sections): Don't use set_got2_shndx. + (Output_data_glink::add_entry,find_entry): Remove shndx param. + (class Glink_sym_ent): Rename from struct Glink_sym_ent. Remove + all references to shndx_. Handle special case for R_PPC_PLTREL24 + here. + (class Glink_sym_ent_hash): Rename from struct Glink_sym_ent_hash. + (Output_data_glink::do_write): Retrieve got2_shdnx from object. + (Target_powerpc::make_plt_entry): Don't special case R_PPC_PLTREL24 + here. + (Target_powerpc::Scan::global): Nor on make_plt_entry call. + (Target_powerpc::Relocate::relocate): Nor on glink->find_entry call. + 2012-08-12 Alan Modra * powerpc.cc: Whitespace fixes. Wrap overly long lines. diff --git a/gold/powerpc.cc b/gold/powerpc.cc index ba6c257c593..eb094b64717 100644 --- a/gold/powerpc.cc +++ b/gold/powerpc.cc @@ -75,15 +75,6 @@ public: return 0; } - void - set_got2_shndx(unsigned int shndx) - { - if (size == 32) - this->got2_section_ = shndx; - else - gold_unreachable(); - } - bool do_find_special_sections(Read_symbols_data* sd); @@ -722,7 +713,7 @@ Powerpc_relobj::do_find_special_sections( if (s != NULL) { unsigned int ndx = (s - pshdrs) / elfcpp::Elf_sizes::shdr_size; - this->set_got2_shndx(ndx); + this->got2_section_ = ndx; } } return Sized_relobj_file::do_find_special_sections(sd); @@ -1117,11 +1108,11 @@ class Output_data_glink : public Output_section_data // Add an entry void add_entry(const Symbol*, const elfcpp::Rela&, - const Sized_relobj*, unsigned int); + const Sized_relobj*); unsigned int find_entry(const Symbol*, const elfcpp::Rela&, - const Sized_relobj*, unsigned int) const; + const Sized_relobj*) const; unsigned int glink_entry_size() const @@ -1150,54 +1141,50 @@ class Output_data_glink : public Output_section_data void do_write(Output_file*); - struct Glink_sym_ent + class Glink_sym_ent { - Glink_sym_ent(const Symbol *sym, + public: + Glink_sym_ent(const Symbol* sym, const elfcpp::Rela& reloc, - const Sized_relobj* object, - unsigned int shndx) - : sym_(sym), object_(0), shndx_(0), addend_(0) + const Sized_relobj* object) + : sym_(sym), object_(0), addend_(0) { if (size != 32) this->addend_ = reloc.get_r_addend(); - else if (parameters->options().output_is_position_independent()) + else if (parameters->options().output_is_position_independent() + && (elfcpp::elf_r_type(reloc.get_r_info()) + == elfcpp::R_PPC_PLTREL24)) { - if (object != NULL && shndx != 0) - this->addend_ = reloc.get_r_addend(); + this->addend_ = reloc.get_r_addend(); if (this->addend_ != 0) - { - this->object_ = object; - this->shndx_ = shndx; - } + this->object_ = object; } } - const Symbol *sym_; + const Symbol* sym_; const Sized_relobj* object_; - unsigned int shndx_; unsigned int addend_; bool operator==(const Glink_sym_ent& that) const { return (this->sym_ == that.sym_ && this->object_ == that.object_ - && this->shndx_ == that.shndx_ - && this->addend_ == that.addend_); + && this->addend_ == that.addend_); } }; - struct Glink_sym_ent_hash + class Glink_sym_ent_hash { + public: size_t operator()(const Glink_sym_ent& ent) const { return (reinterpret_cast(ent.sym_) ^ reinterpret_cast(ent.object_) - ^ ent.shndx_ ^ ent.addend_); } }; - // Set of sym/shndx/addend entries. + // Map sym/object/addend to index. typedef Unordered_map Glink_entries; Glink_entries glink_entries_; @@ -1220,17 +1207,16 @@ Output_data_glink::Output_data_glink( } // Add an entry to glink, if we do not already have one for this -// sym/addend/shndx combo. +// sym/object/addend combo. template void Output_data_glink::add_entry( const Symbol* gsym, const elfcpp::Rela& reloc, - const Sized_relobj* object, - unsigned int shndx) + const Sized_relobj* object) { - Glink_sym_ent ent(gsym, reloc, object, shndx); + Glink_sym_ent ent(gsym, reloc, object); unsigned int indx = this->glink_entries_.size(); this->glink_entries_.insert(std::make_pair(ent, indx)); } @@ -1240,10 +1226,9 @@ unsigned int Output_data_glink::find_entry( const Symbol* gsym, const elfcpp::Rela& reloc, - const Sized_relobj* object, - unsigned int shndx) const + const Sized_relobj* object) const { - Glink_sym_ent ent(gsym, reloc, object, shndx); + Glink_sym_ent ent(gsym, reloc, object); typename Glink_entries::const_iterator p = this->glink_entries_.find(ent); gold_assert(p != this->glink_entries_.end()); return p->second; @@ -1433,11 +1418,15 @@ Output_data_glink::do_write(Output_file* of) p = oview + g->second * this->glink_entry_size(); if (parameters->options().output_is_position_independent()) { - unsigned int got2 = g->first.shndx_; - if (got2) - got_addr = (g->first.object_->output_section(got2)->address() - + g->first.object_->output_section_offset(got2) - + g->first.addend_); + const Powerpc_relobj* object = static_cast + *>(g->first.object_); + if (object != NULL) + { + unsigned int got2 = object->got2_shndx(); + got_addr = (g->first.object_->output_section(got2)->address() + + g->first.object_->output_section_offset(got2) + + g->first.addend_); + } else got_addr = g_o_t; @@ -1572,14 +1561,7 @@ Target_powerpc::make_plt_entry( this->plt_->add_entry(gsym); - unsigned int got2_shndx = 0; - if (size == 32 && object != NULL) - { - const Powerpc_relobj* ppc_obj - = static_cast*>(object); - got2_shndx = ppc_obj->got2_shndx(); - } - this->glink_->add_entry(gsym, reloc, object, got2_shndx); + this->glink_->add_entry(gsym, reloc, object); } // Return the number of entries in the PLT. @@ -2046,12 +2028,7 @@ Target_powerpc::Scan::global( && !(gsym->is_defined() && !gsym->is_from_dynobj() && !gsym->is_preemptible()))) - { - if (r_type == elfcpp::R_PPC_PLTREL24) - target->make_plt_entry(layout, gsym, reloc, object); - else - target->make_plt_entry(layout, gsym, reloc, 0); - } + target->make_plt_entry(layout, gsym, reloc, object); // Make a dynamic relocation if necessary. if (gsym->needs_dynamic_reloc(Scan::get_reference_flags(r_type))) { @@ -2328,10 +2305,7 @@ Target_powerpc::Relocate::relocate( const Output_data_glink* glink; glink = target->glink_section(); - unsigned int shndx = 0; - if (size == 32 && r_type == elfcpp::R_PPC_PLTREL24) - shndx = object->got2_shndx(); - unsigned int glink_index = glink->find_entry(gsym, rela, object, shndx); + unsigned int glink_index = glink->find_entry(gsym, rela, object); value = glink->address() + glink_index * glink->glink_entry_size(); } else -- 2.30.2