X-Git-Url: http://drtracing.org/?a=blobdiff_plain;f=gold%2Fsymtab.cc;h=0f43faa360ecd03522cebca98fc928f6ee3d6b4a;hb=ead1e4244a55707685d105c662a9a1faf5d122fe;hp=a410db3171ca805ba98b2bda3b1f30722f6af466;hpb=14bfc3f55540e60253cc4aae73261325309f750a;p=deliverable%2Fbinutils-gdb.git

diff --git a/gold/symtab.cc b/gold/symtab.cc
index a410db3171..0f43faa360 100644
--- a/gold/symtab.cc
+++ b/gold/symtab.cc
@@ -8,6 +8,8 @@
 #include <utility>
 
 #include "object.h"
+#include "output.h"
+#include "target.h"
 #include "symtab.h"
 
 namespace gold
@@ -15,31 +17,85 @@ namespace gold
 
 // Class Symbol.
 
-Symbol::~Symbol()
+// Initialize fields in Symbol.  This initializes everything except u_
+// and source_.
+
+void
+Symbol::init_fields(const char* name, const char* version,
+		    elfcpp::STT type, elfcpp::STB binding,
+		    elfcpp::STV visibility, unsigned char nonvis)
 {
+  this->name_ = name;
+  this->version_ = version;
+  this->got_offset_ = 0;
+  this->type_ = type;
+  this->binding_ = binding;
+  this->visibility_ = visibility;
+  this->nonvis_ = nonvis;
+  this->is_target_special_ = false;
+  this->is_def_ = false;
+  this->is_forwarder_ = false;
+  this->in_dyn_ = false;
+  this->has_got_offset_ = false;
 }
 
-// Initialize the fields in the base class Symbol.
+// Initialize the fields in the base class Symbol for SYM in OBJECT.
 
 template<int size, bool big_endian>
 void
 Symbol::init_base(const char* name, const char* version, Object* object,
 		  const elfcpp::Sym<size, big_endian>& sym)
 {
-  this->name_ = name;
-  this->version_ = version;
-  this->object_ = object;
-  this->shnum_ = sym.get_st_shndx(); // FIXME: Handle SHN_XINDEX.
-  this->type_ = sym.get_st_type();
-  this->binding_ = sym.get_st_bind();
-  this->visibility_ = sym.get_st_visibility();
-  this->other_ = sym.get_st_nonvis();
-  this->special_ = false;
-  this->def_ = false;
-  this->forwarder_ = false;
+  this->init_fields(name, version, sym.get_st_type(), sym.get_st_bind(),
+		    sym.get_st_visibility(), sym.get_st_nonvis());
+  this->u_.from_object.object = object;
+  // FIXME: Handle SHN_XINDEX.
+  this->u_.from_object.shnum = sym.get_st_shndx();
+  this->source_ = FROM_OBJECT;
+  this->in_dyn_ = object->is_dynamic();
+}
+
+// Initialize the fields in the base class Symbol for a symbol defined
+// in an Output_data.
+
+void
+Symbol::init_base(const char* name, Output_data* od, elfcpp::STT type,
+		  elfcpp::STB binding, elfcpp::STV visibility,
+		  unsigned char nonvis, bool offset_is_from_end)
+{
+  this->init_fields(name, NULL, type, binding, visibility, nonvis);
+  this->u_.in_output_data.output_data = od;
+  this->u_.in_output_data.offset_is_from_end = offset_is_from_end;
+  this->source_ = IN_OUTPUT_DATA;
+}
+
+// Initialize the fields in the base class Symbol for a symbol defined
+// in an Output_segment.
+
+void
+Symbol::init_base(const char* name, Output_segment* os, elfcpp::STT type,
+		  elfcpp::STB binding, elfcpp::STV visibility,
+		  unsigned char nonvis, Segment_offset_base offset_base)
+{
+  this->init_fields(name, NULL, type, binding, visibility, nonvis);
+  this->u_.in_output_segment.output_segment = os;
+  this->u_.in_output_segment.offset_base = offset_base;
+  this->source_ = IN_OUTPUT_SEGMENT;
+}
+
+// Initialize the fields in the base class Symbol for a symbol defined
+// as a constant.
+
+void
+Symbol::init_base(const char* name, elfcpp::STT type,
+		  elfcpp::STB binding, elfcpp::STV visibility,
+		  unsigned char nonvis)
+{
+  this->init_fields(name, NULL, type, binding, visibility, nonvis);
+  this->source_ = CONSTANT;
 }
 
