* options.cc (options::help): Print list of supported targets.
authorIan Lance Taylor <ian@airs.com>
Tue, 25 Mar 2008 21:39:01 +0000 (21:39 +0000)
committerIan Lance Taylor <ian@airs.com>
Tue, 25 Mar 2008 21:39:01 +0000 (21:39 +0000)
* target-select.h: Include <vector>.
(class Target_selector): Make machine_, size_, and is_big_endian_
fields const.  Add bfd_name_ and instantiated_target_ fields.
(Target_selector::Target_selector): Add bfd_name parameter.
(Target_selector::recognize): Make non-virtual, call
do_recognize.
(Target_selector::recognize_by_name): Make non-virtual, call
do_recognize_by_name.
(Target_selector::supported_names): New function.
(Target_selector::bfd_name): New function.
(Target_selector::do_instantiate_target): New pure virtual
function.
(Target_selector::do_recognize): New virtual function.
(Target_selector::do_recognize_by_name): New virtual function.
(Target_selector::instantiate_target): New private function.
(supported_target_names): Declare.
* target-select.cc (Target_selector::Target_selector): Update for
new parameter and fields.
(select_target_by_name): Check that the name matches before
calling recognize_by_name.
(supported_target_names): New function.
* i386.cc (class Target_selector_i386): Update Target_selector
constructor call.  Remove recognize and recognize_by_name.  Add
do_instantiate_target.
* x86_64.cc (class Target_selector_x86_64): Likewise.
* testsuite/testfile.cc (class Target_selector_test): Update for
changes to Target_selector.

gold/ChangeLog
gold/i386.cc
gold/options.cc
gold/target-select.cc
gold/target-select.h
gold/testsuite/testfile.cc
gold/x86_64.cc

index 020419a498751542c616eccc08652ec7090cafec..74d4eb2ae061db52a5b0353888595532f2347e8a 100644 (file)
@@ -1,5 +1,34 @@
 2008-03-25  Ian Lance Taylor  <iant@google.com>
 
+       * options.cc (options::help): Print list of supported targets.
+       * target-select.h: Include <vector>.
+       (class Target_selector): Make machine_, size_, and is_big_endian_
+       fields const.  Add bfd_name_ and instantiated_target_ fields.
+       (Target_selector::Target_selector): Add bfd_name parameter.
+       (Target_selector::recognize): Make non-virtual, call
+       do_recognize.
+       (Target_selector::recognize_by_name): Make non-virtual, call
+       do_recognize_by_name.
+       (Target_selector::supported_names): New function.
+       (Target_selector::bfd_name): New function.
+       (Target_selector::do_instantiate_target): New pure virtual
+       function.
+       (Target_selector::do_recognize): New virtual function.
+       (Target_selector::do_recognize_by_name): New virtual function.
+       (Target_selector::instantiate_target): New private function.
+       (supported_target_names): Declare.
+       * target-select.cc (Target_selector::Target_selector): Update for
+       new parameter and fields.
+       (select_target_by_name): Check that the name matches before
+       calling recognize_by_name.
+       (supported_target_names): New function.
+       * i386.cc (class Target_selector_i386): Update Target_selector
+       constructor call.  Remove recognize and recognize_by_name.  Add
+       do_instantiate_target.
+       * x86_64.cc (class Target_selector_x86_64): Likewise.
+       * testsuite/testfile.cc (class Target_selector_test): Update for
+       changes to Target_selector.
+
        * README: Rewrite, with some notes on unsupported features.
 
 2008-03-24  Cary Coutant  <ccoutant@google.com>
index f12b6cdebf0dbf5a34b1722f83f1ad7d9e90b5e3..b7bf36203a4c39c020aa2b4c83af8a63d54f9c3a 100644 (file)
@@ -2444,40 +2444,14 @@ class Target_selector_i386 : public Target_selector
 {
 public:
   Target_selector_i386()
-    : Target_selector(elfcpp::EM_386, 32, false)
+    : Target_selector(elfcpp::EM_386, 32, false, "elf32-i386")
   { }
 
   Target*
-  recognize(int machine, int osabi, int abiversion);
-
-  Target*
-  recognize_by_name(const char* name);
-
- private:
-  Target_i386* target_;
+  do_instantiate_target()
+  { return new Target_i386(); }
 };
 
-// Recognize an i386 object file when we already know that the machine
-// number is EM_386.
-
-Target*
-Target_selector_i386::recognize(int, int, int)
-{
-  if (this->target_ == NULL)
-    this->target_ = new Target_i386();
-  return this->target_;
-}
-
-Target*
-Target_selector_i386::recognize_by_name(const char* name)
-{
-  if (strcmp(name, "elf32-i386") != 0)
-    return NULL;
-  if (this->target_ == NULL)
-    this->target_ = new Target_i386();
-  return this->target_;
-}
-
 Target_selector_i386 target_selector_i386;
 
 } // End anonymous namespace.
