]> git.ipfire.org Git - thirdparty/binutils-gdb.git/blobdiff - gdb/xcoffread.c
Update years in copyright notice for the GDB files.
[thirdparty/binutils-gdb.git] / gdb / xcoffread.c
index 0e85125791d7439639fa06d39b20dddf2c962d2d..9fe8621448fe01fee92f743ea4183799884ea64f 100644 (file)
@@ -1,7 +1,5 @@
 /* Read AIX xcoff symbol tables and convert to internal format, for GDB.
-   Copyright 1986, 1987, 1988, 1989, 1990, 1991, 1992, 1993, 1994, 1995,
-   1996, 1997, 1998, 1999, 2000, 2001
-   Free Software Foundation, Inc.
+   Copyright (C) 1986-2013 Free Software Foundation, Inc.
    Derived from coffread.c, dbxread.c, and a lot of hacking.
    Contributed by IBM Corporation.
 
@@ -9,7 +7,7 @@
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
-   the Free Software Foundation; either version 2 of the License, or
+   the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
 
    This program is distributed in the hope that it will be useful,
@@ -18,9 +16,7 @@
    GNU General Public License for more details.
 
    You should have received a copy of the GNU General Public License
-   along with this program; if not, write to the Free Software
-   Foundation, Inc., 59 Temple Place - Suite 330,
-   Boston, MA 02111-1307, USA.  */
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 #include "defs.h"
 #include "bfd.h"
@@ -31,7 +27,7 @@
 #include "gdb_string.h"
 
 #include <sys/param.h>
-#ifndef        NO_SYS_FILE
+#ifdef HAVE_SYS_FILE_H
 #include <sys/file.h>
 #endif
 #include "gdb_stat.h"
 #include "coff/xcoff.h"
 #include "libxcoff.h"
 #include "coff/rs6000.h"
+#include "xcoffread.h"
 
 #include "symtab.h"
 #include "gdbtypes.h"
+/* FIXME: ezannoni/2004-02-13 Verify if the include below is really needed.  */
 #include "symfile.h"
 #include "objfiles.h"
 #include "buildsym.h"
 #include "stabsread.h"
 #include "expression.h"
 #include "complaints.h"
+#include "psympriv.h"
 
 #include "gdb-stabs.h"
 
 #include "aout/stab_gnu.h"
 
 \f
+/* Key for XCOFF-associated data.  */
+
+static const struct objfile_data *xcoff_objfile_data_key;
+
 /* We put a pointer to this structure in the read_symtab_private field
    of the psymtab.  */
 
@@ -75,29 +78,30 @@ struct symloc
 
     int numsyms;
 
-    /* Position of the start of the line number information for this psymtab.  */
+    /* Position of the start of the line number information for this
+       psymtab.  */
     unsigned int lineno_off;
   };
 
-/* Remember what we deduced to be the source language of this psymtab. */
+/* Remember what we deduced to be the source language of this psymtab.  */
 
 static enum language psymtab_language = language_unknown;
 \f
 
-/* Simplified internal version of coff symbol table information */
+/* Simplified internal version of coff symbol table information */
 
 struct coff_symbol
   {
     char *c_name;
-    int c_symnum;              /* symbol number of this entry */
-    int c_naux;                        /* 0 if syment only, 1 if syment + auxent */
+    int c_symnum;              /* Symbol number of this entry.  */
+    int c_naux;                        /* 0 if syment only, 1 if syment + auxent */
     long c_value;
     unsigned char c_sclass;
     int c_secnum;
     unsigned int c_type;
   };
 
-/* last function's saved coff symbol `cs' */
+/* Last function's saved coff symbol `cs'.  */
 
 static struct coff_symbol fcn_cs_saved;
 
@@ -105,7 +109,7 @@ static bfd *symfile_bfd;
 
 /* Core address of start and end of text of current source file.
    This is calculated from the first function seen after a C_FILE
-   symbol. */
+   symbol.  */
 
 
 static CORE_ADDR cur_src_end_addr;
@@ -114,7 +118,7 @@ static CORE_ADDR cur_src_end_addr;
 
 static CORE_ADDR first_object_file_end;
 
-/* initial symbol-table-debug-string vector length */
+/* Initial symbol-table-debug-string vector length.  */
 
 #define        INITIAL_STABVECTOR_LENGTH       40
 
@@ -131,8 +135,8 @@ static unsigned local_symesz;
 
 struct coff_symfile_info
   {
-    file_ptr min_lineno_offset;        /* Where in file lowest line#s are */
-    file_ptr max_lineno_offset;        /* 1+last byte of line#s in file */
+    file_ptr min_lineno_offset;        /* Where in file lowest line#s are */
+    file_ptr max_lineno_offset;        /* 1+last byte of line#s in file */
 
     /* Pointer to the string table.  */
     char *strtbl;
@@ -150,17 +154,51 @@ struct coff_symfile_info
     CORE_ADDR toc_offset;
   };
 
-static struct complaint storclass_complaint =
-{"Unexpected storage class: %d", 0, 0};
+/* Convenience macro to access the per-objfile XCOFF data.  */
+
+#define XCOFF_DATA(objfile)                                            \
+  ((struct coff_symfile_info *) objfile_data ((objfile),               \
+                                             xcoff_objfile_data_key))
+
+/* XCOFF names for dwarf sections.  There is no compressed sections.  */
+
+static const struct dwarf2_debug_sections dwarf2_xcoff_names = {
+  { ".dwinfo", NULL },
+  { ".dwabrev", NULL },
+  { ".dwline", NULL },
+  { ".dwloc", NULL },
+  { NULL, NULL }, /* debug_macinfo */
+  { NULL, NULL }, /* debug_macro */
+  { ".dwstr", NULL },
+  { ".dwrnges", NULL },
+  { NULL, NULL }, /* debug_types */
+  { NULL, NULL }, /* debug_addr */
+  { ".dwframe", NULL },
+  { NULL, NULL }, /* eh_frame */
+  { NULL, NULL }, /* gdb_index */
+  23
+};
 
-static struct complaint bf_notfound_complaint =
-{"line numbers off, `.bf' symbol not found", 0, 0};
+static void
+bf_notfound_complaint (void)
+{
+  complaint (&symfile_complaints,
+            _("line numbers off, `.bf' symbol not found"));
+}
 
-static struct complaint ef_complaint =
-{"Mismatched .ef symbol ignored starting at symnum %d", 0, 0};
+static void
+ef_complaint (int arg1)
+{
+  complaint (&symfile_complaints,
+            _("Mismatched .ef symbol ignored starting at symnum %d"), arg1);
+}
 
-static struct complaint eb_complaint =
-{"Mismatched .eb symbol ignored starting at symnum %d", 0, 0};
+static void
+eb_complaint (int arg1)
+{
+  complaint (&symfile_complaints,
+            _("Mismatched .eb symbol ignored starting at symnum %d"), arg1);
+}
 
 static void xcoff_initial_scan (struct objfile *, int);
 
@@ -182,10 +220,8 @@ static void xcoff_new_init (struct objfile *);
 
 static void xcoff_symfile_finish (struct objfile *);
 
-static void
-xcoff_symfile_offsets (struct objfile *, struct section_addr_info *addrs);
-
-static void find_linenos (bfd *, sec_ptr, PTR);
+static void xcoff_symfile_offsets (struct objfile *,
+                                  struct section_addr_info *addrs);
 
 static char *coff_getfilename (union internal_auxent *, struct objfile *);
 
@@ -198,7 +234,7 @@ static CORE_ADDR read_symbol_nvalue (int);
 static struct symbol *process_xcoff_symbol (struct coff_symbol *,
                                            struct objfile *);
 
-static void read_xcoff_symtab (struct partial_symtab *);
+static void read_xcoff_symtab (struct objfile *, struct partial_symtab *);
 
 #if 0
 static void add_stab_to_list (char *, struct pending_stabs **);
@@ -229,10 +265,11 @@ struct find_targ_sec_arg
 static void find_targ_sec (bfd *, asection *, void *);
 
 static void
-find_targ_sec (bfd *abfd, asection *sect, PTR obj)
+find_targ_sec (bfd *abfd, asection *sect, void *obj)
 {
   struct find_targ_sec_arg *args = (struct find_targ_sec_arg *) obj;
   struct objfile *objfile = args->objfile;
+
   if (sect->target_index == args->targ_index)
     {
       /* This is the section.  Figure out what SECT_OFF_* code it is.  */
@@ -246,37 +283,56 @@ find_targ_sec (bfd *abfd, asection *sect, PTR obj)
     }
 }
 
-/* Return the section number (SECT_OFF_*) that CS points to.  */
-static int
-secnum_to_section (int secnum, struct objfile *objfile)
+/* Search all BFD sections for the section whose target_index is
+   equal to N_SCNUM.  Set *BFD_SECT to that section.  The section's
+   associated index in the objfile's section_offset table is also
+   stored in *SECNUM.
+
+   If no match is found, *BFD_SECT is set to NULL, and *SECNUM
+   is set to the text section's number.  */
+
+static void
+xcoff_secnum_to_sections (int n_scnum, struct objfile *objfile,
+                         asection **bfd_sect, int *secnum)
 {
-  int off = SECT_OFF_TEXT (objfile);
-  asection *sect = NULL;
   struct find_targ_sec_arg args;
-  args.targ_index = secnum;
-  args.resultp = &off;
-  args.bfd_sect = &sect;
+
+  args.targ_index = n_scnum;
+  args.resultp = secnum;
+  args.bfd_sect = bfd_sect;
   args.objfile = objfile;
+
+  *bfd_sect = NULL;
+  *secnum = SECT_OFF_TEXT (objfile);
+
   bfd_map_over_sections (objfile->obfd, find_targ_sec, &args);
-  return off;
 }
 
-/* Return the BFD section that CS points to.  */
+/* Return the section number (SECT_OFF_*) that N_SCNUM points to.  */
+
+static int
+secnum_to_section (int n_scnum, struct objfile *objfile)
+{
+  int secnum;
+  asection *ignored;
+
+  xcoff_secnum_to_sections (n_scnum, objfile, &ignored, &secnum);
+  return secnum;
+}
+
+/* Return the BFD section that N_SCNUM points to.  */
+
 static asection *
-secnum_to_bfd_section (int secnum, struct objfile *objfile)
+secnum_to_bfd_section (int n_scnum, struct objfile *objfile)
 {
-  int off = SECT_OFF_TEXT (objfile);
-  asection *sect = NULL;
-  struct find_targ_sec_arg args;
-  args.targ_index = secnum;
-  args.resultp = &off;
-  args.bfd_sect = &sect;
-  args.objfile = objfile;
-  bfd_map_over_sections (objfile->obfd, find_targ_sec, &args);
-  return sect;
+  int ignored;
+  asection *bfd_sect;
+
+  xcoff_secnum_to_sections (n_scnum, objfile, &bfd_sect, &ignored);
+  return bfd_sect;
 }
 \f
-/* add a given stab string into given stab vector. */
+/* add a given stab string into given stab vector.  */
 
 #if 0
 
@@ -296,7 +352,7 @@ add_stab_to_list (char *stabname, struct pending_stabs **stabvector)
       (*stabvector)->length += INITIAL_STABVECTOR_LENGTH;
       *stabvector = (struct pending_stabs *)
        xrealloc ((char *) *stabvector, sizeof (struct pending_stabs) +
-                   (*stabvector)->length * sizeof (char *));
+                 (*stabvector)->length * sizeof (char *));
     }
   (*stabvector)->stab[(*stabvector)->count++] = stabname;
 }
@@ -308,9 +364,9 @@ add_stab_to_list (char *stabname, struct pending_stabs **stabvector)
    Two reasons:
 
    1) xlc (IBM's native c compiler) postpones static function code
-   emission to the end of a compilation unit. This way it can
+   emission to the end of a compilation unit.  This way it can
    determine if those functions (statics) are needed or not, and
-   can do some garbage collection (I think). This makes line
+   can do some garbage collection (I think).  This makes line
    numbers and corresponding addresses unordered, and we end up
    with a line table like:
 
@@ -330,7 +386,7 @@ add_stab_to_list (char *stabname, struct pending_stabs **stabvector)
    60   0x900           
 
    and that breaks gdb's binary search on line numbers, if the
-   above table is not sorted on line numbers. And that sort
+   above table is not sorted on line numbers.  And that sort
    should be on function based, since gcc can emit line numbers
    like:
 
@@ -339,7 +395,7 @@ add_stab_to_list (char *stabname, struct pending_stabs **stabvector)
    30   0x300
    10   0x400   - for the increment part of a for stmt.
 
-   arrange_linetable() will do this sorting.            
+   arrange_linetable() will do this sorting.
 
    2)   aix symbol table might look like:
 
@@ -350,7 +406,7 @@ add_stab_to_list (char *stabname, struct pending_stabs **stabvector)
    .ei
 
    basically, .bi/.ei pairs do not necessarily encapsulate
-   their scope. They need to be recorded, and processed later
+   their scope.  They need to be recorded, and processed later
    on when we come the end of the compilation unit.
    Include table (inclTable) and process_linenos() handle
    that.  */
@@ -358,19 +414,21 @@ add_stab_to_list (char *stabname, struct pending_stabs **stabvector)
 
 
 