-// Initialize the fields in Sized_symbol.
+// Initialize the fields in Sized_symbol for SYM in OBJECT.
 
 template<int size>
 template<bool big_endian>
@@ -49,13 +105,61 @@ Sized_symbol<size>::init(const char* name, const char* version, Object* object,
 {
   this->init_base(name, version, object, sym);
   this->value_ = sym.get_st_value();
-  this->size_ = sym.get_st_size();
+  this->symsize_ = sym.get_st_size();
+}
+
+// Initialize the fields in Sized_symbol for a symbol defined in an
+// Output_data.
+
+template<int size>
+void
+Sized_symbol<size>::init(const char* name, Output_data* od,
+			 Value_type value, Size_type symsize,
+			 elfcpp::STT type, elfcpp::STB binding,
+			 elfcpp::STV visibility, unsigned char nonvis,
+			 bool offset_is_from_end)
+{
+  this->init_base(name, od, type, binding, visibility, nonvis,
+		  offset_is_from_end);
+  this->value_ = value;
+  this->symsize_ = symsize;
+}
+
+// Initialize the fields in Sized_symbol for a symbol defined in an
+// Output_segment.
+
+template<int size>
+void
+Sized_symbol<size>::init(const char* name, Output_segment* os,
+			 Value_type value, Size_type symsize,
+			 elfcpp::STT type, elfcpp::STB binding,
+			 elfcpp::STV visibility, unsigned char nonvis,
+			 Segment_offset_base offset_base)
+{
+  this->init_base(name, os, type, binding, visibility, nonvis, offset_base);
+  this->value_ = value;
+  this->symsize_ = symsize;
+}
+
+// Initialize the fields in Sized_symbol for a symbol defined as a
+// constant.
+
+template<int size>
+void
+Sized_symbol<size>::init(const char* name, Value_type value, Size_type symsize,
+			 elfcpp::STT type, elfcpp::STB binding,
+			 elfcpp::STV visibility, unsigned char nonvis)
+{
+  this->init_base(name, type, binding, visibility, nonvis);
+  this->value_ = value;
+  this->symsize_ = symsize;
 }
 
 // Class Symbol_table.
 
 Symbol_table::Symbol_table()
-  : size_(0), table_(), namepool_(), forwarders_()
+  : size_(0), saw_undefined_(0), offset_(0), table_(), namepool_(),
+    forwarders_(), commons_()
 {
 }
 
@@ -94,6 +198,8 @@ Symbol_table::make_forwarder(Symbol* from, Symbol* to)
   from->set_forwarder();
 }
 
+// Resolve the forwards from FROM, returning the real symbol.
+
 Symbol*
 Symbol_table::resolve_forwards(Symbol* from) const
 {
@@ -104,14 +210,48 @@ Symbol_table::resolve_forwards(Symbol* from) const
   return p->second;
 }
 
+// Look up a symbol by name.
+
+Symbol*
+Symbol_table::lookup(const char* name, const char* version) const
+{
+  name = this->namepool_.find(name);
+  if (name == NULL)
+    return NULL;
+  if (version != NULL)
+    {
+      version = this->namepool_.find(version);
+      if (version == NULL)
+	return NULL;
+    }
+
+  Symbol_table_key key(name, version);
+  Symbol_table::Symbol_table_type::const_iterator p = this->table_.find(key);
+  if (p == this->table_.end())
+    return NULL;
+  return p->second;
+}
+
 // Resolve a Symbol with another Symbol.  This is only used in the
 // unusual case where there are references to both an unversioned
 // symbol and a symbol with a version, and we then discover that that
