Automatic date update in version.in
[deliverable/binutils-gdb.git] / bfd / xcofflink.c
index 6ad148b5836bbbfd1ffafbdff61ca385c4f19269..e18efd4381a8d15e03a0d9b7f6de9b6434c0f6f2 100644 (file)
@@ -1,6 +1,5 @@
 /* POWER/PowerPC XCOFF linker support.
-   Copyright 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004,
-   2005, 2006, 2007, 2008, 2009 Free Software Foundation, Inc.
+   Copyright (C) 1995-2021 Free Software Foundation, Inc.
    Written by Ian Lance Taylor <ian@cygnus.com>, Cygnus Support.
 
    This file is part of BFD, the Binary File Descriptor library.
@@ -29,6 +28,7 @@
 #include "libcoff.h"
 #include "libxcoff.h"
 #include "libiberty.h"
+#include "xcofflink.h"
 
 /* This file holds the XCOFF linker code.  */
 
@@ -134,13 +134,13 @@ struct xcoff_link_hash_table
   unsigned long file_align;
 
   /* Whether the .text section must be read-only.  */
-  bfd_boolean textro;
+  bool textro;
 
   /* Whether -brtl was specified.  */
-  bfd_boolean rtld;
+  bool rtld;
 
   /* Whether garbage collection was done.  */
-  bfd_boolean gc;
+  bool gc;
 
   /* A linked list of symbols for which we have size information.  */
   struct xcoff_link_size_list
@@ -148,7 +148,7 @@ struct xcoff_link_hash_table
     struct xcoff_link_size_list *next;
     struct xcoff_link_hash_entry *h;
     bfd_size_type size;
-  } 
+  }
   *size_list;
 
   /* Information about archives.  */
@@ -199,7 +199,7 @@ struct xcoff_final_link_info
   bfd_byte *external_relocs;
 };
 
-static bfd_boolean xcoff_mark (struct bfd_link_info *, asection *);
+static bool xcoff_mark (struct bfd_link_info *, asection *);
 
 \f
 
@@ -209,16 +209,16 @@ static bfd_boolean xcoff_mark (struct bfd_link_info *, asection *);
 
 /* Read the contents of a section.  */
 
-static bfd_boolean
+static bool
 xcoff_get_section_contents (bfd *abfd, asection *sec)
 {
   if (coff_section_data (abfd, sec) == NULL)
     {
-      bfd_size_type amt = sizeof (struct coff_section_tdata);
+      size_t amt = sizeof (struct coff_section_tdata);
 
       sec->used_by_bfd = bfd_zalloc (abfd, amt);
       if (sec->used_by_bfd == NULL)
-       return FALSE;
+       return false;
     }
 
   if (coff_section_data (abfd, sec)->contents == NULL)
@@ -227,14 +227,13 @@ xcoff_get_section_contents (bfd *abfd, asection *sec)
 
       if (! bfd_malloc_and_get_section (abfd, sec, &contents))
        {
-         if (contents != NULL)
-           free (contents);
-         return FALSE;
+         free (contents);
+         return false;
        }
       coff_section_data (abfd, sec)->contents = contents;
     }
 
-  return TRUE;
+  return true;
 }
 
 /* Get the size required to hold the dynamic symbols.  */
@@ -297,7 +296,7 @@ _bfd_xcoff_canonicalize_dynamic_symtab (bfd *abfd, asymbol **psyms)
     return -1;
   contents = coff_section_data (abfd, lsec)->contents;
 
-  coff_section_data (abfd, lsec)->keep_contents = TRUE;
+  coff_section_data (abfd, lsec)->keep_contents = true;
 
   bfd_xcoff_swap_ldhdr_in (abfd, contents, &ldhdr);
 
