* target.h (class Target): Add osabi_ field.
authorIan Lance Taylor <ian@airs.com>
Tue, 28 Jun 2011 22:25:14 +0000 (22:25 +0000)
committerIan Lance Taylor <ian@airs.com>
Tue, 28 Jun 2011 22:25:14 +0000 (22:25 +0000)
(Target::osabi): New function.
(Target::set_osabi): New function.
(Target::Target): Initialize osabi_.
(Target::do_adjust_elf_header): Make pure virtual.
(Sized_target::do_adjust_elf_header): Declare.
* target.cc (Sized_target::do_adjust_elf_header): New function.
(class Sized_target): Instantiate all versions.
* freebsd.h (class Target_freebsd): Remove.
(Target_selector_freebsd::do_recognize): Call set_osabi on
Target.
(Target_selector_freebsd::do_recognize_by_name): Likewise.
(Target_selector_freebsd::set_osabi): Remove.
* i386.cc (class Target_i386): Inherit from Sized_target rather
than Target_freebsd.
* x86_64.cc (class Target_x86_64): Likewise.

gold/ChangeLog
gold/freebsd.h
gold/i386.cc
gold/target.cc
gold/target.h
gold/x86_64.cc

index 57fbfeaa813edf99633e64dfab0ba09f8d5c7a12..4ea905fe73e1dccfaa13ac998fe154b4448ee9bb 100644 (file)
@@ -1,3 +1,22 @@
+2011-06-28  Ian Lance Taylor  <iant@google.com>
+
+       * target.h (class Target): Add osabi_ field.
+       (Target::osabi): New function.
+       (Target::set_osabi): New function.
+       (Target::Target): Initialize osabi_.
+       (Target::do_adjust_elf_header): Make pure virtual.
+       (Sized_target::do_adjust_elf_header): Declare.
+       * target.cc (Sized_target::do_adjust_elf_header): New function.
+       (class Sized_target): Instantiate all versions.
+       * freebsd.h (class Target_freebsd): Remove.
+       (Target_selector_freebsd::do_recognize): Call set_osabi on
+       Target.
+       (Target_selector_freebsd::do_recognize_by_name): Likewise.
+       (Target_selector_freebsd::set_osabi): Remove.
+       * i386.cc (class Target_i386): Inherit from Sized_target rather
+       than Target_freebsd.
+       * x86_64.cc (class Target_x86_64): Likewise.
+
 2011-06-28  Ian Lance Taylor  <iant@google.com>
 
        * target.h (Target::can_check_for_function_pointers): Rewrite.
index de697357f41f1bb10d89f857bf178f7924d49bc5..bb86432519fbcf10c1090fdf602f7b7d2b8ecf1a 100644 (file)
@@ -1,6 +1,6 @@
 // freebsd.h -- FreeBSD support for gold    -*- C++ -*-
 
-// Copyright 2009 Free Software Foundation, Inc.
+// Copyright 2009, 2011 Free Software Foundation, Inc.
 // Written by Ian Lance Taylor <iant@google.com>.
 
 // This file is part of gold.