-// version is the default version.
+// version is the default version.  Because this is unusual, we do
+// this the slow way, by converting back to an ELF symbol.
 
+template<int size, bool big_endian>
 void
-Symbol_table::resolve(Symbol*, const Symbol*)
+Symbol_table::resolve(Sized_symbol<size>* to, const Sized_symbol<size>* from
+                      ACCEPT_SIZE_ENDIAN)
 {
+  unsigned char buf[elfcpp::Elf_sizes<size>::sym_size];
+  elfcpp::Sym_write<size, big_endian> esym(buf);
+  // We don't bother to set the st_name field.
+  esym.put_st_value(from->value());
+  esym.put_st_size(from->symsize());
+  esym.put_st_info(from->binding(), from->type());
+  esym.put_st_other(from->visibility(), from->nonvis());
+  esym.put_st_shndx(from->shnum());
+  Symbol_table::resolve(to, esym.sym(), from->object());
 }
 
 // Add one symbol from OBJECT to the symbol table.  NAME is symbol
@@ -162,12 +302,19 @@ Symbol_table::add_from_object(Sized_object<size, big_endian>* object,
   // ins.first->second: the value (Symbol*).
   // ins.second: true if new entry was inserted, false if not.
 
-  Symbol* ret;
+  Sized_symbol<size>* ret;
+  bool was_undefined;
+  bool was_common;
   if (!ins.second)
     {
       // We already have an entry for NAME/VERSION.
-      ret = ins.first->second;
+      ret = this->get_sized_symbol SELECT_SIZE_NAME (ins.first->second
+                                                     SELECT_SIZE(size));
       assert(ret != NULL);
+
+      was_undefined = ret->is_undefined();
+      was_common = ret->is_common();
+
       Symbol_table::resolve(ret, sym, object);
 
       if (def)
@@ -182,7 +329,12 @@ Symbol_table::add_from_object(Sized_object<size, big_endian>* object,
 	    {
 	      // This is the unfortunate case where we already have
 	      // entries for both NAME/VERSION and NAME/NULL.
-	      Symbol_table::resolve(ret, insdef.first->second);
+	      const Sized_symbol<size>* sym2;
+	      sym2 = this->get_sized_symbol SELECT_SIZE_NAME (
+		insdef.first->second
+                SELECT_SIZE(size));
+	      Symbol_table::resolve SELECT_SIZE_ENDIAN_NAME (
+		ret, sym2 SELECT_SIZE_ENDIAN(size, big_endian));
 	      this->make_forwarder(insdef.first->second, ret);
 	      insdef.first->second = ret;
 	    }
@@ -192,22 +344,28 @@ Symbol_table::add_from_object(Sized_object<size, big_endian>* object,
     {
       // This is the first time we have seen NAME/VERSION.
       assert(ins.first->second == NULL);
+
+      was_undefined = false;
+      was_common = false;
+
       if (def && !insdef.second)
 	{
 	  // We already have an entry for NAME/NULL.  Make
 	  // NAME/VERSION point to it.
-	  ret = insdef.first->second;
+	  ret = this->get_sized_symbol SELECT_SIZE_NAME (insdef.first->second
+                                                         SELECT_SIZE(size));
 	  Symbol_table::resolve(ret, sym, object);
 	  ins.first->second = ret;
 	}
       else
 	{
-	  Sized_symbol<size>* rs;
 	  Sized_target<size, big_endian>* target = object->sized_target();
-	  if (target->has_make_symbol())
+	  if (!target->has_make_symbol())
+	    ret = new Sized_symbol<size>();
+	  else
 	    {
-	      rs = target->make_symbol();
-	      if (rs == NULL)
+	      ret = target->make_symbol();
+	      if (ret == NULL)
 		{
 		  // This means that we don't want a symbol table
 		  // entry after all.
@@ -222,11 +380,9 @@ Symbol_table::add_from_object(Sized_object<size, big_endian>* object,
 		  return NULL;
 		}
 	    }
-	  else
-	    rs = new Sized_symbol<size>();
-	  rs->init(name, version, object, sym);
 
-	  ret = rs;
+	  ret->init(name, version, object, sym);
+
 	  ins.first->second = ret;
 	  if (def)
 	    {
@@ -238,6 +394,16 @@ Symbol_table::add_from_object(Sized_object<size, big_endian>* object,
 	}
     }
 
+  // Record every time we see a new undefined symbol, to speed up
+  // archive groups.
+  if (!was_undefined && ret->is_undefined())
+    ++this->saw_undefined_;
+
+  // Keep track of common symbols, to speed up common symbol
+  // allocation.
+  if (!was_common && ret->is_common())
+    this->commons_.push_back(ret);
+
   return ret;
 }
 
@@ -264,20 +430,39 @@ Symbol_table::add_from_object(
       gold_exit(false);
     }
 
+  const int sym_size = elfcpp::Elf_sizes<size>::sym_size;
+
   const unsigned char* p = reinterpret_cast<const unsigned char*>(syms);
-  for (size_t i = 0; i < count; ++i)
+  for (size_t i = 0; i < count; ++i, p += sym_size)
     {
       elfcpp::Sym<size, big_endian> sym(p);
+      elfcpp::Sym<size, big_endian>* psym = &sym;
 
-      unsigned int st_name = sym.get_st_name();
+      unsigned int st_name = psym->get_st_name();
       if (st_name >= sym_name_size)
 	{
-	  fprintf(stderr, _("%s: %s: bad symbol name offset %u at %lu\n"),
+	  fprintf(stderr,
+		  _("%s: %s: bad global symbol name offset %u at %lu\n"),
 		  program_name, object->name().c_str(), st_name,
 		  static_cast<unsigned long>(i));
 	  gold_exit(false);
 	}
 
+      // A symbol defined in a section which we are not including must
+      // be treated as an undefined symbol.
+      unsigned char symbuf[sym_size];
+      elfcpp::Sym<size, big_endian> sym2(symbuf);
+      unsigned int st_shndx = psym->get_st_shndx();
+      if (st_shndx != elfcpp::SHN_UNDEF
+	  && st_shndx < elfcpp::SHN_LORESERVE
+	  && !object->is_section_included(st_shndx))
+	{
+	  memcpy(symbuf, p, sym_size);
+	  elfcpp::Sym_write<size, big_endian> sw(symbuf);
+	  sw.put_st_shndx(elfcpp::SHN_UNDEF);
+	  psym = &sym2;
+	}
+
       const char* name = sym_names + st_name;
 
       // In an object file, an '@' in the name separates the symbol
@@ -289,7 +474,7 @@ Symbol_table::add_from_object(
       if (ver == NULL)
 	{
 	  name = this->namepool_.add(name);
-	  res = this->add_from_object(object, name, NULL, false, sym);
+	  res = this->add_from_object(object, name, NULL, false, *psym);
 	}
       else
 	{
@@ -302,13 +487,540 @@ Symbol_table::add_from_object(
 	      ++ver;
 	    }
 	  ver = this->namepool_.add(ver);
-	  res = this->add_from_object(object, name, ver, def, sym);
+	  res = this->add_from_object(object, name, ver, def, *psym);
 	}
 
       *sympointers++ = res;
+    }
+}
+
+// Create and return a specially defined symbol.  If ONLY_IF_REF is
+// true, then only create the symbol if there is a reference to it.
+
+template<int size, bool big_endian>
+Sized_symbol<size>*
+Symbol_table::define_special_symbol(Target* target, const char* name,
+				    bool only_if_ref)
+{
+  assert(this->size_ == size);
+
+  Symbol* oldsym;
+  Sized_symbol<size>* sym;
+
+  if (only_if_ref)
+    {
+      oldsym = this->lookup(name, NULL);
+      if (oldsym == NULL)
+	return NULL;
+      sym = NULL;
+
+      // Canonicalize NAME.
+      name = oldsym->name();
+    }
+  else
+    {
+      // Canonicalize NAME.
+      name = this->namepool_.add(name);
+
+      Symbol* const snull = NULL;
+      const char* const vnull = NULL;
+      std::pair<typename Symbol_table_type::iterator, bool> ins =
+	this->table_.insert(std::make_pair(std::make_pair(name, vnull),
+					   snull));
+
+      if (!ins.second)
+	{
+	  // We already have a symbol table entry for NAME.
+	  oldsym = ins.first->second;
+	  assert(oldsym != NULL);
+	  sym = NULL;
+	}
+      else
+	{
+	  // We haven't seen this symbol before.
+	  assert(ins.first->second == NULL);
+
+	  if (!target->has_make_symbol())
+	    sym = new Sized_symbol<size>();
+	  else
+	    {
+	      assert(target->get_size() == size);
+	      assert(target->is_big_endian() ? big_endian : !big_endian);
+	      typedef Sized_target<size, big_endian> My_target;
+	      My_target* sized_target = static_cast<My_target*>(target);
+	      sym = sized_target->make_symbol();
+	      if (sym == NULL)
+		return NULL;
+	    }
+
+	  ins.first->second = sym;
+	  oldsym = NULL;
+	}
+    }
+
+  if (oldsym != NULL)
+    {
+      assert(sym == NULL);
+
+      sym = this->get_sized_symbol SELECT_SIZE_NAME (oldsym
+						     SELECT_SIZE(size));
+      assert(sym->source() == Symbol::FROM_OBJECT);
+      const int old_shnum = sym->shnum();
+      if (old_shnum != elfcpp::SHN_UNDEF
+	  && old_shnum != elfcpp::SHN_COMMON
+	  && !sym->object()->is_dynamic())
+	{
+	  fprintf(stderr, "%s: linker defined: multiple definition of %s\n",
+		  program_name, name);
+	  // FIXME: Report old location.  Record that we have seen an
+	  // error.
+	  return NULL;
+	}
 
-      p += elfcpp::Elf_sizes<size>::sym_size;
+      // Our new definition is going to override the old reference.
     }
+
+  return sym;
+}
+
+// Define a symbol based on an Output_data.
+
+void
+Symbol_table::define_in_output_data(Target* target, const char* name,
+				    Output_data* od,
+				    uint64_t value, uint64_t symsize,
+				    elfcpp::STT type, elfcpp::STB binding,
+				    elfcpp::STV visibility,
+				    unsigned char nonvis,
+				    bool offset_is_from_end,
+				    bool only_if_ref)
+{
+  assert(target->get_size() == this->size_);
+  if (this->size_ == 32)
+    this->do_define_in_output_data<32>(target, name, od, value, symsize,
+				       type, binding, visibility, nonvis,
+				       offset_is_from_end, only_if_ref);
+  else if (this->size_ == 64)
+    this->do_define_in_output_data<64>(target, name, od, value, symsize,
+				       type, binding, visibility, nonvis,
+				       offset_is_from_end, only_if_ref);
+  else
+    abort();
+}
+
+// Define a symbol in an Output_data, sized version.
+
+template<int size>
+void
+Symbol_table::do_define_in_output_data(
+    Target* target,
+    const char* name,
+    Output_data* od,
+    typename elfcpp::Elf_types<size>::Elf_Addr value,
+    typename elfcpp::Elf_types<size>::Elf_WXword symsize,
+    elfcpp::STT type,
+    elfcpp::STB binding,
+    elfcpp::STV visibility,
+    unsigned char nonvis,
+    bool offset_is_from_end,
+    bool only_if_ref)
+{
+  Sized_symbol<size>* sym;
+
+  if (target->is_big_endian())
+    sym = this->define_special_symbol<size, true>(target, name, only_if_ref);
+  else
+    sym = this->define_special_symbol<size, false>(target, name, only_if_ref);
+
+  if (sym == NULL)
+    return;
+
+  sym->init(name, od, value, symsize, type, binding, visibility, nonvis,
+	    offset_is_from_end);
+}
+
+// Define a symbol based on an Output_segment.
+
+void
+Symbol_table::define_in_output_segment(Target* target, const char* name,
+				       Output_segment* os,
+				       uint64_t value, uint64_t symsize,
+				       elfcpp::STT type, elfcpp::STB binding,
+				       elfcpp::STV visibility,
+				       unsigned char nonvis,
+				       Symbol::Segment_offset_base offset_base,
+				       bool only_if_ref)
+{
+  assert(target->get_size() == this->size_);
+  if (this->size_ == 32)
+    this->do_define_in_output_segment<32>(target, name, os, value, symsize,
+					  type, binding, visibility, nonvis,
+					  offset_base, only_if_ref);
+  else if (this->size_ == 64)
+    this->do_define_in_output_segment<64>(target, name, os, value, symsize,
+					  type, binding, visibility, nonvis,
+					  offset_base, only_if_ref);
+  else
+    abort();
+}
+
+// Define a symbol in an Output_segment, sized version.
+
+template<int size>
+void
+Symbol_table::do_define_in_output_segment(
+    Target* target,
+    const char* name,
+    Output_segment* os,
+    typename elfcpp::Elf_types<size>::Elf_Addr value,
+    typename elfcpp::Elf_types<size>::Elf_WXword symsize,
+    elfcpp::STT type,
+    elfcpp::STB binding,
+    elfcpp::STV visibility,
+    unsigned char nonvis,
+    Symbol::Segment_offset_base offset_base,
+    bool only_if_ref)
+{
+  Sized_symbol<size>* sym;
+
+  if (target->is_big_endian())
+    sym = this->define_special_symbol<size, true>(target, name, only_if_ref);
+  else
+    sym = this->define_special_symbol<size, false>(target, name, only_if_ref);
+
+  if (sym == NULL)
+    return;
+
+  sym->init(name, os, value, symsize, type, binding, visibility, nonvis,
+	    offset_base);
+}
+
+// Define a special symbol with a constant value.  It is a multiple
+// definition error if this symbol is already defined.
+
+void
+Symbol_table::define_as_constant(Target* target, const char* name,
+				 uint64_t value, uint64_t symsize,
+				 elfcpp::STT type, elfcpp::STB binding,
+				 elfcpp::STV visibility, unsigned char nonvis,
+				 bool only_if_ref)
+{
+  assert(target->get_size() == this->size_);
+  if (this->size_ == 32)
+    this->do_define_as_constant<32>(target, name, value, symsize,
+				    type, binding, visibility, nonvis,
+				    only_if_ref);
+  else if (this->size_ == 64)
+    this->do_define_as_constant<64>(target, name, value, symsize,
+				    type, binding, visibility, nonvis,
+				    only_if_ref);
+  else
+    abort();
+}
+
+// Define a symbol as a constant, sized version.
+
+template<int size>
+void
+Symbol_table::do_define_as_constant(
+    Target* target,
+    const char* name,
+    typename elfcpp::Elf_types<size>::Elf_Addr value,
+    typename elfcpp::Elf_types<size>::Elf_WXword symsize,
+    elfcpp::STT type,
+    elfcpp::STB binding,
+    elfcpp::STV visibility,
+    unsigned char nonvis,
+    bool only_if_ref)
+{
+  Sized_symbol<size>* sym;
+
+  if (target->is_big_endian())
+    sym = this->define_special_symbol<size, true>(target, name, only_if_ref);
+  else
+    sym = this->define_special_symbol<size, false>(target, name, only_if_ref);
+
+  if (sym == NULL)
+    return;
+
+  sym->init(name, value, symsize, type, binding, visibility, nonvis);
+}
+
+// Define a set of symbols in output sections.
+
+void
+Symbol_table::define_symbols(const Layout* layout, Target* target, int count,
+			     const Define_symbol_in_section* p)
+{
+  for (int i = 0; i < count; ++i, ++p)
+    {
+      Output_section* os = layout->find_output_section(p->output_section);
+      if (os != NULL)
+	this->define_in_output_data(target, p->name, os, p->value, p->size,
+				    p->type, p->binding, p->visibility,
+				    p->nonvis, p->offset_is_from_end,
+				    p->only_if_ref);
+      else
+	this->define_as_constant(target, p->name, 0, p->size, p->type,
+				 p->binding, p->visibility, p->nonvis,
+				 p->only_if_ref);
+    }
+}
+
+// Define a set of symbols in output segments.
+
+void
+Symbol_table::define_symbols(const Layout* layout, Target* target, int count,
+			     const Define_symbol_in_segment* p)
+{
+  for (int i = 0; i < count; ++i, ++p)
+    {
+      Output_segment* os = layout->find_output_segment(p->segment_type,
+						       p->segment_flags_set,
+						       p->segment_flags_clear);
+      if (os != NULL)
+	this->define_in_output_segment(target, p->name, os, p->value, p->size,
+				       p->type, p->binding, p->visibility,
+				       p->nonvis, p->offset_base,
+				       p->only_if_ref);
+      else
+	this->define_as_constant(target, p->name, 0, p->size, p->type,
+				 p->binding, p->visibility, p->nonvis,
+				 p->only_if_ref);
+    }
+}
+
+// Set the final values for all the symbols.  Record the file offset
+// OFF.  Add their names to POOL.  Return the new file offset.
+
+off_t
+Symbol_table::finalize(off_t off, Stringpool* pool)
+{
+  if (this->size_ == 32)
+    return this->sized_finalize<32>(off, pool);
+  else if (this->size_ == 64)
+    return this->sized_finalize<64>(off, pool);
+  else
+    abort();
+}
+
+// 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(off_t off, Stringpool* pool)
+{
+  off = align_address(off, size >> 3);
+  this->offset_ = off;
+
+  const int sym_size = elfcpp::Elf_sizes<size>::sym_size;
+  Symbol_table_type::iterator p = this->table_.begin();
+  size_t count = 0;
+  while (p != this->table_.end())
+    {
+      Sized_symbol<size>* sym = static_cast<Sized_symbol<size>*>(p->second);
+
+      // FIXME: Here we need to decide which symbols should go into
+      // the output file.
+
+      typename Sized_symbol<size>::Value_type value;
+
+      switch (sym->source())
+	{
+	case Symbol::FROM_OBJECT:
+	  {
+	    unsigned int shnum = sym->shnum();
+
+	    // FIXME: We need some target specific support here.
+	    if (shnum >= elfcpp::SHN_LORESERVE
+		&& shnum != elfcpp::SHN_ABS)
+	      {
+		fprintf(stderr, _("%s: %s: unsupported symbol section 0x%x\n"),
+			program_name, sym->name(), shnum);
+		gold_exit(false);
+	      }
+
+	    if (shnum == elfcpp::SHN_UNDEF)
+	      value = 0;
+	    else if (shnum == elfcpp::SHN_ABS)
+	      value = sym->value();
+	    else
+	      {
+		off_t secoff;
+		Output_section* os = sym->object()->output_section(shnum,
+								   &secoff);
+
+		if (os == NULL)
+		  {
+		    // We should be able to erase this symbol from the
+		    // symbol table, but at least with gcc 4.0.2
+		    // std::unordered_map::erase doesn't appear to return
+		    // the new iterator.
+		    // p = this->table_.erase(p);
+		    ++p;
+		    continue;
+		  }
+
+		value = sym->value() + os->address() + secoff;
+	      }
+	  }
+	  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:
+		abort();
+	      }
+	  }
+	  break;
+
+	case Symbol::CONSTANT:
+	  value = sym->value();
+	  break;
+
+	default:
+	  abort();
+	}
+
+      sym->set_value(value);
+      pool->add(sym->name());
+      ++count;
+      off += sym_size;
+      ++p;
+    }
+
+  this->output_count_ = count;
+
+  return off;
+}
+
+// Write out the global symbols.
+
+void
+Symbol_table::write_globals(const Target* target, const Stringpool* sympool,
+			    Output_file* of) const
+{
+  if (this->size_ == 32)
+    {
+      if (target->is_big_endian())
+	this->sized_write_globals<32, true>(target, sympool, of);
+      else
+	this->sized_write_globals<32, false>(target, sympool, of);
+    }
+  else if (this->size_ == 64)
+    {
+      if (target->is_big_endian())
+	this->sized_write_globals<64, true>(target, sympool, of);
+      else
+	this->sized_write_globals<64, false>(target, sympool, of);
+    }
+  else
+    abort();
+}
+
+// Write out the global symbols.
+
+template<int size, bool big_endian>
+void
+Symbol_table::sized_write_globals(const Target*,
+				  const Stringpool* sympool,
+				  Output_file* of) const
+{
+  const int sym_size = elfcpp::Elf_sizes<size>::sym_size;
+  unsigned char* psyms = of->get_output_view(this->offset_,
+					     this->output_count_ * sym_size);
+  unsigned char* ps = psyms;
+  for (Symbol_table_type::const_iterator p = this->table_.begin();
+       p != this->table_.end();
+       ++p)
+    {
+      Sized_symbol<size>* sym = static_cast<Sized_symbol<size>*>(p->second);
+
+      // FIXME: This repeats sized_finalize().
+
+      unsigned int shndx;
+      switch (sym->source())
+	{
+	case Symbol::FROM_OBJECT:
+	  {
+	    unsigned int shnum = sym->shnum();
+
+	    // FIXME: We need some target specific support here.
+	    if (shnum >= elfcpp::SHN_LORESERVE
+		&& shnum != elfcpp::SHN_ABS)
+	      {
+		fprintf(stderr, _("%s: %s: unsupported symbol section 0x%x\n"),
+			program_name, sym->name(), sym->shnum());
+		gold_exit(false);
+	      }
+
+	    if (shnum == elfcpp::SHN_UNDEF || shnum == elfcpp::SHN_ABS)
+	      shndx = shnum;
+	    else
+	      {
+		off_t secoff;
+		Output_section* os = sym->object()->output_section(shnum,
+								   &secoff);
+		if (os == NULL)
+		  continue;
+
+		shndx = os->out_shndx();
+	      }
+	  }
+	  break;
+
+	case Symbol::IN_OUTPUT_DATA:
+	  shndx = sym->output_data()->out_shndx();
+	  break;
+
+	case Symbol::IN_OUTPUT_SEGMENT:
+	  shndx = elfcpp::SHN_ABS;
+	  break;
+
+	case Symbol::CONSTANT:
+	  shndx = elfcpp::SHN_ABS;
+	  break;
+
+	default:
+	  abort();
+	}
+
+      elfcpp::Sym_write<size, big_endian> osym(ps);
+      osym.put_st_name(sympool->get_offset(sym->name()));
+      osym.put_st_value(sym->value());
+      osym.put_st_size(sym->symsize());
+      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);
+
+      ps += sym_size;
+    }
+
+  of->write_output_view(this->offset_, this->output_count_ * sym_size, psyms);
 }
 
 // Instantiate the templates we need.  We could use the configure