-/* compare line table entry addresses. */
+/* compare line table entry addresses.  */
 
 static int
 compare_lte (const void *lte1p, const void *lte2p)
 {
   struct linetable_entry *lte1 = (struct linetable_entry *) lte1p;
   struct linetable_entry *lte2 = (struct linetable_entry *) lte2p;
+
   return lte1->pc - lte2->pc;
 }
 
-/* Given a line table with function entries are marked, arrange its functions
-   in ascending order and strip off function entry markers and return it in
-   a newly created table. If the old one is good enough, return the old one. */
+/* Given a line table with function entries are marked, arrange its
+   functions in ascending order and strip off function entry markers
+   and return it in a newly created table.  If the old one is good
+   enough, return the old one.  */
 /* FIXME: I think all this stuff can be replaced by just passing
    sort_linevec = 1 to end_symtab.  */
 
@@ -392,15 +450,14 @@ arrange_linetable (struct linetable *oldLineTb)
 
   for (function_count = 0, ii = 0; ii < oldLineTb->nitems; ++ii)
     {
-
       if (oldLineTb->item[ii].line == 0)
-       {                       /* function entry found. */
-
+       {                       /* Function entry found.  */
          if (function_count >= fentry_size)
-           {                   /* make sure you have room. */
+           {                   /* Make sure you have room.  */
              fentry_size *= 2;
              fentry = (struct linetable_entry *)
-               xrealloc (fentry, fentry_size * sizeof (struct linetable_entry));
+               xrealloc (fentry,
+                         fentry_size * sizeof (struct linetable_entry));
            }
          fentry[function_count].line = ii;
          fentry[function_count].pc = oldLineTb->item[ii].pc;
@@ -414,16 +471,17 @@ arrange_linetable (struct linetable *oldLineTb)
       return oldLineTb;
     }
   else if (function_count > 1)
-    qsort (fentry, function_count, sizeof (struct linetable_entry), compare_lte);
+    qsort (fentry, function_count,
+          sizeof (struct linetable_entry), compare_lte);
 
-  /* allocate a new line table. */
+  /* Allocate a new line table.  */
   newLineTb = (struct linetable *)
     xmalloc
     (sizeof (struct linetable) +
     (oldLineTb->nitems - function_count) * sizeof (struct linetable_entry));
 
-  /* if line table does not start with a function beginning, copy up until
-     a function begin. */
+  /* If line table does not start with a function beginning, copy up until
+     a function begin.  */
 
   newline = 0;
   if (oldLineTb->item[0].line != 0)
@@ -431,7 +489,7 @@ arrange_linetable (struct linetable *oldLineTb)
     newline < oldLineTb->nitems && oldLineTb->item[newline].line; ++newline)
       newLineTb->item[newline] = oldLineTb->item[newline];
 
-  /* Now copy function lines one by one. */
+  /* Now copy function lines one by one.  */
 
   for (ii = 0; ii < function_count; ++ii)
     {
@@ -446,9 +504,9 @@ arrange_linetable (struct linetable *oldLineTb)
 }
 
 /* include file support: C_BINCL/C_EINCL pairs will be kept in the 
-   following `IncludeChain'. At the end of each symtab (end_symtab),
+   following `IncludeChain'.  At the end of each symtab (end_symtab),
    we will determine if we should create additional symtab's to
-   represent if (the include files. */
+   represent if (the include files.  */
 
 
 typedef struct _inclTable
@@ -460,7 +518,7 @@ typedef struct _inclTable
   int begin, end;
 
   struct subfile *subfile;
-  unsigned funStartLine;       /* start line # of its function */
+  unsigned funStartLine;       /* Start line # of its function.  */
 }
 InclTable;
 
@@ -483,9 +541,7 @@ record_include_begin (struct coff_symbol *cs)
       /* This can happen with old versions of GCC.
          GCC 2.3.3-930426 does not exhibit this on a test case which
          a user said produced the message for him.  */
-      static struct complaint msg =
-      {"Nested C_BINCL symbols", 0, 0};
-      complain (&msg);
+      complaint (&symfile_complaints, _("Nested C_BINCL symbols"));
     }
   ++inclDepth;
 
@@ -502,9 +558,7 @@ record_include_end (struct coff_symbol *cs)
 
   if (inclDepth == 0)
     {
-      static struct complaint msg =
-      {"Mismatched C_BINCL/C_EINCL pair", 0, 0};
-      complain (&msg);
+      complaint (&symfile_complaints, _("Mismatched C_BINCL/C_EINCL pair"));
     }
 
   allocate_include_entry ();
@@ -542,16 +596,18 @@ allocate_include_entry (void)
    in psymtab to symtab processing.  */
 static struct partial_symtab *this_symtab_psymtab;
 
+/* Objfile related to this_symtab_psymtab; set at the same time.  */
+static struct objfile *this_symtab_objfile;
+
 /* given the start and end addresses of a compilation unit (or a csect,
-   at times) process its lines and create appropriate line vectors. */
+   at times) process its lines and create appropriate line vectors.  */
 
 static void
 process_linenos (CORE_ADDR start, CORE_ADDR end)
 {
   int offset, ii;
-  file_ptr max_offset =
-  ((struct coff_symfile_info *) this_symtab_psymtab->objfile->sym_private)
-  ->max_lineno_offset;
+  file_ptr max_offset
+    = XCOFF_DATA (this_symtab_objfile)->max_lineno_offset;
 
   /* subfile structure for the main compilation unit.  */
   struct subfile main_subfile;
@@ -573,7 +629,8 @@ process_linenos (CORE_ADDR start, CORE_ADDR end)
   memset (&main_subfile, '\0', sizeof (main_subfile));
 
   if (inclIndx == 0)
-    /* All source lines were in the main source file. None in include files. */
+    /* All source lines were in the main source file.  None in include
+       files.  */
 
     enter_line_range (&main_subfile, offset, 0, start, end,
                      &main_source_baseline);
@@ -583,7 +640,7 @@ process_linenos (CORE_ADDR start, CORE_ADDR end)
       /* There was source with line numbers in include files.  */
 
       int linesz =
-       coff_data (this_symtab_psymtab->objfile->obfd)->local_linesz;
+       coff_data (this_symtab_objfile->obfd)->local_linesz;
       main_source_baseline = 0;
 
       for (ii = 0; ii < inclIndx; ++ii)
@@ -598,17 +655,31 @@ process_linenos (CORE_ADDR start, CORE_ADDR end)
                 start, 0, &main_source_baseline);
            }
 
-         /* Have a new subfile for the include file.  */
+         if (strcmp (inclTable[ii].name, last_source_file) == 0)
+           {
+              /* The entry in the include table refers to the main source
+                 file.  Add the lines to the main subfile.  */
+
+             main_source_baseline = inclTable[ii].funStartLine;
+             enter_line_range
+               (&main_subfile, inclTable[ii].begin, inclTable[ii].end,
+                start, 0, &main_source_baseline);
+             inclTable[ii].subfile = &main_subfile;
+           }
+         else
+           {
+             /* Have a new subfile for the include file.  */
 
-         tmpSubfile = inclTable[ii].subfile =
-           (struct subfile *) xmalloc (sizeof (struct subfile));
+             tmpSubfile = inclTable[ii].subfile =
+               (struct subfile *) xmalloc (sizeof (struct subfile));
 
-         memset (tmpSubfile, '\0', sizeof (struct subfile));
-         firstLine = &(inclTable[ii].funStartLine);
+             memset (tmpSubfile, '\0', sizeof (struct subfile));
+             firstLine = &(inclTable[ii].funStartLine);
 
-         /* Enter include file's lines now.  */
-         enter_line_range (tmpSubfile, inclTable[ii].begin,
-                           inclTable[ii].end, start, 0, firstLine);
+             /* Enter include file's lines now.  */
+             enter_line_range (tmpSubfile, inclTable[ii].begin,
+                               inclTable[ii].end, start, 0, firstLine);
+           }
 
          if (offset <= inclTable[ii].end)
            offset = inclTable[ii].end + linesz;
@@ -630,8 +701,8 @@ process_linenos (CORE_ADDR start, CORE_ADDR end)
 
       lv = main_subfile.line_vector;
 
-      /* Line numbers are not necessarily ordered. xlc compilation will
-         put static function to the end. */
+      /* Line numbers are not necessarily ordered.  xlc compilation will
+         put static function to the end.  */
 
       lineTb = arrange_linetable (lv);
       if (lv == lineTb)
