From dd74ae067124f200158633dd96bea9900f5d55c9 Mon Sep 17 00:00:00 2001 From: Cary Coutant Date: Wed, 4 Jan 2012 00:18:23 +0000 Subject: [PATCH] * gold/incremental.cc (Sized_incremental_binary::do_process_got_plt): Use abstract base class for GOT. * gold/output.h (class Output_data_got_base): New abstract base class. (class Output_data_got): Derive from new base class, adjust ctors. (Output_data_got::reserve_slot): Make virtual; rename to do_reserve_slot; Adjust callers. * gold/target.h (Sized_target::init_got_plt_for_update): Return pointer to abstract base class. * gold/x86_64.cc (Target_x86_64::init_got_plt_for_update): Likewise. --- gold/ChangeLog | 12 ++++++++++++ gold/incremental.cc | 2 +- gold/output.cc | 4 ++-- gold/output.h | 40 +++++++++++++++++++++++++++++++--------- gold/target.h | 5 ++--- gold/x86_64.cc | 4 ++-- 6 files changed, 50 insertions(+), 17 deletions(-) diff --git a/gold/ChangeLog b/gold/ChangeLog index 790d8d381d..4adbecfcdf 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,3 +1,15 @@ +2012-01-03 Cary Coutant + + * gold/incremental.cc (Sized_incremental_binary::do_process_got_plt): + Use abstract base class for GOT. + * gold/output.h (class Output_data_got_base): New abstract base class. + (class Output_data_got): Derive from new base class, adjust ctors. + (Output_data_got::reserve_slot): Make virtual; rename to + do_reserve_slot; Adjust callers. + * gold/target.h (Sized_target::init_got_plt_for_update): Return + pointer to abstract base class. + * gold/x86_64.cc (Target_x86_64::init_got_plt_for_update): Likewise. + 2011-12-18 Ian Lance Taylor * object.h (Relobj::local_symbol_value): New function. diff --git a/gold/incremental.cc b/gold/incremental.cc index 75e44c5707..39bad37b5e 100644 --- a/gold/incremental.cc +++ b/gold/incremental.cc @@ -632,7 +632,7 @@ Sized_incremental_binary::do_process_got_plt( // Tell the target how big the GOT and PLT sections are. unsigned int got_count = got_plt_reader.get_got_entry_count(); unsigned int plt_count = got_plt_reader.get_plt_entry_count(); - Output_data_got* got = + Output_data_got_base* got = target->init_got_plt_for_update(symtab, layout, got_count, plt_count); // Read the GOT entries from the base file and build the outgoing GOT. diff --git a/gold/output.cc b/gold/output.cc index 6e46fd5dca..ca190392d1 100644 --- a/gold/output.cc +++ b/gold/output.cc @@ -1582,7 +1582,7 @@ Output_data_got::reserve_local( unsigned int sym_index, unsigned int got_type) { - this->reserve_slot(i); + this->do_reserve_slot(i); object->set_local_got_offset(sym_index, got_type, this->got_offset(i)); } @@ -1595,7 +1595,7 @@ Output_data_got::reserve_global( Symbol* gsym, unsigned int got_type) { - this->reserve_slot(i); + this->do_reserve_slot(i); gsym->set_got_offset(got_type, this->got_offset(i)); } diff --git a/gold/output.h b/gold/output.h index db38236f3d..838ca3d4dd 100644 --- a/gold/output.h +++ b/gold/output.h @@ -2151,20 +2151,42 @@ class Output_data_group : public Output_section_data // needed. The GOT_SIZE template parameter is the size in bits of a // GOT entry, typically 32 or 64. +class Output_data_got_base : public Output_section_data_build +{ + public: + Output_data_got_base(uint64_t align) + : Output_section_data_build(align) + { } + + Output_data_got_base(off_t data_size, uint64_t align) + : Output_section_data_build(data_size, align) + { } + + // Reserve the slot at index I in the GOT. + void + reserve_slot(unsigned int i) + { this->do_reserve_slot(i); } + + protected: + // Reserve the slot at index I in the GOT. + virtual void + do_reserve_slot(unsigned int i) = 0; +}; + template -class Output_data_got : public Output_section_data_build +class Output_data_got : public Output_data_got_base { public: typedef typename elfcpp::Elf_types::Elf_Addr Valtype; Output_data_got() - : Output_section_data_build(Output_data::default_alignment_for_size(got_size)), + : Output_data_got_base(Output_data::default_alignment_for_size(got_size)), entries_(), free_list_() { } Output_data_got(off_t data_size) - : Output_section_data_build(data_size, - Output_data::default_alignment_for_size(got_size)), + : Output_data_got_base(data_size, + Output_data::default_alignment_for_size(got_size)), entries_(), free_list_() { // For an incremental update, we have an existing GOT section. @@ -2231,11 +2253,6 @@ class Output_data_got : public Output_section_data_build return got_offset; } - // Reserve a slot in the GOT. - void - reserve_slot(unsigned int i) - { this->free_list_.remove(i * got_size / 8, (i + 1) * got_size / 8); } - // Reserve a slot in the GOT for a local symbol. void reserve_local(unsigned int i, Relobj* object, unsigned int sym_index, @@ -2255,6 +2272,11 @@ class Output_data_got : public Output_section_data_build do_print_to_mapfile(Mapfile* mapfile) const { mapfile->print_output_data(this, _("** GOT")); } + // Reserve the slot at index I in the GOT. + virtual void + do_reserve_slot(unsigned int i) + { this->free_list_.remove(i * got_size / 8, (i + 1) * got_size / 8); } + private: // This POD class holds a single GOT entry. class Got_entry diff --git a/gold/target.h b/gold/target.h index a378120325..10354273d2 100644 --- a/gold/target.h +++ b/gold/target.h @@ -56,8 +56,7 @@ template class Sized_symbol; class Symbol_table; class Output_data; -template -class Output_data_got; +class Output_data_got_base; class Output_section; class Input_objects; class Task; @@ -845,7 +844,7 @@ class Sized_target : public Target // Create the GOT and PLT sections for an incremental update. // A target needs to implement this to support incremental linking. - virtual Output_data_got* + virtual Output_data_got_base* init_got_plt_for_update(Symbol_table*, Layout*, unsigned int /* got_count */, diff --git a/gold/x86_64.cc b/gold/x86_64.cc index 552e9d10c9..eeb532beae 100644 --- a/gold/x86_64.cc +++ b/gold/x86_64.cc @@ -425,7 +425,7 @@ class Target_x86_64 : public Sized_target<64, false> plt_entry_size() const; // Create the GOT section for an incremental update. - Output_data_got<64, false>* + Output_data_got_base* init_got_plt_for_update(Symbol_table* symtab, Layout* layout, unsigned int got_count, @@ -1463,7 +1463,7 @@ Target_x86_64::plt_entry_size() const // Create the GOT and PLT sections for an incremental update. -Output_data_got<64, false>* +Output_data_got_base* Target_x86_64::init_got_plt_for_update(Symbol_table* symtab, Layout* layout, unsigned int got_count, -- 2.34.1