@@ -30,59 +30,8 @@ namespace gold
 {
 
 // FreeBSD 4.1 and later wants the EI_OSABI field in the ELF header to
-// be set to ELFOSABI_FREEBSD.  This is a subclass of Sized_target
-// which supports that.  The real target would be a subclass of this
-// one.  We permit combining FreeBSD and non-FreeBSD object files.
-// The effect of this target is to set the code in the output file.
-
-template<int size, bool big_endian>
-class Target_freebsd : public Sized_target<size, big_endian>
-{
- public:
-  // Set the value to use for the EI_OSABI field in the ELF header.
-  void
-  set_osabi(elfcpp::ELFOSABI osabi)
-  { this->osabi_ = osabi; }
-
- protected:
-  Target_freebsd(const Target::Target_info* pti)
-    : Sized_target<size, big_endian>(pti),
-      osabi_(elfcpp::ELFOSABI_NONE)
-  { }
-
-  virtual void
-  do_adjust_elf_header(unsigned char* view, int len) const;
-
- private:
-  // Value to store in the EI_OSABI field of the ELF file header.
-  elfcpp::ELFOSABI osabi_;
-};
-
-// Adjust the ELF file header by storing the requested value in the
-// OSABI field.  This is for FreeBSD support.
-
-template<int size, bool big_endian>
-inline void
-Target_freebsd<size, big_endian>::do_adjust_elf_header(unsigned char* view,
-                                                      int len) const
-{
-  if (this->osabi_ != elfcpp::ELFOSABI_NONE)
-    {
-      gold_assert(len == elfcpp::Elf_sizes<size>::ehdr_size);
-
-      elfcpp::Ehdr<size, false> ehdr(view);
-      unsigned char e_ident[elfcpp::EI_NIDENT];
-      memcpy(e_ident, ehdr.get_e_ident(), elfcpp::EI_NIDENT);
-
-      e_ident[elfcpp::EI_OSABI] = this->osabi_;
-
-      elfcpp::Ehdr_write<size, false> oehdr(view);
-      oehdr.put_e_ident(e_ident);
-    }
-}
-
-// A target selector for targets which permit combining both FreeBSD
-// and non-FreeBSD object files.
+// be set to ELFOSABI_FREEBSD.  This is a target selector for targets
+// which permit combining both FreeBSD and non-FreeBSD object files.
 
 class Target_selector_freebsd : public Target_selector
 {
@@ -102,7 +51,7 @@ class Target_selector_freebsd : public Target_selector
   {
     Target* ret = this->instantiate_target();
     if (osabi == elfcpp::ELFOSABI_FREEBSD)
-      this->set_osabi(ret);
+      ret->set_osabi(static_cast<elfcpp::ELFOSABI>(osabi));
     return ret;
   }
   
@@ -115,7 +64,7 @@ class Target_selector_freebsd : public Target_selector
     else if (strcmp(name, this->freebsd_bfd_name_) == 0)
       {
        Target* ret = this->instantiate_target();
-       this->set_osabi(ret);
+       ret->set_osabi(elfcpp::ELFOSABI_FREEBSD);
        return ret;
       }
     else
@@ -131,32 +80,6 @@ class Target_selector_freebsd : public Target_selector
   }
 
  private:
-  // Set the OSABI field.  This is quite ugly.
-  void
-  set_osabi(Target* target)
-  {
-    if (this->get_size() == 32)
-      {
-       if (this->is_big_endian())
-         static_cast<Target_freebsd<32, true>*>(target)->
-           set_osabi(elfcpp::ELFOSABI_FREEBSD);
-       else
-         static_cast<Target_freebsd<32, false>*>(target)->
-           set_osabi(elfcpp::ELFOSABI_FREEBSD);
-      }
-    else if (this->get_size() == 64)
-      {
-       if (this->is_big_endian())
-         static_cast<Target_freebsd<64, true>*>(target)->
-           set_osabi(elfcpp::ELFOSABI_FREEBSD);
-       else
-         static_cast<Target_freebsd<64, false>*>(target)->
-           set_osabi(elfcpp::ELFOSABI_FREEBSD);
-      }
-    else
-      gold_unreachable();
-  }
-
   // The BFD name for the non-Freebsd target.
   const char* bfd_name_;
   // The BFD name for the Freebsd target.
index bd5eaaf7145db28dfe52737a294ac5bd9815c3ab..b158b1f6b7d4da4d1dd491f9c4cfd189f086a091 100644 (file)
@@ -158,13 +158,13 @@ class Output_data_plt_i386 : public Output_section_data
 //   http://people.redhat.com/drepper/tls.pdf
 //   http://www.lsd.ic.unicamp.br/~oliva/writeups/TLS/RFC-TLSDESC-x86.txt
 
-class Target_i386 : public Target_freebsd<32, false>
+class Target_i386 : public Sized_target<32, false>
 {
  public:
   typedef Output_data_reloc<elfcpp::SHT_REL, true, 32, false> Reloc_section;
 
   Target_i386()
-    : Target_freebsd<32, false>(&i386_info),
+    : Sized_target<32, false>(&i386_info),
       got_(NULL), plt_(NULL), got_plt_(NULL), got_tlsdesc_(NULL),
       global_offset_table_(NULL), rel_dyn_(NULL),
       copy_relocs_(elfcpp::R_386_COPY), dynbss_(NULL),
index 0e329cdd0b38e6c5507c713fae273031f2f42c06..4a7af83c161735db77b20a63378387c2fab5daf4 100644 (file)
@@ -1,6 +1,6 @@
-// target.cc
+// target.cc -- target support for gold.
 
-// Copyright 2009, 2010 Free Software Foundation, Inc.
+// Copyright 2009, 2010, 2011 Free Software Foundation, Inc.
 // Written by Doug Kwan <dougkwan@google.com>.
 
 // This file is part of gold.
@@ -200,4 +200,49 @@ Target::set_view_to_nop(unsigned char* view, section_size_type view_size,
     }
 }
 
+// Class Sized_target.
+
+// Set the EI_OSABI field of the ELF header if requested.
+
+template<int size, bool big_endian>
+void
+Sized_target<size, big_endian>::do_adjust_elf_header(unsigned char* view,
+                                                    int len) const
+{
+  elfcpp::ELFOSABI osabi = this->osabi();
+  if (osabi != elfcpp::ELFOSABI_NONE)
+    {
+      gold_assert(len == elfcpp::Elf_sizes<size>::ehdr_size);
+
+      elfcpp::Ehdr<size, false> ehdr(view);
+      unsigned char e_ident[elfcpp::EI_NIDENT];
+      memcpy(e_ident, ehdr.get_e_ident(), elfcpp::EI_NIDENT);
+
+      e_ident[elfcpp::EI_OSABI] = osabi;
+
+      elfcpp::Ehdr_write<size, false> oehdr(view);
+      oehdr.put_e_ident(e_ident);
+    }
+}
+
+#ifdef HAVE_TARGET_32_LITTLE
+template
+class Sized_target<32, false>;
+#endif
+
+#ifdef HAVE_TARGET_32_BIG
+template
+class Sized_target<32, true>;
+#endif
+
+#ifdef HAVE_TARGET_64_LITTLE
+template
+class Sized_target<64, false>;
+#endif
+
+#ifdef HAVE_TARGET_64_BIG
+template
+class Sized_target<64, true>;
+#endif
+
 } // End namespace gold.