@@ -654,14 +725,16 @@ process_linenos (CORE_ADDR start, CORE_ADDR end)
 
   for (ii = 0; ii < inclIndx; ++ii)
     {
-      if ((inclTable[ii].subfile)->line_vector)                /* Useless if!!! FIXMEmgo */
+      if (inclTable[ii].subfile != ((struct subfile *) &main_subfile)
+          && (inclTable[ii].subfile)->line_vector)     /* Useless if!!!
+                                                          FIXMEmgo */
        {
          struct linetable *lineTb, *lv;
 
          lv = (inclTable[ii].subfile)->line_vector;
 
-         /* Line numbers are not necessarily ordered. xlc compilation will
-            put static function to the end. */
+         /* Line numbers are not necessarily ordered.  xlc compilation will
+            put static function to the end.  */
 
          lineTb = arrange_linetable (lv);
 
@@ -676,7 +749,7 @@ process_linenos (CORE_ADDR start, CORE_ADDR end)
             #include "foo.h"
             ......
 
-            while foo.h including code in it. (stupid but possible)
+            while foo.h including code in it.  (stupid but possible)
             Since start_subfile() looks at the name and uses an
             existing one if finds, we need to provide a fake name and
             fool it.  */
@@ -689,6 +762,7 @@ process_linenos (CORE_ADDR start, CORE_ADDR end)
               one when passed to deduce_language_from_filename.  Kludge on
               top of kludge.  */
            char *fakename = strrchr (inclTable[ii].name, '.');
+
            if (fakename == NULL)
              fakename = " ?";
            start_subfile (fakename, (char *) 0);
@@ -726,27 +800,36 @@ return_after_cleanup:
   memset (&main_subfile, '\0', sizeof (struct subfile));
 }
 
-void
+static void
 aix_process_linenos (void)
 {
-  /* process line numbers and enter them into line vector */
+  /* There is no linenos to read if there are only dwarf info.  */
+  if (this_symtab_psymtab == NULL)
+    return;
+
+  /* Process line numbers and enter them into line vector.  */
   process_linenos (last_source_start_addr, cur_src_end_addr);
 }
 
 
 /* Enter a given range of lines into the line vector.
    can be called in the following two ways:
-   enter_line_range (subfile, beginoffset, endoffset, startaddr, 0, firstLine)  or
-   enter_line_range (subfile, beginoffset, 0, startaddr, endaddr, firstLine)
+   enter_line_range (subfile, beginoffset, endoffset,
+                     startaddr, 0, firstLine)  or
+   enter_line_range (subfile, beginoffset, 0, 
+                     startaddr, endaddr, firstLine)
 
    endoffset points to the last line table entry that we should pay
    attention to.  */
 
 static void
-enter_line_range (struct subfile *subfile, unsigned beginoffset, unsigned endoffset,   /* offsets to line table */
+enter_line_range (struct subfile *subfile, unsigned beginoffset,
+                 unsigned endoffset,   /* offsets to line table */
                  CORE_ADDR startaddr,  /* offsets to line table */
                  CORE_ADDR endaddr, unsigned *firstLine)
 {
+  struct objfile *objfile = this_symtab_objfile;
+  struct gdbarch *gdbarch = get_objfile_arch (objfile);
   unsigned int curoffset;
   CORE_ADDR addr;
   void *ext_lnno;
@@ -758,17 +841,14 @@ enter_line_range (struct subfile *subfile, unsigned beginoffset, unsigned endoff
   if (endoffset == 0 && startaddr == 0 && endaddr == 0)
     return;
   curoffset = beginoffset;
-  limit_offset =
-    ((struct coff_symfile_info *) this_symtab_psymtab->objfile->sym_private)
-    ->max_lineno_offset;
+  limit_offset = XCOFF_DATA (objfile)->max_lineno_offset;
 
   if (endoffset != 0)
     {
       if (endoffset >= limit_offset)
        {
-         static struct complaint msg =
-         {"Bad line table offset in C_EINCL directive", 0, 0};
-         complain (&msg);
+         complaint (&symfile_complaints,
+                    _("Bad line table offset in C_EINCL directive"));
          return;
        }
       limit_offset = endoffset;
@@ -776,7 +856,7 @@ enter_line_range (struct subfile *subfile, unsigned beginoffset, unsigned endoff
   else
     limit_offset -= 1;
 
-  abfd = this_symtab_psymtab->objfile->obfd;
+  abfd = objfile->obfd;
   linesz = coff_data (abfd)->local_linesz;
   ext_lnno = alloca (linesz);
 
@@ -790,8 +870,7 @@ enter_line_range (struct subfile *subfile, unsigned beginoffset, unsigned endoff
       addr = (int_lnno.l_lnno
              ? int_lnno.l_addr.l_paddr
              : read_symbol_nvalue (int_lnno.l_addr.l_symndx));
-      addr += ANOFFSET (this_symtab_psymtab->objfile->section_offsets,
-                       SECT_OFF_TEXT (this_symtab_psymtab->objfile));
+      addr += ANOFFSET (objfile->section_offsets, SECT_OFF_TEXT (objfile));
 
       if (addr < startaddr || (endaddr && addr >= endaddr))
        return;
@@ -799,11 +878,12 @@ enter_line_range (struct subfile *subfile, unsigned beginoffset, unsigned endoff
       if (int_lnno.l_lnno == 0)
        {
          *firstLine = read_symbol_lineno (int_lnno.l_addr.l_symndx);
-         record_line (subfile, 0, addr);
+         record_line (subfile, 0, gdbarch_addr_bits_remove (gdbarch, addr));
          --(*firstLine);
        }
       else
-       record_line (subfile, *firstLine + int_lnno.l_lnno, addr);
+       record_line (subfile, *firstLine + int_lnno.l_lnno,
+                    gdbarch_addr_bits_remove (gdbarch, addr));
       curoffset += linesz;
     }
 }
@@ -814,8 +894,8 @@ enter_line_range (struct subfile *subfile, unsigned beginoffset, unsigned endoff
    text address for the file, and SIZE is the number of bytes of text.  */
 
 #define complete_symtab(name, start_addr) {    \
-  last_source_file = savestring (name, strlen (name)); \
-  last_source_start_addr = start_addr;                 \
+  last_source_file = xstrdup (name);           \
+  last_source_start_addr = start_addr;         \
 }
 
 
@@ -825,23 +905,37 @@ enter_line_range (struct subfile *subfile, unsigned beginoffset, unsigned endoff
    This function can read past the end of the symbol table
    (into the string table) but this does no harm.  */
 
-/* Reading symbol table has to be fast! Keep the followings as macros, rather
-   than functions. */
-
-#define        RECORD_MINIMAL_SYMBOL(NAME, ADDR, TYPE, SECTION, OBJFILE) \
-{                                              \
-  char *namestr;                               \
-  namestr = (NAME); \
-  if (namestr[0] == '.') ++namestr; \
-  prim_record_minimal_symbol_and_info (namestr, (ADDR), (TYPE), \
-                                      (char *)NULL, (SECTION), (asection *)NULL, (OBJFILE)); \
-  misc_func_recorded = 1;                                      \
-}
+/* Create a new minimal symbol (using prim_record_minimal_symbol_and_info).
 
+   Arguments are:
 
-/* xcoff has static blocks marked in `.bs', `.es' pairs. They cannot be
-   nested. At any given time, a symbol can only be in one static block.
-   This is the base address of current static block, zero if non exists. */
+   NAME - the symbol's name (but if NAME starts with a period, that
+   leading period is discarded).
+   ADDRESS - the symbol's address.
+   MS_TYPE - the symbol's type.
+   N_SCNUM - the symbol's XCOFF section number.
+   OBJFILE - the objfile associated with the minimal symbol.  */
+
+static void
+record_minimal_symbol (const char *name, CORE_ADDR address,
+                      enum minimal_symbol_type ms_type,
+                      int n_scnum,
+                      struct objfile *objfile)
+{
+  int secnum;
+  asection *bfd_sect;
+
+  if (name[0] == '.')
+    ++name;
+
+  xcoff_secnum_to_sections (n_scnum, objfile, &bfd_sect, &secnum);
+  prim_record_minimal_symbol_and_info (name, address, ms_type,
+                                      secnum, bfd_sect, objfile);
+}
+
+/* xcoff has static blocks marked in `.bs', `.es' pairs.  They cannot be
+   nested.  At any given time, a symbol can only be in one static block.
+   This is the base address of current static block, zero if non exists.  */
 
 static int static_block_base = 0;
 
@@ -849,7 +943,7 @@ static int static_block_base = 0;
 
 static int static_block_section = -1;
 
-/* true if space for symbol name has been allocated. */
+/* true if space for symbol name has been allocated.  */
 
 static int symname_alloced = 0;
 
@@ -864,16 +958,16 @@ static char *
 xcoff_next_symbol_text (struct objfile *objfile)
 {
   struct internal_syment symbol;
-  static struct complaint msg =
-  {"Unexpected symbol continuation", 0, 0};
   char *retval;
-  /* FIXME: is this the same as the passed arg? */
-  objfile = this_symtab_psymtab->objfile;
+
+  /* FIXME: is this the same as the passed arg?  */
+  if (this_symtab_objfile)
+    objfile = this_symtab_objfile;
 
   bfd_coff_swap_sym_in (objfile->obfd, raw_symbol, &symbol);
   if (symbol.n_zeroes)
     {
-      complain (&msg);
+      complaint (&symfile_complaints, _("Unexpected symbol continuation"));
 
       /* Return something which points to '\0' and hope the symbol reading
          code does something reasonable.  */
@@ -881,16 +975,13 @@ xcoff_next_symbol_text (struct objfile *objfile)
     }
   else if (symbol.n_sclass & 0x80)
     {
-      retval =
-       ((struct coff_symfile_info *) objfile->sym_private)->debugsec
-       + symbol.n_offset;
-      raw_symbol +=
-       coff_data (objfile->obfd)->local_symesz;
+      retval = XCOFF_DATA (objfile)->debugsec + symbol.n_offset;
+      raw_symbol += coff_data (objfile->obfd)->local_symesz;
       ++symnum;
     }
   else
     {
-      complain (&msg);
+      complaint (&symfile_complaints, _("Unexpected symbol continuation"));
 
       /* Return something which points to '\0' and hope the symbol reading
          code does something reasonable.  */
@@ -902,15 +993,14 @@ xcoff_next_symbol_text (struct objfile *objfile)
 /* Read symbols for a given partial symbol table.  */
 
 static void
-read_xcoff_symtab (struct partial_symtab *pst)
+read_xcoff_symtab (struct objfile *objfile, struct partial_symtab *pst)
 {
-  struct objfile *objfile = pst->objfile;
   bfd *abfd = objfile->obfd;
-  char *raw_auxptr;            /* Pointer to first raw aux entry for sym */
-  char *strtbl = ((struct coff_symfile_info *) objfile->sym_private)->strtbl;
-  char *debugsec =
-  ((struct coff_symfile_info *) objfile->sym_private)->debugsec;
-  char *debugfmt = bfd_xcoff_is_xcoff64 (abfd) ? "XCOFF64" : "XCOFF";
+  char *raw_auxptr;            /* Pointer to first raw aux entry for sym */
+  struct coff_symfile_info *xcoff = XCOFF_DATA (objfile);
+  char *strtbl = xcoff->strtbl;
+  char *debugsec = xcoff->debugsec;
+  const char *debugfmt = bfd_xcoff_is_xcoff64 (abfd) ? "XCOFF64" : "XCOFF";
 
   struct internal_syment symbol[1];
   union internal_auxent main_aux;
@@ -924,27 +1014,25 @@ read_xcoff_symtab (struct partial_symtab *pst)
   int depth = 0;
   int fcn_start_addr = 0;
 
-  struct coff_symbol fcn_stab_saved;
+  struct coff_symbol fcn_stab_saved = { 0 };
 
-  /* fcn_cs_saved is global because process_xcoff_symbol needs it. */
-  union internal_auxent fcn_aux_saved;
+  /* fcn_cs_saved is global because process_xcoff_symbol needs it.  */
+  union internal_auxent fcn_aux_saved = main_aux;
   struct context_stack *new;
 
-  char *filestring = " _start_ ";      /* Name of the current file. */
+  char *filestring = " _start_ ";      /* Name of the current file.  */
 
-  char *last_csect_name;       /* last seen csect's name and value */
-  CORE_ADDR last_csect_val;
-  int last_csect_sec;
+  const char *last_csect_name; /* Last seen csect's name.  */
 
   this_symtab_psymtab = pst;
+  this_symtab_objfile = objfile;
 
   /* Get the appropriate COFF "constants" related to the file we're
-     handling. */
+     handling.  */
   local_symesz = coff_data (abfd)->local_symesz;
 
   last_source_file = NULL;
   last_csect_name = 0;
-  last_csect_val = 0;
 
   start_stabs ();
   start_symtab (filestring, (char *) NULL, file_start_addr);
@@ -954,21 +1042,18 @@ read_xcoff_symtab (struct partial_symtab *pst)
     symnum + ((struct symloc *) pst->read_symtab_private)->numsyms;
   first_object_file_end = 0;
 
-  raw_symbol =
-    ((struct coff_symfile_info *) objfile->sym_private)->symtbl
-    + symnum * local_symesz;
+  raw_symbol = xcoff->symtbl + symnum * local_symesz;
 
   while (symnum < max_symnum)
     {
-
       QUIT;                    /* make this command interruptable.  */
 
       /* READ_ONE_SYMBOL (symbol, cs, symname_alloced); */
-      /* read one symbol into `cs' structure. After processing the
+      /* read one symbol into `cs' structure.  After processing the
          whole symbol table, only string table will be kept in memory,
-         symbol table and debug section of xcoff will be freed. Thus
+         symbol table and debug section of xcoff will be freed.  Thus
          we can mark symbols with names in string table as
-         `alloced'. */
+         `alloced'.  */
       {
        int ii;
 
@@ -989,7 +1074,8 @@ read_xcoff_symtab (struct partial_symtab *pst)
            if (cs->c_name[E_SYMNMLEN - 1] != '\0')
              {
                char *p;
-               p = obstack_alloc (&objfile->symbol_obstack, E_SYMNMLEN + 1);
+
+               p = obstack_alloc (&objfile->objfile_obstack, E_SYMNMLEN + 1);
                strncpy (p, cs->c_name, E_SYMNMLEN);
                p[E_SYMNMLEN] = '\0';
                cs->c_name = p;
@@ -1026,7 +1112,7 @@ read_xcoff_symtab (struct partial_symtab *pst)
          }
       }
 
-      /* if symbol name starts with ".$" or "$", ignore it. */
+      /* if symbol name starts with ".$" or "$", ignore it.  */
       if (cs->c_name[0] == '$'
          || (cs->c_name[1] == '$' && cs->c_name[0] == '.'))
        continue;
@@ -1035,8 +1121,8 @@ read_xcoff_symtab (struct partial_symtab *pst)
        {
          if (last_source_file)
            {
-             pst->symtab =
-               end_symtab (cur_src_end_addr, objfile, SECT_OFF_TEXT (objfile));
+             pst->symtab = end_symtab (cur_src_end_addr, objfile,
+                                       SECT_OFF_TEXT (objfile));
              end_stabs ();
            }
 
@@ -1044,7 +1130,7 @@ read_xcoff_symtab (struct partial_symtab *pst)
          start_symtab ("_globals_", (char *) NULL, (CORE_ADDR) 0);
          record_debugformat (debugfmt);
          cur_src_end_addr = first_object_file_end;
-         /* done with all files, everything from here on is globals */
+         /* Done with all files, everything from here on is globals.  */
        }
 
       if ((cs->c_sclass == C_EXT || cs->c_sclass == C_HIDEXT)
@@ -1083,7 +1169,7 @@ read_xcoff_symtab (struct partial_symtab *pst)
                         prefers one symtab for each source file.  In case
                         of AIX, one source file might include more than one
                         [PR] csect, and they don't have to be adjacent in
-                        terms of the space they occupy in memory. Thus, one
+                        terms of the space they occupy in memory.  Thus, one
                         single source file might get fragmented in the
                         memory and gdb's file start and end address
                         approach does not work!  GCC (and I think xlc) seem
@@ -1093,7 +1179,8 @@ read_xcoff_symtab (struct partial_symtab *pst)
                        {
                          complete_symtab (filestring, file_start_addr);
                          cur_src_end_addr = file_end_addr;
-                         end_symtab (file_end_addr, objfile, SECT_OFF_TEXT (objfile));
+                         end_symtab (file_end_addr, objfile,
+                                     SECT_OFF_TEXT (objfile));
                          end_stabs ();
                          start_stabs ();
                          /* Give all csects for this source file the same
@@ -1103,7 +1190,7 @@ read_xcoff_symtab (struct partial_symtab *pst)
                        }
 
                      /* If this is the very first csect seen,
-                        basically `__start'. */
+                        basically `__start'.  */
                      if (just_started)
                        {
                          first_object_file_end
@@ -1116,13 +1203,8 @@ read_xcoff_symtab (struct partial_symtab *pst)
                                                SECT_OFF_TEXT (objfile));
                      file_end_addr = file_start_addr + CSECT_LEN (&main_aux);
 
-                     if (cs->c_name && (cs->c_name[0] == '.'
-                                        || cs->c_name[0] == '@'))
-                       {
-                         last_csect_name = cs->c_name;
-                         last_csect_val = cs->c_value;
-                         last_csect_sec = secnum_to_section (cs->c_secnum, objfile);
-                       }
+                     if (cs->c_name && (cs->c_name[0] == '.' || cs->c_name[0] == '@'))
+                       last_csect_name = cs->c_name;
                    }
                    continue;
 
@@ -1150,19 +1232,19 @@ read_xcoff_symtab (struct partial_symtab *pst)
              switch (CSECT_SCLAS (&main_aux))
                {
                case XMC_PR:
-                 /* a function entry point. */
+                 /* a function entry point.  */
                function_entry_point:
 
                  fcn_start_addr = cs->c_value;
 
                  /* save the function header info, which will be used
-                    when `.bf' is seen. */
+                    when `.bf' is seen.  */
                  fcn_cs_saved = *cs;
                  fcn_aux_saved = main_aux;
                  continue;
 
                case XMC_GL:
-                 /* shared library function trampoline code entry point. */
+                 /* shared library function trampoline code entry point.  */
                  continue;
 
                case XMC_DS:
@@ -1174,7 +1256,7 @@ read_xcoff_symtab (struct partial_symtab *pst)
                  /* xlc puts each variable in a separate csect, so we get
                     an XTY_SD for each variable.  But gcc puts several
                     variables in a csect, so that each variable only gets
-                    an XTY_LD. This will typically be XMC_RW; I suspect
+                    an XTY_LD.  This will typically be XMC_RW; I suspect
                     XMC_RO and XMC_BS might be possible too.
                     These variables are put in the minimal symbol table
                     only.  */
@@ -1203,16 +1285,15 @@ read_xcoff_symtab (struct partial_symtab *pst)
 
       switch (cs->c_sclass)
        {
-
        case C_FILE:
 
          /* c_value field contains symnum of next .file entry in table
-            or symnum of first global after last .file. */
+            or symnum of first global after last .file.  */
 
          next_file_symnum = cs->c_value;
 
          /* Complete symbol table for last object file containing
-            debugging information. */
+            debugging information.  */
 
          /* Whether or not there was a csect in the previous file, we
             have to call `end_stabs' and `start_stabs' to reset
@@ -1223,11 +1304,12 @@ read_xcoff_symtab (struct partial_symtab *pst)
          end_symtab (file_end_addr, objfile, SECT_OFF_TEXT (objfile));
          end_stabs ();
 
-         /* XCOFF, according to the AIX 3.2 documentation, puts the filename
-            in cs->c_name.  But xlc 1.3.0.2 has decided to do things the
-            standard COFF way and put it in the auxent.  We use the auxent if
-            the symbol is ".file" and an auxent exists, otherwise use the symbol
-            itself.  Simple enough.  */
+         /* XCOFF, according to the AIX 3.2 documentation, puts the
+            filename in cs->c_name.  But xlc 1.3.0.2 has decided to
+            do things the standard COFF way and put it in the auxent.
+            We use the auxent if the symbol is ".file" and an auxent
+            exists, otherwise use the symbol itself.  Simple
+            enough.  */
          if (!strcmp (cs->c_name, ".file") && cs->c_naux > 0)
            {
              bfd_coff_swap_aux_in (abfd, raw_auxptr, cs->c_type, cs->c_sclass,
@@ -1242,8 +1324,9 @@ read_xcoff_symtab (struct partial_symtab *pst)
          record_debugformat (debugfmt);
          last_csect_name = 0;
 
-         /* reset file start and end addresses. A compilation unit with no text
-            (only data) should have zero file boundaries. */
+         /* reset file start and end addresses.  A compilation unit
+            with no text (only data) should have zero file
+            boundaries.  */
          file_start_addr = file_end_addr = 0;
          break;
 
@@ -1252,10 +1335,11 @@ read_xcoff_symtab (struct partial_symtab *pst)
          break;
 
        case C_FCN:
-         if (STREQ (cs->c_name, ".bf"))
+         if (strcmp (cs->c_name, ".bf") == 0)
            {
              CORE_ADDR off = ANOFFSET (objfile->section_offsets,
                                        SECT_OFF_TEXT (objfile));
+
              bfd_coff_swap_aux_in (abfd, raw_auxptr, cs->c_type, cs->c_sclass,
                                    0, cs->c_naux, &main_aux);
 
@@ -1269,9 +1353,8 @@ read_xcoff_symtab (struct partial_symtab *pst)
              if (new->name != NULL)
                SYMBOL_SECTION (new->name) = SECT_OFF_TEXT (objfile);
            }
-         else if (STREQ (cs->c_name, ".ef"))
+         else if (strcmp (cs->c_name, ".ef") == 0)
            {
-
              bfd_coff_swap_aux_in (abfd, raw_auxptr, cs->c_type, cs->c_sclass,
                                    0, cs->c_naux, &main_aux);
 
@@ -1281,8 +1364,8 @@ read_xcoff_symtab (struct partial_symtab *pst)
                 contains number of lines to '}' */
 
              if (context_stack_depth <= 0)
-               {               /* We attempted to pop an empty context stack */
-                 complain (&ef_complaint, cs->c_symnum);
+               {       /* We attempted to pop an empty context stack.  */
+                 ef_complaint (cs->c_symnum);
                  within_function = 0;
                  break;
                }
@@ -1290,7 +1373,7 @@ read_xcoff_symtab (struct partial_symtab *pst)
              /* Stack must be empty now.  */
              if (context_stack_depth > 0 || new == NULL)
                {
-                 complain (&ef_complaint, cs->c_symnum);
+                 ef_complaint (cs->c_symnum);
                  within_function = 0;
                  break;
                }
@@ -1332,9 +1415,9 @@ read_xcoff_symtab (struct partial_symtab *pst)
        case C_UNTAG:
        case C_ENTAG:
          {
-           static struct complaint msg =
-           {"Unrecognized storage class %d.", 0, 0};
-           complain (&msg, cs->c_sclass);
+           complaint (&symfile_complaints,
+                      _("Unrecognized storage class %d."),
+                      cs->c_sclass);
          }
          break;
 
@@ -1350,9 +1433,9 @@ read_xcoff_symtab (struct partial_symtab *pst)
        case C_BINCL:
          /* beginning of include file */
          /* In xlc output, C_BINCL/C_EINCL pair doesn't show up in sorted
-            order. Thus, when wee see them, we might not know enough info
-            to process them. Thus, we'll be saving them into a table 
-            (inclTable) and postpone their processing. */
+            order.  Thus, when wee see them, we might not know enough info
+            to process them.  Thus, we'll be saving them into a table 
+            (inclTable) and postpone their processing.  */
 
          record_include_begin (cs);
          break;
@@ -1364,7 +1447,7 @@ read_xcoff_symtab (struct partial_symtab *pst)
          break;
 
        case C_BLOCK:
-         if (STREQ (cs->c_name, ".bb"))
+         if (strcmp (cs->c_name, ".bb") == 0)
            {
              depth++;
              new = push_context (depth,
@@ -1372,17 +1455,17 @@ read_xcoff_symtab (struct partial_symtab *pst)
                                   + ANOFFSET (objfile->section_offsets,
                                               SECT_OFF_TEXT (objfile))));
            }
-         else if (STREQ (cs->c_name, ".eb"))
+         else if (strcmp (cs->c_name, ".eb") == 0)
            {
              if (context_stack_depth <= 0)
-               {               /* We attempted to pop an empty context stack */
-                 complain (&eb_complaint, cs->c_symnum);
+               {       /* We attempted to pop an empty context stack.  */
+                 eb_complaint (cs->c_symnum);
                  break;
                }
              new = pop_context ();
              if (depth-- != new->depth)
                {
-                 complain (&eb_complaint, cs->c_symnum);
+                 eb_complaint (cs->c_symnum);
                  break;
                }
              if (local_symbols && context_stack_depth > 0)
@@ -1424,24 +1507,22 @@ read_xcoff_symtab (struct partial_symtab *pst)
 
 #define        SYMBOL_DUP(SYMBOL1, SYMBOL2)    \
   (SYMBOL2) = (struct symbol *)                \
-       obstack_alloc (&objfile->symbol_obstack, sizeof (struct symbol)); \
+       obstack_alloc (&objfile->objfile_obstack, sizeof (struct symbol)); \
   *(SYMBOL2) = *(SYMBOL1);
 
 
 #define        SYMNAME_ALLOC(NAME, ALLOCED)    \
-  (ALLOCED) ? (NAME) : obsavestring ((NAME), strlen (NAME), &objfile->symbol_obstack);
-
+  ((ALLOCED) ? (NAME) : obsavestring ((NAME), strlen (NAME), \
+                                     &objfile->objfile_obstack))
 
-static struct type *func_symbol_type;
-static struct type *var_symbol_type;
 
-/* process one xcoff symbol. */
+/* process one xcoff symbol.  */
 
 static struct symbol *
-process_xcoff_symbol (register struct coff_symbol *cs, struct objfile *objfile)
+process_xcoff_symbol (struct coff_symbol *cs, struct objfile *objfile)
 {
   struct symbol onesymbol;
-  register struct symbol *sym = &onesymbol;
+  struct symbol *sym = &onesymbol;
   struct symbol *sym2 = NULL;
   char *name, *pp;
 
@@ -1469,7 +1550,7 @@ process_xcoff_symbol (register struct coff_symbol *cs, struct objfile *objfile)
 
   /* default assumptions */
   SYMBOL_VALUE_ADDRESS (sym) = cs->c_value + off;
-  SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
+  SYMBOL_DOMAIN (sym) = VAR_DOMAIN;
   SYMBOL_SECTION (sym) = secnum_to_section (cs->c_secnum, objfile);
 
   if (ISFCN (cs->c_type))
@@ -1478,8 +1559,8 @@ process_xcoff_symbol (register struct coff_symbol *cs, struct objfile *objfile)
          will be patched with the type from its stab entry later on in
          patch_block_stabs (), unless the file was compiled without -g.  */
 
-      SYMBOL_NAME (sym) = SYMNAME_ALLOC (name, symname_alloced);
-      SYMBOL_TYPE (sym) = func_symbol_type;
+      SYMBOL_SET_LINKAGE_NAME (sym, SYMNAME_ALLOC (name, symname_alloced));
+      SYMBOL_TYPE (sym) = objfile_type (objfile)->nodebug_text_symbol;
 
       SYMBOL_CLASS (sym) = LOC_BLOCK;
       SYMBOL_DUP (sym, sym2);
@@ -1491,8 +1572,8 @@ process_xcoff_symbol (register struct coff_symbol *cs, struct objfile *objfile)
     }
   else
     {
-      /* In case we can't figure out the type, provide default. */
-      SYMBOL_TYPE (sym) = var_symbol_type;
+      /* In case we can't figure out the type, provide default.  */
+      SYMBOL_TYPE (sym) = objfile_type (objfile)->nodebug_data_symbol;
 
       switch (cs->c_sclass)
        {
@@ -1520,7 +1601,8 @@ process_xcoff_symbol (register struct coff_symbol *cs, struct objfile *objfile)
          break;
 
        default:
-         complain (&storclass_complaint, cs->c_sclass);
+         complaint (&symfile_complaints, _("Unexpected storage class: %d"),
+                    cs->c_sclass);
          /* FALLTHROUGH */
 
        case C_DECL:
@@ -1549,7 +1631,11 @@ process_xcoff_symbol (register struct coff_symbol *cs, struct objfile *objfile)
             where we need to, which is not necessarily super-clean,
             but seems workable enough.  */
 
-         if (*name == ':' || (pp = (char *) strchr (name, ':')) == NULL)
+         if (*name == ':')
+           return NULL;
+
+         pp = strchr (name, ':');
+         if (pp == NULL)
            return NULL;
 
          ++pp;
@@ -1580,9 +1666,8 @@ coff_getfilename (union internal_auxent *aux_entry, struct objfile *objfile)
   static char buffer[BUFSIZ];
 
   if (aux_entry->x_file.x_n.x_zeroes == 0)
-    strcpy (buffer,
-           ((struct coff_symfile_info *) objfile->sym_private)->strtbl
-           + aux_entry->x_file.x_n.x_offset);
+    strcpy (buffer, (XCOFF_DATA (objfile)->strtbl
+                    + aux_entry->x_file.x_n.x_offset));
   else
     {
       strncpy (buffer, aux_entry->x_file.x_fname, FILNMLEN);
@@ -1595,22 +1680,18 @@ coff_getfilename (union internal_auxent *aux_entry, struct objfile *objfile)
 static void
 read_symbol (struct internal_syment *symbol, int symno)
 {
-  int nsyms =
-  ((struct coff_symfile_info *) this_symtab_psymtab->objfile->sym_private)
-  ->symtbl_num_syms;
-  char *stbl =
-  ((struct coff_symfile_info *) this_symtab_psymtab->objfile->sym_private)
-  ->symtbl;
+  struct coff_symfile_info *xcoff = XCOFF_DATA (this_symtab_objfile);
+  int nsyms = xcoff->symtbl_num_syms;
+  char *stbl = xcoff->symtbl;
+
   if (symno < 0 || symno >= nsyms)
     {
-      static struct complaint msg =
-      {"Invalid symbol offset", 0, 0};
-      complain (&msg);
+      complaint (&symfile_complaints, _("Invalid symbol offset"));
       symbol->n_value = 0;
       symbol->n_scnum = -1;
       return;
     }
-  bfd_coff_swap_sym_in (this_symtab_psymtab->objfile->obfd,
+  bfd_coff_swap_sym_in (this_symtab_objfile->obfd,
                        stbl + (symno * local_symesz),
                        symbol);
 }
@@ -1633,11 +1714,10 @@ read_symbol_nvalue (int symno)
 static int
 read_symbol_lineno (int symno)
 {
-  struct objfile *objfile = this_symtab_psymtab->objfile;
-  boolean xcoff64 = bfd_xcoff_is_xcoff64 (objfile->obfd);
+  struct objfile *objfile = this_symtab_objfile;
+  int xcoff64 = bfd_xcoff_is_xcoff64 (objfile->obfd);
 
-  struct coff_symfile_info *info =
-    (struct coff_symfile_info *)objfile->sym_private;
+  struct coff_symfile_info *info = XCOFF_DATA (objfile);
   int nsyms = info->symtbl_num_syms;
   char *stbl = info->symtbl;
   char *strtbl = info->strtbl;
@@ -1647,7 +1727,7 @@ read_symbol_lineno (int symno)
 
   if (symno < 0)
     {
-      complain (&bf_notfound_complaint);
+      bf_notfound_complaint ();
       return 0;
     }
 
@@ -1674,17 +1754,18 @@ read_symbol_lineno (int symno)
       if (symbol->n_sclass == C_FCN)
        {
          char *name = xcoff64 ? strtbl + symbol->n_offset : symbol->n_name;
-         if (STREQ (name, ".bf"))
+
+         if (strcmp (name, ".bf") == 0)
            goto gotit;
        }
       symno += symbol->n_numaux + 1;
     }
 
-  complain (&bf_notfound_complaint);
+  bf_notfound_complaint ();
   return 0;
 
 gotit:
-  /* take aux entry and return its lineno */
+  /* Take aux entry and return its lineno.  */
   symno++;
   bfd_coff_swap_aux_in (objfile->obfd, stbl + symno * local_symesz,
                        symbol->n_type, symbol->n_sclass,
@@ -1693,14 +1774,14 @@ gotit:
   return main_aux->x_sym.x_misc.x_lnsz.x_lnno;
 }
 
-/* Support for line number handling */
+/* Support for line number handling */
 
 /* This function is called for every section; it finds the outer limits
  * of the line table (minimum and maximum file offset) so that the
  * mainline code can read the whole thing for efficiency.
  */
 static void
-find_linenos (bfd *abfd, sec_ptr asect, PTR vpinfo)
+find_linenos (struct bfd *abfd, struct bfd_section *asect, void *vpinfo)
 {
   struct coff_symfile_info *info;
   int size, count;
@@ -1708,7 +1789,7 @@ find_linenos (bfd *abfd, sec_ptr asect, PTR vpinfo)
 
   count = asect->lineno_count;
 
-  if (!STREQ (asect->name, ".text") || count == 0)
+  if (strcmp (asect->name, ".text") != 0 || count == 0)
     return;
 
   size = count * coff_data (abfd)->local_linesz;
@@ -1723,10 +1804,8 @@ find_linenos (bfd *abfd, sec_ptr asect, PTR vpinfo)
     info->max_lineno_offset = maxoff;
 }
 \f
-static void xcoff_psymtab_to_symtab_1 (struct partial_symtab *);
-
 static void
-xcoff_psymtab_to_symtab_1 (struct partial_symtab *pst)
+xcoff_psymtab_to_symtab_1 (struct objfile *objfile, struct partial_symtab *pst)
 {
   struct cleanup *old_chain;
   int i;
@@ -1742,7 +1821,7 @@ xcoff_psymtab_to_symtab_1 (struct partial_symtab *pst)
       return;
     }
 
-  /* Read in all partial symtabs on which this one is dependent */
+  /* Read in all partial symtabs on which this one is dependent */
   for (i = 0; i < pst->number_of_dependencies; i++)
     if (!pst->dependencies[i]->readin)
       {
@@ -1757,7 +1836,7 @@ xcoff_psymtab_to_symtab_1 (struct partial_symtab *pst)
            wrap_here ("");     /* Flush output */
            gdb_flush (gdb_stdout);
          }
-       xcoff_psymtab_to_symtab_1 (pst->dependencies[i]);
+       xcoff_psymtab_to_symtab_1 (objfile, pst->dependencies[i]);
       }
 
   if (((struct symloc *) pst->read_symtab_private)->numsyms != 0)
@@ -1767,8 +1846,7 @@ xcoff_psymtab_to_symtab_1 (struct partial_symtab *pst)
       buildsym_init ();
       old_chain = make_cleanup (really_free_pendings, 0);
 
-      read_xcoff_symtab (pst);
-      sort_symtab_syms (pst->symtab);
+      read_xcoff_symtab (objfile, pst);
 
       do_cleanups (old_chain);
     }
@@ -1776,16 +1854,12 @@ xcoff_psymtab_to_symtab_1 (struct partial_symtab *pst)
   pst->readin = 1;
 }
 
-static void xcoff_psymtab_to_symtab (struct partial_symtab *);
-
 /* Read in all of the symbols for a given psymtab for real.
    Be verbose about it if the user wants that.  */
 
 static void
-xcoff_psymtab_to_symtab (struct partial_symtab *pst)
+xcoff_psymtab_to_symtab (struct objfile *objfile, struct partial_symtab *pst)
 {
-  bfd *sym_bfd;
-
   if (!pst)
     return;
 
@@ -1808,15 +1882,13 @@ xcoff_psymtab_to_symtab (struct partial_symtab *pst)
          gdb_flush (gdb_stdout);
        }
 
-      sym_bfd = pst->objfile->obfd;
-
       next_symbol_text_func = xcoff_next_symbol_text;
 
-      xcoff_psymtab_to_symtab_1 (pst);
+      xcoff_psymtab_to_symtab_1 (objfile, pst);
 
       /* Match with global symbols.  This only needs to be done once,
          after all of the symtabs and dependencies have been read in.   */
-      scan_file_globals (pst->objfile);
+      scan_file_globals (objfile);
 
       /* Finish up the debug error message.  */
       if (info_verbose)
@@ -1840,31 +1912,26 @@ xcoff_new_init (struct objfile *objfile)
 static void
 xcoff_symfile_init (struct objfile *objfile)
 {
-  /* Allocate struct to keep track of the symfile */
-  objfile->sym_private = xmmalloc (objfile->md,
-                                  sizeof (struct coff_symfile_info));
+  struct coff_symfile_info *xcoff;
+
+  /* Allocate struct to keep track of the symfile.  */
+  xcoff = XNEW (struct coff_symfile_info);
+  set_objfile_data (objfile, xcoff_objfile_data_key, xcoff);
 
   /* XCOFF objects may be reordered, so set OBJF_REORDERED.  If we
      find this causes a significant slowdown in gdb then we could
      set it in the debug symbol readers only when necessary.  */
   objfile->flags |= OBJF_REORDERED;
-
-  init_entry_point_info (objfile);
 }
 
 /* Perform any local cleanups required when we are done with a particular
    objfile.  I.E, we are in the process of discarding all symbol information
    for an objfile, freeing up all memory held for it, and unlinking the
-   objfile struct from the global list of known objfiles. */
+   objfile struct from the global list of known objfiles.  */
 
 static void
 xcoff_symfile_finish (struct objfile *objfile)
 {
-  if (objfile->sym_private != NULL)
-    {
-      xmfree (objfile->md, objfile->sym_private);
-    }
-
   /* Start with a fresh include table for the next objfile.  */
   if (inclTable)
     {
@@ -1872,6 +1939,8 @@ xcoff_symfile_finish (struct objfile *objfile)
       inclTable = NULL;
     }
   inclIndx = inclLength = inclDepth = 0;
+
+  dwarf2_free_objfile (objfile);
 }
 
 
@@ -1882,11 +1951,12 @@ init_stringtab (bfd *abfd, file_ptr offset, struct objfile *objfile)
   int val;
   unsigned char lengthbuf[4];
   char *strtbl;
+  struct coff_symfile_info *xcoff = XCOFF_DATA (objfile);
 
-  ((struct coff_symfile_info *) objfile->sym_private)->strtbl = NULL;
+  xcoff->strtbl = NULL;
 
   if (bfd_seek (abfd, offset, SEEK_SET) < 0)
-    error ("cannot seek to string table in %s: %s",
+    error (_("cannot seek to string table in %s: %s"),
           bfd_get_filename (abfd), bfd_errmsg (bfd_get_error ()));
 
   val = bfd_bread ((char *) lengthbuf, sizeof lengthbuf, abfd);
@@ -1898,11 +1968,11 @@ init_stringtab (bfd *abfd, file_ptr offset, struct objfile *objfile)
   if (val != sizeof lengthbuf || length < sizeof lengthbuf)
     return;
 
-  /* Allocate string table from symbol_obstack. We will need this table
-     as long as we have its symbol table around. */
+  /* Allocate string table from objfile_obstack.  We will need this table
+     as long as we have its symbol table around.  */
 
-  strtbl = (char *) obstack_alloc (&objfile->symbol_obstack, length);
-  ((struct coff_symfile_info *) objfile->sym_private)->strtbl = strtbl;
+  strtbl = (char *) obstack_alloc (&objfile->objfile_obstack, length);
+  xcoff->strtbl = strtbl;
 
   /* Copy length buffer, the first byte is usually zero and is
      used for stabs with a name length of zero.  */
@@ -1913,10 +1983,11 @@ init_stringtab (bfd *abfd, file_ptr offset, struct objfile *objfile)
   val = bfd_bread (strtbl + sizeof lengthbuf, length - sizeof lengthbuf, abfd);
 
   if (val != length - sizeof lengthbuf)
-    error ("cannot read string table from %s: %s",
+    error (_("cannot read string table from %s: %s"),
           bfd_get_filename (abfd), bfd_errmsg (bfd_get_error ()));
   if (strtbl[length - 1] != '\0')
-    error ("bad symbol file: string table does not end with null character");
+    error (_("bad symbol file: string table "
+            "does not end with null character"));
 
   return;
 }
@@ -1926,45 +1997,38 @@ init_stringtab (bfd *abfd, file_ptr offset, struct objfile *objfile)
    for the psymtab.  */
 static unsigned int first_fun_line_offset;
 
-static struct partial_symtab *xcoff_start_psymtab
-  (struct objfile *, char *, int,
-   struct partial_symbol **, struct partial_symbol **);
-
 /* Allocate and partially fill a partial symtab.  It will be
    completely filled at the end of the symbol list.
 
    SYMFILE_NAME is the name of the symbol-file we are reading from, and ADDR
    is the address relative to which its symbols are (incremental) or 0
-   (normal). */
+   (normal).  */
 
 static struct partial_symtab *
-xcoff_start_psymtab (struct objfile *objfile, char *filename, int first_symnum,
+xcoff_start_psymtab (struct objfile *objfile,
+                    const char *filename, int first_symnum,
                     struct partial_symbol **global_syms,
                     struct partial_symbol **static_syms)
 {
   struct partial_symtab *result =
-  start_psymtab_common (objfile, objfile->section_offsets,
-                       filename,
-                       /* We fill in textlow later.  */
-                       0,
-                       global_syms, static_syms);
-
-  result->read_symtab_private = (char *)
-    obstack_alloc (&objfile->psymbol_obstack, sizeof (struct symloc));
+    start_psymtab_common (objfile, objfile->section_offsets,
+                         filename,
+                         /* We fill in textlow later.  */
+                         0,
+                         global_syms, static_syms);
+
+  result->read_symtab_private = obstack_alloc (&objfile->objfile_obstack,
+                                              sizeof (struct symloc));
   ((struct symloc *) result->read_symtab_private)->first_symnum = first_symnum;
   result->read_symtab = xcoff_psymtab_to_symtab;
 
-  /* Deduce the source language from the filename for this psymtab. */
+  /* Deduce the source language from the filename for this psymtab.  */
   psymtab_language = deduce_language_from_filename (filename);
 
   return result;
 }
 
-static struct partial_symtab *xcoff_end_psymtab
-  (struct partial_symtab *, char **, int, int,
-   struct partial_symtab **, int, int);
-
-/* Close off the current usage of PST.  
+/* Close off the current usage of PST.
    Returns PST, or NULL if the partial symtab was empty and thrown away.
 
    CAPPING_SYMBOL_NUMBER is the end of pst (exclusive).
@@ -1973,13 +2037,13 @@ static struct partial_symtab *xcoff_end_psymtab
    are the information for includes and dependencies.  */
 
 static struct partial_symtab *
-xcoff_end_psymtab (struct partial_symtab *pst, char **include_list,
-                  int num_includes, int capping_symbol_number,
+xcoff_end_psymtab (struct objfile *objfile, struct partial_symtab *pst,
+                  const char **include_list, int num_includes,
+                  int capping_symbol_number,
                   struct partial_symtab **dependency_list,
                   int number_dependencies, int textlow_not_set)
 {
   int i;
-  struct objfile *objfile = pst->objfile;
 
   if (capping_symbol_number != -1)
     ((struct symloc *) pst->read_symtab_private)->numsyms =
@@ -1988,16 +2052,16 @@ xcoff_end_psymtab (struct partial_symtab *pst, char **include_list,
   ((struct symloc *) pst->read_symtab_private)->lineno_off =
     first_fun_line_offset;
   first_fun_line_offset = 0;
-  pst->n_global_syms =
-    objfile->global_psymbols.next - (objfile->global_psymbols.list + pst->globals_offset);
-  pst->n_static_syms =
-    objfile->static_psymbols.next - (objfile->static_psymbols.list + pst->statics_offset);
+  pst->n_global_syms = objfile->global_psymbols.next
+    - (objfile->global_psymbols.list + pst->globals_offset);
+  pst->n_static_syms = objfile->static_psymbols.next
+    - (objfile->static_psymbols.list + pst->statics_offset);
 
   pst->number_of_dependencies = number_dependencies;
   if (number_dependencies)
     {
       pst->dependencies = (struct partial_symtab **)
-       obstack_alloc (&objfile->psymbol_obstack,
+       obstack_alloc (&objfile->objfile_obstack,
                    number_dependencies * sizeof (struct partial_symtab *));
       memcpy (pst->dependencies, dependency_list,
              number_dependencies * sizeof (struct partial_symtab *));
@@ -2008,12 +2072,11 @@ xcoff_end_psymtab (struct partial_symtab *pst, char **include_list,
   for (i = 0; i < num_includes; i++)
     {
       struct partial_symtab *subpst =
-      allocate_psymtab (include_list[i], objfile);
+       allocate_psymtab (include_list[i], objfile);
 
       subpst->section_offsets = pst->section_offsets;
-      subpst->read_symtab_private =
-       (char *) obstack_alloc (&objfile->psymbol_obstack,
-                               sizeof (struct symloc));
+      subpst->read_symtab_private = obstack_alloc (&objfile->objfile_obstack,
+                                                  sizeof (struct symloc));
       ((struct symloc *) subpst->read_symtab_private)->first_symnum = 0;
       ((struct symloc *) subpst->read_symtab_private)->numsyms = 0;
       subpst->textlow = 0;
@@ -2022,7 +2085,7 @@ xcoff_end_psymtab (struct partial_symtab *pst, char **include_list,
       /* We could save slight bits of space by only making one of these,
          shared by the entire set of include files.  FIXME-someday.  */
       subpst->dependencies = (struct partial_symtab **)
-       obstack_alloc (&objfile->psymbol_obstack,
+       obstack_alloc (&objfile->objfile_obstack,
                       sizeof (struct partial_symtab *));
       subpst->dependencies[0] = pst;
       subpst->number_of_dependencies = 1;
@@ -2037,12 +2100,7 @@ xcoff_end_psymtab (struct partial_symtab *pst, char **include_list,
       subpst->read_symtab = pst->read_symtab;
     }
 
-  sort_pst_symbols (pst);
-
-  /* If there is already a psymtab or symtab for a file of this name,
-     remove it.  (If there is a symtab, more drastic things also
-     happen.)  This happens in VxWorks.  */
-  free_named_symtabs (pst->filename);
+  sort_pst_symbols (objfile, pst);
 
   if (num_includes == 0
       && number_dependencies == 0
@@ -2054,7 +2112,7 @@ xcoff_end_psymtab (struct partial_symtab *pst, char **include_list,
       /* Empty psymtabs happen as a result of header files which don't have
          any symbols in them.  There can be a lot of them.  */
 
-      discard_psymtab (pst);
+      discard_psymtab (objfile, pst);
 
       /* Indicate that psymtab was thrown away.  */
       pst = (struct partial_symtab *) NULL;
@@ -2062,17 +2120,13 @@ xcoff_end_psymtab (struct partial_symtab *pst, char **include_list,
   return pst;
 }
 
-static void swap_sym (struct internal_syment *,
-                     union internal_auxent *, char **, char **,
-                     unsigned int *, struct objfile *);
-
 /* Swap raw symbol at *RAW and put the name in *NAME, the symbol in
    *SYMBOL, the first auxent in *AUX.  Advance *RAW and *SYMNUMP over
    the symbol and its auxents.  */
 
 static void
 swap_sym (struct internal_syment *symbol, union internal_auxent *aux,
-         char **name, char **raw, unsigned int *symnump,
+         const char **name, char **raw, unsigned int *symnump,
          struct objfile *objfile)
 {
   bfd_coff_swap_sym_in (objfile->obfd, *raw, symbol);
@@ -2085,7 +2139,8 @@ swap_sym (struct internal_syment *symbol, union internal_auxent *aux,
          /* FIXME: wastes memory for symbols which we don't end up putting
             into the minimal symbols.  */
          char *p;
-         p = obstack_alloc (&objfile->psymbol_obstack, E_SYMNMLEN + 1);
+
+         p = obstack_alloc (&objfile->objfile_obstack, E_SYMNMLEN + 1);
          strncpy (p, symbol->n_name, E_SYMNMLEN);
          p[E_SYMNMLEN] = '\0';
          *name = p;
@@ -2097,13 +2152,11 @@ swap_sym (struct internal_syment *symbol, union internal_auxent *aux,
     }
   else if (symbol->n_sclass & 0x80)
     {
-      *name = ((struct coff_symfile_info *) objfile->sym_private)->debugsec
-       + symbol->n_offset;
+      *name = XCOFF_DATA (objfile)->debugsec + symbol->n_offset;
     }
   else
     {
-      *name = ((struct coff_symfile_info *) objfile->sym_private)->strtbl
-       + symbol->n_offset;
+      *name = XCOFF_DATA (objfile)->strtbl + symbol->n_offset;
     }
   ++*symnump;
   *raw += coff_data (objfile->obfd)->local_symesz;
@@ -2117,13 +2170,23 @@ swap_sym (struct internal_syment *symbol, union internal_auxent *aux,
     }
 }
 
+static void
+function_outside_compilation_unit_complaint (const char *arg1)
+{
+  complaint (&symfile_complaints,
+            _("function `%s' appears to be defined "
+              "outside of all compilation units"),
+            arg1);
+}
+
 static void
 scan_xcoff_symtab (struct objfile *objfile)
 {
-  CORE_ADDR toc_offset = 0;    /* toc offset value in data section. */
-  char *filestring = NULL;
+  struct gdbarch *gdbarch = get_objfile_arch (objfile);
+  CORE_ADDR toc_offset = 0;    /* toc offset value in data section.  */
+  const char *filestring = NULL;
 
-  char *namestring;
+  const char *namestring;
   int past_first_source_file = 0;
   bfd *abfd;
   asection *bfd_sect;
@@ -2132,12 +2195,12 @@ scan_xcoff_symtab (struct objfile *objfile)
   /* Current partial symtab */
   struct partial_symtab *pst;
 
-  /* List of current psymtab's include files */
-  char **psymtab_include_list;
+  /* List of current psymtab's include files */
+  const char **psymtab_include_list;
   int includes_allocated;
   int includes_used;
 
-  /* Index within current psymtab dependency list */
+  /* Index within current psymtab dependency list */
   struct partial_symtab **dependency_list;
   int dependencies_used, dependencies_allocated;
 
@@ -2146,18 +2209,18 @@ scan_xcoff_symtab (struct objfile *objfile)
   union internal_auxent main_aux[5];
   unsigned int ssymnum;
 
-  char *last_csect_name = NULL;        /* last seen csect's name and value */
+  const char *last_csect_name = NULL; /* Last seen csect's name and value.  */
   CORE_ADDR last_csect_val = 0;
   int last_csect_sec = 0;
-  int misc_func_recorded = 0;  /* true if any misc. function */
+  int misc_func_recorded = 0;  /* true if any misc. function */
   int textlow_not_set = 1;
 
   pst = (struct partial_symtab *) 0;
 
   includes_allocated = 30;
   includes_used = 0;
-  psymtab_include_list = (char **) alloca (includes_allocated *
-                                          sizeof (char *));
+  psymtab_include_list = (const char **) alloca (includes_allocated *
+                                                sizeof (const char *));
 
   dependencies_allocated = 30;
   dependencies_used = 0;
@@ -2168,9 +2231,10 @@ scan_xcoff_symtab (struct objfile *objfile)
   last_source_file = NULL;
 
   abfd = objfile->obfd;
+  next_symbol_text_func = xcoff_next_symbol_text;
 
-  sraw_symbol = ((struct coff_symfile_info *) objfile->sym_private)->symtbl;
-  nsyms = ((struct coff_symfile_info *) objfile->sym_private)->symtbl_num_syms;
+  sraw_symbol = XCOFF_DATA (objfile)->symtbl;
+  nsyms = XCOFF_DATA (objfile)->symtbl_num_syms;
   ssymnum = 0;
   while (ssymnum < nsyms)
     {
@@ -2220,16 +2284,16 @@ scan_xcoff_symtab (struct objfile *objfile)
                    if (last_csect_name)
                      {
                        /* If no misc. function recorded in the last
-                          seen csect, enter it as a function. This
+                          seen csect, enter it as a function.  This
                           will take care of functions like strcmp()
                           compiled by xlc.  */
 
                        if (!misc_func_recorded)
                          {
-                           RECORD_MINIMAL_SYMBOL
+                           record_minimal_symbol
                              (last_csect_name, last_csect_val,
-                              mst_text, last_csect_sec,
-                              objfile);
+                              mst_text, last_csect_sec, objfile);
+                           misc_func_recorded = 1;
                          }
 
                        if (pst != NULL)
@@ -2238,8 +2302,8 @@ scan_xcoff_symtab (struct objfile *objfile)
                               each program csect, because their text
                               sections need not be adjacent.  */
                            xcoff_end_psymtab
-                             (pst, psymtab_include_list, includes_used,
-                              symnum_before, dependency_list,
+                             (objfile, pst, psymtab_include_list,
+                              includes_used, symnum_before, dependency_list,
                               dependencies_used, textlow_not_set);
                            includes_used = 0;
                            dependencies_used = 0;
@@ -2261,13 +2325,13 @@ scan_xcoff_symtab (struct objfile *objfile)
                      {
                        last_csect_name = namestring;
                        last_csect_val = symbol.n_value;
-                       last_csect_sec =
-                         secnum_to_section (symbol.n_scnum, objfile);
+                       last_csect_sec = symbol.n_scnum;
                      }
                    if (pst != NULL)
                      {
                        CORE_ADDR highval =
-                       symbol.n_value + csect_aux.x_csect.x_scnlen.l;
+                         symbol.n_value + csect_aux.x_csect.x_scnlen.l;
+
                        if (highval > pst->texthigh)
                          pst->texthigh = highval;
                        if (pst->textlow == 0 || symbol.n_value < pst->textlow)
@@ -2284,16 +2348,18 @@ scan_xcoff_symtab (struct objfile *objfile)
                      prim_record_minimal_symbol_and_info
                        (namestring, symbol.n_value,
                         sclass == C_HIDEXT ? mst_file_data : mst_data,
-                        NULL, secnum_to_section (symbol.n_scnum, objfile),
-                        NULL, objfile);
+                        secnum_to_section (symbol.n_scnum, objfile),
+                        secnum_to_bfd_section (symbol.n_scnum, objfile),
+                        objfile);
                    break;
 
                  case XMC_TC0:
                    if (toc_offset)
-                     warning ("More than one XMC_TC0 symbol found.");
+                     warning (_("More than one XMC_TC0 symbol found."));
                    toc_offset = symbol.n_value;
 
-                   /* Make TOC offset relative to start address of section.  */
+                   /* Make TOC offset relative to start address of
+                      section.  */
                    bfd_sect = secnum_to_bfd_section (symbol.n_scnum, objfile);
                    if (bfd_sect)
                      toc_offset -= bfd_section_vma (objfile->obfd, bfd_sect);
@@ -2318,26 +2384,27 @@ scan_xcoff_symtab (struct objfile *objfile)
                    if (first_fun_line_offset == 0 && symbol.n_numaux > 1)
                      first_fun_line_offset =
                        main_aux[0].x_sym.x_fcnary.x_fcn.x_lnnoptr;
-                   RECORD_MINIMAL_SYMBOL
-                     (namestring, symbol.n_value,
-                      sclass == C_HIDEXT ? mst_file_text : mst_text,
-                      secnum_to_section (symbol.n_scnum, objfile),
-                      objfile);
+                     {
+                       record_minimal_symbol
+                         (namestring, symbol.n_value,
+                          sclass == C_HIDEXT ? mst_file_text : mst_text,
+                          symbol.n_scnum, objfile);
+                       misc_func_recorded = 1;
+                     }
                    break;
 
                  case XMC_GL:
                    /* shared library function trampoline code entry
-                      point. */
+                      point.  */
 
                    /* record trampoline code entries as
                       mst_solib_trampoline symbol.  When we lookup mst
                       symbols, we will choose mst_text over
-                      mst_solib_trampoline. */
-                   RECORD_MINIMAL_SYMBOL
+                      mst_solib_trampoline.  */
+                   record_minimal_symbol
                      (namestring, symbol.n_value,
-                      mst_solib_trampoline,
-                      secnum_to_section (symbol.n_scnum, objfile),
-                      objfile);
+                      mst_solib_trampoline, symbol.n_scnum, objfile);
+                   misc_func_recorded = 1;
                    break;
 
                  case XMC_DS:
@@ -2359,8 +2426,9 @@ scan_xcoff_symtab (struct objfile *objfile)
                      prim_record_minimal_symbol_and_info
                        (namestring, symbol.n_value,
                         sclass == C_HIDEXT ? mst_file_data : mst_data,
-                        NULL, secnum_to_section (symbol.n_scnum, objfile),
-                        NULL, objfile);
+                        secnum_to_section (symbol.n_scnum, objfile),
+                        secnum_to_bfd_section (symbol.n_scnum, objfile),
+                        objfile);
                    break;
                  }
                break;
@@ -2376,8 +2444,9 @@ scan_xcoff_symtab (struct objfile *objfile)
                      prim_record_minimal_symbol_and_info
                        (namestring, symbol.n_value,
                         sclass == C_HIDEXT ? mst_file_bss : mst_bss,
-                        NULL, secnum_to_section (symbol.n_scnum, objfile),
-                        NULL, objfile);
+                        secnum_to_section (symbol.n_scnum, objfile),
+                        secnum_to_bfd_section (symbol.n_scnum, objfile),
+                        objfile);
                    break;
                  }
                break;
@@ -2399,21 +2468,21 @@ scan_xcoff_symtab (struct objfile *objfile)
 
            if (last_csect_name && !misc_func_recorded)
              {
-
                /* If no misc. function recorded in the last seen csect, enter
                   it as a function.  This will take care of functions like
                   strcmp() compiled by xlc.  */
 
-               RECORD_MINIMAL_SYMBOL
-                 (last_csect_name, last_csect_val,
-                  mst_text, last_csect_sec, objfile);
+               record_minimal_symbol (last_csect_name, last_csect_val,
+                                      mst_text, last_csect_sec, objfile);
+               misc_func_recorded = 1;
              }
 
            if (pst)
              {
-               xcoff_end_psymtab (pst, psymtab_include_list, includes_used,
-                                  symnum_before, dependency_list,
-                                  dependencies_used, textlow_not_set);
+               xcoff_end_psymtab (objfile, pst, psymtab_include_list,
+                                  includes_used, symnum_before,
+                                  dependency_list, dependencies_used,
+                                  textlow_not_set);
                includes_used = 0;
                dependencies_used = 0;
              }
@@ -2442,9 +2511,9 @@ scan_xcoff_symtab (struct objfile *objfile)
 
        default:
          {
-           static struct complaint msg =
-           {"Storage class %d not recognized during scan", 0, 0};
-           complain (&msg, sclass);
+           complaint (&symfile_complaints,
+                      _("Storage class %d not recognized during scan"),
+                      sclass);
          }
          /* FALLTHROUGH */
 
@@ -2494,6 +2563,7 @@ scan_xcoff_symtab (struct objfile *objfile)
            /* We probably could save a few instructions by assuming that
               C_LSYM, C_PSYM, etc., never have auxents.  */
            int naux1 = symbol.n_numaux + 1;
+
            ssymnum += naux1;
            sraw_symbol += bfd_coff_symesz (abfd) * naux1;
          }
@@ -2501,8 +2571,9 @@ scan_xcoff_symtab (struct objfile *objfile)
 
        case C_BINCL:
          {
-           /* Mark down an include file in the current psymtab */
+           /* Mark down an include file in the current psymtab */
            enum language tmp_language;
+
            swap_sym (&symbol, &main_aux[0], &namestring, &sraw_symbol,
                      &ssymnum, objfile);
 
@@ -2519,7 +2590,7 @@ scan_xcoff_symtab (struct objfile *objfile)
 
            /* In C++, one may expect the same filename to come round many
               times, when code is coming alternately from the main file
-              and from inline functions in other files. So I check to see
+              and from inline functions in other files.  So I check to see
               if this is a file we've seen before -- either the main
               source file, or a previously included file.
 
@@ -2527,12 +2598,14 @@ scan_xcoff_symtab (struct objfile *objfile)
               things like "break c-exp.y:435" need to work (I
               suppose the psymtab_include_list could be hashed or put
               in a binary tree, if profiling shows this is a major hog).  */
-           if (pst && STREQ (namestring, pst->filename))
+           if (pst && strcmp (namestring, pst->filename) == 0)
              continue;
+
            {
-             register int i;
+             int i;
+
              for (i = 0; i < includes_used; i++)
-               if (STREQ (namestring, psymtab_include_list[i]))
+               if (strcmp (namestring, psymtab_include_list[i]) == 0)
                  {
                    i = -1;
                    break;
@@ -2543,13 +2616,13 @@ scan_xcoff_symtab (struct objfile *objfile)
            psymtab_include_list[includes_used++] = namestring;
            if (includes_used >= includes_allocated)
              {
-               char **orig = psymtab_include_list;
+               const char **orig = psymtab_include_list;
 
-               psymtab_include_list = (char **)
+               psymtab_include_list = (const char **)
                  alloca ((includes_allocated *= 2) *
-                         sizeof (char *));
-               memcpy ((PTR) psymtab_include_list, (PTR) orig,
-                       includes_used * sizeof (char *));
+                         sizeof (const char *));
+               memcpy (psymtab_include_list, orig,
+                       includes_used * sizeof (const char *));
              }
            continue;
          }
@@ -2564,16 +2637,12 @@ scan_xcoff_symtab (struct objfile *objfile)
        case C_DECL:
        case C_STSYM:
          {
-
-           static struct complaint function_outside_compilation_unit = {
-             "function `%s' appears to be defined outside of all compilation units", 0, 0
-           };
-
            char *p;
+
            swap_sym (&symbol, &main_aux[0], &namestring, &sraw_symbol,
                      &ssymnum, objfile);
 
-           p = (char *) strchr (namestring, ':');
+           p = strchr (namestring, ':');
            if (!p)
              continue;                 /* Not a debugging symbol.   */
 
@@ -2587,23 +2656,27 @@ scan_xcoff_symtab (struct objfile *objfile)
            switch (p[1])
              {
              case 'S':
-               symbol.n_value += ANOFFSET (objfile->section_offsets, SECT_OFF_DATA (objfile));
-#ifdef STATIC_TRANSFORM_NAME
-               namestring = STATIC_TRANSFORM_NAME (namestring);
-#endif
-               add_psymbol_to_list (namestring, p - namestring,
-                                    VAR_NAMESPACE, LOC_STATIC,
+               symbol.n_value += ANOFFSET (objfile->section_offsets,
+                                           SECT_OFF_DATA (objfile));
+
+               if (gdbarch_static_transform_name_p (gdbarch))
+                 namestring = gdbarch_static_transform_name
+                                (gdbarch, namestring);
+
+               add_psymbol_to_list (namestring, p - namestring, 1,
+                                    VAR_DOMAIN, LOC_STATIC,
                                     &objfile->static_psymbols,
                                     0, symbol.n_value,
                                     psymtab_language, objfile);
                continue;
 
              case 'G':
-               symbol.n_value += ANOFFSET (objfile->section_offsets, SECT_OFF_DATA (objfile));
+               symbol.n_value += ANOFFSET (objfile->section_offsets,
+                                           SECT_OFF_DATA (objfile));
                /* The addresses in these entries are reported to be
-                  wrong.  See the code that reads 'G's for symtabs. */
-               add_psymbol_to_list (namestring, p - namestring,
-                                    VAR_NAMESPACE, LOC_STATIC,
+                  wrong.  See the code that reads 'G's for symtabs.  */
+               add_psymbol_to_list (namestring, p - namestring, 1,
+                                    VAR_DOMAIN, LOC_STATIC,
                                     &objfile->global_psymbols,
                                     0, symbol.n_value,
                                     psymtab_language, objfile);
@@ -2620,43 +2693,29 @@ scan_xcoff_symtab (struct objfile *objfile)
                    || (p == namestring + 1
                        && namestring[0] != ' '))
                  {
-                   add_psymbol_to_list (namestring, p - namestring,
-                                        STRUCT_NAMESPACE, LOC_TYPEDEF,
+                   add_psymbol_to_list (namestring, p - namestring, 1,
+                                        STRUCT_DOMAIN, LOC_TYPEDEF,
                                         &objfile->static_psymbols,
                                         symbol.n_value, 0,
                                         psymtab_language, objfile);
                    if (p[2] == 't')
                      {
                        /* Also a typedef with the same name.  */
-                       add_psymbol_to_list (namestring, p - namestring,
-                                            VAR_NAMESPACE, LOC_TYPEDEF,
+                       add_psymbol_to_list (namestring, p - namestring, 1,
+                                            VAR_DOMAIN, LOC_TYPEDEF,
                                             &objfile->static_psymbols,
                                             symbol.n_value, 0,
                                             psymtab_language, objfile);
                        p += 1;
                      }
-                   /* The semantics of C++ state that "struct foo { ... }"
-                      also defines a typedef for "foo".  Unfortuantely, cfront
-                      never makes the typedef when translating from C++ to C.
-                      We make the typedef here so that "ptype foo" works as
-                      expected for cfront translated code.  */
-                   else if (psymtab_language == language_cplus)
-                     {
-                       /* Also a typedef with the same name.  */
-                       add_psymbol_to_list (namestring, p - namestring,
-                                            VAR_NAMESPACE, LOC_TYPEDEF,
-                                            &objfile->static_psymbols,
-                                            symbol.n_value, 0,
-                                            psymtab_language, objfile);
-                     }
                  }
                goto check_enum;
 
              case 't':
-               if (p != namestring)    /* a name is there, not just :T... */
+               if (p != namestring)    /* a name is there, not just :T...  */
                  {
-                   add_psymbol_to_list (namestring, p - namestring,
-                                        VAR_NAMESPACE, LOC_TYPEDEF,
+                   add_psymbol_to_list (namestring, p - namestring, 1,
+                                        VAR_DOMAIN, LOC_TYPEDEF,
                                         &objfile->static_psymbols,
                                         symbol.n_value, 0,
                                         psymtab_language, objfile);
@@ -2686,7 +2745,8 @@ scan_xcoff_symtab (struct objfile *objfile)
 
                if (*p++ == 'e')
                  {
-                   /* The aix4 compiler emits extra crud before the members.  */
+                   /* The aix4 compiler emits extra crud before the
+                      members.  */
                    if (*p == '-')
                      {
                        /* Skip over the type (?).  */
@@ -2717,8 +2777,8 @@ scan_xcoff_symtab (struct objfile *objfile)
                          ;
                        /* Note that the value doesn't matter for
                           enum constants in psymtabs, just in symtabs.  */
-                       add_psymbol_to_list (p, q - p,
-                                            VAR_NAMESPACE, LOC_CONST,
+                       add_psymbol_to_list (p, q - p, 1,
+                                            VAR_DOMAIN, LOC_CONST,
                                             &objfile->static_psymbols, 0,
                                             0, psymtab_language, objfile);
                        /* Point past the name.  */
@@ -2735,8 +2795,8 @@ scan_xcoff_symtab (struct objfile *objfile)
 
              case 'c':
                /* Constant, e.g. from "const" in Pascal.  */
-               add_psymbol_to_list (namestring, p - namestring,
-                                    VAR_NAMESPACE, LOC_CONST,
+               add_psymbol_to_list (namestring, p - namestring, 1,
+                                    VAR_DOMAIN, LOC_CONST,
                                     &objfile->static_psymbols, symbol.n_value,
                                     0, psymtab_language, objfile);
                continue;
@@ -2746,14 +2806,16 @@ scan_xcoff_symtab (struct objfile *objfile)
                  {
                    int name_len = p - namestring;
                    char *name = xmalloc (name_len + 1);
+
                    memcpy (name, namestring, name_len);
                    name[name_len] = '\0';
-                   complain (&function_outside_compilation_unit, name);
+                   function_outside_compilation_unit_complaint (name);
                    xfree (name);
                  }
-               symbol.n_value += ANOFFSET (objfile->section_offsets, SECT_OFF_TEXT (objfile));
-               add_psymbol_to_list (namestring, p - namestring,
-                                    VAR_NAMESPACE, LOC_BLOCK,
+               symbol.n_value += ANOFFSET (objfile->section_offsets,
+                                           SECT_OFF_TEXT (objfile));
+               add_psymbol_to_list (namestring, p - namestring, 1,
+                                    VAR_DOMAIN, LOC_BLOCK,
                                     &objfile->static_psymbols,
                                     0, symbol.n_value,
                                     psymtab_language, objfile);
@@ -2767,14 +2829,24 @@ scan_xcoff_symtab (struct objfile *objfile)
                  {
                    int name_len = p - namestring;
                    char *name = xmalloc (name_len + 1);
+
                    memcpy (name, namestring, name_len);
                    name[name_len] = '\0';
-                   complain (&function_outside_compilation_unit, name);
+                   function_outside_compilation_unit_complaint (name);
                    xfree (name);
                  }
-               symbol.n_value += ANOFFSET (objfile->section_offsets, SECT_OFF_TEXT (objfile));
-               add_psymbol_to_list (namestring, p - namestring,
-                                    VAR_NAMESPACE, LOC_BLOCK,
+
+               /* We need only the minimal symbols for these
+                  loader-generated definitions.  Keeping the global
+                  symbols leads to "in psymbols but not in symbols"
+                  errors.  */
+               if (strncmp (namestring, "@FIX", 4) == 0)
+                 continue;
+
+               symbol.n_value += ANOFFSET (objfile->section_offsets,
+                                           SECT_OFF_TEXT (objfile));
+               add_psymbol_to_list (namestring, p - namestring, 1,
+                                    VAR_DOMAIN, LOC_BLOCK,
                                     &objfile->global_psymbols,
                                     0, symbol.n_value,
                                     psymtab_language, objfile);
@@ -2796,10 +2868,8 @@ scan_xcoff_symtab (struct objfile *objfile)
              case '8':
              case '9':
              case '-':
-             case '#':         /* for symbol identification (used in live ranges) */
-               /* added to support cfront stabs strings */
-             case 'Z':         /* for definition continuations */
-             case 'P':         /* for prototypes */
+             case '#':         /* For symbol identification (used in
+                                  live ranges).  */
                continue;
 
              case ':':
@@ -2813,14 +2883,16 @@ scan_xcoff_symtab (struct objfile *objfile)
                continue;
 
              default:
-               /* Unexpected symbol descriptor.  The second and subsequent stabs
-                  of a continued stab can show up here.  The question is
-                  whether they ever can mimic a normal stab--it would be
-                  nice if not, since we certainly don't want to spend the
-                  time searching to the end of every string looking for
-                  a backslash.  */
+               /* Unexpected symbol descriptor.  The second and
+                  subsequent stabs of a continued stab can show up
+                  here.  The question is whether they ever can mimic
+                  a normal stab--it would be nice if not, since we
+                  certainly don't want to spend the time searching to
+                  the end of every string looking for a
+                  backslash.  */
 
-               complain (&unknown_symchar_complaint, p[1]);
+               complaint (&symfile_complaints,
+                          _("unknown symbol descriptor `%c'"), p[1]);
 
                /* Ignore it; perhaps it is an extension that we don't
                   know about.  */
@@ -2832,25 +2904,26 @@ scan_xcoff_symtab (struct objfile *objfile)
 
   if (pst)
     {
-      xcoff_end_psymtab (pst, psymtab_include_list, includes_used,
+      xcoff_end_psymtab (objfile, pst, psymtab_include_list, includes_used,
                         ssymnum, dependency_list,
                         dependencies_used, textlow_not_set);
     }
 
-  /* Record the toc offset value of this symbol table into objfile structure.
-     If no XMC_TC0 is found, toc_offset should be zero. Another place to obtain
-     this information would be file auxiliary header. */
+  /* Record the toc offset value of this symbol table into objfile
+     structure.  If no XMC_TC0 is found, toc_offset should be zero.
+     Another place to obtain this information would be file auxiliary
+     header.  */
 
-  ((struct coff_symfile_info *) objfile->sym_private)->toc_offset = toc_offset;
+  XCOFF_DATA (objfile)->toc_offset = toc_offset;
 }
 
 /* Return the toc offset value for a given objfile.  */
 
 CORE_ADDR
-get_toc_offset (struct objfile *objfile)
+xcoff_get_toc_offset (struct objfile *objfile)
 {
   if (objfile)
-    return ((struct coff_symfile_info *) objfile->sym_private)->toc_offset;
+    return XCOFF_DATA (objfile)->toc_offset;
   return 0;
 }
 
@@ -2860,12 +2933,11 @@ get_toc_offset (struct objfile *objfile)
    hung off the objfile structure.
 
    SECTION_OFFSETS contains offsets relative to which the symbols in the
-   various sections are (depending where the sections were actually loaded).
-   MAINLINE is true if we are reading the main symbol
-   table (as opposed to a shared lib or dynamically loaded file).  */
+   various sections are (depending where the sections were actually
+   loaded).  */
 
 static void
-xcoff_initial_scan (struct objfile *objfile, int mainline)
+xcoff_initial_scan (struct objfile *objfile, int symfile_flags)
 {
   bfd *abfd;
   int val;
@@ -2877,7 +2949,7 @@ xcoff_initial_scan (struct objfile *objfile, int mainline)
   char *name;
   unsigned int size;
 
-  info = (struct coff_symfile_info *) objfile->sym_private;
+  info = XCOFF_DATA (objfile);
   symfile_bfd = abfd = objfile->obfd;
   name = objfile->name;
 
@@ -2897,9 +2969,9 @@ xcoff_initial_scan (struct objfile *objfile, int mainline)
 
       /* Read the .debug section, if present.  */
       {
-       sec_ptr secp;
+       struct bfd_section *secp;
        bfd_size_type length;
-       char *debugsec = NULL;
+       bfd_byte *debugsec = NULL;
 
        secp = bfd_get_section_by_name (abfd, ".debug");
        if (secp)
@@ -2907,19 +2979,16 @@ xcoff_initial_scan (struct objfile *objfile, int mainline)
            length = bfd_section_size (abfd, secp);
            if (length)
              {
-               debugsec =
-                 (char *) obstack_alloc (&objfile->symbol_obstack, length);
+               debugsec = obstack_alloc (&objfile->objfile_obstack, length);
 
-               if (!bfd_get_section_contents (abfd, secp, debugsec,
-                                              (file_ptr) 0, length))
+               if (!bfd_get_full_section_contents (abfd, secp, &debugsec))
                  {
-                   error ("Error reading .debug section of `%s': %s",
+                   error (_("Error reading .debug section of `%s': %s"),
                           name, bfd_errmsg (bfd_get_error ()));
                  }
              }
          }
-       ((struct coff_symfile_info *) objfile->sym_private)->debugsec =
-         debugsec;
+       info->debugsec = debugsec;
       }
     }
 
@@ -2927,23 +2996,18 @@ xcoff_initial_scan (struct objfile *objfile, int mainline)
      access them randomly in read_symbol*.  */
   val = bfd_seek (abfd, symtab_offset, SEEK_SET);
   if (val < 0)
-    error ("Error reading symbols from %s: %s",
+    error (_("Error reading symbols from %s: %s"),
           name, bfd_errmsg (bfd_get_error ()));
   size = coff_data (abfd)->local_symesz * num_symbols;
-  ((struct coff_symfile_info *) objfile->sym_private)->symtbl =
-    obstack_alloc (&objfile->symbol_obstack, size);
-  ((struct coff_symfile_info *) objfile->sym_private)->symtbl_num_syms =
-    num_symbols;
+  info->symtbl = obstack_alloc (&objfile->objfile_obstack, size);
+  info->symtbl_num_syms = num_symbols;
 
-  val = bfd_bread (((struct coff_symfile_info *) objfile->sym_private)->symtbl,
-                  size, abfd);
+  val = bfd_bread (info->symtbl, size, abfd);
   if (val != size)
-    perror_with_name ("reading symbol table");
+    perror_with_name (_("reading symbol table"));
 
-  /* If we are reinitializing, or if we have never loaded syms yet, init */
-  if (mainline
-      || (objfile->global_psymbols.size == 0
-         && objfile->static_psymbols.size == 0))
+  /* If we are reinitializing, or if we have never loaded syms yet, init.  */
+  if (objfile->global_psymbols.size == 0 && objfile->static_psymbols.size == 0)
     /* I'm not sure how how good num_symbols is; the rule of thumb in
        init_psymbol_list was developed for a.out.  On the one hand,
        num_symbols includes auxents.  On the other hand, it doesn't
@@ -2962,24 +3026,33 @@ xcoff_initial_scan (struct objfile *objfile, int mainline)
   scan_xcoff_symtab (objfile);
 
   /* Install any minimal symbols that have been collected as the current
-     minimal symbols for this objfile. */
+     minimal symbols for this objfile.  */
 
   install_minimal_symbols (objfile);
 
+  /* DWARF2 sections.  */
+
+  if (dwarf2_has_info (objfile, &dwarf2_xcoff_names))
+    dwarf2_build_psymtabs (objfile);
+
+  dwarf2_build_frame_info (objfile);
+
   do_cleanups (back_to);
 }
 \f
 static void
-xcoff_symfile_offsets (struct objfile *objfile, struct section_addr_info *addrs)
+xcoff_symfile_offsets (struct objfile *objfile,
+                      struct section_addr_info *addrs)
 {
   asection *sect = NULL;
   int i;
 
-  objfile->num_sections = SECT_OFF_MAX;
+  objfile->num_sections = bfd_count_sections (objfile->obfd);
   objfile->section_offsets = (struct section_offsets *)
-    obstack_alloc (&objfile->psymbol_obstack, SIZEOF_SECTION_OFFSETS);
+    obstack_alloc (&objfile->objfile_obstack, 
+                  SIZEOF_N_SECTION_OFFSETS (objfile->num_sections));
 
-  /* Initialize the section indexes for future use. */
+  /* Initialize the section indexes for future use.  */
   sect = bfd_get_section_by_name (objfile->obfd, ".text");
   if (sect) 
     objfile->sect_index_text = sect->index;
@@ -3013,7 +3086,7 @@ xcoff_symfile_offsets (struct objfile *objfile, struct section_addr_info *addrs)
 
 /* Register our ability to parse symbols for xcoff BFD files.  */
 
-static struct sym_fns xcoff_sym_fns =
+static const struct sym_fns xcoff_sym_fns =
 {
 
   /* It is possible that coff and xcoff should be merged as
@@ -3027,23 +3100,96 @@ static struct sym_fns xcoff_sym_fns =
 
   bfd_target_xcoff_flavour,
 
-  xcoff_new_init,              /* sym_new_init: init anything gbl to entire symtab */
-  xcoff_symfile_init,          /* sym_init: read initial info, setup for sym_read() */
-  xcoff_initial_scan,          /* sym_read: read a symbol file into symtab */
-  xcoff_symfile_finish,                /* sym_finish: finished with file, cleanup */
-  xcoff_symfile_offsets,       /* sym_offsets: xlate offsets ext->int form */
-  NULL                         /* next: pointer to next struct sym_fns */
+  xcoff_new_init,              /* init anything gbl to entire symtab */
+  xcoff_symfile_init,          /* read initial info, setup for sym_read() */
+  xcoff_initial_scan,          /* read a symbol file into symtab */
+  NULL,                                /* sym_read_psymbols */
+  xcoff_symfile_finish,                /* finished with file, cleanup */
+  xcoff_symfile_offsets,       /* xlate offsets ext->int form */
+  default_symfile_segments,    /* Get segment information from a file.  */
+  aix_process_linenos,
+  default_symfile_relocate,    /* Relocate a debug section.  */
+  NULL,                                /* sym_probe_fns */
+  &psym_functions
 };
 
+/* Same as xcoff_get_n_import_files, but for core files.  */
+
+static int
+xcoff_get_core_n_import_files (bfd *abfd)
+{
+  asection *sect = bfd_get_section_by_name (abfd, ".ldinfo");
+  gdb_byte buf[4];
+  file_ptr offset = 0;
+  int n_entries = 0;
+
+  if (sect == NULL)
+    return -1;  /* Not a core file.  */
+
+  for (offset = 0; offset < bfd_get_section_size (sect);)
+    {
+      int next;
+
+      n_entries++;
+
+      if (!bfd_get_section_contents (abfd, sect, buf, offset, 4))
+       return -1;
+      next = bfd_get_32 (abfd, buf);
+      if (next == 0)
+       break;  /* This is the last entry.  */
+      offset += next;
+    }
+
+  /* Return the number of entries, excluding the first one, which is
+     the path to the executable that produced this core file.  */
+  return n_entries - 1;
+}
+
+/* Return the number of import files (shared libraries) that the given
+   BFD depends on.  Return -1 if this number could not be computed.  */
+
+int
+xcoff_get_n_import_files (bfd *abfd)
+{
+  asection *sect = bfd_get_section_by_name (abfd, ".loader");
+  gdb_byte buf[4];
+  int l_nimpid;
+
+  /* If the ".loader" section does not exist, the objfile is probably
+     not an executable.  Might be a core file...  */
+  if (sect == NULL)
+    return xcoff_get_core_n_import_files (abfd);
+
+  /* The number of entries in the Import Files Table is stored in
+     field l_nimpid.  This field is always at offset 16, and is
+     always 4 bytes long.  Read those 4 bytes.  */
+
+  if (!bfd_get_section_contents (abfd, sect, buf, 16, 4))
+    return -1;
+  l_nimpid = bfd_get_32 (abfd, buf);
+
+  /* By convention, the first entry is the default LIBPATH value
+     to be used by the system loader, so it does not count towards
+     the number of import files.  */
+  return l_nimpid - 1;
+}
+
+/* Free the per-objfile xcoff data.  */
+
+static void
+xcoff_free_info (struct objfile *objfile, void *arg)
+{
+  xfree (arg);
+}
+
+/* Provide a prototype to silence -Wmissing-prototypes.  */
+extern initialize_file_ftype _initialize_xcoffread;
+
 void
 _initialize_xcoffread (void)
 {
   add_symtab_fns (&xcoff_sym_fns);
 
-  func_symbol_type = init_type (TYPE_CODE_FUNC, 1, 0,
-                               "<function, no debug info>", NULL);
-  TYPE_TARGET_TYPE (func_symbol_type) = builtin_type_int;
-  var_symbol_type =
-    init_type (TYPE_CODE_INT, TARGET_INT_BIT / HOST_CHAR_BIT, 0,
-              "<variable, no debug info>", NULL);
+  xcoff_objfile_data_key = register_objfile_data_with_cleanup (NULL,
+                                                              xcoff_free_info);
 }