@@ -526,7 +525,7 @@ xcoff_get_archive_info (struct bfd_link_info *info, bfd *archive)
   entryp = *slot;
   if (!entryp)
     {
-      entryp = bfd_zalloc (archive, sizeof (entry));
+      entryp = bfd_zalloc (info->output_bfd, sizeof (entry));
       if (!entryp)
        return NULL;
 
@@ -572,15 +571,31 @@ xcoff_link_hash_newfunc (struct bfd_hash_entry *entry,
   return (struct bfd_hash_entry *) ret;
 }
 
-/* Create a XCOFF link hash table.  */
+/* Destroy an XCOFF link hash table.  */
+
+static void
+_bfd_xcoff_bfd_link_hash_table_free (bfd *obfd)
+{
+  struct xcoff_link_hash_table *ret;
+
+  ret = (struct xcoff_link_hash_table *) obfd->link.hash;
+  if (ret->archive_info)
+    htab_delete (ret->archive_info);
+  if (ret->debug_strtab)
+    _bfd_stringtab_free (ret->debug_strtab);
+  _bfd_generic_link_hash_table_free (obfd);
+}
+
+/* Create an XCOFF link hash table.  */
 
 struct bfd_link_hash_table *
 _bfd_xcoff_bfd_link_hash_table_create (bfd *abfd)
 {
   struct xcoff_link_hash_table *ret;
-  bfd_size_type amt = sizeof (* ret);
+  bool isxcoff64 = false;
+  size_t amt = sizeof (* ret);
 
-  ret = bfd_malloc (amt);
+  ret = bfd_zmalloc (amt);
   if (ret == NULL)
     return NULL;
   if (!_bfd_link_hash_table_init (&ret->root, abfd, xcoff_link_hash_newfunc,
@@ -590,41 +605,25 @@ _bfd_xcoff_bfd_link_hash_table_create (bfd *abfd)
       return NULL;
     }
 
-  ret->debug_strtab = _bfd_xcoff_stringtab_init ();
-  ret->debug_section = NULL;
-  ret->loader_section = NULL;
-  ret->ldrel_count = 0;
-  memset (&ret->ldhdr, 0, sizeof (struct internal_ldhdr));
-  ret->linkage_section = NULL;
-  ret->toc_section = NULL;
-  ret->descriptor_section = NULL;
-  ret->imports = NULL;
-  ret->file_align = 0;
-  ret->textro = FALSE;
-  ret->gc = FALSE;
+  isxcoff64 = bfd_coff_debug_string_prefix_length (abfd) == 4;
+
+  ret->debug_strtab = _bfd_xcoff_stringtab_init (isxcoff64);
   ret->archive_info = htab_create (37, xcoff_archive_info_hash,
                                   xcoff_archive_info_eq, NULL);
-  memset (ret->special_sections, 0, sizeof ret->special_sections);
+  if (!ret->debug_strtab || !ret->archive_info)
+    {
+      _bfd_xcoff_bfd_link_hash_table_free (abfd);
+      return NULL;
+    }
+  ret->root.hash_table_free = _bfd_xcoff_bfd_link_hash_table_free;
 
   /* The linker will always generate a full a.out header.  We need to
      record that fact now, before the sizeof_headers routine could be
      called.  */
-  xcoff_data (abfd)->full_aouthdr = TRUE;
+  xcoff_data (abfd)->full_aouthdr = true;
 
   return &ret->root;
 }
-
-/* Free a XCOFF link hash table.  */
-
-void
-_bfd_xcoff_bfd_link_hash_table_free (struct bfd_link_hash_table *hash)
-{
-  struct xcoff_link_hash_table *ret = (struct xcoff_link_hash_table *) hash;
-
-  _bfd_stringtab_free (ret->debug_strtab);
-  bfd_hash_table_free (&ret->root.table);
-  free (ret);
-}
 \f
 /* Read internal relocs for an XCOFF csect.  This is a wrapper around
    _bfd_coff_read_internal_relocs which tries to take advantage of any
@@ -633,9 +632,9 @@ _bfd_xcoff_bfd_link_hash_table_free (struct bfd_link_hash_table *hash)
 static struct internal_reloc *
 xcoff_read_internal_relocs (bfd *abfd,
                            asection *sec,
-                           bfd_boolean cache,
+                           bool cache,
                            bfd_byte *external_relocs,
-                           bfd_boolean require_internal,
+                           bool require_internal,
                            struct internal_reloc *internal_relocs)
 {
   if (coff_section_data (abfd, sec) != NULL
@@ -652,8 +651,8 @@ xcoff_read_internal_relocs (bfd *abfd,
          && cache
          && enclosing->reloc_count > 0)
        {
-         if (_bfd_coff_read_internal_relocs (abfd, enclosing, TRUE,
-                                             external_relocs, FALSE, NULL)
+         if (_bfd_coff_read_internal_relocs (abfd, enclosing, true,
+                                             external_relocs, false, NULL)
              == NULL)
            return NULL;
        }
@@ -683,7 +682,7 @@ xcoff_read_internal_relocs (bfd *abfd,
 /* Split FILENAME into an import path and an import filename,
    storing them in *IMPPATH and *IMPFILE respectively.  */
 
-bfd_boolean
+bool
 bfd_xcoff_split_import_path (bfd *abfd, const char *filename,
                             const char **imppath, const char **impfile)
 {
@@ -706,19 +705,19 @@ bfd_xcoff_split_import_path (bfd *abfd, const char *filename,
         of the string; the native linker doesn't do that either.  */
       path = bfd_alloc (abfd, length);
       if (path == NULL)
-       return FALSE;
+       return false;
       memcpy (path, filename, length - 1);
       path[length - 1] = 0;
       *imppath = path;
     }
   *impfile = base;
-  return TRUE;
+  return true;
 }
 
 /* Set ARCHIVE's import path as though its filename had been given
    as FILENAME.  */
 
-bfd_boolean
+bool
 bfd_xcoff_set_archive_import_path (struct bfd_link_info *info,
                                   bfd *archive, const char *filename)
 {
@@ -735,7 +734,7 @@ bfd_xcoff_set_archive_import_path (struct bfd_link_info *info,
    to IMPATH, IMPFILE and IMPMEMBER respectively.  All three are null if
    no specific import module is specified.  */
 
-static bfd_boolean
+static bool
 xcoff_set_import_path (struct bfd_link_info *info,
                       struct xcoff_link_hash_entry *h,
                       const char *imppath, const char *impfile,
@@ -758,20 +757,20 @@ xcoff_set_import_path (struct bfd_link_info *info,
           *pp != NULL;
           pp = &(*pp)->next, ++c)
        {
-         if (strcmp ((*pp)->path, imppath) == 0
-             && strcmp ((*pp)->file, impfile) == 0
-             && strcmp ((*pp)->member, impmember) == 0)
+         if (filename_cmp ((*pp)->path, imppath) == 0
+             && filename_cmp ((*pp)->file, impfile) == 0
+             && filename_cmp ((*pp)->member, impmember) == 0)
            break;
        }
 
       if (*pp == NULL)
        {
          struct xcoff_import_file *n;
-         bfd_size_type amt = sizeof (* n);
+         size_t amt = sizeof (*n);
 
          n = bfd_alloc (info->output_bfd, amt);
          if (n == NULL)
-           return FALSE;
+           return false;
          n->next = NULL;
          n->path = imppath;
          n->file = impfile;
@@ -780,20 +779,20 @@ xcoff_set_import_path (struct bfd_link_info *info,
        }
       h->ldindx = c;
     }
-  return TRUE;
+  return true;
 }
 \f
 /* H is the bfd symbol associated with exported .loader symbol LDSYM.
    Return true if LDSYM defines H.  */
 
-static bfd_boolean
+static bool
 xcoff_dynamic_definition_p (struct xcoff_link_hash_entry *h,
                            struct internal_ldsym *ldsym)
 {
   /* If we didn't know about H before processing LDSYM, LDSYM
      definitely defines H.  */
   if (h->root.type == bfd_link_hash_new)
-    return TRUE;
+    return true;
 
   /* If H is currently a weak dynamic symbol, and if LDSYM is a strong
      dynamic symbol, LDSYM trumps the current definition of H.  */
@@ -802,21 +801,21 @@ xcoff_dynamic_definition_p (struct xcoff_link_hash_entry *h,
       && (h->flags & XCOFF_DEF_REGULAR) == 0
       && (h->root.type == bfd_link_hash_defweak
          || h->root.type == bfd_link_hash_undefweak))
-    return TRUE;
+    return true;
 
   /* If H is currently undefined, LDSYM defines it.  */
   if ((h->flags & XCOFF_DEF_DYNAMIC) == 0
       && (h->root.type == bfd_link_hash_undefined
          || h->root.type == bfd_link_hash_undefweak))
-    return TRUE;
+    return true;
 
-  return FALSE;
+  return false;
 }
 
 /* This function is used to add symbols from a dynamic object to the
    global symbol table.  */
 
-static bfd_boolean
+static bool
 xcoff_link_add_dynamic_symbols (bfd *abfd, struct bfd_link_info *info)
 {
   asection *lsec;
@@ -832,11 +831,11 @@ xcoff_link_add_dynamic_symbols (bfd *abfd, struct bfd_link_info *info)
      output file.  */
    if (info->output_bfd->xvec != abfd->xvec)
     {
-      (*_bfd_error_handler)
-       (_("%s: XCOFF shared object when not producing XCOFF output"),
-        bfd_get_filename (abfd));
+      _bfd_error_handler
+       (_("%pB: XCOFF shared object when not producing XCOFF output"),
+        abfd);
       bfd_set_error (bfd_error_invalid_operation);
-      return FALSE;
+      return false;
     }
 
   /* The symbols we use from a dynamic object are not the symbols in
@@ -853,15 +852,15 @@ xcoff_link_add_dynamic_symbols (bfd *abfd, struct bfd_link_info *info)
   lsec = bfd_get_section_by_name (abfd, ".loader");
   if (lsec == NULL)
     {
-      (*_bfd_error_handler)
-       (_("%s: dynamic object with no .loader section"),
-        bfd_get_filename (abfd));
+      _bfd_error_handler
+       (_("%pB: dynamic object with no .loader section"),
+        abfd);
       bfd_set_error (bfd_error_no_symbols);
-      return FALSE;
+      return false;
     }
 
   if (! xcoff_get_section_contents (abfd, lsec))
-    return FALSE;
+    return false;
   contents = coff_section_data (abfd, lsec)->contents;
 
   /* Remove the sections from this object, so that they do not get
@@ -903,10 +902,10 @@ xcoff_link_add_dynamic_symbols (bfd *abfd, struct bfd_link_info *info)
         table.  However, we verified above that we are using an XCOFF
         hash table.  */
 
-      h = xcoff_link_hash_lookup (xcoff_hash_table (info), name, TRUE,
-                                 TRUE, TRUE);
+      h = xcoff_link_hash_lookup (xcoff_hash_table (info), name, true,
+                                 true, true);
       if (h == NULL)
-       return FALSE;
+       return false;
 
       if (!xcoff_dynamic_definition_p (h, &ldsym))
        continue;
@@ -952,14 +951,14 @@ xcoff_link_add_dynamic_symbols (bfd *abfd, struct bfd_link_info *info)
 
              dsnm = bfd_malloc ((bfd_size_type) strlen (name) + 2);
              if (dsnm == NULL)
-               return FALSE;
+               return false;
              dsnm[0] = '.';
              strcpy (dsnm + 1, name);
              hds = xcoff_link_hash_lookup (xcoff_hash_table (info), dsnm,
-                                           TRUE, TRUE, TRUE);
+                                           true, true, true);
              free (dsnm);
              if (hds == NULL)
-               return FALSE;
+               return false;
 
              hds->descriptor = h;
              h->descriptor = hds;
@@ -998,14 +997,14 @@ xcoff_link_add_dynamic_symbols (bfd *abfd, struct bfd_link_info *info)
   /* Record this file in the import files.  */
   n = bfd_alloc (abfd, (bfd_size_type) sizeof (struct xcoff_import_file));
   if (n == NULL)
-    return FALSE;
+    return false;
   n->next = NULL;
 
-  if (abfd->my_archive == NULL)
+  if (abfd->my_archive == NULL || bfd_is_thin_archive (abfd->my_archive))
     {
-      if (!bfd_xcoff_split_import_path (abfd, abfd->filename,
+      if (!bfd_xcoff_split_import_path (abfd, bfd_get_filename (abfd),
                                        &n->path, &n->file))
-       return FALSE;
+       return false;
       n->member = "";
     }
   else
@@ -1016,10 +1015,11 @@ xcoff_link_add_dynamic_symbols (bfd *abfd, struct bfd_link_info *info)
       if (!archive_info->impfile)
        {
          if (!bfd_xcoff_split_import_path (archive_info->archive,
-                                           archive_info->archive->filename,
+                                           bfd_get_filename (archive_info
+                                                             ->archive),
                                            &archive_info->imppath,
                                            &archive_info->impfile))
-           return FALSE;
+           return false;
        }
       n->path = archive_info->imppath;
       n->file = archive_info->impfile;
@@ -1036,17 +1036,17 @@ xcoff_link_add_dynamic_symbols (bfd *abfd, struct bfd_link_info *info)
 
   xcoff_data (abfd)->import_file_id = c;
 
-  return TRUE;
+  return true;
 }
 
 /* xcoff_link_create_extra_sections
 
    Takes care of creating the .loader, .gl, .ds, .debug and sections.  */
 
-static bfd_boolean
+static bool
 xcoff_link_create_extra_sections (bfd * abfd, struct bfd_link_info *info)
 {
-  bfd_boolean return_value = FALSE;
+  bool return_value = false;
 
   if (info->output_bfd->xvec == abfd->xvec)
     {
@@ -1054,7 +1054,7 @@ xcoff_link_create_extra_sections (bfd * abfd, struct bfd_link_info *info)
         won't work if we're producing an XCOFF output file with no
         XCOFF input files.  FIXME.  */
 
-      if (!info->relocatable
+      if (!bfd_link_relocatable (info)
          && xcoff_hash_table (info)->loader_section == NULL)
        {
          asection *lsec;
@@ -1127,7 +1127,7 @@ xcoff_link_create_extra_sections (bfd * abfd, struct bfd_link_info *info)
        }
     }
 
-  return_value = TRUE;
+  return_value = true;
 
  end_return:
 
@@ -1201,12 +1201,12 @@ xcoff_find_reloc (struct internal_reloc *relocs,
    relocation entries carefully, since the only way to tell which
    csect they belong to is to examine the address.  */
 
-static bfd_boolean
+static bool
 xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
 {
   unsigned int n_tmask;
   unsigned int n_btshft;
-  bfd_boolean default_copy;
+  bool default_copy;
   bfd_size_type symcount;
   struct xcoff_link_hash_entry **sym_hash;
   asection **csect_cache;
@@ -1214,7 +1214,7 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
   bfd_size_type linesz;
   asection *o;
   asection *last_real;
-  bfd_boolean keep_syms;
+  bool keep_syms;
   asection *csect;
   unsigned int csect_index;
   asection *first_csect;
@@ -1235,7 +1235,7 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
       && ! info->static_link)
     {
       if (! xcoff_link_add_dynamic_symbols (abfd, info))
-       return FALSE;
+       return false;
     }
 
   /* Create the loader, toc, gl, ds and debug sections, if needed.  */
@@ -1244,7 +1244,7 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
 
   if ((abfd->flags & DYNAMIC) != 0
       && ! info->static_link)
-    return TRUE;
+    return true;
 
   n_tmask = coff_data (abfd)->local_n_tmask;
   n_btshft = coff_data (abfd)->local_n_btshft;
@@ -1254,9 +1254,9 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
 #define N_BTSHFT n_btshft
 
   if (info->keep_memory)
-    default_copy = FALSE;
+    default_copy = false;
   else
-    default_copy = TRUE;
+    default_copy = true;
 
   symcount = obj_raw_syment_count (abfd);
 
@@ -1307,7 +1307,7 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
       if ((o->flags & SEC_RELOC) != 0)
        {
          reloc_info[o->target_index].relocs =
-           xcoff_read_internal_relocs (abfd, o, TRUE, NULL, FALSE, NULL);
+           xcoff_read_internal_relocs (abfd, o, true, NULL, false, NULL);
          amt = o->reloc_count;
          amt *= sizeof (asection *);
          reloc_info[o->target_index].csects = bfd_zmalloc (amt);
@@ -1320,19 +1320,22 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
        {
          bfd_byte *linenos;
 
-         amt = linesz * o->lineno_count;
-         linenos = bfd_malloc (amt);
+         if (bfd_seek (abfd, o->line_filepos, SEEK_SET) != 0)
+           goto error_return;
+         if (_bfd_mul_overflow (linesz, o->lineno_count, &amt))
+           {
+             bfd_set_error (bfd_error_file_too_big);
+             goto error_return;
+           }
+         linenos = _bfd_malloc_and_read (abfd, amt, amt);
          if (linenos == NULL)
            goto error_return;
          reloc_info[o->target_index].linenos = linenos;
-         if (bfd_seek (abfd, o->line_filepos, SEEK_SET) != 0
-             || bfd_bread (linenos, amt, abfd) != amt)
-           goto error_return;
        }
     }
 
   /* Don't let the linker relocation routines discard the symbols.  */
-  obj_coff_keep_syms (abfd) = TRUE;
+  obj_coff_keep_syms (abfd) = true;
 
   csect = NULL;
   csect_index = 0;
@@ -1365,11 +1368,12 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
             If C_FILE or first time, handle special
 
             Advance esym, sym_hash, csect_hash ptrs.  */
-         if (sym.n_sclass == C_FILE)
+         if (sym.n_sclass == C_FILE || sym.n_sclass == C_DWARF)
            csect = NULL;
          if (csect != NULL)
            *csect_cache = csect;
-         else if (first_csect == NULL || sym.n_sclass == C_FILE)
+         else if (first_csect == NULL
+                  || sym.n_sclass == C_FILE || sym.n_sclass == C_DWARF)
            *csect_cache = coff_section_from_bfd_index (abfd, sym.n_scnum);
          else
            *csect_cache = NULL;
@@ -1411,8 +1415,9 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
              enclosing = xcoff_section_data (abfd, csect)->enclosing;
              if (enclosing == NULL)
                {
-                 (*_bfd_error_handler)
-                   (_("%B: `%s' has line numbers but no enclosing section"),
+                 _bfd_error_handler
+                   /* xgettext:c-format */
+                   (_("%pB: `%s' has line numbers but no enclosing section"),
                     abfd, name);
                  bfd_set_error (bfd_error_bad_value);
                  goto error_return;
@@ -1463,8 +1468,9 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
       /* Pick up the csect auxiliary information.  */
       if (sym.n_numaux == 0)
        {
-         (*_bfd_error_handler)
-           (_("%B: class %d symbol `%s' has no aux entries"),
+         _bfd_error_handler
+           /* xgettext:c-format */
+           (_("%pB: class %d symbol `%s' has no aux entries"),
             abfd, sym.n_sclass, name);
          bfd_set_error (bfd_error_bad_value);
          goto error_return;
@@ -1485,8 +1491,9 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
       switch (smtyp)
        {
        default:
-         (*_bfd_error_handler)
-           (_("%B: symbol `%s' has unrecognized csect type %d"),
+         _bfd_error_handler
+           /* xgettext:c-format */
+           (_("%pB: symbol `%s' has unrecognized csect type %d"),
             abfd, name, smtyp);
          bfd_set_error (bfd_error_bad_value);
          goto error_return;
@@ -1497,10 +1504,12 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
              || sym.n_scnum != N_UNDEF
              || aux.x_csect.x_scnlen.l != 0)
            {
-             (*_bfd_error_handler)
-               (_("%B: bad XTY_ER symbol `%s': class %d scnum %d scnlen %d"),
+             _bfd_error_handler
+               /* xgettext:c-format */
+               (_("%pB: bad XTY_ER symbol `%s': class %d scnum %d "
+                  "scnlen %" PRId64),
                 abfd, name, sym.n_sclass, sym.n_scnum,
-                aux.x_csect.x_scnlen.l);
+                (int64_t) aux.x_csect.x_scnlen.l);
              bfd_set_error (bfd_error_bad_value);
              goto error_return;
            }
@@ -1526,9 +1535,10 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
              if (sym.n_sclass != C_HIDEXT
                  || aux.x_csect.x_scnlen.l != 0)
                {
-                 (*_bfd_error_handler)
-                   (_("%B: XMC_TC0 symbol `%s' is class %d scnlen %d"),
-                    abfd, name, sym.n_sclass, aux.x_csect.x_scnlen.l);
+                 _bfd_error_handler
+                   /* xgettext:c-format */
+                   (_("%pB: XMC_TC0 symbol `%s' is class %d scnlen %" PRId64),
+                    abfd, name, sym.n_sclass, (int64_t) aux.x_csect.x_scnlen.l);
                  bfd_set_error (bfd_error_bad_value);
                  goto error_return;
                }
@@ -1592,7 +1602,7 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
                    {
                      const char *relname;
                      char relbuf[SYMNMLEN + 1];
-                     bfd_boolean copy;
+                     bool copy;
                      struct xcoff_link_hash_entry *h;
 
                      /* At this point we know that the TOC entry is
@@ -1613,8 +1623,8 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
                                  || relsym._n._n_n._n_zeroes != 0
                                  || relsym._n._n_n._n_offset == 0);
                          h = xcoff_link_hash_lookup (xcoff_hash_table (info),
-                                                     relname, TRUE, copy,
-                                                     FALSE);
+                                                     relname, true, copy,
+                                                     false);
                          if (h == NULL)
                            goto error_return;
 
@@ -1672,8 +1682,9 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
                    || ((bfd_vma) sym.n_value + aux.x_csect.x_scnlen.l
                        > enclosing->vma + enclosing->size)))
              {
-               (*_bfd_error_handler)
-                 (_("%B: csect `%s' not in enclosing section"),
+               _bfd_error_handler
+                 /* xgettext:c-format */
+                 (_("%pB: csect `%s' not in enclosing section"),
                   abfd, name);
                bfd_set_error (bfd_error_bad_value);
                goto error_return;
@@ -1763,29 +1774,30 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
             follow its appropriate XTY_SD symbol.  The .set pseudo op can
             cause the XTY_LD to not follow the XTY_SD symbol. */
          {
-           bfd_boolean bad;
+           bool bad;
 
-           bad = FALSE;
+           bad = false;
            if (aux.x_csect.x_scnlen.l < 0
                || (aux.x_csect.x_scnlen.l
                    >= esym - (bfd_byte *) obj_coff_external_syms (abfd)))
-             bad = TRUE;
+             bad = true;
            if (! bad)
              {
                section = xcoff_data (abfd)->csects[aux.x_csect.x_scnlen.l];
                if (section == NULL
                    || (section->flags & SEC_HAS_CONTENTS) == 0)
-                 bad = TRUE;
+                 bad = true;
              }
            if (bad)
              {
-               (*_bfd_error_handler)
-                 (_("%B: misplaced XTY_LD `%s'"),
+               _bfd_error_handler
+                 /* xgettext:c-format */
+                 (_("%pB: misplaced XTY_LD `%s'"),
                   abfd, name);
                bfd_set_error (bfd_error_bad_value);
                goto error_return;
              }
-           csect = section;
+           csect = section;
            value = sym.n_value - csect->vma;
          }
          break;
@@ -1805,6 +1817,12 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
              csect = bfd_make_section_anyway_with_flags (abfd, ".td",
                                                          SEC_ALLOC);
            }
+         else if (aux.x_csect.x_smclas == XMC_UL)
+           {
+             /* This is a thread-local unitialized csect.  */
+             csect = bfd_make_section_anyway_with_flags (abfd, ".tbss",
+                                                         SEC_ALLOC | SEC_THREAD_LOCAL);
+           }
          else
            csect = bfd_make_section_anyway_with_flags (abfd, ".bss",
                                                        SEC_ALLOC);
@@ -1877,7 +1895,7 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
 
       if (EXTERN_SYM_P (sym.n_sclass))
        {
-         bfd_boolean copy;
+         bool copy, ok;
          flagword flags;
 
          BFD_ASSERT (section != NULL);
@@ -1887,7 +1905,7 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
          copy = default_copy;
          if (sym._n._n_n._n_zeroes != 0
              || sym._n._n_n._n_offset == 0)
-           copy = TRUE;
+           copy = true;
 
          /* Ignore global linkage code when linking statically.  */
          if (info->static_link
@@ -1922,17 +1940,17 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
             We also have to handle the case of statically linking a
             shared object, which will cause symbol redefinitions,
             although this is an easier case to detect.  */
-         else if (info->output_bfd->xvec == abfd->xvec)
+         else if (info->output_bfd->xvec == abfd->xvec)
            {
              if (! bfd_is_und_section (section))
                *sym_hash = xcoff_link_hash_lookup (xcoff_hash_table (info),
-                                                   name, TRUE, copy, FALSE);
+                                                   name, true, copy, false);
              else
                /* Make a copy of the symbol name to prevent problems with
                   merging symbols.  */
                *sym_hash = ((struct xcoff_link_hash_entry *)
                             bfd_wrapped_link_hash_lookup (abfd, info, name,
-                                                          TRUE, TRUE, FALSE));
+                                                          true, true, false));
 
              if (*sym_hash == NULL)
                goto error_return;
@@ -1995,13 +2013,10 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
                     sure the XCOFF linker is wholly prepared to
                     handle them, and that would only be a warning,
                     not an error.  */
-                 if (! ((*info->callbacks->multiple_definition)
-                        (info, (*sym_hash)->root.root.string,
-                         NULL, NULL, (bfd_vma) 0,
-                         (*sym_hash)->root.u.def.section->owner,
-                         (*sym_hash)->root.u.def.section,
-                         (*sym_hash)->root.u.def.value)))
-                   goto error_return;
+                 (*info->callbacks->multiple_definition) (info,
+                                                          &(*sym_hash)->root,
+                                                          NULL, NULL,
+                                                          (bfd_vma) 0);
                  /* Try not to give this error too many times.  */
                  (*sym_hash)->flags &= ~XCOFF_MULTIPLY_DEFINED;
                }
@@ -2020,12 +2035,12 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
          BFD_ASSERT (last_real->next == first_csect);
          last_real->next = NULL;
          flags = (sym.n_sclass == C_EXT ? BSF_GLOBAL : BSF_WEAK);
-         if (! (_bfd_generic_link_add_one_symbol
-                (info, abfd, name, flags, section, value,
-                 NULL, copy, TRUE,
-                 (struct bfd_link_hash_entry **) sym_hash)))
-           goto error_return;
+         ok = (_bfd_generic_link_add_one_symbol
+               (info, abfd, name, flags, section, value, NULL, copy, true,
+                (struct bfd_link_hash_entry **) sym_hash));
          last_real->next = first_csect;
+         if (!ok)
+           goto error_return;
 
          if (smtyp == XTY_CM)
            {
@@ -2038,7 +2053,7 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
                  = csect->alignment_power;
            }
 
-         if (info->output_bfd->xvec == abfd->xvec)
+         if (info->output_bfd->xvec == abfd->xvec)
            {
              int flag;
 
@@ -2077,11 +2092,15 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
   /* Make sure that we have seen all the relocs.  */
   for (o = abfd->sections; o != first_csect; o = o->next)
     {
+      /* Debugging sections have no csects.  */
+      if (bfd_section_flags (o) & SEC_DEBUGGING)
+       continue;
+
       /* Reset the section size and the line number count, since the
         data is now attached to the csects.  Don't reset the size of
         the .debug section, since we need to read it below in
         bfd_xcoff_size_dynamic_sections.  */
-      if (strcmp (bfd_get_section_name (abfd, o), ".debug") != 0)
+      if (strcmp (bfd_section_name (o), ".debug") != 0)
        o->size = 0;
       o->lineno_count = 0;
 
@@ -2098,9 +2117,10 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
            {
              if (*rel_csect == NULL)
                {
-                 (*_bfd_error_handler)
-                   (_("%B: reloc %s:%d not in csect"),
-                    abfd, o->name, i);
+                 _bfd_error_handler
+                   /* xgettext:c-format */
+                   (_("%pB: reloc %s:%" PRId64 " not in csect"),
+                    abfd, o->name, (int64_t) i);
                  bfd_set_error (bfd_error_bad_value);
                  goto error_return;
                }
@@ -2108,7 +2128,7 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
              /* We identify all function symbols that are the target
                 of a relocation, so that we can create glue code for
                 functions imported from dynamic objects.  */
-             if (info->output_bfd->xvec == abfd->xvec
+             if (info->output_bfd->xvec == abfd->xvec
                  && *rel_csect != bfd_und_section_ptr
                  && obj_xcoff_sym_hashes (abfd)[rel->r_symndx] != NULL)
                {
@@ -2132,7 +2152,7 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
 
                      hds = xcoff_link_hash_lookup (xcoff_hash_table (info),
                                                    h->root.root.string + 1,
-                                                   TRUE, FALSE, TRUE);
+                                                   true, false, true);
                      if (hds == NULL)
                        goto error_return;
                      if (hds->root.type == bfd_link_hash_new)
@@ -2141,8 +2161,8 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
                          if (! (_bfd_generic_link_add_one_symbol
                                 (info, abfd, hds->root.root.string,
                                  (flagword) 0, bfd_und_section_ptr,
-                                 (bfd_vma) 0, NULL, FALSE,
-                                 TRUE, &bh)))
+                                 (bfd_vma) 0, NULL, false,
+                                 true, &bh)))
                            goto error_return;
                          hds = (struct xcoff_link_hash_entry *) bh;
                        }
@@ -2167,7 +2187,6 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
          /* If we are not keeping memory, free the reloc information.  */
          if (! info->keep_memory
              && coff_section_data (abfd, o) != NULL
-             && coff_section_data (abfd, o)->relocs != NULL
              && ! coff_section_data (abfd, o)->keep_relocs)
            {
              free (coff_section_data (abfd, o)->relocs);
@@ -2177,33 +2196,28 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
 
       /* Free up the line numbers.  FIXME: We could cache these
         somewhere for the final link, to avoid reading them again.  */
-      if (reloc_info[o->target_index].linenos != NULL)
-       {
-         free (reloc_info[o->target_index].linenos);
-         reloc_info[o->target_index].linenos = NULL;
-       }
+      free (reloc_info[o->target_index].linenos);
+      reloc_info[o->target_index].linenos = NULL;
     }
 
   free (reloc_info);
 
   obj_coff_keep_syms (abfd) = keep_syms;
 
-  return TRUE;
+  return true;
 
  error_return:
   if (reloc_info != NULL)
     {
       for (o = abfd->sections; o != NULL; o = o->next)
        {
-         if (reloc_info[o->target_index].csects != NULL)
-           free (reloc_info[o->target_index].csects);
-         if (reloc_info[o->target_index].linenos != NULL)
-           free (reloc_info[o->target_index].linenos);
+         free (reloc_info[o->target_index].csects);
+         free (reloc_info[o->target_index].linenos);
        }
       free (reloc_info);
     }
   obj_coff_keep_syms (abfd) = keep_syms;
-  return FALSE;
+  return false;
 }
 
 #undef N_TMASK
@@ -2211,29 +2225,30 @@ xcoff_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
 
 /* Add symbols from an XCOFF object file.  */
 
-static bfd_boolean
+static bool
 xcoff_link_add_object_symbols (bfd *abfd, struct bfd_link_info *info)
 {
   if (! _bfd_coff_get_external_symbols (abfd))
-    return FALSE;
+    return false;
   if (! xcoff_link_add_symbols (abfd, info))
-    return FALSE;
+    return false;
   if (! info->keep_memory)
     {
       if (! _bfd_coff_free_symbols (abfd))
-       return FALSE;
+       return false;
     }
-  return TRUE;
+  return true;
 }
 
 /* Look through the loader symbols to see if this dynamic object
    should be included in the link.  The native linker uses the loader
    symbols, not the normal symbol table, so we do too.  */
 
-static bfd_boolean
+static bool
 xcoff_link_check_dynamic_ar_symbols (bfd *abfd,
                                     struct bfd_link_info *info,
-                                    bfd_boolean *pneeded)
+                                    bool *pneeded,
+                                    bfd **subsbfd)
 {
   asection *lsec;
   bfd_byte *contents;
@@ -2241,15 +2256,15 @@ xcoff_link_check_dynamic_ar_symbols (bfd *abfd,
   const char *strings;
   bfd_byte *elsym, *elsymend;
 
-  *pneeded = FALSE;
+  *pneeded = false;
 
   lsec = bfd_get_section_by_name (abfd, ".loader");
   if (lsec == NULL)
     /* There are no symbols, so don't try to include it.  */
-    return TRUE;
+    return true;
 
   if (! xcoff_get_section_contents (abfd, lsec))
-    return FALSE;
+    return false;
   contents = coff_section_data (abfd, lsec)->contents;
 
   bfd_xcoff_swap_ldhdr_in (abfd, contents, &ldhdr);
@@ -2281,7 +2296,7 @@ xcoff_link_check_dynamic_ar_symbols (bfd *abfd,
          name = nambuf;
        }
 
-      h = bfd_link_hash_lookup (info->hash, name, FALSE, FALSE, TRUE);
+      h = bfd_link_hash_lookup (info->hash, name, false, false, true);
 
       /* We are only interested in symbols that are currently
         undefined.  At this point we know that we are using an XCOFF
@@ -2291,10 +2306,11 @@ xcoff_link_check_dynamic_ar_symbols (bfd *abfd,
          && (((struct xcoff_link_hash_entry *) h)->flags
              & XCOFF_DEF_DYNAMIC) == 0)
        {
-         if (! (*info->callbacks->add_archive_element) (info, abfd, name))
-           return FALSE;
-         *pneeded = TRUE;
-         return TRUE;
+         if (!(*info->callbacks
+               ->add_archive_element) (info, abfd, name, subsbfd))
+           continue;
+         *pneeded = true;
+         return true;
        }
     }
 
@@ -2305,27 +2321,28 @@ xcoff_link_check_dynamic_ar_symbols (bfd *abfd,
       coff_section_data (abfd, lsec)->contents = NULL;
     }
 
-  return TRUE;
+  return true;
 }
 
 /* Look through the symbols to see if this object file should be
    included in the link.  */
 
-static bfd_boolean
+static bool
 xcoff_link_check_ar_symbols (bfd *abfd,
                             struct bfd_link_info *info,
-                            bfd_boolean *pneeded)
+                            bool *pneeded,
+                            bfd **subsbfd)
 {
   bfd_size_type symesz;
   bfd_byte *esym;
   bfd_byte *esym_end;
 
-  *pneeded = FALSE;
+  *pneeded = false;
 
   if ((abfd->flags & DYNAMIC) != 0
       && ! info->static_link
       && info->output_bfd->xvec == abfd->xvec)
-    return xcoff_link_check_dynamic_ar_symbols (abfd, info, pneeded);
+    return xcoff_link_check_dynamic_ar_symbols (abfd, info, pneeded, subsbfd);
 
   symesz = bfd_coff_symesz (abfd);
   esym = (bfd_byte *) obj_coff_external_syms (abfd);
@@ -2335,6 +2352,7 @@ xcoff_link_check_ar_symbols (bfd *abfd,
       struct internal_syment sym;
 
       bfd_coff_swap_sym_in (abfd, (void *) esym, (void *) &sym);
+      esym += (sym.n_numaux + 1) * symesz;
 
       if (EXTERN_SYM_P (sym.n_sclass) && sym.n_scnum != N_UNDEF)
        {
@@ -2347,8 +2365,8 @@ xcoff_link_check_ar_symbols (bfd *abfd,
          name = _bfd_coff_internal_syment_name (abfd, &sym, buf);
 
          if (name == NULL)
-           return FALSE;
-         h = bfd_link_hash_lookup (info->hash, name, FALSE, FALSE, TRUE);
+           return false;
+         h = bfd_link_hash_lookup (info->hash, name, false, false, true);
 
          /* We are only interested in symbols that are currently
             undefined.  If a symbol is currently known to be common,
@@ -2357,22 +2375,21 @@ xcoff_link_check_ar_symbols (bfd *abfd,
             undefined references in shared objects.  */
          if (h != NULL
              && h->type == bfd_link_hash_undefined
-             && (info->output_bfd->xvec != abfd->xvec
+             && (info->output_bfd->xvec != abfd->xvec
                  || (((struct xcoff_link_hash_entry *) h)->flags
                      & XCOFF_DEF_DYNAMIC) == 0))
            {
-             if (! (*info->callbacks->add_archive_element) (info, abfd, name))
-               return FALSE;
-             *pneeded = TRUE;
-             return TRUE;
+             if (!(*info->callbacks
+                   ->add_archive_element) (info, abfd, name, subsbfd))
+               continue;
+             *pneeded = true;
+             return true;
            }
        }
-
-      esym += (sym.n_numaux + 1) * symesz;
     }
 
   /* We do not need this object file.  */
-  return TRUE;
+  return true;
 }
 
 /* Check a single archive element to see if we need to include it in
@@ -2380,41 +2397,56 @@ xcoff_link_check_ar_symbols (bfd *abfd,
    needed in the link or not.  This is called via
    _bfd_generic_link_add_archive_symbols.  */
 
-static bfd_boolean
+static bool
 xcoff_link_check_archive_element (bfd *abfd,
                                  struct bfd_link_info *info,
-                                 bfd_boolean *pneeded)
+                                 struct bfd_link_hash_entry *h ATTRIBUTE_UNUSED,
+                                 const char *name ATTRIBUTE_UNUSED,
+                                 bool *pneeded)
 {
-  bfd_boolean keep_syms_p;
+  bool keep_syms_p;
+  bfd *oldbfd;
 
   keep_syms_p = (obj_coff_external_syms (abfd) != NULL);
-  if (! _bfd_coff_get_external_symbols (abfd))
-    return FALSE;
+  if (!_bfd_coff_get_external_symbols (abfd))
+    return false;
 
-  if (! xcoff_link_check_ar_symbols (abfd, info, pneeded))
-    return FALSE;
+  oldbfd = abfd;
+  if (!xcoff_link_check_ar_symbols (abfd, info, pneeded, &abfd))
+    return false;
 
   if (*pneeded)
     {
-      if (! xcoff_link_add_symbols (abfd, info))
-       return FALSE;
+      /* Potentially, the add_archive_element hook may have set a
+        substitute BFD for us.  */
+      if (abfd != oldbfd)
+       {
+         if (!keep_syms_p
+             && !_bfd_coff_free_symbols (oldbfd))
+           return false;
+         keep_syms_p = (obj_coff_external_syms (abfd) != NULL);
+         if (!_bfd_coff_get_external_symbols (abfd))
+           return false;
+       }
+      if (!xcoff_link_add_symbols (abfd, info))
+       return false;
       if (info->keep_memory)
-       keep_syms_p = TRUE;
+       keep_syms_p = true;
     }
 
   if (!keep_syms_p)
     {
-      if (! _bfd_coff_free_symbols (abfd))
-       return FALSE;
+      if (!_bfd_coff_free_symbols (abfd))
+       return false;
     }
 
-  return TRUE;
+  return true;
 }
 
 /* Given an XCOFF BFD, add symbols to the global hash table as
    appropriate.  */
 
-bfd_boolean
+bool
 _bfd_xcoff_bfd_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
 {
   switch (bfd_get_format (abfd))
@@ -2433,7 +2465,7 @@ _bfd_xcoff_bfd_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
        {
          if (! (_bfd_generic_link_add_archive_symbols
                 (abfd, info, xcoff_link_check_archive_element)))
-           return FALSE;
+           return false;
        }
 
       {
@@ -2446,11 +2478,11 @@ _bfd_xcoff_bfd_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
                && (info->output_bfd->xvec == member->xvec)
                && (! bfd_has_map (abfd) || (member->flags & DYNAMIC) != 0))
              {
-               bfd_boolean needed;
+               bool needed;
 
                if (! xcoff_link_check_archive_element (member, info,
-                                                       &needed))
-                 return FALSE;
+                                                       NULL, NULL, &needed))
+                 return false;
                if (needed)
                  member->archive_pass = -1;
              }
@@ -2458,15 +2490,15 @@ _bfd_xcoff_bfd_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
          }
       }
 
-      return TRUE;
+      return true;
 
     default:
       bfd_set_error (bfd_error_wrong_format);
-      return FALSE;
+      return false;
     }
 }
 \f
-bfd_boolean
+bool
 _bfd_xcoff_define_common_symbol (bfd *output_bfd ATTRIBUTE_UNUSED,
                                 struct bfd_link_info *info ATTRIBUTE_UNUSED,
                                 struct bfd_link_hash_entry *harg)
@@ -2474,17 +2506,17 @@ _bfd_xcoff_define_common_symbol (bfd *output_bfd ATTRIBUTE_UNUSED,
   struct xcoff_link_hash_entry *h;
 
   if (!bfd_generic_define_common_symbol (output_bfd, info, harg))
-    return FALSE;
+    return false;
 
   h = (struct xcoff_link_hash_entry *) harg;
   h->flags |= XCOFF_DEF_REGULAR;
-  return TRUE;
+  return true;
 }
 \f
 /* If symbol H has not been interpreted as a function descriptor,
    see whether it should be.  Set up its descriptor information if so.  */
 
-static bfd_boolean
+static bool
 xcoff_find_function (struct bfd_link_info *info,
                     struct xcoff_link_hash_entry *h)
 {
@@ -2493,16 +2525,16 @@ xcoff_find_function (struct bfd_link_info *info,
     {
       char *fnname;
       struct xcoff_link_hash_entry *hfn;
-      bfd_size_type amt;
+      size_t amt;
 
       amt = strlen (h->root.root.string) + 2;
       fnname = bfd_malloc (amt);
       if (fnname == NULL)
-       return FALSE;
+       return false;
       fnname[0] = '.';
       strcpy (fnname + 1, h->root.root.string);
       hfn = xcoff_link_hash_lookup (xcoff_hash_table (info),
-                                   fnname, FALSE, FALSE, TRUE);
+                                   fnname, false, false, true);
       free (fnname);
       if (hfn != NULL
          && hfn->smclas == XMC_PR
@@ -2514,12 +2546,12 @@ xcoff_find_function (struct bfd_link_info *info,
          hfn->descriptor = h;
        }
     }
-  return TRUE;
+  return true;
 }
 \f
 /* Return true if the given bfd contains at least one shared object.  */
 
-static bfd_boolean
+static bool
 xcoff_archive_contains_shared_object_p (struct bfd_link_info *info,
                                        bfd *archive)
 {
@@ -2542,12 +2574,12 @@ xcoff_archive_contains_shared_object_p (struct bfd_link_info *info,
 /* Symbol H qualifies for export by -bexpfull.  Return true if it also
    qualifies for export by -bexpall.  */
 
-static bfd_boolean
+static bool
 xcoff_covered_by_expall_p (struct xcoff_link_hash_entry *h)
 {
   /* Exclude symbols beginning with '_'.  */
   if (h->root.root.string[0] == '_')
-    return FALSE;
+    return false;
 
   /* Exclude archive members that would otherwise be unreferenced.  */
   if ((h->flags & XCOFF_MARK) == 0
@@ -2555,30 +2587,30 @@ xcoff_covered_by_expall_p (struct xcoff_link_hash_entry *h)
          || h->root.type == bfd_link_hash_defweak)
       && h->root.u.def.section->owner != NULL
       && h->root.u.def.section->owner->my_archive != NULL)
-    return FALSE;
+    return false;
 
-  return TRUE;
+  return true;
 }
 
 /* Return true if symbol H qualifies for the forms of automatic export
    specified by AUTO_EXPORT_FLAGS.  */
 
-static bfd_boolean
+static bool
 xcoff_auto_export_p (struct bfd_link_info *info,
                     struct xcoff_link_hash_entry *h,
                     unsigned int auto_export_flags)
 {
   /* Don't automatically export things that were explicitly exported.  */
   if ((h->flags & XCOFF_EXPORT) != 0)
-    return FALSE;
+    return false;
 
   /* Don't export things that we don't define.  */
   if ((h->flags & XCOFF_DEF_REGULAR) == 0)
-    return FALSE;
+    return false;
 
   /* Don't export functions; export their descriptors instead.  */
   if (h->root.root.string[0] == '.')
-    return FALSE;
+    return false;
 
   /* We don't export a symbol which is being defined by an object
      included from an archive which contains a shared object.  The
@@ -2603,31 +2635,31 @@ xcoff_auto_export_p (struct bfd_link_info *info,
       if (owner != NULL
          && owner->my_archive != NULL
          && xcoff_archive_contains_shared_object_p (info, owner->my_archive))
-       return FALSE;
+       return false;
     }
 
   /* Otherwise, all symbols are exported by -bexpfull.  */
   if ((auto_export_flags & XCOFF_EXPFULL) != 0)
-    return TRUE;
+    return true;
 
   /* Despite its name, -bexpall exports most but not all symbols.  */
   if ((auto_export_flags & XCOFF_EXPALL) != 0
       && xcoff_covered_by_expall_p (h))
-    return TRUE;
+    return true;
 
-  return FALSE;
+  return false;
 }
 \f
 /* Return true if relocation REL needs to be copied to the .loader section.
    If REL is against a global symbol, H is that symbol, otherwise it
    is null.  */
 
-static bfd_boolean
+static bool
 xcoff_need_ldrel_p (struct bfd_link_info *info, struct internal_reloc *rel,
-                   struct xcoff_link_hash_entry *h)
+                   struct xcoff_link_hash_entry *h, asection *ssec)
 {
   if (!xcoff_hash_table (info)->loader_section)
-    return FALSE;
+    return false;
 
   switch (rel->r_type)
     {
@@ -2637,7 +2669,7 @@ xcoff_need_ldrel_p (struct bfd_link_info *info, struct internal_reloc *rel,
     case R_TRL:
     case R_TRLA:
       /* We should never need a .loader reloc for a TOC-relative reloc.  */
-      return FALSE;
+      return false;
 
     default:
       /* In this case, relocations against defined symbols can be resolved
@@ -2646,14 +2678,14 @@ xcoff_need_ldrel_p (struct bfd_link_info *info, struct internal_reloc *rel,
          || h->root.type == bfd_link_hash_defined
          || h->root.type == bfd_link_hash_defweak
          || h->root.type == bfd_link_hash_common)
-       return FALSE;
+       return false;
 
       /* We will always provide a local definition of function symbols,
         even if we don't have one yet.  */
       if ((h->flags & XCOFF_CALLED) != 0)
-       return FALSE;
+       return false;
 
-      return TRUE;
+      return true;
 
     case R_POS:
     case R_NEG:
@@ -2664,27 +2696,48 @@ xcoff_need_ldrel_p (struct bfd_link_info *info, struct internal_reloc *rel,
       if (h != NULL
          && (h->root.type == bfd_link_hash_defined
              || h->root.type == bfd_link_hash_defweak)
-         && bfd_is_abs_section (h->root.u.def.section))
-       return FALSE;
+         && !h->root.rel_from_abs)
+       {
+         asection *sec = h->root.u.def.section;
+         if (bfd_is_abs_section (sec)
+             || (sec != NULL
+                 && bfd_is_abs_section (sec->output_section)))
+           return false;
+       }
 
-      return TRUE;
+      /* Absolute relocations from read-only sections are forbidden
+        by AIX loader. However, they can appear in their section's
+         relocations.  */
+      if (ssec != NULL
+         && (ssec->output_section->flags & SEC_READONLY) != 0)
+       return false;
+
+      return true;
+
+    case R_TLS:
+    case R_TLS_LE:
+    case R_TLS_IE:
+    case R_TLS_LD:
+    case R_TLSM:
+    case R_TLSML:
+      return true;
     }
 }
 \f
 /* Mark a symbol as not being garbage, including the section in which
    it is defined.  */
 
-static inline bfd_boolean
+static inline bool
 xcoff_mark_symbol (struct bfd_link_info *info, struct xcoff_link_hash_entry *h)
 {
   if ((h->flags & XCOFF_MARK) != 0)
-    return TRUE;
+    return true;
 
   h->flags |= XCOFF_MARK;
 
   /* If we're marking an undefined symbol, try find some way of
      defining it.  */
-  if (!info->relocatable
+  if (!bfd_link_relocatable (info)
       && (h->flags & XCOFF_IMPORT) == 0
       && (h->flags & XCOFF_DEF_REGULAR) == 0
       && (h->root.type == bfd_link_hash_undefined
@@ -2693,7 +2746,7 @@ xcoff_mark_symbol (struct bfd_link_info *info, struct xcoff_link_hash_entry *h)
       /* First check whether this symbol can be interpreted as an
         undefined function descriptor for a defined function symbol.  */
       if (!xcoff_find_function (info, h))
-       return FALSE;
+       return false;
 
       if ((h->flags & XCOFF_DESCRIPTOR) != 0
          && (h->descriptor->root.type == bfd_link_hash_defined
@@ -2726,12 +2779,12 @@ xcoff_mark_symbol (struct bfd_link_info *info, struct xcoff_link_hash_entry *h)
 
          /* Mark the function itself.  */
          if (!xcoff_mark_symbol (info, h->descriptor))
-           return FALSE;
+           return false;
 
          /* Mark the TOC section, so that we get an anchor
             to relocate against.  */
          if (!xcoff_mark (info, xcoff_hash_table (info)->toc_section))
-           return FALSE;
+           return false;
 
          /* We handle writing out the contents of the descriptor in
             xcoff_write_global_symbol.  */
@@ -2753,7 +2806,7 @@ xcoff_mark_symbol (struct bfd_link_info *info, struct xcoff_link_hash_entry *h)
                       || hds->root.type == bfd_link_hash_undefweak)
                      && (hds->flags & XCOFF_DEF_REGULAR) == 0);
          if (!xcoff_mark_symbol (info, hds))
-           return FALSE;
+           return false;
 
          /* Treat this symbol as undefined if the descriptor was.  */
          if ((hds->flags & XCOFF_WAS_UNDEFINED) != 0)
@@ -2782,14 +2835,14 @@ xcoff_mark_symbol (struct bfd_link_info *info, struct xcoff_link_hash_entry *h)
              else if (bfd_xcoff_is_xcoff32 (info->output_bfd))
                byte_size = 4;
              else
-               return FALSE;
+               return false;
 
              /* Allocate room in the fallback TOC section.  */
              hds->toc_section = xcoff_hash_table (info)->toc_section;
              hds->u.toc_offset = hds->toc_section->size;
              hds->toc_section->size += byte_size;
              if (!xcoff_mark (info, hds->toc_section))
-               return FALSE;
+               return false;
 
              /* Allocate room for a static and dynamic R_TOC
                 relocation.  */
@@ -2810,12 +2863,12 @@ xcoff_mark_symbol (struct bfd_link_info *info, struct xcoff_link_hash_entry *h)
          if (xcoff_hash_table (info)->rtld)
            {
              if (!xcoff_set_import_path (info, h, "", "..", ""))
-               return FALSE;
+               return false;
            }
          else
            {
              if (!xcoff_set_import_path (info, h, NULL, NULL, NULL))
-               return FALSE;
+               return false;
            }
        }
     }
@@ -2830,7 +2883,7 @@ xcoff_mark_symbol (struct bfd_link_info *info, struct xcoff_link_hash_entry *h)
          && (hsec->flags & SEC_MARK) == 0)
        {
          if (! xcoff_mark (info, hsec))
-           return FALSE;
+           return false;
        }
     }
 
@@ -2838,23 +2891,23 @@ xcoff_mark_symbol (struct bfd_link_info *info, struct xcoff_link_hash_entry *h)
       && (h->toc_section->flags & SEC_MARK) == 0)
     {
       if (! xcoff_mark (info, h->toc_section))
-       return FALSE;
+       return false;
     }
 
-  return TRUE;
+  return true;
 }
 
 /* Look for a symbol called NAME.  If the symbol is defined, mark it.
    If the symbol exists, set FLAGS.  */
 
-static bfd_boolean
+static bool
 xcoff_mark_symbol_by_name (struct bfd_link_info *info,
                           const char *name, unsigned int flags)
 {
   struct xcoff_link_hash_entry *h;
 
   h = xcoff_link_hash_lookup (xcoff_hash_table (info), name,
-                             FALSE, FALSE, TRUE);
+                             false, false, true);
   if (h != NULL)
     {
       h->flags |= flags;
@@ -2862,10 +2915,10 @@ xcoff_mark_symbol_by_name (struct bfd_link_info *info,
          || h->root.type == bfd_link_hash_defweak)
        {
          if (!xcoff_mark (info, h->root.u.def.section))
-           return FALSE;
+           return false;
        }
     }
-  return TRUE;
+  return true;
 }
 
 /* The mark phase of garbage collection.  For a given section, mark
@@ -2874,12 +2927,12 @@ xcoff_mark_symbol_by_name (struct bfd_link_info *info,
    the number of relocs which need to be copied into the .loader
    section.  */
 
-static bfd_boolean
+static bool
 xcoff_mark (struct bfd_link_info *info, asection *sec)
 {
-  if (bfd_is_abs_section (sec)
+  if (bfd_is_const_section (sec)
       || (sec->flags & SEC_MARK) != 0)
-    return TRUE;
+    return true;
 
   sec->flags |= SEC_MARK;
 
@@ -2903,17 +2956,17 @@ xcoff_mark (struct bfd_link_info *info, asection *sec)
            && (syms[i]->flags & XCOFF_MARK) == 0)
          {
            if (!xcoff_mark_symbol (info, syms[i]))
-             return FALSE;
+             return false;
          }
 
       /* Look through the section relocs.  */
       if ((sec->flags & SEC_RELOC) != 0
          && sec->reloc_count > 0)
        {
-         rel = xcoff_read_internal_relocs (sec->owner, sec, TRUE,
-                                           NULL, FALSE, NULL);
+         rel = xcoff_read_internal_relocs (sec->owner, sec, true,
+                                           NULL, false, NULL);
          if (rel == NULL)
-           return FALSE;
+           return false;
          relend = rel + sec->reloc_count;
          for (; rel < relend; rel++)
            {
@@ -2929,7 +2982,7 @@ xcoff_mark (struct bfd_link_info *info, asection *sec)
                  if ((h->flags & XCOFF_MARK) == 0)
                    {
                      if (!xcoff_mark_symbol (info, h))
-                       return FALSE;
+                       return false;
                    }
                }
              else
@@ -2941,13 +2994,13 @@ xcoff_mark (struct bfd_link_info *info, asection *sec)
                      && (rsec->flags & SEC_MARK) == 0)
                    {
                      if (!xcoff_mark (info, rsec))
-                       return FALSE;
+                       return false;
                    }
                }
 
              /* See if this reloc needs to be copied into the .loader
                 section.  */
-             if (xcoff_need_ldrel_p (info, rel, h))
+             if (xcoff_need_ldrel_p (info, rel, h, sec))
                {
                  ++xcoff_hash_table (info)->ldrel_count;
                  if (h != NULL)
@@ -2957,7 +3010,6 @@ xcoff_mark (struct bfd_link_info *info, asection *sec)
 
          if (! info->keep_memory
              && coff_section_data (sec->owner, sec) != NULL
-             && coff_section_data (sec->owner, sec)->relocs != NULL
              && ! coff_section_data (sec->owner, sec)->keep_relocs)
            {
              free (coff_section_data (sec->owner, sec)->relocs);
@@ -2966,7 +3018,7 @@ xcoff_mark (struct bfd_link_info *info, asection *sec)
        }
     }
 
-  return TRUE;
+  return true;
 }
 
 /* Routines that are called after all the input files have been
@@ -2980,7 +3032,7 @@ xcoff_sweep (struct bfd_link_info *info)
 {
   bfd *sub;
 
-  for (sub = info->input_bfds; sub != NULL; sub = sub->link_next)
+  for (sub = info->input_bfds; sub != NULL; sub = sub->link.next)
     {
       asection *o;
 
@@ -2996,6 +3048,7 @@ xcoff_sweep (struct bfd_link_info *info)
                  || o == xcoff_hash_table (info)->loader_section
                  || o == xcoff_hash_table (info)->linkage_section
                  || o == xcoff_hash_table (info)->descriptor_section
+                 || (bfd_section_flags (o) & SEC_DEBUGGING)
                  || strcmp (o->name, ".debug") == 0)
                o->flags |= SEC_MARK;
              else
@@ -3011,7 +3064,7 @@ xcoff_sweep (struct bfd_link_info *info)
 /* Record the number of elements in a set.  This is used to output the
    correct csect length.  */
 
-bfd_boolean
+bool
 bfd_xcoff_link_record_set (bfd *output_bfd,
                           struct bfd_link_info *info,
                           struct bfd_link_hash_entry *harg,
@@ -3019,10 +3072,10 @@ bfd_xcoff_link_record_set (bfd *output_bfd,
 {
   struct xcoff_link_hash_entry *h = (struct xcoff_link_hash_entry *) harg;
   struct xcoff_link_size_list *n;
-  bfd_size_type amt;
+  size_t amt;
 
   if (bfd_get_flavour (output_bfd) != bfd_target_xcoff_flavour)
-    return TRUE;
+    return true;
 
   /* This will hardly ever be called.  I don't want to burn four bytes
      per global symbol, so instead the size is kept on a linked list
@@ -3030,7 +3083,7 @@ bfd_xcoff_link_record_set (bfd *output_bfd,
   amt = sizeof (* n);
   n = bfd_alloc (output_bfd, amt);
   if (n == NULL)
-    return FALSE;
+    return false;
   n->next = xcoff_hash_table (info)->size_list;
   n->h = h;
   n->size = size;
@@ -3038,12 +3091,12 @@ bfd_xcoff_link_record_set (bfd *output_bfd,
 
   h->flags |= XCOFF_HAS_SIZE;
 
-  return TRUE;
+  return true;
 }
 
 /* Import a symbol.  */
 
-bfd_boolean
+bool
 bfd_xcoff_import_symbol (bfd *output_bfd,
                         struct bfd_link_info *info,
                         struct bfd_link_hash_entry *harg,
@@ -3056,7 +3109,7 @@ bfd_xcoff_import_symbol (bfd *output_bfd,
   struct xcoff_link_hash_entry *h = (struct xcoff_link_hash_entry *) harg;
 
   if (bfd_get_flavour (output_bfd) != bfd_target_xcoff_flavour)
-    return TRUE;
+    return true;
 
   /* A symbol name which starts with a period is the code for a
      function.  If the symbol is undefined, then add an undefined
@@ -3072,9 +3125,9 @@ bfd_xcoff_import_symbol (bfd *output_bfd,
        {
          hds = xcoff_link_hash_lookup (xcoff_hash_table (info),
                                        h->root.root.string + 1,
-                                       TRUE, FALSE, TRUE);
+                                       true, false, true);
          if (hds == NULL)
-           return FALSE;
+           return false;
          if (hds->root.type == bfd_link_hash_new)
            {
              hds->root.type = bfd_link_hash_undefined;
@@ -3097,16 +3150,9 @@ bfd_xcoff_import_symbol (bfd *output_bfd,
 
   if (val != (bfd_vma) -1)
     {
-      if (h->root.type == bfd_link_hash_defined
-         && (! bfd_is_abs_section (h->root.u.def.section)
-             || h->root.u.def.value != val))
-       {
-         if (! ((*info->callbacks->multiple_definition)
-                (info, h->root.root.string, h->root.u.def.section->owner,
-                 h->root.u.def.section, h->root.u.def.value,
-                 output_bfd, bfd_abs_section_ptr, val)))
-           return FALSE;
-       }
+      if (h->root.type == bfd_link_hash_defined)
+       (*info->callbacks->multiple_definition) (info, &h->root, output_bfd,
+                                                bfd_abs_section_ptr, val);
 
       h->root.type = bfd_link_hash_defined;
       h->root.u.def.section = bfd_abs_section_ptr;
@@ -3115,14 +3161,14 @@ bfd_xcoff_import_symbol (bfd *output_bfd,
     }
 
   if (!xcoff_set_import_path (info, h, imppath, impfile, impmember))
-    return FALSE;
+    return false;
 
-  return TRUE;
+  return true;
 }
 
 /* Export a symbol.  */
 
-bfd_boolean
+bool
 bfd_xcoff_export_symbol (bfd *output_bfd,
                         struct bfd_link_info *info,
                         struct bfd_link_hash_entry *harg)
@@ -3130,7 +3176,7 @@ bfd_xcoff_export_symbol (bfd *output_bfd,
   struct xcoff_link_hash_entry *h = (struct xcoff_link_hash_entry *) harg;
 
   if (bfd_get_flavour (output_bfd) != bfd_target_xcoff_flavour)
-    return TRUE;
+    return true;
 
   h->flags |= XCOFF_EXPORT;
 
@@ -3139,7 +3185,7 @@ bfd_xcoff_export_symbol (bfd *output_bfd,
 
   /* Make sure we don't garbage collect this symbol.  */
   if (! xcoff_mark_symbol (info, h))
-    return FALSE;
+    return false;
 
   /* If this is a function descriptor, make sure we don't garbage
      collect the associated function code.  We normally don't have to
@@ -3149,17 +3195,17 @@ bfd_xcoff_export_symbol (bfd *output_bfd,
   if ((h->flags & XCOFF_DESCRIPTOR) != 0)
     {
       if (! xcoff_mark_symbol (info, h->descriptor))
-       return FALSE;
+       return false;
     }
 
-  return TRUE;
+  return true;
 }
 
 /* Count a reloc against a symbol.  This is called for relocs
    generated by the linker script, typically for global constructors
    and destructors.  */
 
-bfd_boolean
+bool
 bfd_xcoff_link_count_reloc (bfd *output_bfd,
                            struct bfd_link_info *info,
                            const char *name)
@@ -3167,16 +3213,16 @@ bfd_xcoff_link_count_reloc (bfd *output_bfd,
   struct xcoff_link_hash_entry *h;
 
   if (bfd_get_flavour (output_bfd) != bfd_target_xcoff_flavour)
-    return TRUE;
+    return true;
 
   h = ((struct xcoff_link_hash_entry *)
-       bfd_wrapped_link_hash_lookup (output_bfd, info, name, FALSE, FALSE,
-                                    FALSE));
+       bfd_wrapped_link_hash_lookup (output_bfd, info, name, false, false,
+                                    false));
   if (h == NULL)
     {
-      (*_bfd_error_handler) (_("%s: no such symbol"), name);
+      _bfd_error_handler (_("%s: no such symbol"), name);
       bfd_set_error (bfd_error_no_symbols);
-      return FALSE;
+      return false;
     }
 
   h->flags |= XCOFF_REF_REGULAR;
@@ -3188,15 +3234,21 @@ bfd_xcoff_link_count_reloc (bfd *output_bfd,
 
   /* Mark the symbol to avoid garbage collection.  */
   if (! xcoff_mark_symbol (info, h))
-    return FALSE;
+    return false;
 
-  return TRUE;
+  return true;
 }
 
 /* This function is called for each symbol to which the linker script
-   assigns a value.  */
-
-bfd_boolean
+   assigns a value.
+   FIXME: In cases like the linker test ld-scripts/defined5 where a
+   symbol is defined both by an input object file and the script,
+   the script definition doesn't override the object file definition
+   as is usual for other targets.  At least not when the symbol is
+   output.  Other uses of the symbol value by the linker do use the
+   script value.  */
+
+bool
 bfd_xcoff_record_link_assignment (bfd *output_bfd,
                                  struct bfd_link_info *info,
                                  const char *name)
@@ -3204,23 +3256,23 @@ bfd_xcoff_record_link_assignment (bfd *output_bfd,
   struct xcoff_link_hash_entry *h;
 
   if (bfd_get_flavour (output_bfd) != bfd_target_xcoff_flavour)
-    return TRUE;
+    return true;
 
-  h = xcoff_link_hash_lookup (xcoff_hash_table (info), name, TRUE, TRUE,
-                             FALSE);
+  h = xcoff_link_hash_lookup (xcoff_hash_table (info), name, true, true,
+                             false);
   if (h == NULL)
-    return FALSE;
+    return false;
 
   h->flags |= XCOFF_DEF_REGULAR;
 
-  return TRUE;
+  return true;
 }
 
 /* An xcoff_link_hash_traverse callback for which DATA points to an
    xcoff_loader_info.  Mark all symbols that should be automatically
    exported.  */
 
-static bfd_boolean
+static bool
 xcoff_mark_auto_exports (struct xcoff_link_hash_entry *h, void *data)
 {
   struct xcoff_loader_info *ldinfo;
@@ -3229,9 +3281,9 @@ xcoff_mark_auto_exports (struct xcoff_link_hash_entry *h, void *data)
   if (xcoff_auto_export_p (ldinfo->info, h, ldinfo->auto_export_flags))
     {
       if (!xcoff_mark_symbol (ldinfo->info, h))
-       ldinfo->failed = TRUE;
+       ldinfo->failed = true;
     }
-  return TRUE;
+  return true;
 }
 
 /* Add a symbol to the .loader symbols, if necessary.  */
@@ -3239,7 +3291,7 @@ xcoff_mark_auto_exports (struct xcoff_link_hash_entry *h, void *data)
 /* INPUT_BFD has an external symbol associated with hash table entry H
    and csect CSECT.   Return true if INPUT_BFD defines H.  */
 
-static bfd_boolean
+static bool
 xcoff_final_definition_p (bfd *input_bfd, struct xcoff_link_hash_entry *h,
                          asection *csect)
 {
@@ -3259,7 +3311,7 @@ xcoff_final_definition_p (bfd *input_bfd, struct xcoff_link_hash_entry *h,
     case bfd_link_hash_undefweak:
       /* We can't treat undef.abfd as the owner because that bfd
         might be a dynamic object.  Allow any bfd to claim it.  */
-      return TRUE;
+      return true;
 
     default:
       abort ();
@@ -3268,20 +3320,20 @@ xcoff_final_definition_p (bfd *input_bfd, struct xcoff_link_hash_entry *h,
 
 /* See if H should have a loader symbol associated with it.  */
 
-static bfd_boolean
+static bool
 xcoff_build_ldsym (struct xcoff_loader_info *ldinfo,
                   struct xcoff_link_hash_entry *h)
 {
-  bfd_size_type amt;
+  size_t amt;
 
   /* Warn if this symbol is exported but not defined.  */
   if ((h->flags & XCOFF_EXPORT) != 0
       && (h->flags & XCOFF_WAS_UNDEFINED) != 0)
     {
-      (*_bfd_error_handler)
+      _bfd_error_handler
        (_("warning: attempt to export undefined symbol `%s'"),
         h->root.root.string);
-      return TRUE;
+      return true;
     }
 
   /* We need to add a symbol to the .loader section if it is mentioned
@@ -3294,7 +3346,7 @@ xcoff_build_ldsym (struct xcoff_loader_info *ldinfo,
        || h->root.type == bfd_link_hash_common)
       && (h->flags & XCOFF_ENTRY) == 0
       && (h->flags & XCOFF_EXPORT) == 0)
-    return TRUE;
+    return true;
 
   /* We need to add this symbol to the .loader symbols.  */
 
@@ -3303,8 +3355,8 @@ xcoff_build_ldsym (struct xcoff_loader_info *ldinfo,
   h->ldsym = bfd_zalloc (ldinfo->output_bfd, amt);
   if (h->ldsym == NULL)
     {
-      ldinfo->failed = TRUE;
-      return FALSE;
+      ldinfo->failed = true;
+      return false;
     }
 
   if ((h->flags & XCOFF_IMPORT) != 0)
@@ -3323,26 +3375,23 @@ xcoff_build_ldsym (struct xcoff_loader_info *ldinfo,
 
   if (! bfd_xcoff_put_ldsymbol_name (ldinfo->output_bfd, ldinfo,
                                     h->ldsym, h->root.root.string))
-    return FALSE;
+    return false;
 
   h->flags |= XCOFF_BUILT_LDSYM;
-  return TRUE;
+  return true;
 }
 
 /* An xcoff_htab_traverse callback that is called for each symbol
    once garbage collection is complete.  */
 
-static bfd_boolean
+static bool
 xcoff_post_gc_symbol (struct xcoff_link_hash_entry *h, void * p)
 {
   struct xcoff_loader_info *ldinfo = (struct xcoff_loader_info *) p;
 
-  if (h->root.type == bfd_link_hash_warning)
-    h = (struct xcoff_link_hash_entry *) h->root.u.i.link;
-
   /* __rtinit, this symbol has special handling. */
   if (h->flags & XCOFF_RTINIT)
-    return TRUE;
+    return true;
 
   /* We don't want to garbage collect symbols which are not defined in
      XCOFF files.  This is a convenient place to mark them.  */
@@ -3358,7 +3407,7 @@ xcoff_post_gc_symbol (struct xcoff_link_hash_entry *h, void * p)
   /* Skip discarded symbols.  */
   if (xcoff_hash_table (ldinfo->info)->gc
       && (h->flags & XCOFF_MARK) == 0)
-    return TRUE;
+    return true;
 
   /* If this is still a common symbol, and it wasn't garbage
      collected, we need to actually allocate space for it in the .bss
@@ -3376,14 +3425,14 @@ xcoff_post_gc_symbol (struct xcoff_link_hash_entry *h, void * p)
        h->flags |= XCOFF_EXPORT;
 
       if (!xcoff_build_ldsym (ldinfo, h))
-       return FALSE;
+       return false;
     }
 
-  return TRUE;
+  return true;
 }
 
 /* INPUT_BFD includes XCOFF symbol ISYM, which is associated with linker
-   hash table entry H and csect CSECT.  AUX contains ISYM's auxillary
+   hash table entry H and csect CSECT.  AUX contains ISYM's auxiliary
    csect information, if any.  NAME is the function's name if the name
    is stored in the .debug section, otherwise it is null.
 
@@ -3460,7 +3509,7 @@ xcoff_keep_symbol_p (struct bfd_link_info *info, bfd *input_bfd,
        }
 
       if (info->strip == strip_some
-         && bfd_hash_lookup (info->keep_hash, name, FALSE, FALSE) == NULL)
+         && bfd_hash_lookup (info->keep_hash, name, false, false) == NULL)
        return 0;
 
       if (info->discard == discard_l
@@ -3476,7 +3525,7 @@ xcoff_keep_symbol_p (struct bfd_link_info *info, bfd *input_bfd,
 /* Lay out the .loader section, filling in the header and the import paths.
    LIBPATH is as for bfd_xcoff_size_dynamic_sections.  */
 
-static bfd_boolean
+static bool
 xcoff_build_loader_section (struct xcoff_loader_info *ldinfo,
                            const char *libpath)
 {
@@ -3539,7 +3588,7 @@ xcoff_build_loader_section (struct xcoff_loader_info *ldinfo,
   lsec->size = stoff + ldhdr->l_stlen;
   lsec->contents = bfd_zalloc (output_bfd, lsec->size);
   if (lsec->contents == NULL)
-    return FALSE;
+    return false;
 
   /* Set up the header.  */
   bfd_xcoff_swap_ldhdr_out (output_bfd, ldhdr, lsec->contents);
@@ -3583,7 +3632,7 @@ xcoff_build_loader_section (struct xcoff_loader_info *ldinfo,
      when the corresponding normal relocs are handled in
      xcoff_link_input_bfd.  */
 
-  return TRUE;
+  return true;
 }
 
 /* Build the .loader section.  This is called by the XCOFF linker
@@ -3602,7 +3651,7 @@ xcoff_build_loader_section (struct xcoff_loader_info *ldinfo,
    is a mask of XCOFF_EXPALL and XCOFF_EXPFULL.  SPECIAL_SECTIONS
    is set by this routine to csects with magic names like _end.  */
 
-bfd_boolean
+bool
 bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
                                 struct bfd_link_info *info,
                                 const char *libpath,
@@ -3610,12 +3659,12 @@ bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
                                 unsigned long file_align,
                                 unsigned long maxstack,
                                 unsigned long maxdata,
-                                bfd_boolean gc,
+                                bool gc,
                                 int modtype,
-                                bfd_boolean textro,
+                                bool textro,
                                 unsigned int auto_export_flags,
                                 asection **special_sections,
-                                bfd_boolean rtld)
+                                bool rtld)
 {
   struct xcoff_loader_info ldinfo;
   int i;
@@ -3623,16 +3672,16 @@ bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
   bfd *sub;
   struct bfd_strtab_hash *debug_strtab;
   bfd_byte *debug_contents = NULL;
-  bfd_size_type amt;
+  size_t amt;
 
   if (bfd_get_flavour (output_bfd) != bfd_target_xcoff_flavour)
     {
       for (i = 0; i < XCOFF_NUMBER_OF_SPECIAL_SECTIONS; i++)
        special_sections[i] = NULL;
-      return TRUE;
+      return true;
     }
 
-  ldinfo.failed = FALSE;
+  ldinfo.failed = false;
   ldinfo.output_bfd = output_bfd;
   ldinfo.info = info;
   ldinfo.auto_export_flags = auto_export_flags;
@@ -3657,12 +3706,12 @@ bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
       struct internal_ldsym *ldsym;
 
       hsym = xcoff_link_hash_lookup (xcoff_hash_table (info),
-                                    "__rtinit", FALSE, FALSE, TRUE);
+                                    "__rtinit", false, false, true);
       if (hsym == NULL)
        {
-         (*_bfd_error_handler)
+         _bfd_error_handler
            (_("error: undefined symbol __rtinit"));
-         return FALSE;
+         return false;
        }
 
       xcoff_mark_symbol (info, hsym);
@@ -3692,7 +3741,7 @@ bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
 
       if (! bfd_xcoff_put_ldsymbol_name (ldinfo.output_bfd, &ldinfo,
                                         hsym->ldsym, hsym->root.root.string))
-       return FALSE;
+       return false;
 
       /* This symbol is written out by xcoff_write_global_symbol
         Set stuff up so xcoff_write_global_symbol logic works.  */
@@ -3702,14 +3751,14 @@ bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
     }
 
   /* Garbage collect unused sections.  */
-  if (info->relocatable || !gc)
+  if (bfd_link_relocatable (info) || !gc)
     {
-      gc = FALSE;
-      xcoff_hash_table (info)->gc = FALSE;
+      gc = false;
+      xcoff_hash_table (info)->gc = false;
 
       /* We still need to call xcoff_mark, in order to set ldrel_count
         correctly.  */
-      for (sub = info->input_bfds; sub != NULL; sub = sub->link_next)
+      for (sub = info->input_bfds; sub != NULL; sub = sub->link.next)
        {
          asection *o;
 
@@ -3747,7 +3796,7 @@ bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
            goto error_return;
        }
       xcoff_sweep (info);
-      xcoff_hash_table (info)->gc = TRUE;
+      xcoff_hash_table (info)->gc = true;
     }
 
   /* Return special sections to the caller.  */
@@ -3765,7 +3814,7 @@ bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
 
   if (info->input_bfds == NULL)
     /* I'm not sure what to do in this bizarre case.  */
-    return TRUE;
+    return true;
 
   xcoff_link_hash_traverse (xcoff_hash_table (info), xcoff_post_gc_symbol,
                            (void *) &ldinfo);
@@ -3803,7 +3852,7 @@ bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
      and figure out the contents of the .debug section.  */
   debug_strtab = xcoff_hash_table (info)->debug_strtab;
 
-  for (sub = info->input_bfds; sub != NULL; sub = sub->link_next)
+  for (sub = info->input_bfds; sub != NULL; sub = sub->link.next)
     {
       asection *subdeb;
       bfd_size_type symcount;
@@ -3892,7 +3941,7 @@ bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
          keep_p = xcoff_keep_symbol_p (info, sub, &sym, &aux,
                                        *sym_hash, csect, name);
          if (keep_p < 0)
-           return FALSE;
+           return false;
 
          if (!keep_p)
            /* Use a debug_index of -2 to record that a symbol should
@@ -3906,7 +3955,7 @@ bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
                {
                  bfd_size_type indx;
 
-                 indx = _bfd_stringtab_add (debug_strtab, name, TRUE, TRUE);
+                 indx = _bfd_stringtab_add (debug_strtab, name, true, true);
                  if (indx == (bfd_size_type) -1)
                    goto error_return;
                  *debug_index = indx;
@@ -3947,47 +3996,47 @@ bfd_xcoff_size_dynamic_sections (bfd *output_bfd,
     xcoff_hash_table (info)->debug_section->size =
       _bfd_stringtab_size (debug_strtab);
 
-  return TRUE;
+  return true;
 
  error_return:
-  if (ldinfo.strings != NULL)
-    free (ldinfo.strings);
-  if (debug_contents != NULL)
-    free (debug_contents);
-  return FALSE;
+  free (ldinfo.strings);
+  free (debug_contents);
+  return false;
 }
 
-bfd_boolean
+bool
 bfd_xcoff_link_generate_rtinit (bfd *abfd,
                                const char *init,
                                const char *fini,
-                               bfd_boolean rtld)
+                               bool rtld)
 {
   struct bfd_in_memory *bim;
 
   bim = bfd_malloc ((bfd_size_type) sizeof (* bim));
   if (bim == NULL)
-    return FALSE;
+    return false;
 
   bim->size = 0;
   bim->buffer = 0;
 
-  abfd->link_next = 0;
+  abfd->link.next = 0;
   abfd->format = bfd_object;
   abfd->iostream = (void *) bim;
   abfd->flags = BFD_IN_MEMORY;
+  abfd->iovec = &_bfd_memory_iovec;
   abfd->direction = write_direction;
+  abfd->origin = 0;
   abfd->where = 0;
 
   if (! bfd_xcoff_generate_rtinit (abfd, init, fini, rtld))
-    return FALSE;
+    return false;
 
   /* need to reset to unknown or it will not be read back in correctly */
   abfd->format = bfd_unknown;
   abfd->direction = read_direction;
   abfd->where = 0;
 
-  return TRUE;
+  return true;
 }
 \f
 /* Return the section that defines H.  Return null if no section does.  */
@@ -4015,8 +4064,8 @@ xcoff_symbol_section (struct xcoff_link_hash_entry *h)
    symbol that IREL is against, or null if it isn't against a global symbol.
    REFERENCE_BFD is the bfd to use in error messages about the relocation.  */
 
-static bfd_boolean
-xcoff_create_ldrel (bfd *output_bfd, struct xcoff_final_link_info *finfo,
+static bool
+xcoff_create_ldrel (bfd *output_bfd, struct xcoff_final_link_info *flinfo,
                    asection *output_section, bfd *reference_bfd,
                    struct internal_reloc *irel, asection *hsec,
                    struct xcoff_link_hash_entry *h)
@@ -4035,24 +4084,30 @@ xcoff_create_ldrel (bfd *output_bfd, struct xcoff_final_link_info *finfo,
        ldrel.l_symndx = 1;
       else if (strcmp (secname, ".bss") == 0)
        ldrel.l_symndx = 2;
+      else if (strcmp (secname, ".tdata") == 0)
+       ldrel.l_symndx = -1;
+      else if (strcmp (secname, ".tbss") == 0)
+       ldrel.l_symndx = -2;
       else
        {
-         (*_bfd_error_handler)
-           (_("%B: loader reloc in unrecognized section `%s'"),
+         _bfd_error_handler
+           /* xgettext:c-format */
+           (_("%pB: loader reloc in unrecognized section `%s'"),
             reference_bfd, secname);
          bfd_set_error (bfd_error_nonrepresentable_section);
-         return FALSE;
+         return false;
        }
     }
   else if (h != NULL)
     {
       if (h->ldindx < 0)
        {
-         (*_bfd_error_handler)
-           (_("%B: `%s' in loader reloc but not loader sym"),
+         _bfd_error_handler
+           /* xgettext:c-format */
+           (_("%pB: `%s' in loader reloc but not loader sym"),
             reference_bfd, h->root.root.string);
          bfd_set_error (bfd_error_bad_value);
-         return FALSE;
+         return false;
        }
       ldrel.l_symndx = h->ldindx;
     }
@@ -4061,25 +4116,26 @@ xcoff_create_ldrel (bfd *output_bfd, struct xcoff_final_link_info *finfo,
 
   ldrel.l_rtype = (irel->r_size << 8) | irel->r_type;
   ldrel.l_rsecnm = output_section->target_index;
-  if (xcoff_hash_table (finfo->info)->textro
+  if (xcoff_hash_table (flinfo->info)->textro
       && strcmp (output_section->name, ".text") == 0)
     {
-      (*_bfd_error_handler)
-       (_("%B: loader reloc in read-only section %A"),
+      _bfd_error_handler
+       /* xgettext:c-format */
+       (_("%pB: loader reloc in read-only section %pA"),
         reference_bfd, output_section);
       bfd_set_error (bfd_error_invalid_operation);
-      return FALSE;
+      return false;
     }
-  bfd_xcoff_swap_ldrel_out (output_bfd, &ldrel, finfo->ldrel);
-  finfo->ldrel += bfd_xcoff_ldrelsz (output_bfd);
-  return TRUE;
+  bfd_xcoff_swap_ldrel_out (output_bfd, &ldrel, flinfo->ldrel);
+  flinfo->ldrel += bfd_xcoff_ldrelsz (output_bfd);
+  return true;
 }
 
 /* Link an input file into the linker output file.  This function
    handles all the sections and relocations of the input file at once.  */
 
-static bfd_boolean
-xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
+static bool
+xcoff_link_input_bfd (struct xcoff_final_link_info *flinfo,
                      bfd *input_bfd)
 {
   bfd *output_bfd;
@@ -4087,7 +4143,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
   bfd_size_type syment_base;
   unsigned int n_tmask;
   unsigned int n_btshft;
-  bfd_boolean copy, hash;
+  bool copy, hash;
   bfd_size_type isymesz;
   bfd_size_type osymesz;
   bfd_size_type linesz;
@@ -4103,16 +4159,16 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
   bfd_byte *outsym;
   unsigned int incls;
   asection *oline;
-  bfd_boolean keep_syms;
+  bool keep_syms;
   asection *o;
 
   /* We can just skip DYNAMIC files, unless this is a static link.  */
   if ((input_bfd->flags & DYNAMIC) != 0
-      && ! finfo->info->static_link)
-    return TRUE;
+      && ! flinfo->info->static_link)
+    return true;
 
   /* Move all the symbols to the output file.  */
-  output_bfd = finfo->output_bfd;
+  output_bfd = flinfo->output_bfd;
   strings = NULL;
   syment_base = obj_raw_syment_count (output_bfd);
   isymesz = bfd_coff_symesz (input_bfd);
@@ -4127,15 +4183,15 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
 #define N_TMASK n_tmask
 #define N_BTSHFT n_btshft
 
-  copy = FALSE;
-  if (! finfo->info->keep_memory)
-    copy = TRUE;
-  hash = TRUE;
-  if ((output_bfd->flags & BFD_TRADITIONAL_FORMAT) != 0)
-    hash = FALSE;
+  copy = false;
+  if (! flinfo->info->keep_memory)
+    copy = true;
+  hash = true;
+  if (flinfo->info->traditional_format)
+    hash = false;
 
   if (! _bfd_coff_get_external_symbols (input_bfd))
-    return FALSE;
+    return false;
 
   /* Make one pass over the symbols and assign indices to symbols that
      we have decided to keep.  Also use create .loader symbol information
@@ -4145,8 +4201,8 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
   sym_hash = obj_xcoff_sym_hashes (input_bfd);
   csectpp = xcoff_data (input_bfd)->csects;
   debug_index = xcoff_data (input_bfd)->debug_indices;
-  isymp = finfo->internal_syms;
-  indexp = finfo->sym_indices;
+  isymp = flinfo->internal_syms;
+  indexp = flinfo->sym_indices;
   output_index = syment_base;
   while (esym < esym_end)
     {
@@ -4236,7 +4292,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                    ldsym->l_ifile = 0;
                  else
                    {
-                     BFD_ASSERT (impbfd->xvec == finfo->output_bfd->xvec);
+                     BFD_ASSERT (impbfd->xvec == flinfo->output_bfd->xvec);
                      ldsym->l_ifile = xcoff_data (impbfd)->import_file_id;
                    }
                }
@@ -4245,10 +4301,10 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
          ldsym->l_parm = 0;
 
          BFD_ASSERT (h->ldindx >= 0);
-         bfd_xcoff_swap_ldsym_out (finfo->output_bfd, ldsym,
-                                   (finfo->ldsym
+         bfd_xcoff_swap_ldsym_out (flinfo->output_bfd, ldsym,
+                                   (flinfo->ldsym
                                     + ((h->ldindx - 3)
-                                       * bfd_xcoff_ldsymsz (finfo->output_bfd))));
+                                       * bfd_xcoff_ldsymsz (flinfo->output_bfd))));
          h->ldsym = NULL;
 
          /* Fill in snentry now that we know the target_index.  */
@@ -4307,12 +4363,12 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
   esym = (bfd_byte *) obj_coff_external_syms (input_bfd);
   esym_end = esym + obj_raw_syment_count (input_bfd) * isymesz;
   sym_hash = obj_xcoff_sym_hashes (input_bfd);
-  isymp = finfo->internal_syms;
-  indexp = finfo->sym_indices;
+  isymp = flinfo->internal_syms;
+  indexp = flinfo->sym_indices;
   csectpp = xcoff_data (input_bfd)->csects;
   lineno_counts = xcoff_data (input_bfd)->lineno_counts;
   debug_index = xcoff_data (input_bfd)->debug_indices;
-  outsym = finfo->outsyms;
+  outsym = flinfo->outsyms;
   incls = 0;
   oline = NULL;
   while (esym < esym_end)
@@ -4346,10 +4402,10 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                  name = _bfd_coff_internal_syment_name (input_bfd, &isym, NULL);
 
                  if (name == NULL)
-                   return FALSE;
-                 indx = _bfd_stringtab_add (finfo->strtab, name, hash, copy);
+                   return false;
+                 indx = _bfd_stringtab_add (flinfo->strtab, name, hash, copy);
                  if (indx == (bfd_size_type) -1)
-                   return FALSE;
+                   return false;
                  isym._n._n_n._n_offset = STRING_SIZE_SIZE + indx;
                }
            }
@@ -4371,18 +4427,18 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
             have to write the symbol out twice.  */
          if (isym.n_sclass == C_FILE)
            {
-             if (finfo->last_file_index != -1
-                 && finfo->last_file.n_value != (bfd_vma) *indexp)
+             if (flinfo->last_file_index != -1
+                 && flinfo->last_file.n_value != (bfd_vma) *indexp)
                {
                  /* We must correct the value of the last C_FILE entry.  */
-                 finfo->last_file.n_value = *indexp;
-                 if ((bfd_size_type) finfo->last_file_index >= syment_base)
+                 flinfo->last_file.n_value = *indexp;
+                 if ((bfd_size_type) flinfo->last_file_index >= syment_base)
                    {
                      /* The last C_FILE symbol is in this input file.  */
                      bfd_coff_swap_sym_out (output_bfd,
-                                            (void *) &finfo->last_file,
-                                            (void *) (finfo->outsyms
-                                                   + ((finfo->last_file_index
+                                            (void *) &flinfo->last_file,
+                                            (void *) (flinfo->outsyms
+                                                   + ((flinfo->last_file_index
                                                        - syment_base)
                                                       * osymesz)));
                    }
@@ -4394,20 +4450,20 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                      file_ptr pos;
 
                      bfd_coff_swap_sym_out (output_bfd,
-                                            (void *) &finfo->last_file,
+                                            (void *) &flinfo->last_file,
                                             (void *) outsym);
 
                      pos = obj_sym_filepos (output_bfd);
-                     pos += finfo->last_file_index * osymesz;
+                     pos += flinfo->last_file_index * osymesz;
                      if (bfd_seek (output_bfd, pos, SEEK_SET) != 0
                          || (bfd_bwrite (outsym, osymesz, output_bfd)
                              != osymesz))
-                       return FALSE;
+                       return false;
                    }
                }
 
-             finfo->last_file_index = *indexp;
-             finfo->last_file = isym;
+             flinfo->last_file_index = *indexp;
+             flinfo->last_file = isym;
            }
 
          /* The value of a C_BINCL or C_EINCL symbol is a file offset
@@ -4416,7 +4472,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
          if (isym.n_sclass == C_BINCL
              || isym.n_sclass == C_EINCL)
            {
-             isym.n_value = finfo->line_filepos;
+             isym.n_value = flinfo->line_filepos;
              ++incls;
            }
          /* The value of a C_BSTAT symbol is the symbol table
@@ -4430,7 +4486,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                {
                  long symindx;
 
-                 symindx = finfo->sym_indices[indx];
+                 symindx = flinfo->sym_indices[indx];
                  if (symindx < 0)
                    isym.n_value = 0;
                  else
@@ -4478,13 +4534,16 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                        {
                          strings = _bfd_coff_read_string_table (input_bfd);
                          if (strings == NULL)
-                           return FALSE;
+                           return false;
                        }
-                     filename = strings + aux.x_file.x_n.x_offset;
-                     indx = _bfd_stringtab_add (finfo->strtab, filename,
+                     if ((bfd_size_type) aux.x_file.x_n.x_offset >= obj_coff_strings_len (input_bfd))
+                       filename = _("<corrupt>");
+                     else
+                       filename = strings + aux.x_file.x_n.x_offset;
+                     indx = _bfd_stringtab_add (flinfo->strtab, filename,
                                                 hash, copy);
                      if (indx == (bfd_size_type) -1)
-                       return FALSE;
+                       return false;
                      aux.x_file.x_n.x_offset = STRING_SIZE_SIZE + indx;
                    }
                }
@@ -4509,7 +4568,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                        {
                          long symindx;
 
-                         symindx = finfo->sym_indices[indx];
+                         symindx = flinfo->sym_indices[indx];
                          if (symindx < 0)
                            {
                              aux.x_csect.x_scnlen.l = 0;
@@ -4538,13 +4597,13 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                             the index of the next symbol we are going
                             to include.  I don't know if this is
                             entirely right.  */
-                         while (finfo->sym_indices[indx] < 0
+                         while (flinfo->sym_indices[indx] < 0
                                 && indx < obj_raw_syment_count (input_bfd))
                            ++indx;
                          if (indx >= obj_raw_syment_count (input_bfd))
                            indx = output_index;
                          else
-                           indx = finfo->sym_indices[indx];
+                           indx = flinfo->sym_indices[indx];
                          aux.x_sym.x_fcnary.x_fcn.x_endndx.l = indx;
 
                        }
@@ -4555,7 +4614,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                    {
                      long symindx;
 
-                     symindx = finfo->sym_indices[indx];
+                     symindx = flinfo->sym_indices[indx];
                      if (symindx < 0)
                        aux.x_sym.x_tagndx.l = 0;
                      else
@@ -4597,9 +4656,9 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                          pos = enclosing->line_filepos;
                          amt = linesz * enc_count;
                          if (bfd_seek (input_bfd, pos, SEEK_SET) != 0
-                             || (bfd_bread (finfo->linenos, amt, input_bfd)
+                             || (bfd_bread (flinfo->linenos, amt, input_bfd)
                                  != amt))
-                           return FALSE;
+                           return false;
                          oline = enclosing;
                        }
 
@@ -4607,7 +4666,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                         symbol index.  */
                      linoff = (aux.x_sym.x_fcnary.x_fcn.x_lnnoptr
                                - enclosing->line_filepos);
-                     linp = finfo->linenos + linoff;
+                     linp = flinfo->linenos + linoff;
                      bfd_coff_swap_lineno_in (input_bfd, linp, &lin);
                      lin.l_addr.l_symndx = *indexp;
                      bfd_coff_swap_lineno_out (output_bfd, &lin, linp);
@@ -4629,9 +4688,9 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                             + o->output_section->lineno_count * linesz);
                      amt = linesz * *lineno_counts;
                      if (bfd_seek (output_bfd, pos, SEEK_SET) != 0
-                         || bfd_bwrite (finfo->linenos + linoff,
+                         || bfd_bwrite (flinfo->linenos + linoff,
                                         amt, output_bfd) != amt)
-                       return FALSE;
+                       return false;
                      o->output_section->lineno_count += *lineno_counts;
 
                      /* Record the offset of the symbol's line numbers
@@ -4649,10 +4708,10 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                          /* Update any C_BINCL or C_EINCL symbols
                             that refer to a line number in the
                             range we just output.  */
-                         iisp = finfo->internal_syms;
+                         iisp = flinfo->internal_syms;
                          iispend = iisp + obj_raw_syment_count (input_bfd);
-                         iindp = finfo->sym_indices;
-                         oos = finfo->outsyms;
+                         iindp = flinfo->sym_indices;
+                         oos = flinfo->outsyms;
                          range_start = enclosing->line_filepos + linoff;
                          range_end = range_start + *lineno_counts * linesz;
                          while (iisp < iispend)
@@ -4704,27 +4763,27 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
      symbol will be the first symbol in the next input file.  In the
      normal case, this will save us from writing out the C_FILE symbol
      again.  */
-  if (finfo->last_file_index != -1
-      && (bfd_size_type) finfo->last_file_index >= syment_base)
+  if (flinfo->last_file_index != -1
+      && (bfd_size_type) flinfo->last_file_index >= syment_base)
     {
-      finfo->last_file.n_value = output_index;
-      bfd_coff_swap_sym_out (output_bfd, (void *) &finfo->last_file,
-                            (void *) (finfo->outsyms
-                                   + ((finfo->last_file_index - syment_base)
-                                      * osymesz)));
+      flinfo->last_file.n_value = output_index;
+      bfd_coff_swap_sym_out (output_bfd, (void *) &flinfo->last_file,
+                            (void *) (flinfo->outsyms
+                                   + ((flinfo->last_file_index - syment_base)
+                                      * osymesz)));
     }
 
   /* Write the modified symbols to the output file.  */
-  if (outsym > finfo->outsyms)
+  if (outsym > flinfo->outsyms)
     {
       file_ptr pos = obj_sym_filepos (output_bfd) + syment_base * osymesz;
-      bfd_size_type amt = outsym - finfo->outsyms;
+      bfd_size_type amt = outsym - flinfo->outsyms;
       if (bfd_seek (output_bfd, pos, SEEK_SET) != 0
-         || bfd_bwrite (finfo->outsyms, amt, output_bfd) != amt)
-       return FALSE;
+         || bfd_bwrite (flinfo->outsyms, amt, output_bfd) != amt)
+       return false;
 
       BFD_ASSERT ((obj_raw_syment_count (output_bfd)
-                  + (outsym - finfo->outsyms) / osymesz)
+                  + (outsym - flinfo->outsyms) / osymesz)
                  == output_index);
 
       obj_raw_syment_count (output_bfd) = output_index;
@@ -4732,7 +4791,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
 
   /* Don't let the linker relocation routines discard the symbols.  */
   keep_syms = obj_coff_keep_syms (input_bfd);
-  obj_coff_keep_syms (input_bfd) = TRUE;
+  obj_coff_keep_syms (input_bfd) = true;
 
   /* Relocate the contents of each section.  */
   for (o = input_bfd->sections; o != NULL; o = o->next)
@@ -4756,9 +4815,9 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
       else
        {
          bfd_size_type sz = o->rawsize ? o->rawsize : o->size;
-         if (!bfd_get_section_contents (input_bfd, o, finfo->contents, 0, sz))
-           return FALSE;
-         contents = finfo->contents;
+         if (!bfd_get_section_contents (input_bfd, o, flinfo->contents, 0, sz))
+           goto err_out;
+         contents = flinfo->contents;
        }
 
       if ((o->flags & SEC_RELOC) != 0)
@@ -4774,27 +4833,27 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
          /* Read in the relocs.  */
          target_index = o->output_section->target_index;
          internal_relocs = (xcoff_read_internal_relocs
-                            (input_bfd, o, FALSE, finfo->external_relocs,
-                             TRUE,
-                             (finfo->section_info[target_index].relocs
+                            (input_bfd, o, false, flinfo->external_relocs,
+                             true,
+                             (flinfo->section_info[target_index].relocs
                               + o->output_section->reloc_count)));
          if (internal_relocs == NULL)
-           return FALSE;
+           goto err_out;
 
          /* Call processor specific code to relocate the section
             contents.  */
-         if (! bfd_coff_relocate_section (output_bfd, finfo->info,
+         if (! bfd_coff_relocate_section (output_bfd, flinfo->info,
                                           input_bfd, o,
                                           contents,
                                           internal_relocs,
-                                          finfo->internal_syms,
+                                          flinfo->internal_syms,
                                           xcoff_data (input_bfd)->csects))
-           return FALSE;
+           goto err_out;
 
          offset = o->output_section->vma + o->output_offset - o->vma;
          irel = internal_relocs;
          irelend = irel + o->reloc_count;
-         rel_hash = (finfo->section_info[target_index].rel_hashes
+         rel_hash = (flinfo->section_info[target_index].rel_hashes
                      + o->output_section->reloc_count);
          for (; irel < irelend; irel++, rel_hash++)
            {
@@ -4813,7 +4872,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
              else
                h = obj_xcoff_sym_hashes (input_bfd)[r_symndx];
 
-             if (r_symndx != -1 && finfo->info->strip != strip_all)
+             if (r_symndx != -1 && flinfo->info->strip != strip_all)
                {
                  if (h != NULL
                      && h->smclas != XMC_TD
@@ -4836,13 +4895,13 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                        {
                          struct xcoff_toc_rel_hash *n;
                          struct xcoff_link_section_info *si;
-                         bfd_size_type amt;
+                         size_t amt;
 
                          amt = sizeof (* n);
-                         n = bfd_alloc (finfo->output_bfd, amt);
+                         n = bfd_alloc (flinfo->output_bfd, amt);
                          if (n == NULL)
-                           return FALSE;
-                         si = finfo->section_info + target_index;
+                           goto err_out;
+                         si = flinfo->section_info + target_index;
                          n->next = si->toc_rel_hashes;
                          n->h = h;
                          n->rel = irel;
@@ -4870,7 +4929,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                    {
                      long indx;
 
-                     indx = finfo->sym_indices[r_symndx];
+                     indx = flinfo->sym_indices[r_symndx];
 
                      if (indx == -1)
                        {
@@ -4879,7 +4938,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                          /* Relocations against a TC0 TOC anchor are
                             automatically transformed to be against
                             the TOC anchor in the output file.  */
-                         is = finfo->internal_syms + r_symndx;
+                         is = flinfo->internal_syms + r_symndx;
                          if (is->n_sclass == C_HIDEXT
                              && is->n_numaux > 0)
                            {
@@ -4898,7 +4957,7 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                                                    (void *) &aux);
                              if (SMTYP_SMTYP (aux.x_csect.x_smtyp) == XTY_SD
                                  && aux.x_csect.x_smclas == XMC_TC0)
-                               indx = finfo->toc_symindx;
+                               indx = flinfo->toc_symindx;
                            }
                        }
 
@@ -4915,23 +4974,26 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                          /* This reloc is against a symbol we are
                             stripping.  It would be possible to handle
                             this case, but I don't think it's worth it.  */
-                         is = finfo->internal_syms + r_symndx;
+                         is = flinfo->internal_syms + r_symndx;
 
-                         name = (_bfd_coff_internal_syment_name
-                                 (input_bfd, is, buf));
+                         if (is->n_sclass != C_DWARF)
+                           {
+                             name = (_bfd_coff_internal_syment_name
+                                     (input_bfd, is, buf));
 
-                         if (name == NULL)
-                           return FALSE;
+                             if (name == NULL)
+                               goto err_out;
 
-                         if (! ((*finfo->info->callbacks->unattached_reloc)
-                                (finfo->info, name, input_bfd, o,
-                                 irel->r_vaddr)))
-                           return FALSE;
+                             (*flinfo->info->callbacks->unattached_reloc)
+                               (flinfo->info, name,
+                                input_bfd, o, irel->r_vaddr);
+                           }
                        }
                    }
                }
 
-             if (xcoff_need_ldrel_p (finfo->info, irel, h))
+             if ((o->flags & SEC_DEBUGGING) == 0
+                 && xcoff_need_ldrel_p (flinfo->info, irel, h, o))
                {
                  asection *sec;
 
@@ -4941,10 +5003,10 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
                    sec = xcoff_data (input_bfd)->csects[r_symndx];
                  else
                    sec = xcoff_symbol_section (h);
-                 if (!xcoff_create_ldrel (output_bfd, finfo,
+                 if (!xcoff_create_ldrel (output_bfd, flinfo,
                                           o->output_section, input_bfd,
                                           irel, sec, h))
-                   return FALSE;
+                   goto err_out;
                }
            }
 
@@ -4955,18 +5017,22 @@ xcoff_link_input_bfd (struct xcoff_final_link_info *finfo,
       if (! bfd_set_section_contents (output_bfd, o->output_section,
                                      contents, (file_ptr) o->output_offset,
                                      o->size))
-       return FALSE;
+       goto err_out;
     }
 
   obj_coff_keep_syms (input_bfd) = keep_syms;
 
-  if (! finfo->info->keep_memory)
+  if (! flinfo->info->keep_memory)
     {
       if (! _bfd_coff_free_symbols (input_bfd))
-       return FALSE;
+       return false;
     }
 
-  return TRUE;
+  return true;
+
+ err_out:
+  obj_coff_keep_syms (input_bfd) = keep_syms;
+  return false;
 }
 
 #undef N_TMASK
@@ -4990,7 +5056,7 @@ xcoff_sort_relocs (const void * p1, const void * p2)
 
 /* Return true if section SEC is a TOC section.  */
 
-static inline bfd_boolean
+static inline bool
 xcoff_toc_section_p (asection *sec)
 {
   const char *name;
@@ -5001,22 +5067,22 @@ xcoff_toc_section_p (asection *sec)
       if (name[2] == 'c')
        {
          if (name[3] == '0' && name[4] == 0)
-           return TRUE;
+           return true;
          if (name[3] == 0)
-           return TRUE;
+           return true;
        }
       if (name[2] == 'd' && name[3] == 0)
-       return TRUE;
+       return true;
     }
-  return FALSE;
+  return false;
 }
 
 /* See if the link requires a TOC (it usually does!).  If so, find a
    good place to put the TOC anchor csect, and write out the associated
    symbol.  */
 
-static bfd_boolean
-xcoff_find_tc0 (bfd *output_bfd, struct xcoff_final_link_info *finfo)
+static bool
+xcoff_find_tc0 (bfd *output_bfd, struct xcoff_final_link_info *flinfo)
 {
   bfd_vma toc_start, toc_end, start, end, best_address;
   asection *sec;
@@ -5032,9 +5098,9 @@ xcoff_find_tc0 (bfd *output_bfd, struct xcoff_final_link_info *finfo)
   toc_start = ~(bfd_vma) 0;
   toc_end = 0;
   section_index = -1;
-  for (input_bfd = finfo->info->input_bfds;
+  for (input_bfd = flinfo->info->input_bfds;
        input_bfd != NULL;
-       input_bfd = input_bfd->link_next)
+       input_bfd = input_bfd->link.next)
     for (sec = input_bfd->sections; sec != NULL; sec = sec->next)
       if ((sec->flags & SEC_MARK) != 0 && xcoff_toc_section_p (sec))
        {
@@ -5054,7 +5120,7 @@ xcoff_find_tc0 (bfd *output_bfd, struct xcoff_final_link_info *finfo)
   if (toc_end < toc_start)
     {
       xcoff_data (output_bfd)->toc = toc_start;
-      return TRUE;
+      return true;
     }
 
   if (toc_end - toc_start < 0x8000)
@@ -5064,9 +5130,9 @@ xcoff_find_tc0 (bfd *output_bfd, struct xcoff_final_link_info *finfo)
     {
       /* Find the lowest TOC csect that is still within range of TOC_END.  */
       best_address = toc_end;
-      for (input_bfd = finfo->info->input_bfds;
+      for (input_bfd = flinfo->info->input_bfds;
           input_bfd != NULL;
-          input_bfd = input_bfd->link_next)
+          input_bfd = input_bfd->link.next)
        for (sec = input_bfd->sections; sec != NULL; sec = sec->next)
          if ((sec->flags & SEC_MARK) != 0 && xcoff_toc_section_p (sec))
            {
@@ -5082,78 +5148,80 @@ xcoff_find_tc0 (bfd *output_bfd, struct xcoff_final_link_info *finfo)
       /* Make sure that the start of the TOC is also within range.  */
       if (best_address > toc_start + 0x8000)
        {
-         (*_bfd_error_handler)
-           (_("TOC overflow: 0x%lx > 0x10000; try -mminimal-toc "
+         _bfd_error_handler
+           (_("TOC overflow: %#" PRIx64 " > 0x10000; try -mminimal-toc "
               "when compiling"),
-            (unsigned long) (toc_end - toc_start));
+            (uint64_t) (toc_end - toc_start));
          bfd_set_error (bfd_error_file_too_big);
-         return FALSE;
+         return false;
        }
     }
 
   /* Record the chosen TOC value.  */
-  finfo->toc_symindx = obj_raw_syment_count (output_bfd);
+  flinfo->toc_symindx = obj_raw_syment_count (output_bfd);
   xcoff_data (output_bfd)->toc = best_address;
   xcoff_data (output_bfd)->sntoc = section_index;
 
   /* Fill out the TC0 symbol.  */
-  if (!bfd_xcoff_put_symbol_name (output_bfd, finfo->strtab, &irsym, "TOC"))
-    return FALSE;
+  if (!bfd_xcoff_put_symbol_name (output_bfd, flinfo->info, flinfo->strtab,
+                                 &irsym, "TOC"))
+    return false;
   irsym.n_value = best_address;
   irsym.n_scnum = section_index;
   irsym.n_sclass = C_HIDEXT;
   irsym.n_type = T_NULL;
   irsym.n_numaux = 1;
-  bfd_coff_swap_sym_out (output_bfd, &irsym, finfo->outsyms);
+  bfd_coff_swap_sym_out (output_bfd, &irsym, flinfo->outsyms);
 
-  /* Fill out the auxillary csect information.  */
+  /* Fill out the auxiliary csect information.  */
   memset (&iraux, 0, sizeof iraux);
   iraux.x_csect.x_smtyp = XTY_SD;
   iraux.x_csect.x_smclas = XMC_TC0;
   iraux.x_csect.x_scnlen.l = 0;
   bfd_coff_swap_aux_out (output_bfd, &iraux, T_NULL, C_HIDEXT, 0, 1,
-                        finfo->outsyms + bfd_coff_symesz (output_bfd));
+                        flinfo->outsyms + bfd_coff_symesz (output_bfd));
 
   /* Write the contents to the file.  */
   pos = obj_sym_filepos (output_bfd);
   pos += obj_raw_syment_count (output_bfd) * bfd_coff_symesz (output_bfd);
   size = 2 * bfd_coff_symesz (output_bfd);
   if (bfd_seek (output_bfd, pos, SEEK_SET) != 0
-      || bfd_bwrite (finfo->outsyms, size, output_bfd) != size)
-    return FALSE;
+      || bfd_bwrite (flinfo->outsyms, size, output_bfd) != size)
+    return false;
   obj_raw_syment_count (output_bfd) += 2;
 
-  return TRUE;
+  return true;
 }
 
 /* Write out a non-XCOFF global symbol.  */
 
-static bfd_boolean
-xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
+static bool
+xcoff_write_global_symbol (struct bfd_hash_entry *bh, void * inf)
 {
-  struct xcoff_final_link_info *finfo = (struct xcoff_final_link_info *) inf;
+  struct xcoff_link_hash_entry *h = (struct xcoff_link_hash_entry *) bh;
+  struct xcoff_final_link_info *flinfo = (struct xcoff_final_link_info *) inf;
   bfd *output_bfd;
   bfd_byte *outsym;
   struct internal_syment isym;
   union internal_auxent aux;
-  bfd_boolean result;
+  bool result;
   file_ptr pos;
   bfd_size_type amt;
 
-  output_bfd = finfo->output_bfd;
-  outsym = finfo->outsyms;
+  output_bfd = flinfo->output_bfd;
+  outsym = flinfo->outsyms;
 
   if (h->root.type == bfd_link_hash_warning)
     {
       h = (struct xcoff_link_hash_entry *) h->root.u.i.link;
       if (h->root.type == bfd_link_hash_new)
-       return TRUE;
+       return true;
     }
 
   /* If this symbol was garbage collected, just skip it.  */
-  if (xcoff_hash_table (finfo->info)->gc
+  if (xcoff_hash_table (flinfo->info)->gc
       && (h->flags & XCOFF_MARK) == 0)
-    return TRUE;
+    return true;
 
   /* If we need a .loader section entry, write it out.  */
   if (h->ldsym != NULL)
@@ -5252,16 +5320,16 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
       BFD_ASSERT (h->ldindx >= 0);
 
       bfd_xcoff_swap_ldsym_out (output_bfd, ldsym,
-                               (finfo->ldsym +
+                               (flinfo->ldsym +
                                 (h->ldindx - 3)
-                                * bfd_xcoff_ldsymsz(finfo->output_bfd)));
+                                * bfd_xcoff_ldsymsz(flinfo->output_bfd)));
       h->ldsym = NULL;
     }
 
   /* If this symbol needs global linkage code, write it out.  */
   if (h->root.type == bfd_link_hash_defined
       && (h->root.u.def.section
-         == xcoff_hash_table (finfo->info)->linkage_section))
+         == xcoff_hash_table (flinfo->info)->linkage_section))
     {
       bfd_byte *p;
       bfd_vma tocoff;
@@ -5279,7 +5347,7 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
        tocoff += h->descriptor->u.toc_offset;
 
       /* The first instruction in the glink code needs to be
-        cooked to to hold the correct offset in the toc.  The
+        cooked to hold the correct offset in the toc.  The
         rest are just output raw.  */
       bfd_put_32 (output_bfd,
                  bfd_xcoff_glink_code(output_bfd, 0) | (tocoff & 0xffff), p);
@@ -5307,7 +5375,7 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
       tocsec = h->toc_section;
       osec = tocsec->output_section;
       oindx = osec->target_index;
-      irel = finfo->section_info[oindx].relocs + osec->reloc_count;
+      irel = flinfo->section_info[oindx].relocs + osec->reloc_count;
       irel->r_vaddr = (osec->vma
                       + tocsec->output_offset
                       + h->u.toc_offset);
@@ -5347,24 +5415,25 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
          iraux.x_csect.x_scnlen.l = 4;
        }
       else
-       return FALSE;
+       return false;
 
       irel->r_type = R_POS;
-      finfo->section_info[oindx].rel_hashes[osec->reloc_count] = NULL;
+      flinfo->section_info[oindx].rel_hashes[osec->reloc_count] = NULL;
       ++osec->reloc_count;
 
-      if (!xcoff_create_ldrel (output_bfd, finfo, osec,
+      if (!xcoff_create_ldrel (output_bfd, flinfo, osec,
                               output_bfd, irel, NULL, h))
-       return FALSE;
+       return false;
 
       /* We need to emit a symbol to define a csect which holds
         the reloc.  */
-      if (finfo->info->strip != strip_all)
+      if (flinfo->info->strip != strip_all)
        {
-         result = bfd_xcoff_put_symbol_name (output_bfd, finfo->strtab,
+         result = bfd_xcoff_put_symbol_name (output_bfd, flinfo->info,
+                                             flinfo->strtab,
                                              &irsym, h->root.root.string);
          if (!result)
-           return FALSE;
+           return false;
 
          irsym.n_value = irel->r_vaddr;
          irsym.n_scnum = osec->target_index;
@@ -5387,14 +5456,14 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
              pos = obj_sym_filepos (output_bfd);
              pos += (obj_raw_syment_count (output_bfd)
                      * bfd_coff_symesz (output_bfd));
-             amt = outsym - finfo->outsyms;
+             amt = outsym - flinfo->outsyms;
              if (bfd_seek (output_bfd, pos, SEEK_SET) != 0
-                 || bfd_bwrite (finfo->outsyms, amt, output_bfd) != amt)
-               return FALSE;
+                 || bfd_bwrite (flinfo->outsyms, amt, output_bfd) != amt)
+               return false;
              obj_raw_syment_count (output_bfd) +=
-               (outsym - finfo->outsyms) / bfd_coff_symesz (output_bfd);
+               (outsym - flinfo->outsyms) / bfd_coff_symesz (output_bfd);
 
-             outsym = finfo->outsyms;
+             outsym = flinfo->outsyms;
            }
        }
     }
@@ -5412,7 +5481,7 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
   if ((h->flags & XCOFF_DESCRIPTOR) != 0
       && h->root.type == bfd_link_hash_defined
       && (h->root.u.def.section
-         == xcoff_hash_table (finfo->info)->descriptor_section))
+         == xcoff_hash_table (flinfo->info)->descriptor_section))
     {
       asection *sec;
       asection *osec;
@@ -5435,7 +5504,7 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
          byte_size = 4;
        }
       else
-       return FALSE;
+       return false;
 
       sec = h->root.u.def.section;
       osec = sec->output_section;
@@ -5448,19 +5517,19 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
                      || hentry->root.type == bfd_link_hash_defweak));
       esec = hentry->root.u.def.section;
 
-      irel = finfo->section_info[oindx].relocs + osec->reloc_count;
+      irel = flinfo->section_info[oindx].relocs + osec->reloc_count;
       irel->r_vaddr = (osec->vma
                       + sec->output_offset
                       + h->root.u.def.value);
       irel->r_symndx = esec->output_section->target_index;
       irel->r_type = R_POS;
       irel->r_size = reloc_size;
-      finfo->section_info[oindx].rel_hashes[osec->reloc_count] = NULL;
+      flinfo->section_info[oindx].rel_hashes[osec->reloc_count] = NULL;
       ++osec->reloc_count;
 
-      if (!xcoff_create_ldrel (output_bfd, finfo, osec,
+      if (!xcoff_create_ldrel (output_bfd, flinfo, osec,
                               output_bfd, irel, esec, NULL))
-       return FALSE;
+       return false;
 
       /* There are three items to write out,
         the address of the code
@@ -5500,45 +5569,45 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
       irel->r_symndx = tsec->output_section->target_index;
       irel->r_type = R_POS;
       irel->r_size = reloc_size;
-      finfo->section_info[oindx].rel_hashes[osec->reloc_count] = NULL;
+      flinfo->section_info[oindx].rel_hashes[osec->reloc_count] = NULL;
       ++osec->reloc_count;
 
-      if (!xcoff_create_ldrel (output_bfd, finfo, osec,
+      if (!xcoff_create_ldrel (output_bfd, flinfo, osec,
                               output_bfd, irel, tsec, NULL))
-       return FALSE;
+       return false;
     }
 
-  if (h->indx >= 0 || finfo->info->strip == strip_all)
+  if (h->indx >= 0 || flinfo->info->strip == strip_all)
     {
-      BFD_ASSERT (outsym == finfo->outsyms);
-      return TRUE;
+      BFD_ASSERT (outsym == flinfo->outsyms);
+      return true;
     }
 
   if (h->indx != -2
-      && (finfo->info->strip == strip_all
-         || (finfo->info->strip == strip_some
-             && bfd_hash_lookup (finfo->info->keep_hash, h->root.root.string,
-                                 FALSE, FALSE) == NULL)))
+      && (flinfo->info->strip == strip_all
+         || (flinfo->info->strip == strip_some
+             && bfd_hash_lookup (flinfo->info->keep_hash, h->root.root.string,
+                                 false, false) == NULL)))
     {
-      BFD_ASSERT (outsym == finfo->outsyms);
-      return TRUE;
+      BFD_ASSERT (outsym == flinfo->outsyms);
+      return true;
     }
 
   if (h->indx != -2
       && (h->flags & (XCOFF_REF_REGULAR | XCOFF_DEF_REGULAR)) == 0)
     {
-      BFD_ASSERT (outsym == finfo->outsyms);
-      return TRUE;
+      BFD_ASSERT (outsym == flinfo->outsyms);
+      return true;
     }
 
   memset (&aux, 0, sizeof aux);
 
   h->indx = obj_raw_syment_count (output_bfd);
 
-  result = bfd_xcoff_put_symbol_name (output_bfd, finfo->strtab, &isym,
-                                     h->root.root.string);
+  result = bfd_xcoff_put_symbol_name (output_bfd, flinfo->info, flinfo->strtab,
+                                     &isym, h->root.root.string);
   if (!result)
-    return FALSE;
+    return false;
 
   if (h->root.type == bfd_link_hash_undefined
       || h->root.type == bfd_link_hash_undefweak)
@@ -5556,10 +5625,10 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
            || h->root.type == bfd_link_hash_defweak)
           && h->smclas == XMC_XO)
     {
-      BFD_ASSERT (bfd_is_abs_section (h->root.u.def.section));
+      BFD_ASSERT (bfd_is_abs_symbol (&h->root));
       isym.n_value = h->root.u.def.value;
       isym.n_scnum = N_UNDEF;
-      if (h->root.type == bfd_link_hash_undefweak
+      if (h->root.type == bfd_link_hash_defweak
          && C_WEAKEXT == C_AIX_WEAKEXT)
        isym.n_sclass = C_WEAKEXT;
       else
@@ -5583,7 +5652,7 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
 
       if ((h->flags & XCOFF_HAS_SIZE) != 0)
        {
-         for (l = xcoff_hash_table (finfo->info)->size_list;
+         for (l = xcoff_hash_table (flinfo->info)->size_list;
               l != NULL;
               l = l->next)
            {
@@ -5625,7 +5694,7 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
       /* We just output an SD symbol.  Now output an LD symbol.  */
       h->indx += 2;
 
-      if (h->root.type == bfd_link_hash_undefweak
+      if (h->root.type == bfd_link_hash_defweak
          && C_WEAKEXT == C_AIX_WEAKEXT)
        isym.n_sclass = C_WEAKEXT;
       else
@@ -5642,21 +5711,21 @@ xcoff_write_global_symbol (struct xcoff_link_hash_entry *h, void * inf)
 
   pos = obj_sym_filepos (output_bfd);
   pos += obj_raw_syment_count (output_bfd) * bfd_coff_symesz (output_bfd);
-  amt = outsym - finfo->outsyms;
+  amt = outsym - flinfo->outsyms;
   if (bfd_seek (output_bfd, pos, SEEK_SET) != 0
-      || bfd_bwrite (finfo->outsyms, amt, output_bfd) != amt)
-    return FALSE;
+      || bfd_bwrite (flinfo->outsyms, amt, output_bfd) != amt)
+    return false;
   obj_raw_syment_count (output_bfd) +=
-    (outsym - finfo->outsyms) / bfd_coff_symesz (output_bfd);
+    (outsym - flinfo->outsyms) / bfd_coff_symesz (output_bfd);
 
-  return TRUE;
+  return true;
 }
 
 /* Handle a link order which is supposed to generate a reloc.  */
 
-static bfd_boolean
+static bool
 xcoff_reloc_link_order (bfd *output_bfd,
-                       struct xcoff_final_link_info *finfo,
+                       struct xcoff_final_link_info *flinfo,
                        asection *output_section,
                        struct bfd_link_order *link_order)
 {
@@ -5679,19 +5748,18 @@ xcoff_reloc_link_order (bfd *output_bfd,
   if (howto == NULL)
     {
       bfd_set_error (bfd_error_bad_value);
-      return FALSE;
+      return false;
     }
 
   h = ((struct xcoff_link_hash_entry *)
-       bfd_wrapped_link_hash_lookup (output_bfd, finfo->info,
+       bfd_wrapped_link_hash_lookup (output_bfd, flinfo->info,
                                     link_order->u.reloc.p->u.name,
-                                    FALSE, FALSE, TRUE));
+                                    false, false, true));
   if (h == NULL)
     {
-      if (! ((*finfo->info->callbacks->unattached_reloc)
-            (finfo->info, link_order->u.reloc.p->u.name, NULL, NULL, (bfd_vma) 0)))
-       return FALSE;
-      return TRUE;
+      (*flinfo->info->callbacks->unattached_reloc)
+       (flinfo->info, link_order->u.reloc.p->u.name, NULL, NULL, (bfd_vma) 0);
+      return true;
     }
 
   hsec = xcoff_symbol_section (h);
@@ -5712,12 +5780,12 @@ xcoff_reloc_link_order (bfd *output_bfd,
       bfd_size_type size;
       bfd_byte *buf;
       bfd_reloc_status_type rstat;
-      bfd_boolean ok;
+      bool ok;
 
       size = bfd_get_reloc_size (howto);
       buf = bfd_zmalloc (size);
-      if (buf == NULL)
-       return FALSE;
+      if (buf == NULL && size != 0)
+       return false;
 
       rstat = _bfd_relocate_contents (howto, output_bfd, addend, buf);
       switch (rstat)
@@ -5728,27 +5796,23 @@ xcoff_reloc_link_order (bfd *output_bfd,
        case bfd_reloc_outofrange:
          abort ();
        case bfd_reloc_overflow:
-         if (! ((*finfo->info->callbacks->reloc_overflow)
-                (finfo->info, NULL, link_order->u.reloc.p->u.name,
-                 howto->name, addend, NULL, NULL, (bfd_vma) 0)))
-           {
-             free (buf);
-             return FALSE;
-           }
+         (*flinfo->info->callbacks->reloc_overflow)
+           (flinfo->info, NULL, link_order->u.reloc.p->u.name,
+            howto->name, addend, NULL, NULL, (bfd_vma) 0);
          break;
        }
       ok = bfd_set_section_contents (output_bfd, output_section, (void *) buf,
                                     (file_ptr) link_order->offset, size);
       free (buf);
       if (! ok)
-       return FALSE;
+       return false;
     }
 
   /* Store the reloc information in the right place.  It will get
      swapped and written out at the end of the final_link routine.  */
-  irel = (finfo->section_info[output_section->target_index].relocs
+  irel = (flinfo->section_info[output_section->target_index].relocs
          + output_section->reloc_count);
-  rel_hash_ptr = (finfo->section_info[output_section->target_index].rel_hashes
+  rel_hash_ptr = (flinfo->section_info[output_section->target_index].rel_hashes
                  + output_section->reloc_count);
 
   memset (irel, 0, sizeof (struct internal_reloc));
@@ -5774,23 +5838,23 @@ xcoff_reloc_link_order (bfd *output_bfd,
   ++output_section->reloc_count;
 
   /* Now output the reloc to the .loader section.  */
-  if (xcoff_hash_table (finfo->info)->loader_section)
+  if (xcoff_hash_table (flinfo->info)->loader_section)
     {
-      if (!xcoff_create_ldrel (output_bfd, finfo, output_section,
+      if (!xcoff_create_ldrel (output_bfd, flinfo, output_section,
                               output_bfd, irel, hsec, h))
-       return FALSE;
+       return false;
     }
 
-  return TRUE;
+  return true;
 }
 
 /* Do the final link step.  */
 
-bfd_boolean
+bool
 _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
 {
   bfd_size_type symesz;
-  struct xcoff_final_link_info finfo;
+  struct xcoff_final_link_info flinfo;
   asection *o;
   struct bfd_link_order *p;
   bfd_size_type max_contents_size;
@@ -5808,43 +5872,43 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
   file_ptr pos;
   bfd_size_type amt;
 
-  if (info->shared)
+  if (bfd_link_pic (info))
     abfd->flags |= DYNAMIC;
 
   symesz = bfd_coff_symesz (abfd);
 
-  finfo.info = info;
-  finfo.output_bfd = abfd;
-  finfo.strtab = NULL;
-  finfo.section_info = NULL;
-  finfo.last_file_index = -1;
-  finfo.toc_symindx = -1;
-  finfo.internal_syms = NULL;
-  finfo.sym_indices = NULL;
-  finfo.outsyms = NULL;
-  finfo.linenos = NULL;
-  finfo.contents = NULL;
-  finfo.external_relocs = NULL;
+  flinfo.info = info;
+  flinfo.output_bfd = abfd;
+  flinfo.strtab = NULL;
+  flinfo.section_info = NULL;
+  flinfo.last_file_index = -1;
+  flinfo.toc_symindx = -1;
+  flinfo.internal_syms = NULL;
+  flinfo.sym_indices = NULL;
+  flinfo.outsyms = NULL;
+  flinfo.linenos = NULL;
+  flinfo.contents = NULL;
+  flinfo.external_relocs = NULL;
 
   if (xcoff_hash_table (info)->loader_section)
     {
-      finfo.ldsym = (xcoff_hash_table (info)->loader_section->contents
+      flinfo.ldsym = (xcoff_hash_table (info)->loader_section->contents
                     + bfd_xcoff_ldhdrsz (abfd));
-      finfo.ldrel = (xcoff_hash_table (info)->loader_section->contents
+      flinfo.ldrel = (xcoff_hash_table (info)->loader_section->contents
                     + bfd_xcoff_ldhdrsz (abfd)
                     + (xcoff_hash_table (info)->ldhdr.l_nsyms
                        * bfd_xcoff_ldsymsz (abfd)));
     }
   else
     {
-      finfo.ldsym = NULL;
-      finfo.ldrel = NULL;
+      flinfo.ldsym = NULL;
+      flinfo.ldrel = NULL;
     }
 
   xcoff_data (abfd)->coff.link_info = info;
 
-  finfo.strtab = _bfd_stringtab_init ();
-  if (finfo.strtab == NULL)
+  flinfo.strtab = _bfd_stringtab_init ();
+  if (flinfo.strtab == NULL)
     goto error_return;
 
   /* Count the relocation entries required for the output file.
@@ -5868,14 +5932,17 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
                 link.  This will normally be every section.  We need
                 to do this so that we can identify any sections which
                 the linker has decided to not include.  */
-             sec->linker_mark = TRUE;
+             sec->linker_mark = true;
 
              o->reloc_count += sec->reloc_count;
 
-             if (sec->rawsize > max_contents_size)
-               max_contents_size = sec->rawsize;
-             if (sec->size > max_contents_size)
-               max_contents_size = sec->size;
+             if ((sec->flags & SEC_IN_MEMORY) == 0)
+               {
+                 if (sec->rawsize > max_contents_size)
+                   max_contents_size = sec->rawsize;
+                 if (sec->size > max_contents_size)
+                   max_contents_size = sec->size;
+               }
              if (coff_section_data (sec->owner, sec) != NULL
                  && xcoff_section_data (sec->owner, sec) != NULL
                  && (xcoff_section_data (sec->owner, sec)->lineno_count
@@ -5904,23 +5971,23 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
       file_align = xcoff_hash_table (info)->file_align;
       if (file_align != 0)
        {
-         bfd_boolean saw_contents;
+         bool saw_contents;
          int indx;
          file_ptr sofar;
 
          /* Insert .pad sections before every section which has
             contents and is loaded, if it is preceded by some other
             section which has contents and is loaded.  */
-         saw_contents = TRUE;
+         saw_contents = true;
          for (o = abfd->sections; o != NULL; o = o->next)
            {
              if (strcmp (o->name, ".pad") == 0)
-               saw_contents = FALSE;
+               saw_contents = false;
              else if ((o->flags & SEC_HAS_CONTENTS) != 0
                       && (o->flags & SEC_LOAD) != 0)
                {
                  if (! saw_contents)
-                   saw_contents = TRUE;
+                   saw_contents = true;
                  else
                    {
                      asection *n;
@@ -5935,7 +6002,7 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
 
                      bfd_section_list_remove (abfd, n);
                      bfd_section_list_insert_before (abfd, o, n);
-                     saw_contents = FALSE;
+                     saw_contents = false;
                    }
                }
            }
@@ -6001,14 +6068,14 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
        the target_index fields are 1 based.  */
     amt = abfd->section_count + 1;
     amt *= sizeof (struct xcoff_link_section_info);
-    finfo.section_info = bfd_malloc (amt);
-    if (finfo.section_info == NULL)
+    flinfo.section_info = bfd_malloc (amt);
+    if (flinfo.section_info == NULL)
       goto error_return;
     for (i = 0; i <= abfd->section_count; i++)
       {
-       finfo.section_info[i].relocs = NULL;
-       finfo.section_info[i].rel_hashes = NULL;
-       finfo.section_info[i].toc_rel_hashes = NULL;
+       flinfo.section_info[i].relocs = NULL;
+       flinfo.section_info[i].rel_hashes = NULL;
+       flinfo.section_info[i].toc_rel_hashes = NULL;
       }
   }
 
@@ -6049,14 +6116,14 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
             would be slow.  */
          amt = o->reloc_count;
          amt *= sizeof (struct internal_reloc);
-         finfo.section_info[o->target_index].relocs = bfd_malloc (amt);
+         flinfo.section_info[o->target_index].relocs = bfd_malloc (amt);
 
          amt = o->reloc_count;
          amt *= sizeof (struct xcoff_link_hash_entry *);
-         finfo.section_info[o->target_index].rel_hashes = bfd_malloc (amt);
+         flinfo.section_info[o->target_index].rel_hashes = bfd_malloc (amt);
 
-         if (finfo.section_info[o->target_index].relocs == NULL
-             || finfo.section_info[o->target_index].rel_hashes == NULL)
+         if (flinfo.section_info[o->target_index].relocs == NULL
+             || flinfo.section_info[o->target_index].rel_hashes == NULL)
            goto error_return;
 
          if (o->reloc_count > max_output_reloc_count)
@@ -6067,7 +6134,7 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
   /* We now know the size of the relocs, so we can determine the file
      positions of the line numbers.  */
   line_filepos = rel_filepos;
-  finfo.line_filepos = line_filepos;
+  flinfo.line_filepos = line_filepos;
   linesz = bfd_coff_linesz (abfd);
   for (o = abfd->sections; o != NULL; o = o->next)
     {
@@ -6092,11 +6159,11 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
      input BFD's.  We want at least 6 symbols, since that is the
      number which xcoff_write_global_symbol may need.  */
   max_sym_count = 6;
-  for (sub = info->input_bfds; sub != NULL; sub = sub->link_next)
+  for (sub = info->input_bfds; sub != NULL; sub = sub->link.next)
     {
       bfd_size_type sz;
 
-      sub->output_has_begun = FALSE;
+      sub->output_has_begun = false;
       sz = obj_raw_syment_count (sub);
       if (sz > max_sym_count)
        max_sym_count = sz;
@@ -6104,35 +6171,35 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
 
   /* Allocate some buffers used while linking.  */
   amt = max_sym_count * sizeof (struct internal_syment);
-  finfo.internal_syms = bfd_malloc (amt);
+  flinfo.internal_syms = bfd_malloc (amt);
 
   amt = max_sym_count * sizeof (long);
-  finfo.sym_indices = bfd_malloc (amt);
+  flinfo.sym_indices = bfd_malloc (amt);
 
   amt = (max_sym_count + 1) * symesz;
-  finfo.outsyms = bfd_malloc (amt);
+  flinfo.outsyms = bfd_malloc (amt);
 
   amt = max_lineno_count * bfd_coff_linesz (abfd);
-  finfo.linenos = bfd_malloc (amt);
+  flinfo.linenos = bfd_malloc (amt);
 
   amt = max_contents_size;
-  finfo.contents = bfd_malloc (amt);
+  flinfo.contents = bfd_malloc (amt);
 
   amt = max_reloc_count * relsz;
-  finfo.external_relocs = bfd_malloc (amt);
-
-  if ((finfo.internal_syms == NULL && max_sym_count > 0)
-      || (finfo.sym_indices == NULL && max_sym_count > 0)
-      || finfo.outsyms == NULL
-      || (finfo.linenos == NULL && max_lineno_count > 0)
-      || (finfo.contents == NULL && max_contents_size > 0)
-      || (finfo.external_relocs == NULL && max_reloc_count > 0))
+  flinfo.external_relocs = bfd_malloc (amt);
+
+  if ((flinfo.internal_syms == NULL && max_sym_count > 0)
+      || (flinfo.sym_indices == NULL && max_sym_count > 0)
+      || flinfo.outsyms == NULL
+      || (flinfo.linenos == NULL && max_lineno_count > 0)
+      || (flinfo.contents == NULL && max_contents_size > 0)
+      || (flinfo.external_relocs == NULL && max_reloc_count > 0))
     goto error_return;
 
   obj_raw_syment_count (abfd) = 0;
 
   /* Find a TOC symbol, if we need one.  */
-  if (!xcoff_find_tc0 (abfd, &finfo))
+  if (!xcoff_find_tc0 (abfd, &flinfo))
     goto error_return;
 
   /* We now know the position of everything in the file, except that
@@ -6150,15 +6217,15 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
              sub = p->u.indirect.section->owner;
              if (! sub->output_has_begun)
                {
-                 if (! xcoff_link_input_bfd (&finfo, sub))
+                 if (! xcoff_link_input_bfd (&flinfo, sub))
                    goto error_return;
-                 sub->output_has_begun = TRUE;
+                 sub->output_has_begun = true;
                }
            }
          else if (p->type == bfd_section_reloc_link_order
                   || p->type == bfd_symbol_reloc_link_order)
            {
-             if (! xcoff_reloc_link_order (abfd, &finfo, o, p))
+             if (! xcoff_reloc_link_order (abfd, &flinfo, o, p))
                goto error_return;
            }
          else
@@ -6170,56 +6237,36 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
     }
 
   /* Free up the buffers used by xcoff_link_input_bfd.  */
-  if (finfo.internal_syms != NULL)
-    {
-      free (finfo.internal_syms);
-      finfo.internal_syms = NULL;
-    }
-  if (finfo.sym_indices != NULL)
-    {
-      free (finfo.sym_indices);
-      finfo.sym_indices = NULL;
-    }
-  if (finfo.linenos != NULL)
-    {
-      free (finfo.linenos);
-      finfo.linenos = NULL;
-    }
-  if (finfo.contents != NULL)
-    {
-      free (finfo.contents);
-      finfo.contents = NULL;
-    }
-  if (finfo.external_relocs != NULL)
-    {
-      free (finfo.external_relocs);
-      finfo.external_relocs = NULL;
-    }
+  free (flinfo.internal_syms);
+  flinfo.internal_syms = NULL;
+  free (flinfo.sym_indices);
+  flinfo.sym_indices = NULL;
+  free (flinfo.linenos);
+  flinfo.linenos = NULL;
+  free (flinfo.contents);
+  flinfo.contents = NULL;
+  free (flinfo.external_relocs);
+  flinfo.external_relocs = NULL;
 
   /* The value of the last C_FILE symbol is supposed to be -1.  Write
      it out again.  */
-  if (finfo.last_file_index != -1)
+  if (flinfo.last_file_index != -1)
     {
-      finfo.last_file.n_value = -(bfd_vma) 1;
-      bfd_coff_swap_sym_out (abfd, (void *) &finfo.last_file,
-                            (void *) finfo.outsyms);
-      pos = obj_sym_filepos (abfd) + finfo.last_file_index * symesz;
+      flinfo.last_file.n_value = -(bfd_vma) 1;
+      bfd_coff_swap_sym_out (abfd, (void *) &flinfo.last_file,
+                            (void *) flinfo.outsyms);
+      pos = obj_sym_filepos (abfd) + flinfo.last_file_index * symesz;
       if (bfd_seek (abfd, pos, SEEK_SET) != 0
-         || bfd_bwrite (finfo.outsyms, symesz, abfd) != symesz)
+         || bfd_bwrite (flinfo.outsyms, symesz, abfd) != symesz)
        goto error_return;
     }
 
   /* Write out all the global symbols which do not come from XCOFF
      input files.  */
-  xcoff_link_hash_traverse (xcoff_hash_table (info),
-                           xcoff_write_global_symbol,
-                           (void *) &finfo);
+  bfd_hash_traverse (&info->hash->table, xcoff_write_global_symbol, &flinfo);
 
-  if (finfo.outsyms != NULL)
-    {
-      free (finfo.outsyms);
-      finfo.outsyms = NULL;
-    }
+  free (flinfo.outsyms);
+  flinfo.outsyms = NULL;
 
   /* Now that we have written out all the global symbols, we know the
      symbol indices to use for relocs against them, and we can finally
@@ -6248,35 +6295,33 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
       if (o->reloc_count == 0)
        continue;
 
-      irel = finfo.section_info[o->target_index].relocs;
+      irel = flinfo.section_info[o->target_index].relocs;
       irelend = irel + o->reloc_count;
-      rel_hash = finfo.section_info[o->target_index].rel_hashes;
-      for (; irel < irelend; irel++, rel_hash++, erel += relsz)
+      rel_hash = flinfo.section_info[o->target_index].rel_hashes;
+      for (; irel < irelend; irel++, rel_hash++)
        {
          if (*rel_hash != NULL)
            {
              if ((*rel_hash)->indx < 0)
                {
-                 if (! ((*info->callbacks->unattached_reloc)
-                        (info, (*rel_hash)->root.root.string,
-                         NULL, o, irel->r_vaddr)))
-                   goto error_return;
+                 (*info->callbacks->unattached_reloc)
+                   (info, (*rel_hash)->root.root.string,
+                    NULL, o, irel->r_vaddr);
                  (*rel_hash)->indx = 0;
                }
              irel->r_symndx = (*rel_hash)->indx;
            }
        }
 
-      for (toc_rel_hash = finfo.section_info[o->target_index].toc_rel_hashes;
+      for (toc_rel_hash = flinfo.section_info[o->target_index].toc_rel_hashes;
           toc_rel_hash != NULL;
           toc_rel_hash = toc_rel_hash->next)
        {
          if (toc_rel_hash->h->u.toc_indx < 0)
            {
-             if (! ((*info->callbacks->unattached_reloc)
-                    (info, toc_rel_hash->h->root.root.string,
-                     NULL, o, toc_rel_hash->rel->r_vaddr)))
-               goto error_return;
+             (*info->callbacks->unattached_reloc)
+               (info, toc_rel_hash->h->root.root.string,
+                NULL, o, toc_rel_hash->rel->r_vaddr);
              toc_rel_hash->h->u.toc_indx = 0;
            }
          toc_rel_hash->rel->r_symndx = toc_rel_hash->h->u.toc_indx;
@@ -6287,11 +6332,11 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
         appear in the symbol table, which is not necessarily by
         address.  So we sort them here.  There may be a better way to
         do this.  */
-      qsort ((void *) finfo.section_info[o->target_index].relocs,
+      qsort ((void *) flinfo.section_info[o->target_index].relocs,
             o->reloc_count, sizeof (struct internal_reloc),
             xcoff_sort_relocs);
 
-      irel = finfo.section_info[o->target_index].relocs;
+      irel = flinfo.section_info[o->target_index].relocs;
       irelend = irel + o->reloc_count;
       erel = external_relocs;
       for (; irel < irelend; irel++, rel_hash++, erel += relsz)
@@ -6303,33 +6348,30 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
        goto error_return;
     }
 
-  if (external_relocs != NULL)
-    {
-      free (external_relocs);
-      external_relocs = NULL;
-    }
+  free (external_relocs);
+  external_relocs = NULL;
 
   /* Free up the section information.  */
-  if (finfo.section_info != NULL)
+  if (flinfo.section_info != NULL)
     {
       unsigned int i;
 
       for (i = 0; i < abfd->section_count; i++)
        {
-         if (finfo.section_info[i].relocs != NULL)
-           free (finfo.section_info[i].relocs);
-         if (finfo.section_info[i].rel_hashes != NULL)
-           free (finfo.section_info[i].rel_hashes);
+         free (flinfo.section_info[i].relocs);
+         free (flinfo.section_info[i].rel_hashes);
        }
-      free (finfo.section_info);
-      finfo.section_info = NULL;
+      free (flinfo.section_info);
+      flinfo.section_info = NULL;
     }
 
   /* Write out the loader section contents.  */
   o = xcoff_hash_table (info)->loader_section;
-  if (o)
+  if (o != NULL
+      && o->size != 0
+      && o->output_section != bfd_abs_section_ptr)
     {
-      BFD_ASSERT ((bfd_byte *) finfo.ldrel
+      BFD_ASSERT ((bfd_byte *) flinfo.ldrel
                  == (xcoff_hash_table (info)->loader_section->contents
                      + xcoff_hash_table (info)->ldhdr.l_impoff));
       if (!bfd_set_section_contents (abfd, o->output_section, o->contents,
@@ -6339,19 +6381,25 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
 
   /* Write out the magic sections.  */
   o = xcoff_hash_table (info)->linkage_section;
-  if (o->size > 0
+  if (o != NULL
+      && o->size != 0
+      && o->output_section != bfd_abs_section_ptr
       && ! bfd_set_section_contents (abfd, o->output_section, o->contents,
                                     (file_ptr) o->output_offset,
                                     o->size))
     goto error_return;
   o = xcoff_hash_table (info)->toc_section;
-  if (o->size > 0
+  if (o != NULL
+      && o->size != 0
+      && o->output_section != bfd_abs_section_ptr
       && ! bfd_set_section_contents (abfd, o->output_section, o->contents,
                                     (file_ptr) o->output_offset,
                                     o->size))
     goto error_return;
   o = xcoff_hash_table (info)->descriptor_section;
-  if (o->size > 0
+  if (o != NULL
+      && o->size != 0
+      && o->output_section != bfd_abs_section_ptr
       && ! bfd_set_section_contents (abfd, o->output_section, o->contents,
                                     (file_ptr) o->output_offset,
                                     o->size))
@@ -6362,19 +6410,21 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
   if (bfd_seek (abfd, pos, SEEK_SET) != 0)
     goto error_return;
   H_PUT_32 (abfd,
-           _bfd_stringtab_size (finfo.strtab) + STRING_SIZE_SIZE,
+           _bfd_stringtab_size (flinfo.strtab) + STRING_SIZE_SIZE,
            strbuf);
   amt = STRING_SIZE_SIZE;
   if (bfd_bwrite (strbuf, amt, abfd) != amt)
     goto error_return;
-  if (! _bfd_stringtab_emit (abfd, finfo.strtab))
+  if (! _bfd_stringtab_emit (abfd, flinfo.strtab))
     goto error_return;
 
-  _bfd_stringtab_free (finfo.strtab);
+  _bfd_stringtab_free (flinfo.strtab);
 
   /* Write out the debugging string table.  */
   o = xcoff_hash_table (info)->debug_section;
-  if (o != NULL)
+  if (o != NULL
+      && o->size != 0
+      && o->output_section != bfd_abs_section_ptr)
     {
       struct bfd_strtab_hash *debug_strtab;
 
@@ -6388,43 +6438,34 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
        goto error_return;
     }
 
-  /* Setting bfd_get_symcount to 0 will cause write_object_contents to
+  /* Setting symcount to 0 will cause write_object_contents to
      not try to write out the symbols.  */
-  bfd_get_symcount (abfd) = 0;
+  abfd->symcount = 0;
 
-  return TRUE;
+  return true;
 
  error_return:
-  if (finfo.strtab != NULL)
-    _bfd_stringtab_free (finfo.strtab);
+  if (flinfo.strtab != NULL)
+    _bfd_stringtab_free (flinfo.strtab);
 
-  if (finfo.section_info != NULL)
+  if (flinfo.section_info != NULL)
     {
       unsigned int i;
 
       for (i = 0; i < abfd->section_count; i++)
        {
-         if (finfo.section_info[i].relocs != NULL)
-           free (finfo.section_info[i].relocs);
-         if (finfo.section_info[i].rel_hashes != NULL)
-           free (finfo.section_info[i].rel_hashes);
+         free (flinfo.section_info[i].relocs);
+         free (flinfo.section_info[i].rel_hashes);
        }
-      free (finfo.section_info);
+      free (flinfo.section_info);
     }
 
-  if (finfo.internal_syms != NULL)
-    free (finfo.internal_syms);
-  if (finfo.sym_indices != NULL)
-    free (finfo.sym_indices);
-  if (finfo.outsyms != NULL)
-    free (finfo.outsyms);
-  if (finfo.linenos != NULL)
-    free (finfo.linenos);
-  if (finfo.contents != NULL)
-    free (finfo.contents);
-  if (finfo.external_relocs != NULL)
-    free (finfo.external_relocs);
-  if (external_relocs != NULL)
-    free (external_relocs);
-  return FALSE;
+  free (flinfo.internal_syms);
+  free (flinfo.sym_indices);
+  free (flinfo.outsyms);
+  free (flinfo.linenos);
+  free (flinfo.contents);
+  free (flinfo.external_relocs);
+  free (external_relocs);
+  return false;
 }
This page took 0.084922 seconds and 4 git commands to generate.