index 46d84cf55364cf47c8ee936552c7cc7d2e81e1b3..5d565324b8f8f10f3e4c62d5fc629cdc157aa7c7 100644 (file)
@@ -142,6 +142,17 @@ help()
   std::vector<const One_option*>::const_iterator it;
   for (it = registered_options.begin(); it != registered_options.end(); ++it)
     (*it)->print();
+
+  // config.guess and libtool.m4 look in ld --help output for the
+  // string "supported targets".
+  printf(_("%s: supported targets:"), gold::program_name);
+  std::vector<const char*> supported_names;
+  gold::supported_target_names(&supported_names);
+  for (std::vector<const char*>::const_iterator p = supported_names.begin();
+       p != supported_names.end();
+       ++p)
+    printf(" %s", *p);
+  printf("\n");
 }
 
 // For bool, arg will be NULL (boolean options take no argument);
index 8a7570807891a699dacf2071d30ec753dd4eb410..092a1df46fa8f25dd70d047b550bfc222fe7b65e 100644 (file)
@@ -41,8 +41,11 @@ namespace gold
 // list.  This runs at global constructor time, so we want it to be
 // fast.
 
-Target_selector::Target_selector(int machine, int size, bool is_big_endian)
-  : machine_(machine), size_(size), is_big_endian_(is_big_endian)
+Target_selector::Target_selector(int machine, int size, bool is_big_endian,
+                                const char* bfd_name)
+  : machine_(machine), size_(size), is_big_endian_(is_big_endian),
+    bfd_name_(bfd_name), instantiated_target_(NULL)
+    
 {
   this->next_ = target_selectors;
   target_selectors = this;
@@ -77,11 +80,24 @@ select_target_by_name(const char* name)
 {
   for (Target_selector* p = target_selectors; p != NULL; p = p->next())
     {
-      Target* ret = p->recognize_by_name(name);
-      if (ret != NULL)
-       return ret;
+      const char* pname = p->bfd_name();
+      if (pname == NULL || strcmp(pname, name) == 0)
+       {
+         Target* ret = p->recognize_by_name(name);
+         if (ret != NULL)
+           return ret;
+       }
     }
   return NULL;
 }
 
+// Push all the supported BFD names onto a vector.
+
+void
+supported_target_names(std::vector<const char*>* names)
+{
+  for (Target_selector* p = target_selectors; p != NULL; p = p->next())
+    p->supported_names(names);
+}
+
 } // End namespace gold.
index c8f6c55c18180a3769310c44eafd282c3d13284b..0582e2580a174e8a27a63785110525b01b6730d2 100644 (file)
@@ -23,6 +23,8 @@
 #ifndef GOLD_TARGET_SELECT_H
 #define GOLD_TARGET_SELECT_H
 
+#include <vector>
+
 namespace gold
 {
 
@@ -41,29 +43,42 @@ class Target_selector
  public:
   // Create a target selector for a specific machine number, size (32
   // or 64), and endianness.  The machine number can be EM_NONE to
-  // test for any machine number.
-  Target_selector(int machine, int size, bool is_big_endian);
+  // test for any machine number.  BFD_NAME is the name of the target
+  // used by the GNU linker, for backward compatibility; it may be
+  // NULL.
+  Target_selector(int machine, int size, bool is_big_endian,
+                 const char* bfd_name);
 
   virtual ~Target_selector()
   { }
 
   // If we can handle this target, return a pointer to a target
   // structure.  The size and endianness are known.
-  virtual Target*
-  recognize(int machine, int osabi, int abiversion) = 0;
+  Target*
+  recognize(int machine, int osabi, int abiversion)
+  { return this->do_recognize(machine, osabi, abiversion); }
 
   // If NAME matches the target, return a pointer to a target
   // structure.
-  virtual Target*
-  recognize_by_name(const char* name) = 0;
+  Target*
+  recognize_by_name(const char* name)
+  { return this->do_recognize_by_name(name); }
+
+  // Push all supported names onto the vector.  This is only used for
+  // help output.
+  void
+  supported_names(std::vector<const char*>* names)
+  { this->do_supported_names(names); }
 
   // Return the next Target_selector in the linked list.
   Target_selector*
   next() const
   { return this->next_; }
 
-  // Return the machine number this selector is looking for, which can
-  // be EM_NONE to match any machine number.
+  // Return the machine number this selector is looking for.  This can
+  // be EM_NONE to match any machine number, in which case the
+  // do_recognize hook will be responsible for matching the machine
+  // number.
   int
   machine() const
   { return this->machine_; }
@@ -78,21 +93,88 @@ class Target_selector
   is_big_endian() const
   { return this->is_big_endian_; }
 
+  // Return the BFD name.  This may return NULL, in which case the
+  // do_recognize_by_name hook will be responsible for matching the
+  // BFD name.
+  const char*
+  bfd_name() const
+  { return this->bfd_name_; }
+
+ protected:
+  // Return an instance of the real target.  This must be implemented
+  // by the child class.
+  virtual Target*
+  do_instantiate_target() = 0;
+
+  // Recognize an object file given a machine code, size, and
+  // endianness.  When this is called we already know that they match
+  // the machine_, size_, and is_big_endian_ fields.  The child class
+  // may implement a different version of this to do additional
+  // checks, or to check for multiple machine codes if the machine_
+  // field is EM_NONE.
+  virtual Target*
+  do_recognize(int, int, int)
+  { return this->instantiate_target(); }
+
+  // Recognize a target by name.  When this is called we already know
+  // that the name matches (or that the bfd_name_ field is NULL).  The
+  // child class may implement a different version of this to
+  // recognize more than one name.
+  virtual Target*
+  do_recognize_by_name(const char*)
+  { return this->instantiate_target(); }
+
+  // Return a list of supported BFD names.  The child class may
+  // implement a different version of this to handle more than one
+  // name.
+  virtual void
+  do_supported_names(std::vector<const char*>* names)
+  {
+    gold_assert(this->bfd_name_ != NULL);
+    names->push_back(this->bfd_name_);
+  }
+
  private:
-  int machine_;
-  int size_;
-  bool is_big_endian_;
+  // Instantiate the target and return it.
+  Target*
+  instantiate_target()
+  {
+    if (this->instantiated_target_ == NULL)
+      this->instantiated_target_ = this->do_instantiate_target();
+    return this->instantiated_target_;
+  }
+
+  // ELF machine code.
+  const int machine_;
+  // Target size--32 or 64.
+  const int size_;
+  // Whether the target is big endian.
+  const bool is_big_endian_;
+  // BFD name of target, for compatibility.
+  const char* const bfd_name_;
+  // Next entry in list built at global constructor time.
   Target_selector* next_;
+  // The singleton Target structure--this points to an instance of the
+  // real implementation.
+  Target* instantiated_target_;
 };
 
 // Select the target for an ELF file.
 
