From e96caa791095788fe6b3cbbdc54547278ca70db7 Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Tue, 25 Mar 2008 21:39:01 +0000 Subject: [PATCH] * options.cc (options::help): Print list of supported targets. * target-select.h: Include . (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 | 29 ++++++++++++ gold/i386.cc | 32 ++----------- gold/options.cc | 11 +++++ gold/target-select.cc | 26 ++++++++--- gold/target-select.h | 110 +++++++++++++++++++++++++++++++++++++++------ gold/testsuite/testfile.cc | 17 +++++-- gold/x86_64.cc | 32 ++----------- 7 files changed, 177 insertions(+), 80 deletions(-) diff --git a/gold/ChangeLog b/gold/ChangeLog index 020419a..74d4eb2 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,5 +1,34 @@ 2008-03-25 Ian Lance Taylor + * options.cc (options::help): Print list of supported targets. + * target-select.h: Include . + (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 diff --git a/gold/i386.cc b/gold/i386.cc index f12b6cd..b7bf362 100644 --- a/gold/i386.cc +++ b/gold/i386.cc @@ -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. diff --git a/gold/options.cc b/gold/options.cc index 46d84cf..5d56532 100644 --- a/gold/options.cc +++ b/gold/options.cc @@ -142,6 +142,17 @@ help() std::vector::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 supported_names; + gold::supported_target_names(&supported_names); + for (std::vector::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); diff --git a/gold/target-select.cc b/gold/target-select.cc index 8a75708..092a1df 100644 --- a/gold/target-select.cc +++ b/gold/target-select.cc @@ -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* names) +{ + for (Target_selector* p = target_selectors; p != NULL; p = p->next()) + p->supported_names(names); +} + } // End namespace gold. diff --git a/gold/target-select.h b/gold/target-select.h index c8f6c55..0582e25 100644 --- a/gold/target-select.h +++ b/gold/target-select.h @@ -23,6 +23,8 @@ #ifndef GOLD_TARGET_SELECT_H #define GOLD_TARGET_SELECT_H +#include + 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* 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* 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*); } // End namespace gold. diff --git a/gold/testsuite/testfile.cc b/gold/testsuite/testfile.cc index 1289fea..d33078f 100644 --- a/gold/testsuite/testfile.cc +++ b/gold/testsuite/testfile.cc @@ -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*) + { } }; // Register the test target selectors. These don't need to be diff --git a/gold/x86_64.cc b/gold/x86_64.cc index 72c0779..26f7348 100644 --- a/gold/x86_64.cc +++ b/gold/x86_64.cc @@ -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. -- 2.7.4