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.
This commit is contained in:
Sriraman Tallam 2011-07-22 22:38:42 +00:00
parent d474311490
commit 804eb4807b
3 changed files with 13 additions and 5 deletions

View File

@ -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.

View File

@ -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();

View File

@ -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