this->has_plt_offset_ = false;
this->has_warning_ = false;
this->is_copied_from_dynobj_ = false;
+ this->is_forced_local_ = false;
}
// Return the demangled version of the symbol's name, but only
if (this->needs_dynsym_entry())
return true;
+ // If the symbol was forced local in a version script, do not add it.
+ if (this->is_forced_local())
+ return false;
+
// If exporting all symbols or building a shared library,
// and the symbol is defined in a regular object and is
// externally visible, we need to add it.
Symbol_table::Symbol_table(unsigned int count,
const Version_script_info& version_script)
: saw_undefined_(0), offset_(0), table_(count), namepool_(),
- forwarders_(), commons_(), warnings_(), version_script_(version_script)
+ forwarders_(), commons_(), forced_locals_(), warnings_(),
+ version_script_(version_script)
{
namepool_.reserve(count);
}
to->set_in_dyn();
}
+// Record that a symbol is forced to be local by a version script.
+
+void
+Symbol_table::force_local(Symbol* sym)
+{
+ if (!sym->is_defined() && !sym->is_common())
+ return;
+ if (sym->is_forced_local())
+ {
+ // We already got this one.
+ return;
+ }
+ sym->set_is_forced_local();
+ this->forced_locals_.push_back(sym);
+}
+
// Add one symbol from OBJECT to the symbol table. NAME is symbol
// name and VERSION is the version; both are canonicalized. DEF is
// whether this is the default version.
// this is the default version.
const char* ver = strchr(name, '@');
int namelen = 0;
+ // DEF: is the version default? LOCAL: is the symbol forced local?
bool def = false;
+ bool local = false;
if (ver != NULL)
{
// the version script may assign a version anyway.
namelen = strlen(name);
def = true;
+ // Check the global: entries from the version script.
const std::string& version =
version_script_.get_symbol_version(name);
if (!version.empty())
ver = version.c_str();
+ // Check the local: entries from the version script
+ if (version_script_.symbol_is_local(name))
+ local = true;
}
Sized_symbol<size>* res;
name = this->namepool_.add(name, true, &name_key);
res = this->add_from_object(relobj, name, name_key, NULL, 0,
false, *psym, sym);
+ if (local)
+ this->force_local(res);
}
else
{
bool add_to_table = false;
typename Symbol_table_type::iterator add_loc = this->table_.end();
+ // If the caller didn't give us a version, see if we get one from
+ // the version script.
+ if (*pversion == NULL)
+ {
+ const std::string& v(this->version_script_.get_symbol_version(*pname));
+ if (!v.empty())
+ *pversion = v.c_str();
+ }
+
if (only_if_ref)
{
oldsym = this->lookup(*pname, *pversion);
offset_is_from_end);
if (oldsym == NULL)
- return sym;
+ {
+ if (binding == elfcpp::STB_LOCAL
+ || this->version_script_.symbol_is_local(name))
+ this->force_local(sym);
+ return sym;
+ }
if (Symbol_table::should_override_with_special(oldsym))
this->override_with_special(oldsym, sym);
offset_base);
if (oldsym == NULL)
- return sym;
+ {
+ if (binding == elfcpp::STB_LOCAL
+ || this->version_script_.symbol_is_local(name))
+ this->force_local(sym);
+ return sym;
+ }
if (Symbol_table::should_override_with_special(oldsym))
this->override_with_special(oldsym, sym);
sym->init(name, value, symsize, type, binding, visibility, nonvis);
if (oldsym == NULL)
- return sym;
+ {
+ if (binding == elfcpp::STB_LOCAL
+ || this->version_script_.symbol_is_local(name))
+ this->force_local(sym);
+ return sym;
+ }
if (Symbol_table::should_override_with_special(oldsym))
this->override_with_special(oldsym, sym);
}
// Set the final values for all the symbols. The index of the first
-// global symbol in the output file is INDEX. Record the file offset
-// OFF. Add their names to POOL. Return the new file offset.
+// global symbol in the output file is *PLOCAL_SYMCOUNT. Record the
+// file offset OFF. Add their names to POOL. Return the new file
+// offset. Update *PLOCAL_SYMCOUNT if necessary.
off_t
-Symbol_table::finalize(unsigned int index, off_t off, off_t dynoff,
- size_t dyn_global_index, size_t dyncount,
- Stringpool* pool)
+Symbol_table::finalize(off_t off, off_t dynoff, size_t dyn_global_index,
+ size_t dyncount, Stringpool* pool,
+ unsigned int *plocal_symcount)
{
off_t ret;
- gold_assert(index != 0);
- this->first_global_index_ = index;
+ gold_assert(*plocal_symcount != 0);
+ this->first_global_index_ = *plocal_symcount;
this->dynamic_offset_ = dynoff;
this->first_dynamic_global_index_ = dyn_global_index;
if (parameters->get_size() == 32)
{
#if defined(HAVE_TARGET_32_BIG) || defined(HAVE_TARGET_32_LITTLE)
- ret = this->sized_finalize<32>(index, off, pool);
+ ret = this->sized_finalize<32>(off, pool, plocal_symcount);
#else
gold_unreachable();
#endif
else if (parameters->get_size() == 64)
{
#if defined(HAVE_TARGET_64_BIG) || defined(HAVE_TARGET_64_LITTLE)
- ret = this->sized_finalize<64>(index, off, pool);
+ ret = this->sized_finalize<64>(off, pool, plocal_symcount);
#else
gold_unreachable();
#endif
return ret;
}
+// SYM is going into the symbol table at *PINDEX. Add the name to
+// POOL, update *PINDEX and *POFF.
+
+template<int size>
+void
+Symbol_table::add_to_final_symtab(Symbol* sym, Stringpool* pool,
+ unsigned int* pindex, off_t* poff)
+{
+ sym->set_symtab_index(*pindex);
+ pool->add(sym->name(), false, NULL);
+ ++*pindex;
+ *poff += elfcpp::Elf_sizes<size>::sym_size;
+}
+
// Set the final value for all the symbols. This is called after
// Layout::finalize, so all the output sections have their final
// address.
template<int size>
off_t
-Symbol_table::sized_finalize(unsigned index, off_t off, Stringpool* pool)
+Symbol_table::sized_finalize(off_t off, Stringpool* pool,
+ unsigned int* plocal_symcount)
{
off = align_address(off, size >> 3);
this->offset_ = off;
- size_t orig_index = index;
+ unsigned int index = *plocal_symcount;
+ const unsigned int orig_index = index;
- const int sym_size = elfcpp::Elf_sizes<size>::sym_size;
+ // First do all the symbols which have been forced to be local, as
+ // they must appear before all global symbols.
+ for (Forced_locals::iterator p = this->forced_locals_.begin();
+ p != this->forced_locals_.end();
+ ++p)
+ {
+ Symbol* sym = *p;
+ gold_assert(sym->is_forced_local());
+ if (this->sized_finalize_symbol<size>(sym))
+ {
+ this->add_to_final_symtab<size>(sym, pool, &index, &off);
+ ++*plocal_symcount;
+ }
+ }
+
+ // Now do all the remaining symbols.
for (Symbol_table_type::iterator p = this->table_.begin();
p != this->table_.end();
++p)
{
- Sized_symbol<size>* sym = static_cast<Sized_symbol<size>*>(p->second);
+ Symbol* sym = p->second;
+ if (this->sized_finalize_symbol<size>(sym))
+ this->add_to_final_symtab<size>(sym, pool, &index, &off);
+ }
- // FIXME: Here we need to decide which symbols should go into
- // the output file, based on --strip.
+ this->output_count_ = index - orig_index;
- // The default version of a symbol may appear twice in the
- // symbol table. We only need to finalize it once.
- if (sym->has_symtab_index())
- continue;
+ return off;
+}
- if (!sym->in_reg())
- {
- gold_assert(!sym->has_symtab_index());
- sym->set_symtab_index(-1U);
- gold_assert(sym->dynsym_index() == -1U);
- continue;
- }
+// Finalize the symbol SYM. This returns true if the symbol should be
+// added to the symbol table, false otherwise.
- typename Sized_symbol<size>::Value_type value;
+template<int size>
+bool
+Symbol_table::sized_finalize_symbol(Symbol* unsized_sym)
+{
+ Sized_symbol<size>* sym = static_cast<Sized_symbol<size>*>(unsized_sym);
- switch (sym->source())
- {
- case Symbol::FROM_OBJECT:
- {
- unsigned int shndx = sym->shndx();
+ // The default version of a symbol may appear twice in the symbol
+ // table. We only need to finalize it once.
+ if (sym->has_symtab_index())
+ return false;
- // FIXME: We need some target specific support here.
- if (shndx >= elfcpp::SHN_LORESERVE
- && shndx != elfcpp::SHN_ABS)
- {
- gold_error(_("%s: unsupported symbol section 0x%x"),
- sym->demangled_name().c_str(), shndx);
- shndx = elfcpp::SHN_UNDEF;
- }
+ if (!sym->in_reg())
+ {
+ gold_assert(!sym->has_symtab_index());
+ sym->set_symtab_index(-1U);
+ gold_assert(sym->dynsym_index() == -1U);
+ return false;
+ }
- Object* symobj = sym->object();
- if (symobj->is_dynamic())
- {
- value = 0;
- shndx = elfcpp::SHN_UNDEF;
- }
- else if (shndx == elfcpp::SHN_UNDEF)
- value = 0;
- else if (shndx == elfcpp::SHN_ABS)
- value = sym->value();
- else
- {
- Relobj* relobj = static_cast<Relobj*>(symobj);
- section_offset_type secoff;
- Output_section* os = relobj->output_section(shndx, &secoff);
+ typename Sized_symbol<size>::Value_type value;
- if (os == NULL)
- {
- sym->set_symtab_index(-1U);
- gold_assert(sym->dynsym_index() == -1U);
- continue;
- }
+ switch (sym->source())
+ {
+ case Symbol::FROM_OBJECT:
+ {
+ unsigned int shndx = sym->shndx();
- if (sym->type() == elfcpp::STT_TLS)
- value = sym->value() + os->tls_offset() + secoff;
- else
- value = sym->value() + os->address() + secoff;
- }
+ // FIXME: We need some target specific support here.
+ if (shndx >= elfcpp::SHN_LORESERVE
+ && shndx != elfcpp::SHN_ABS)
+ {
+ gold_error(_("%s: unsupported symbol section 0x%x"),
+ sym->demangled_name().c_str(), shndx);
+ shndx = elfcpp::SHN_UNDEF;
}
- break;
- case Symbol::IN_OUTPUT_DATA:
+ Object* symobj = sym->object();
+ if (symobj->is_dynamic())
{
- Output_data* od = sym->output_data();
- value = sym->value() + od->address();
- if (sym->offset_is_from_end())
- value += od->data_size();
+ value = 0;
+ shndx = elfcpp::SHN_UNDEF;
}
- break;
-
- case Symbol::IN_OUTPUT_SEGMENT:
+ else if (shndx == elfcpp::SHN_UNDEF)
+ value = 0;
+ else if (shndx == elfcpp::SHN_ABS)
+ value = sym->value();
+ else
{
- Output_segment* os = sym->output_segment();
- value = sym->value() + os->vaddr();
- switch (sym->offset_base())
+ Relobj* relobj = static_cast<Relobj*>(symobj);
+ section_offset_type secoff;
+ Output_section* os = relobj->output_section(shndx, &secoff);
+
+ if (os == NULL)
{
- case Symbol::SEGMENT_START:
- break;
- case Symbol::SEGMENT_END:
- value += os->memsz();
- break;
- case Symbol::SEGMENT_BSS:
- value += os->filesz();
- break;
- default:
- gold_unreachable();
+ sym->set_symtab_index(-1U);
+ gold_assert(sym->dynsym_index() == -1U);
+ return false;
}
+
+ if (sym->type() == elfcpp::STT_TLS)
+ value = sym->value() + os->tls_offset() + secoff;
+ else
+ value = sym->value() + os->address() + secoff;
}
- break;
+ }
+ break;
+
+ case Symbol::IN_OUTPUT_DATA:
+ {
+ Output_data* od = sym->output_data();
+ value = sym->value() + od->address();
+ if (sym->offset_is_from_end())
+ value += od->data_size();
+ }
+ break;
+
+ case Symbol::IN_OUTPUT_SEGMENT:
+ {
+ Output_segment* os = sym->output_segment();
+ value = sym->value() + os->vaddr();
+ switch (sym->offset_base())
+ {
+ case Symbol::SEGMENT_START:
+ break;
+ case Symbol::SEGMENT_END:
+ value += os->memsz();
+ break;
+ case Symbol::SEGMENT_BSS:
+ value += os->filesz();
+ break;
+ default:
+ gold_unreachable();
+ }
+ }
+ break;
- case Symbol::CONSTANT:
- value = sym->value();
- break;
+ case Symbol::CONSTANT:
+ value = sym->value();
+ break;
- default:
- gold_unreachable();
- }
+ default:
+ gold_unreachable();
+ }
- sym->set_value(value);
+ sym->set_value(value);
- if (parameters->strip_all())
- sym->set_symtab_index(-1U);
- else
- {
- sym->set_symtab_index(index);
- pool->add(sym->name(), false, NULL);
- ++index;
- off += sym_size;
- }
+ if (parameters->strip_all())
+ {
+ sym->set_symtab_index(-1U);
+ return false;
}
- this->output_count_ = index - orig_index;
-
- return off;
+ return true;
}
// Write out the global symbols.
const Target* const target = input_objects->target();
const int sym_size = elfcpp::Elf_sizes<size>::sym_size;
- unsigned int index = this->first_global_index_;
- const off_t oview_size = this->output_count_ * sym_size;
+
+ const unsigned int output_count = this->output_count_;
+ const section_size_type oview_size = output_count * sym_size;
+ const unsigned int first_global_index = this->first_global_index_;
unsigned char* const psyms = of->get_output_view(this->offset_, oview_size);
- unsigned int dynamic_count = this->dynamic_count_;
- off_t dynamic_size = dynamic_count * sym_size;
- unsigned int first_dynamic_global_index = this->first_dynamic_global_index_;
+ const unsigned int dynamic_count = this->dynamic_count_;
+ const section_size_type dynamic_size = dynamic_count * sym_size;
+ const unsigned int first_dynamic_global_index =
+ this->first_dynamic_global_index_;
unsigned char* dynamic_view;
if (this->dynamic_offset_ == 0)
dynamic_view = NULL;
else
dynamic_view = of->get_output_view(this->dynamic_offset_, dynamic_size);
- unsigned char* ps = psyms;
for (Symbol_table_type::const_iterator p = this->table_.begin();
p != this->table_.end();
++p)
continue;
}
- if (sym_index == index)
- ++index;
- else if (sym_index != -1U)
- {
- // We have already seen this symbol, because it has a
- // default version.
- gold_assert(sym_index < index);
- if (dynsym_index == -1U)
- continue;
- sym_index = -1U;
- }
-
unsigned int shndx;
typename elfcpp::Elf_types<32>::Elf_Addr value = sym->value();
switch (sym->source())
if (sym_index != -1U)
{
+ sym_index -= first_global_index;
+ gold_assert(sym_index < output_count);
+ unsigned char* ps = psyms + (sym_index * sym_size);
this->sized_write_symbol SELECT_SIZE_ENDIAN_NAME(size, big_endian) (
sym, sym->value(), shndx, sympool, ps
SELECT_SIZE_ENDIAN(size, big_endian));
- ps += sym_size;
}
if (dynsym_index != -1U)
}
}
- gold_assert(ps - psyms == oview_size);
-
of->write_output_view(this->offset_, oview_size, psyms);
if (dynamic_view != NULL)
of->write_output_view(this->dynamic_offset_, dynamic_size, dynamic_view);
osym.put_st_name(pool->get_offset(sym->name()));
osym.put_st_value(value);
osym.put_st_size(sym->symsize());
- osym.put_st_info(elfcpp::elf_st_info(sym->binding(), sym->type()));
+ // A version script may have overridden the default binding.
+ if (sym->is_forced_local())
+ osym.put_st_info(elfcpp::elf_st_info(elfcpp::STB_LOCAL, sym->type()));
+ else
+ osym.put_st_info(elfcpp::elf_st_info(sym->binding(), sym->type()));
osym.put_st_other(elfcpp::elf_st_other(sym->visibility(), sym->nonvis()));
osym.put_st_shndx(shndx);
}
return (this->visibility_ != elfcpp::STV_INTERNAL
&& this->visibility_ != elfcpp::STV_HIDDEN
&& this->visibility_ != elfcpp::STV_PROTECTED
+ && !this->is_forced_local_
&& parameters->output_is_shared()
&& !parameters->symbolic());
}
set_is_copied_from_dynobj()
{ this->is_copied_from_dynobj_ = true; }
+ // Return whether this symbol is forced to visibility STB_LOCAL
+ // by a "local:" entry in a version script.
+ bool
+ is_forced_local() const
+ { return this->is_forced_local_; }
+
+ // Mark this symbol as forced to STB_LOCAL visibility.
+ void
+ set_is_forced_local()
+ { this->is_forced_local_ = true; }
+
protected:
// Instances of this class should always be created at a specific
// size.
// True if we are using a COPY reloc for this symbol, so that the
// real definition lives in a dynamic object.
bool is_copied_from_dynobj_ : 1;
+ // True if this symbol was forced to local visibility by a version
+ // script.
+ bool is_forced_local_ : 1;
};
// The parts of a symbol which are size specific. Using a template
// Finalize the symbol table after we have set the final addresses
// of all the input sections. This sets the final symbol indexes,
- // values and adds the names to *POOL. INDEX is the index of the
- // first global symbol. OFF is the file offset of the global symbol
- // table, DYNOFF is the offset of the globals in the dynamic symbol
- // table, DYN_GLOBAL_INDEX is the index of the first global dynamic
- // symbol, and DYNCOUNT is the number of global dynamic symbols.
- // This records the parameters, and returns the new file offset.
+ // values and adds the names to *POOL. *PLOCAL_SYMCOUNT is the
+ // index of the first global symbol. OFF is the file offset of the
+ // global symbol table, DYNOFF is the offset of the globals in the
+ // dynamic symbol table, DYN_GLOBAL_INDEX is the index of the first
+ // global dynamic symbol, and DYNCOUNT is the number of global
+ // dynamic symbols. This records the parameters, and returns the
+ // new file offset. It updates *PLOCAL_SYMCOUNT if it created any
+ // local symbols.
off_t
- finalize(unsigned int index, off_t off, off_t dynoff,
- size_t dyn_global_index, size_t dyncount, Stringpool* pool);
+ finalize(off_t off, off_t dynoff, size_t dyn_global_index, size_t dyncount,
+ Stringpool* pool, unsigned int *plocal_symcount);
// Write out the global symbols.
void
resolve(Sized_symbol<size>* to, const Sized_symbol<size>* from,
const char* version ACCEPT_SIZE_ENDIAN);
+ // Record that a symbol is forced to be local by a version script.
+ void
+ force_local(Symbol*);
+
// Whether we should override a symbol, based on flags in
// resolve.cc.
static bool
// Finalize symbols specialized for size.
template<int size>
off_t
- sized_finalize(unsigned int, off_t, Stringpool*);
+ sized_finalize(off_t, Stringpool*, unsigned int*);
+
+ // Finalize a symbol. Return whether it should be added to the
+ // symbol table.
+ template<int size>
+ bool
+ sized_finalize_symbol(Symbol*);
+
+ // Add a symbol the final symtab by setting its index.
+ template<int size>
+ void
+ add_to_final_symtab(Symbol*, Stringpool*, unsigned int* pindex, off_t* poff);
// Write globals specialized for size and endianness.
template<int size, bool big_endian>
// The type of the list of common symbols.
typedef std::vector<Symbol*> Commons_type;
+ // The type of the list of symbols which have been forced local.
+ typedef std::vector<Symbol*> Forced_locals;
+
// A map from symbols with COPY relocs to the dynamic objects where
// they are defined.
typedef Unordered_map<const Symbol*, Dynobj*> Copied_symbol_dynobjs;
// write the table.
off_t offset_;
// The number of global symbols we want to write out.
- size_t output_count_;
+ unsigned int output_count_;
// The file offset of the global dynamic symbols, or 0 if none.
off_t dynamic_offset_;
// The index of the first global dynamic symbol.
unsigned int first_dynamic_global_index_;
// The number of global dynamic symbols, or 0 if none.
- off_t dynamic_count_;
+ unsigned int dynamic_count_;
// The symbol hash table.
Symbol_table_type table_;
// A pool of symbol names. This is used for all global symbols.
// symbol is no longer a common symbol. It may also have become a
// forwarder.
Commons_type commons_;
+ // A list of symbols which have been forced to be local. We don't
+ // expect there to be very many of them, so we keep a list of them
+ // rather than walking the whole table to find them.
+ Forced_locals forced_locals_;
// Manage symbol warnings.
Warnings warnings_;
// Manage potential One Definition Rule (ODR) violations.
projects / binutils-gdb.git / commitdiff