diff options
author | Sriraman Tallam <tmsriram@google.com> | 2011-07-22 22:38:42 +0000 |
---|---|---|
committer | Sriraman Tallam <tmsriram@google.com> | 2011-07-22 22:38:42 +0000 |
commit | 804eb4807b7103135911ff0af4f7688d9458e4e0 (patch) | |
tree | ba7d562a55734c20a5cae3b32c141a6bdf251aeb /gold | |
parent | d474311490a5cf44059606c48c667a2e0f73732e (diff) | |
download | binutils-804eb4807b7103135911ff0af4f7688d9458e4e0.zip binutils-804eb4807b7103135911ff0af4f7688d9458e4e0.tar.gz binutils-804eb4807b7103135911ff0af4f7688d9458e4e0.tar.bz2 |
2011-07-22 Sriraman Tallam <tmsriram@google.com>
* symtab.cc (Symbol_table::add_from_relobj): Mark symbol as referenced
only after checking if it cannot be forced local.
* symtab.h (is_externally_visible): Check if the symbol is not forced
local.
Diffstat (limited to 'gold')
-rw-r--r-- | gold/ChangeLog | 7 | ||||
-rw-r--r-- | gold/symtab.cc | 6 | ||||
-rw-r--r-- | gold/symtab.h | 5 |
3 files changed, 13 insertions, 5 deletions
diff --git a/gold/ChangeLog b/gold/ChangeLog index 9d509c2..c894c71 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,3 +1,10 @@ +2011-07-22 Sriraman Tallam <tmsriram@google.com> + + * symtab.cc (Symbol_table::add_from_relobj): Mark symbol as referenced + only after checking if it cannot be forced local. + * symtab.h (is_externally_visible): Check if the symbol is not forced + local. + 2011-07-15 Ian Lance Taylor <iant@google.com> * options.h (class General_options): Add --print-output-format. diff --git a/gold/symtab.cc b/gold/symtab.cc index 26d8032..e289f79 100644 --- a/gold/symtab.cc +++ b/gold/symtab.cc @@ -1238,15 +1238,15 @@ Symbol_table::add_from_relobj( is_default_version, *psym, st_shndx, is_ordinary, orig_st_shndx); + if (is_forced_local) + this->force_local(res); + // If building a shared library using garbage collection, do not // treat externally visible symbols as garbage. if (parameters->options().gc_sections() && parameters->options().shared()) this->gc_mark_symbol_for_shlib(res); - if (is_forced_local) - this->force_local(res); - if (is_defined_in_discarded_section) res->set_is_defined_in_discarded_section(); diff --git a/gold/symtab.h b/gold/symtab.h index e2aa6a7..b9b9e00 100644 --- a/gold/symtab.h +++ b/gold/symtab.h @@ -538,8 +538,9 @@ class Symbol bool is_externally_visible() const { - return (this->visibility_ == elfcpp::STV_DEFAULT - || this->visibility_ == elfcpp::STV_PROTECTED); + return ((this->visibility_ == elfcpp::STV_DEFAULT + || this->visibility_ == elfcpp::STV_PROTECTED) + && !this->is_forced_local_); } // Return true if this symbol can be preempted by a definition in |