-extern Target* select_target(int machine, int size, bool big_endian,
-                            int osabi, int abiversion);
+extern Target*
+select_target(int machine, int size, bool big_endian, int osabi,
+             int abiversion);
 
 // Select a target using a BFD name.
 
-extern Target* select_target_by_name(const char* name);
+extern Target*
+select_target_by_name(const char* name);
+
+// Fill in a vector with the list of supported targets.  This returns
+// a list of BFD names.
+
+extern void
+supported_target_names(std::vector<const char*>*);
 
 } // End namespace gold.
 
index 1289fea8e6daa62510f5ea061b308eabf6d805db..d33078f5f801ab9190531bb9b12b29da300d1f05 100644 (file)
@@ -137,11 +137,18 @@ class Target_selector_test : public Target_selector
 {
  public:
   Target_selector_test()
-    : Target_selector(0xffff, size, big_endian)
+    : Target_selector(0xffff, size, big_endian, NULL)
   { }
 
   Target*
-  recognize(int, int, int)
+  do_instantiate_target()
+  {
+    gold_unreachable();
+    return NULL;
+  }
+
+  Target*
+  do_recognize(int, int, int)
   {
     if (size == 32)
       {
@@ -178,8 +185,12 @@ class Target_selector_test : public Target_selector
   }
 
   Target*
-  recognize_by_name(const char*)
+  do_recognize_by_name(const char*)
   { return NULL; }
+
+  void
+  do_supported_names(std::vector<const char*>*)
+  { }
 };
 
 // Register the test target selectors.  These don't need to be
index 72c077970a5c7ee3ba4907f931f6a36611d53716..26f73480924ea9f298c6d198acf0de6a4a8afcfe 100644 (file)
@@ -2219,40 +2219,14 @@ class Target_selector_x86_64 : public Target_selector
 {
 public:
   Target_selector_x86_64()
-    : Target_selector(elfcpp::EM_X86_64, 64, false)
+    : Target_selector(elfcpp::EM_X86_64, 64, false, "elf64-x86-64")
   { }
 
   Target*
-  recognize(int machine, int osabi, int abiversion);
-
-  Target*
-  recognize_by_name(const char*);
-
- private:
-  Target_x86_64* target_;
+  do_instantiate_target()
+  { return new Target_x86_64(); }
 };
 
-// Recognize an x86_64 object file when we already know that the machine
-// number is EM_X86_64.
-
-Target*
-Target_selector_x86_64::recognize(int, int, int)
-{
-  if (this->target_ == NULL)
-    this->target_ = new Target_x86_64();
-  return this->target_;
-}
-
-Target*
-Target_selector_x86_64::recognize_by_name(const char* name)
-{
-  if (strcmp(name, "elf64-x86-64") != 0)
-    return NULL;
-  if (this->target_ == NULL)
-    this->target_ = new Target_x86_64();
-  return this->target_;
-}
-
 Target_selector_x86_64 target_selector_x86_64;
 
 } // End anonymous namespace.