From 31365f57aed5fff65b2e88dd8d4ed7412097ae38 Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Wed, 12 Sep 2007 00:30:59 +0000 Subject: [PATCH] Don't try to find version definition index when creating an executable. --- gold/dynobj.cc | 18 ++++++++++++++---- gold/dynobj.h | 6 ++++-- gold/layout.cc | 4 ++-- 3 files changed, 20 insertions(+), 8 deletions(-) diff --git a/gold/dynobj.cc b/gold/dynobj.cc index 96488d8f840..10f8fbefb85 100644 --- a/gold/dynobj.cc +++ b/gold/dynobj.cc @@ -1366,7 +1366,8 @@ Versions::finalize(const Target* target, Symbol_table* symtab, // pointers. unsigned int -Versions::version_index(const Stringpool* dynpool, const Symbol* sym) const +Versions::version_index(const General_options* options, + const Stringpool* dynpool, const Symbol* sym) const { Stringpool::Key version_key; const char* version = dynpool->find(sym->version(), &version_key); @@ -1374,7 +1375,11 @@ Versions::version_index(const Stringpool* dynpool, const Symbol* sym) const Key k; if (!sym->is_from_dynobj()) - k = Key(version_key, 0); + { + if (!options->is_shared()) + return elfcpp::VER_NDX_GLOBAL; + k = Key(version_key, 0); + } else { Object* object = sym->object(); @@ -1399,7 +1404,8 @@ Versions::version_index(const Stringpool* dynpool, const Symbol* sym) const template void -Versions::symbol_section_contents(const Stringpool* dynpool, +Versions::symbol_section_contents(const General_options* options, + const Stringpool* dynpool, unsigned int local_symcount, const std::vector& syms, unsigned char** pp, @@ -1424,7 +1430,7 @@ Versions::symbol_section_contents(const Stringpool* dynpool, if (version == NULL) version_index = elfcpp::VER_NDX_GLOBAL; else - version_index = this->version_index(dynpool, *p); + version_index = this->version_index(options, dynpool, *p); elfcpp::Swap<16, big_endian>::writeval(pbuf + (*p)->dynsym_index() * 2, version_index); } @@ -1548,6 +1554,7 @@ class Sized_dynobj<64, true>; template void Versions::symbol_section_contents<32, false>( + const General_options*, const Stringpool*, unsigned int, const std::vector&, @@ -1560,6 +1567,7 @@ Versions::symbol_section_contents<32, false>( template void Versions::symbol_section_contents<32, true>( + const General_options*, const Stringpool*, unsigned int, const std::vector&, @@ -1572,6 +1580,7 @@ Versions::symbol_section_contents<32, true>( template void Versions::symbol_section_contents<64, false>( + const General_options*, const Stringpool*, unsigned int, const std::vector&, @@ -1584,6 +1593,7 @@ Versions::symbol_section_contents<64, false>( template void Versions::symbol_section_contents<64, true>( + const General_options*, const Stringpool*, unsigned int, const std::vector&, diff --git a/gold/dynobj.h b/gold/dynobj.h index d63aa6af0b2..7575acd804e 100644 --- a/gold/dynobj.h +++ b/gold/dynobj.h @@ -415,7 +415,8 @@ class Versions // version section (.gnu.version). template void - symbol_section_contents(const Stringpool*, unsigned int local_symcount, + symbol_section_contents(const General_options*, const Stringpool*, + unsigned int local_symcount, const std::vector& syms, unsigned char**, unsigned int* ACCEPT_SIZE_ENDIAN) const; @@ -455,7 +456,8 @@ class Versions // Return the version index to use for SYM. unsigned int - version_index(const Stringpool*, const Symbol* sym) const; + version_index(const General_options*, const Stringpool*, + const Symbol* sym) const; // We keep a hash table mapping canonicalized name/version pairs to // a version base. diff --git a/gold/layout.cc b/gold/layout.cc index d5337226875..3e8a223c7fd 100644 --- a/gold/layout.cc +++ b/gold/layout.cc @@ -1056,8 +1056,8 @@ Layout::sized_create_version_sections( unsigned char* vbuf; unsigned int vsize; versions->symbol_section_contents SELECT_SIZE_ENDIAN_NAME(size, big_endian)( - &this->dynpool_, local_symcount, dynamic_symbols, &vbuf, &vsize - SELECT_SIZE_ENDIAN(size, big_endian)); + &this->options_, &this->dynpool_, local_symcount, dynamic_symbols, + &vbuf, &vsize SELECT_SIZE_ENDIAN(size, big_endian)); Output_section_data* vdata = new Output_data_const_buffer(vbuf, vsize, 2); -- 2.30.2