index 1d523b87e496047fd73b0f8ec6179dbd8eb18831..f9b8c0459294f43be56c34764ef6e8ff9c34782d 100644 (file)
@@ -375,6 +375,17 @@ class Target
   select_as_default_target()
   { this->do_select_as_default_target(); } 
 
+  // Return the value to store in the EI_OSABI field in the ELF
+  // header.
+  elfcpp::ELFOSABI
+  osabi() const
+  { return this->osabi_; }
+
+  // Set the value to store in the EI_OSABI field in the ELF header.
+  void
+  set_osabi(elfcpp::ELFOSABI osabi)
+  { this->osabi_ = osabi; }
+
  protected:
   // This struct holds the constant information for a child class.  We
   // use a struct to avoid the overhead of virtual function calls for
@@ -427,7 +438,7 @@ class Target
 
   Target(const Target_info* pti)
     : pti_(pti), processor_specific_flags_(0),
-      are_processor_specific_flags_set_(false)
+      are_processor_specific_flags_set_(false), osabi_(elfcpp::ELFOSABI_NONE)
   { }
 
   // Virtual function which may be implemented by the child class.
@@ -459,10 +470,10 @@ class Target
   // Adjust the output file header before it is written out.  VIEW
   // points to the header in external form.  LEN is the length, and
   // will be one of the values of elfcpp::Elf_sizes<size>::ehdr_size.
-  // By default, we do nothing.
+  // By default, we set the EI_OSABI field if requested (in
+  // Sized_target).
   virtual void
-  do_adjust_elf_header(unsigned char*, int) const
-  { }
+  do_adjust_elf_header(unsigned char*, int) const = 0;
 
   // Virtual function which may be overridden by the child class.
   virtual bool
@@ -622,6 +633,10 @@ class Target
   elfcpp::Elf_Word processor_specific_flags_;
   // Whether the processor-specific flags are set at least once.
   bool are_processor_specific_flags_set_;
+  // If not ELFOSABI_NONE, the value to put in the EI_OSABI field of
+  // the ELF header.  This is handled at this level because it is
+  // OS-specific rather than processor-specific.
+  elfcpp::ELFOSABI osabi_;
 };
 
 // The abstract class for a specific size and endianness of target.
@@ -873,6 +888,10 @@ class Sized_target : public Target
     gold_assert(pti->size == size);
     gold_assert(pti->is_big_endian ? big_endian : !big_endian);
   }
+
+  // Set the EI_OSABI field if requested.
+  virtual void
+  do_adjust_elf_header(unsigned char*, int) const;
 };
 
 } // End namespace gold.
index a6e9ddd727fba02f5eb76d8b56c71da527ac3ff0..6d04dc7fa28530613212aea34d69644982050519 100644 (file)
@@ -201,7 +201,7 @@ class Output_data_plt_x86_64 : public Output_section_data
 //   http://people.redhat.com/drepper/tls.pdf
 //   http://www.lsd.ic.unicamp.br/~oliva/writeups/TLS/RFC-TLSDESC-x86.txt
 
-class Target_x86_64 : public Target_freebsd<64, false>
+class Target_x86_64 : public Sized_target<64, false>
 {
  public:
   // In the x86_64 ABI (p 68), it says "The AMD64 ABI architectures
@@ -209,7 +209,7 @@ class Target_x86_64 : public Target_freebsd<64, false>
   typedef Output_data_reloc<elfcpp::SHT_RELA, true, 64, false> Reloc_section;
 
   Target_x86_64()
-    : Target_freebsd<64, false>(&x86_64_info),
+    : Sized_target<64, false>(&x86_64_info),
       got_(NULL), plt_(NULL), got_plt_(NULL), got_tlsdesc_(NULL),
       global_offset_table_(NULL), rela_dyn_(NULL),
       copy_relocs_(elfcpp::R_X86_64_COPY), dynbss_(NULL),