Introduce assign_operation
[deliverable/binutils-gdb.git] / gdb / xcoffread.c
index e7941acff792df86ca0b4e1c73d8a90f396cdc70..b1768702e26fe414f545a790114b09093af3e0f4 100644 (file)
@@ -1,7 +1,5 @@
 /* Read AIX xcoff symbol tables and convert to internal format, for GDB.
-   Copyright (C) 1986, 1987, 1988, 1989, 1990, 1991, 1992, 1993, 1994, 1995,
-   1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2007, 2008
-   Free Software Foundation, Inc.
+   Copyright (C) 1986-2021 Free Software Foundation, Inc.
    Derived from coffread.c, dbxread.c, and a lot of hacking.
    Contributed by IBM Corporation.
 
 #include <sys/types.h>
 #include <fcntl.h>
 #include <ctype.h>
-#include "gdb_string.h"
-
-#include <sys/param.h>
 #ifdef HAVE_SYS_FILE_H
 #include <sys/file.h>
 #endif
-#include "gdb_stat.h"
+#include <sys/stat.h>
+#include <algorithm>
 
 #include "coff/internal.h"
 #include "libcoff.h"           /* FIXME, internal data from BFD */
 #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 "buildsym-legacy.h"
 #include "stabsread.h"
 #include "expression.h"
 #include "complaints.h"
+#include "psympriv.h"
+#include "dwarf2/sect-names.h"
 
 #include "gdb-stabs.h"
 
@@ -74,29 +73,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 */
-    long c_value;
+    int c_symnum;              /* Symbol number of this entry.  */
+    int c_naux;                        /* 0 if syment only, 1 if syment + auxent */
+    CORE_ADDR 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;
 
@@ -104,7 +104,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;
@@ -113,67 +113,98 @@ 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
 
-/* Nonzero if within a function (so symbols should be local,
-   if nothing says specifically).  */
-
-int within_function;
-
 /* Size of a COFF symbol.  I think it is always 18, so I'm not sure
    there is any reason not to just use a #define, but might as well
    ask BFD for the size and store it here, I guess.  */
 
 static unsigned local_symesz;
 
-struct coff_symfile_info
+struct xcoff_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;
+    char *strtbl = nullptr;
 
     /* Pointer to debug section.  */
-    char *debugsec;
+    char *debugsec = nullptr;
 
     /* Pointer to the a.out symbol table.  */
-    char *symtbl;
+    char *symtbl = nullptr;
 
     /* Number of symbols in symtbl.  */
-    int symtbl_num_syms;
+    int symtbl_num_syms = 0;
 
     /* Offset in data section to TOC anchor.  */
-    CORE_ADDR toc_offset;
+    CORE_ADDR toc_offset = 0;
   };
 
+/* Key for XCOFF-associated data.  */
+
+static const struct objfile_key<xcoff_symfile_info> xcoff_objfile_data_key;
+
+/* Convenience macro to access the per-objfile XCOFF data.  */
+
+#define XCOFF_DATA(objfile)                                            \
+  xcoff_objfile_data_key.get (objfile)
+
+/* 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_loclists */
+  /* AIX XCOFF defines one, named DWARF section for macro debug information.
+     XLC does not generate debug_macinfo for DWARF4 and below.
+     The section is assigned to debug_macro for DWARF5 and above. */
+  { NULL, NULL },
+  { ".dwmac", NULL },
+  { ".dwstr", NULL },
+  { NULL, NULL }, /* debug_str_offsets */
+  { NULL, NULL }, /* debug_line_str */
+  { ".dwrnges", NULL },
+  { NULL, NULL }, /* debug_rnglists */
+  { ".dwpbtyp", NULL },
+  { NULL, NULL }, /* debug_addr */
+  { ".dwframe", NULL },
+  { NULL, NULL }, /* eh_frame */
+  { NULL, NULL }, /* gdb_index */
+  { NULL, NULL }, /* debug_names */
+  { NULL, NULL }, /* debug_aranges */
+  23
+};
+
 static void
 bf_notfound_complaint (void)
 {
-  complaint (&symfile_complaints, _("line numbers off, `.bf' symbol not found"));
+  complaint (_("line numbers off, `.bf' symbol not found"));
 }
 
 static void
 ef_complaint (int arg1)
 {
-  complaint (&symfile_complaints,
-            _("Mismatched .ef symbol ignored starting at symnum %d"), arg1);
+  complaint (_("Mismatched .ef symbol ignored starting at symnum %d"), arg1);
 }
 
 static void
 eb_complaint (int arg1)
 {
-  complaint (&symfile_complaints,
-            _("Mismatched .eb symbol ignored starting at symnum %d"), arg1);
+  complaint (_("Mismatched .eb symbol ignored starting at symnum %d"), arg1);
 }
 
-static void xcoff_initial_scan (struct objfile *, int);
+static void xcoff_initial_scan (struct objfile *, symfile_add_flags);
 
-static void scan_xcoff_symtab (struct objfile *);
+static void scan_xcoff_symtab (minimal_symbol_reader &,
+                              struct objfile *);
 
-static char *xcoff_next_symbol_text (struct objfile *);
+static const char *xcoff_next_symbol_text (struct objfile *);
 
 static void record_include_begin (struct coff_symbol *);
 
@@ -189,9 +220,6 @@ 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 char *coff_getfilename (union internal_auxent *, struct objfile *);
 
 static void read_symbol (struct internal_syment *, int);
@@ -203,14 +231,12 @@ 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 *, legacy_psymtab *);
 
 #if 0
 static void add_stab_to_list (char *, struct pending_stabs **);
 #endif
 
-static int compare_lte (const void *, const void *);
-
 static struct linetable *arrange_linetable (struct linetable *);
 
 static void record_include_end (struct coff_symbol *);
@@ -238,50 +264,70 @@ 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.  */
-      if (bfd_get_section_flags (abfd, sect) & SEC_CODE)
+      if (bfd_section_flags (sect) & SEC_CODE)
        *args->resultp = SECT_OFF_TEXT (objfile);
-      else if (bfd_get_section_flags (abfd, sect) & SEC_LOAD)
+      else if (bfd_section_flags (sect) & SEC_LOAD)
        *args->resultp = SECT_OFF_DATA (objfile);
       else
-       *args->resultp = sect->index;
+       *args->resultp = gdb_bfd_section_index (abfd, sect);
       *args->bfd_sect = sect;
     }
 }
 
-/* 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
 
@@ -301,7 +347,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;
 }
@@ -313,9 +359,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:
 
@@ -335,7 +381,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:
 
@@ -344,7 +390,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:
 
@@ -355,27 +401,17 @@ 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.  */
 /* *INDENT-ON* */
 
 
-
-/* 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.  */
 
@@ -388,28 +424,37 @@ arrange_linetable (struct linetable *oldLineTb)
   struct linetable_entry *fentry;      /* function entry vector */
   int fentry_size;             /* # of function entries */
   struct linetable *newLineTb; /* new line table */
+  int extra_lines = 0;
 
 #define NUM_OF_FUNCTIONS 20
 
   fentry_size = NUM_OF_FUNCTIONS;
-  fentry = (struct linetable_entry *)
-    xmalloc (fentry_size * sizeof (struct linetable_entry));
+  fentry = XNEWVEC (struct linetable_entry, fentry_size);
 
   for (function_count = 0, ii = 0; ii < oldLineTb->nitems; ++ii)
     {
+      if (oldLineTb->item[ii].is_stmt == 0)
+       continue;
 
       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].is_stmt = 1;
          fentry[function_count].pc = oldLineTb->item[ii].pc;
          ++function_count;
+
+         /* If the function was compiled with XLC, we may have to add an
+            extra line entry later.  Reserve space for that.  */
+         if (ii + 1 < oldLineTb->nitems
+             && oldLineTb->item[ii].pc != oldLineTb->item[ii + 1].pc)
+           extra_lines++;
        }
     }
 
@@ -419,16 +464,18 @@ arrange_linetable (struct linetable *oldLineTb)
       return oldLineTb;
     }
   else if (function_count > 1)
-    qsort (fentry, function_count, sizeof (struct linetable_entry), compare_lte);
+    std::sort (fentry, fentry + function_count,
+              [] (const linetable_entry &lte1, const linetable_entry& lte2)
+               { return lte1.pc < lte2.pc; });
 
-  /* 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));
+    (oldLineTb->nitems - function_count + extra_lines) * 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)
@@ -436,24 +483,37 @@ 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)
     {
+      /* If the function was compiled with XLC, we may have to add an
+        extra line to cover the function prologue.  */
+      jj = fentry[ii].line;
+      if (jj + 1 < oldLineTb->nitems
+         && oldLineTb->item[jj].pc != oldLineTb->item[jj + 1].pc)
+       {
+         newLineTb->item[newline] = oldLineTb->item[jj];
+         newLineTb->item[newline].line = oldLineTb->item[jj + 1].line;
+         newline++;
+       }
+
       for (jj = fentry[ii].line + 1;
           jj < oldLineTb->nitems && oldLineTb->item[jj].line != 0;
           ++jj, ++newline)
        newLineTb->item[newline] = oldLineTb->item[jj];
     }
   xfree (fentry);
-  newLineTb->nitems = oldLineTb->nitems - function_count;
+  /* The number of items in the line table must include these
+     extra lines which were added in case of XLC compiled functions.  */
+  newLineTb->nitems = oldLineTb->nitems - function_count + extra_lines;
   return newLineTb;
 }
 
 /* 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
@@ -465,7 +525,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,12 +543,12 @@ record_include_begin (struct coff_symbol *cs)
   if (inclDepth)
     {
       /* In xcoff, we assume include files cannot be nested (not in .c files
-         of course, but in corresponding .s files.).  */
+        of course, but in corresponding .s files.).  */
 
       /* 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.  */
-      complaint (&symfile_complaints, _("Nested C_BINCL symbols"));
+        GCC 2.3.3-930426 does not exhibit this on a test case which
+        a user said produced the message for him.  */
+      complaint (_("Nested C_BINCL symbols"));
     }
   ++inclDepth;
 
@@ -505,7 +565,7 @@ record_include_end (struct coff_symbol *cs)
 
   if (inclDepth == 0)
     {
-      complaint (&symfile_complaints, _("Mismatched C_BINCL/C_EINCL pair"));
+      complaint (_("Mismatched C_BINCL/C_EINCL pair"));
     }
 
   allocate_include_entry ();
@@ -522,18 +582,14 @@ allocate_include_entry (void)
 {
   if (inclTable == NULL)
     {
-      inclTable = (InclTable *)
-       xmalloc (sizeof (InclTable) * INITIAL_INCLUDE_TABLE_LENGTH);
-      memset (inclTable,
-             '\0', sizeof (InclTable) * INITIAL_INCLUDE_TABLE_LENGTH);
+      inclTable = XCNEWVEC (InclTable, INITIAL_INCLUDE_TABLE_LENGTH);
       inclLength = INITIAL_INCLUDE_TABLE_LENGTH;
       inclIndx = 0;
     }
   else if (inclIndx >= inclLength)
     {
       inclLength += INITIAL_INCLUDE_TABLE_LENGTH;
-      inclTable = (InclTable *)
-       xrealloc (inclTable, sizeof (InclTable) * inclLength);
+      inclTable = XRESIZEVEC (InclTable, inclTable, inclLength);
       memset (inclTable + inclLength - INITIAL_INCLUDE_TABLE_LENGTH,
              '\0', sizeof (InclTable) * INITIAL_INCLUDE_TABLE_LENGTH);
     }
@@ -541,18 +597,20 @@ allocate_include_entry (void)
 
 /* Global variable to pass the psymtab down to all the routines involved
    in psymtab to symtab processing.  */
-static struct partial_symtab *this_symtab_psymtab;
+static legacy_psymtab *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->deprecated_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;
@@ -574,7 +632,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);
@@ -584,7 +643,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)
@@ -599,24 +658,37 @@ 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, get_last_source_file ()) == 0)
+           {
+             /* The entry in the include table refers to the main source
+                file.  Add the lines to the main subfile.  */
 
-         tmpSubfile = inclTable[ii].subfile =
-           (struct subfile *) xmalloc (sizeof (struct 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.  */
 
-         memset (tmpSubfile, '\0', sizeof (struct subfile));
-         firstLine = &(inclTable[ii].funStartLine);
+             tmpSubfile = inclTable[ii].subfile = XNEW (struct subfile);
 
-         /* Enter include file's lines now.  */
-         enter_line_range (tmpSubfile, inclTable[ii].begin,
-                           inclTable[ii].end, start, 0, firstLine);
+             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);
+           }
 
          if (offset <= inclTable[ii].end)
            offset = inclTable[ii].end + linesz;
        }
 
       /* All the include files' line have been processed at this point.  Now,
-         enter remaining lines of the main file, if any left.  */
+        enter remaining lines of the main file, if any left.  */
       if (offset < max_offset + 1 - linesz)
        {
          enter_line_range (&main_subfile, offset, 0, start, end,
@@ -631,9 +703,10 @@ 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.  */
 
+      struct subfile *current_subfile = get_current_subfile ();
       lineTb = arrange_linetable (lv);
       if (lv == lineTb)
        {
@@ -655,14 +728,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);
 
@@ -677,24 +752,26 @@ 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.  */
 
 #if 0
-         start_subfile (inclTable[ii].name, (char *) 0);
+         start_subfile (inclTable[ii].name);
 #else
          {
            /* Pick a fake name that will produce the same results as this
               one when passed to deduce_language_from_filename.  Kludge on
               top of kludge.  */
-           char *fakename = strrchr (inclTable[ii].name, '.');
+           const char *fakename = strrchr (inclTable[ii].name, '.');
+
            if (fakename == NULL)
              fakename = " ?";
-           start_subfile (fakename, (char *) 0);
-           xfree (current_subfile->name);
+           start_subfile (fakename);
+           xfree (get_current_subfile ()->name);
          }
+         struct subfile *current_subfile = get_current_subfile ();
          current_subfile->name = xstrdup (inclTable[ii].name);
 #endif
 
@@ -714,7 +791,7 @@ process_linenos (CORE_ADDR start, CORE_ADDR end)
 
          current_subfile->line_vector_length =
            current_subfile->line_vector->nitems;
-         start_subfile (pop_subfile (), (char *) 0);
+         start_subfile (pop_subfile ());
        }
     }
 
@@ -722,32 +799,38 @@ return_after_cleanup:
 
   /* We don't want to keep alloc/free'ing the global include file table.  */
   inclIndx = 0;
-
-  /* Start with a fresh subfile structure for the next file.  */
-  memset (&main_subfile, '\0', sizeof (struct subfile));
 }
 
 static void
-aix_process_linenos (void)
+aix_process_linenos (struct objfile *objfile)
 {
-  /* process line numbers and enter them into line vector */
-  process_linenos (last_source_start_addr, cur_src_end_addr);
+  /* 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 (get_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 = objfile->arch ();
   unsigned int curoffset;
   CORE_ADDR addr;
   void *ext_lnno;
@@ -759,16 +842,13 @@ 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->deprecated_sym_private)
-    ->max_lineno_offset;
+  limit_offset = XCOFF_DATA (objfile)->max_lineno_offset;
 
   if (endoffset != 0)
     {
       if (endoffset >= limit_offset)
        {
-         complaint (&symfile_complaints,
-                    _("Bad line table offset in C_EINCL directive"));
+         complaint (_("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 += objfile->text_section_offset ();
 
       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;                 \
+  set_last_source_file (name);                 \
+  set_last_source_start_addr (start_addr);     \
 }
 
 
@@ -825,23 +905,39 @@ 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), \
-                                      (SECTION), (asection *)NULL, (OBJFILE)); \
-  misc_func_recorded = 1;                                      \
-}
+/* Create a new minimal symbol (using record_with_info).
+
+   Creation of all new minimal symbols should go through this function
+   rather than calling the various record functions in order
+   to make sure that all symbol addresses get properly relocated.
 
+   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, prior to relocation.  This function
+      relocates the address before recording the minimal symbol.
+   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 (minimal_symbol_reader &reader,
+                      const char *name, CORE_ADDR address,
+                      enum minimal_symbol_type ms_type,
+                      int n_scnum,
+                      struct objfile *objfile)
+{
+  if (name[0] == '.')
+    ++name;
+
+  reader.record_with_info (name, address, ms_type,
+                          secnum_to_section (n_scnum, 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 +945,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;
 
@@ -860,39 +956,37 @@ static char *raw_symbol;
 /* This is the function which stabsread.c calls to get symbol
    continuations.  */
 
-static char *
+static const char *
 xcoff_next_symbol_text (struct objfile *objfile)
 {
   struct internal_syment symbol;
-  char *retval;
-  /* FIXME: is this the same as the passed arg? */
-  if (this_symtab_psymtab)
-    objfile = this_symtab_psymtab->objfile;
+  const char *retval;
+
+  /* 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)
     {
-      complaint (&symfile_complaints, _("Unexpected symbol continuation"));
+      complaint (_("Unexpected symbol continuation"));
 
       /* Return something which points to '\0' and hope the symbol reading
-         code does something reasonable.  */
+        code does something reasonable.  */
       retval = "";
     }
   else if (symbol.n_sclass & 0x80)
     {
-      retval =
-       ((struct coff_symfile_info *) objfile->deprecated_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
     {
-      complaint (&symfile_complaints, _("Unexpected symbol continuation"));
+      complaint (_("Unexpected symbol continuation"));
 
       /* Return something which points to '\0' and hope the symbol reading
-         code does something reasonable.  */
+        code does something reasonable.  */
       retval = "";
     }
   return retval;
@@ -901,15 +995,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, legacy_psymtab *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->deprecated_sym_private)->strtbl;
-  char *debugsec =
-  ((struct coff_symfile_info *) objfile->deprecated_sym_private)->debugsec;
-  char *debugfmt = bfd_xcoff_is_xcoff64 (abfd) ? "XCOFF64" : "XCOFF";
+  char *raw_auxptr;            /* Pointer to first raw aux entry for sym */
+  struct xcoff_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;
@@ -921,53 +1014,51 @@ read_xcoff_symtab (struct partial_symtab *pst)
   unsigned int max_symnum;
   int just_started = 1;
   int depth = 0;
-  int fcn_start_addr = 0;
+  CORE_ADDR fcn_start_addr = 0;
+  enum language pst_symtab_language;
 
   struct coff_symbol fcn_stab_saved = { 0 };
 
-  /* fcn_cs_saved is global because process_xcoff_symbol needs it. */
-  union internal_auxent fcn_aux_saved;
-  struct context_stack *new;
+  /* fcn_cs_saved is global because process_xcoff_symbol needs it.  */
+  union internal_auxent fcn_aux_saved = main_aux;
+  struct context_stack *newobj;
 
-  char *filestring = " _start_ ";      /* Name of the current file. */
+  const char *filestring = pst->filename;      /* 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;
+  set_last_source_file (NULL);
   last_csect_name = 0;
-  last_csect_val = 0;
+  pst_symtab_language = deduce_language_from_filename (filestring);
 
   start_stabs ();
-  start_symtab (filestring, (char *) NULL, file_start_addr);
+  start_symtab (objfile, filestring, NULL, file_start_addr,
+               pst_symtab_language);
   record_debugformat (debugfmt);
   symnum = ((struct symloc *) pst->read_symtab_private)->first_symnum;
   max_symnum =
     symnum + ((struct symloc *) pst->read_symtab_private)->numsyms;
   first_object_file_end = 0;
 
-  raw_symbol =
-    ((struct coff_symfile_info *) objfile->deprecated_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
-         whole symbol table, only string table will be kept in memory,
-         symbol table and debug section of xcoff will be freed. Thus
-         we can mark symbols with names in string table as
-         `alloced'. */
+      /* 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
+        we can mark symbols with names in string table as
+        `alloced'.  */
       {
        int ii;
 
@@ -988,7 +1079,9 @@ read_xcoff_symtab (struct partial_symtab *pst)
            if (cs->c_name[E_SYMNMLEN - 1] != '\0')
              {
                char *p;
-               p = obstack_alloc (&objfile->objfile_obstack, E_SYMNMLEN + 1);
+
+               p = (char *) obstack_alloc (&objfile->objfile_obstack,
+                                           E_SYMNMLEN + 1);
                strncpy (p, cs->c_name, E_SYMNMLEN);
                p[E_SYMNMLEN] = '\0';
                cs->c_name = p;
@@ -1025,29 +1118,30 @@ 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;
 
       if (cs->c_symnum == next_file_symnum && cs->c_sclass != C_FILE)
        {
-         if (last_source_file)
+         if (get_last_source_file ())
            {
-             pst->symtab =
-               end_symtab (cur_src_end_addr, objfile, SECT_OFF_TEXT (objfile));
+             pst->compunit_symtab = end_symtab (cur_src_end_addr,
+                                                SECT_OFF_TEXT (objfile));
              end_stabs ();
            }
 
          start_stabs ();
-         start_symtab ("_globals_", (char *) NULL, (CORE_ADDR) 0);
+         start_symtab (objfile, "_globals_", NULL,
+                       0, pst_symtab_language);
          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)
-         && cs->c_naux == 1)
+      if (cs->c_sclass == C_EXT || cs->c_sclass == C_HIDEXT ||
+         cs->c_sclass == C_WEAKEXT)
        {
          /* Dealing with a symbol with a csect entry.  */
 
@@ -1057,9 +1151,41 @@ read_xcoff_symtab (struct partial_symtab *pst)
 #define        CSECT_SMTYP(PP) (SMTYP_SMTYP(CSECT(PP).x_smtyp))
 #define        CSECT_SCLAS(PP) (CSECT(PP).x_smclas)
 
-         /* Convert the auxent to something we can access.  */
-         bfd_coff_swap_aux_in (abfd, raw_auxptr, cs->c_type, cs->c_sclass,
-                               0, cs->c_naux, &main_aux);
+         /* Convert the auxent to something we can access.
+            XCOFF can have more than one auxiliary entries.
+
+            Actual functions will have two auxiliary entries, one to have the
+            function size and other to have the smtype/smclass (LD/PR).
+
+            c_type value of main symbol table will be set only in case of
+            C_EXT/C_HIDEEXT/C_WEAKEXT storage class symbols.
+            Bit 10 of type is set if symbol is a function, ie the value is set
+            to 32(0x20). So we need to read the first function auxiliary entry
+            which contains the size. */
+         if (cs->c_naux > 1 && ISFCN (cs->c_type))
+         {
+           /* a function entry point.  */
+
+           fcn_start_addr = cs->c_value;
+
+           /* save the function header info, which will be used
+              when `.bf' is seen.  */
+           fcn_cs_saved = *cs;
+
+           /* Convert the auxent to something we can access.  */
+           bfd_coff_swap_aux_in (abfd, raw_auxptr, cs->c_type, cs->c_sclass,
+                                 0, cs->c_naux, &fcn_aux_saved);
+           continue;
+         }
+         /* Read the csect auxiliary header, which is always the last by
+            convention. */
+         bfd_coff_swap_aux_in (abfd,
+                              raw_auxptr
+                              + ((coff_data (abfd)->local_symesz)
+                              * (cs->c_naux - 1)),
+                              cs->c_type, cs->c_sclass,
+                              cs->c_naux - 1, cs->c_naux,
+                              &main_aux);
 
          switch (CSECT_SMTYP (&main_aux))
            {
@@ -1078,31 +1204,32 @@ read_xcoff_symtab (struct partial_symtab *pst)
                    {
 
                      /* A program csect is seen.  We have to allocate one
-                        symbol table for each program csect.  Normally gdb
-                        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
-                        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
-                        to put all the code in the unnamed program csect.  */
+                        symbol table for each program csect.  Normally gdb
+                        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
+                        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
+                        to put all the code in the unnamed program csect.  */
 
                      if (last_csect_name)
                        {
                          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, SECT_OFF_TEXT (objfile));
                          end_stabs ();
                          start_stabs ();
                          /* Give all csects for this source file the same
                             name.  */
-                         start_symtab (filestring, NULL, (CORE_ADDR) 0);
+                         start_symtab (objfile, filestring, NULL,
+                                       0, pst_symtab_language);
                          record_debugformat (debugfmt);
                        }
 
                      /* If this is the very first csect seen,
-                        basically `__start'. */
+                        basically `__start'.  */
                      if (just_started)
                        {
                          first_object_file_end
@@ -1111,17 +1238,11 @@ read_xcoff_symtab (struct partial_symtab *pst)
                        }
 
                      file_start_addr =
-                       cs->c_value + ANOFFSET (objfile->section_offsets,
-                                               SECT_OFF_TEXT (objfile));
+                       cs->c_value + objfile->text_section_offset ();
                      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;
 
@@ -1148,20 +1269,15 @@ read_xcoff_symtab (struct partial_symtab *pst)
 
              switch (CSECT_SCLAS (&main_aux))
                {
+               /* We never really come to this part as this case has been
+                  handled in ISFCN check above.
+                  This and other cases of XTY_LD are kept just for
+                  reference. */
                case XMC_PR:
-                 /* 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. */
-                 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:
@@ -1173,7 +1289,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.  */
@@ -1190,28 +1306,17 @@ read_xcoff_symtab (struct partial_symtab *pst)
            }
        }
 
-      /* If explicitly specified as a function, treat is as one.  This check
-        evaluates to true for @FIX* bigtoc CSECT symbols, so it must occur
-        after the above CSECT check.  */
-      if (ISFCN (cs->c_type) && cs->c_sclass != C_TPDEF)
-       {
-         bfd_coff_swap_aux_in (abfd, raw_auxptr, cs->c_type, cs->c_sclass,
-                               0, cs->c_naux, &main_aux);
-         goto function_entry_point;
-       }
-
       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
@@ -1219,14 +1324,15 @@ 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, 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,
@@ -1237,12 +1343,13 @@ read_xcoff_symtab (struct partial_symtab *pst)
            filestring = cs->c_name;
 
          start_stabs ();
-         start_symtab (filestring, (char *) NULL, (CORE_ADDR) 0);
+         start_symtab (objfile, filestring, NULL, 0, pst_symtab_language);
          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;
 
@@ -1253,54 +1360,51 @@ read_xcoff_symtab (struct partial_symtab *pst)
        case C_FCN:
          if (strcmp (cs->c_name, ".bf") == 0)
            {
-             CORE_ADDR off = ANOFFSET (objfile->section_offsets,
-                                       SECT_OFF_TEXT (objfile));
+             CORE_ADDR off = objfile->text_section_offset ();
+
              bfd_coff_swap_aux_in (abfd, raw_auxptr, cs->c_type, cs->c_sclass,
                                    0, cs->c_naux, &main_aux);
 
              within_function = 1;
 
-             new = push_context (0, fcn_start_addr + off);
+             newobj = push_context (0, fcn_start_addr + off);
 
-             new->name = define_symbol
+             newobj->name = define_symbol
                (fcn_cs_saved.c_value + off,
                 fcn_stab_saved.c_name, 0, 0, objfile);
-             if (new->name != NULL)
-               SYMBOL_SECTION (new->name) = SECT_OFF_TEXT (objfile);
+             if (newobj->name != NULL)
+               newobj->name->set_section_index (SECT_OFF_TEXT (objfile));
            }
          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);
 
              /* The value of .ef is the address of epilogue code;
-                not useful for gdb.  */
+                not useful for gdb.  */
              /* { main_aux.x_sym.x_misc.x_lnsz.x_lnno
-                contains number of lines to '}' */
+                contains number of lines to '}' */
 
-             if (context_stack_depth <= 0)
-               {               /* We attempted to pop an empty context stack */
+             if (outermost_context_p ())
+               {       /* We attempted to pop an empty context stack.  */
                  ef_complaint (cs->c_symnum);
                  within_function = 0;
                  break;
                }
-             new = pop_context ();
+             struct context_stack cstk = pop_context ();
              /* Stack must be empty now.  */
-             if (context_stack_depth > 0 || new == NULL)
+             if (!outermost_context_p ())
                {
                  ef_complaint (cs->c_symnum);
                  within_function = 0;
                  break;
                }
 
-             finish_block (new->name, &local_symbols, new->old_blocks,
-                           new->start_addr,
+             finish_block (cstk.name, cstk.old_blocks,
+                           NULL, cstk.start_addr,
                            (fcn_cs_saved.c_value
                             + fcn_aux_saved.x_sym.x_misc.x_fsize
-                            + ANOFFSET (objfile->section_offsets,
-                                        SECT_OFF_TEXT (objfile))),
-                           objfile);
+                            + objfile->text_section_offset ()));
              within_function = 0;
            }
          break;
@@ -1308,12 +1412,12 @@ read_xcoff_symtab (struct partial_symtab *pst)
        case C_BSTAT:
          /* Begin static block.  */
          {
-           struct internal_syment symbol;
+           struct internal_syment static_symbol;
 
-           read_symbol (&symbol, cs->c_value);
-           static_block_base = symbol.n_value;
+           read_symbol (&static_symbol, cs->c_value);
+           static_block_base = static_symbol.n_value;
            static_block_section =
-             secnum_to_section (symbol.n_scnum, objfile);
+             secnum_to_section (static_symbol.n_scnum, objfile);
          }
          break;
 
@@ -1331,7 +1435,7 @@ read_xcoff_symtab (struct partial_symtab *pst)
        case C_UNTAG:
        case C_ENTAG:
          {
-           complaint (&symfile_complaints, _("Unrecognized storage class %d."),
+           complaint (_("Unrecognized storage class %d."),
                       cs->c_sclass);
          }
          break;
@@ -1348,9 +1452,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;
@@ -1365,35 +1469,33 @@ read_xcoff_symtab (struct partial_symtab *pst)
          if (strcmp (cs->c_name, ".bb") == 0)
            {
              depth++;
-             new = push_context (depth,
+             newobj = push_context (depth,
                                  (cs->c_value
-                                  + ANOFFSET (objfile->section_offsets,
-                                              SECT_OFF_TEXT (objfile))));
+                                  + objfile->text_section_offset ()));
            }
          else if (strcmp (cs->c_name, ".eb") == 0)
            {
-             if (context_stack_depth <= 0)
-               {               /* We attempted to pop an empty context stack */
+             if (outermost_context_p ())
+               {       /* We attempted to pop an empty context stack.  */
                  eb_complaint (cs->c_symnum);
                  break;
                }
-             new = pop_context ();
-             if (depth-- != new->depth)
+             struct context_stack cstk = pop_context ();
+             if (depth-- != cstk.depth)
                {
                  eb_complaint (cs->c_symnum);
                  break;
                }
-             if (local_symbols && context_stack_depth > 0)
+             if (*get_local_symbols () && !outermost_context_p ())
                {
                  /* Make a block for the local symbols within.  */
-                 finish_block (new->name, &local_symbols, new->old_blocks,
-                               new->start_addr,
+                 finish_block (cstk.name,
+                               cstk.old_blocks, NULL,
+                               cstk.start_addr,
                                (cs->c_value
-                                + ANOFFSET (objfile->section_offsets,
-                                            SECT_OFF_TEXT (objfile))),
-                               objfile);
+                                + objfile->text_section_offset ()));
                }
-             local_symbols = new->locals;
+             *get_local_symbols () = cstk.locals;
            }
          break;
 
@@ -1403,39 +1505,33 @@ read_xcoff_symtab (struct partial_symtab *pst)
        }
     }
 
-  if (last_source_file)
+  if (get_last_source_file ())
     {
-      struct symtab *s;
+      struct compunit_symtab *cust;
 
       complete_symtab (filestring, file_start_addr);
       cur_src_end_addr = file_end_addr;
-      s = end_symtab (file_end_addr, objfile, SECT_OFF_TEXT (objfile));
+      cust = end_symtab (file_end_addr, SECT_OFF_TEXT (objfile));
       /* When reading symbols for the last C_FILE of the objfile, try
-         to make sure that we set pst->symtab to the symtab for the
-         file, not to the _globals_ symtab.  I'm not sure whether this
-         actually works right or when/if it comes up.  */
-      if (pst->symtab == NULL)
-       pst->symtab = s;
+        to make sure that we set pst->compunit_symtab to the symtab for the
+        file, not to the _globals_ symtab.  I'm not sure whether this
+        actually works right or when/if it comes up.  */
+      if (pst->compunit_symtab == NULL)
+       pst->compunit_symtab = cust;
       end_stabs ();
     }
 }
 
-#define        SYMBOL_DUP(SYMBOL1, SYMBOL2)    \
-  (SYMBOL2) = (struct symbol *)                \
-       obstack_alloc (&objfile->objfile_obstack, sizeof (struct symbol)); \
-  *(SYMBOL2) = *(SYMBOL1);
-
-
 #define        SYMNAME_ALLOC(NAME, ALLOCED)    \
-  ((ALLOCED) ? (NAME) : obsavestring ((NAME), strlen (NAME), &objfile->objfile_obstack))
+  ((ALLOCED) ? (NAME) : obstack_strdup (&objfile->objfile_obstack, \
+                                       (NAME)))
 
 
-/* process one xcoff symbol. */
+/* process one xcoff symbol.  */
 
 static struct symbol *
 process_xcoff_symbol (struct coff_symbol *cs, struct objfile *objfile)
 {
-  struct gdbarch *gdbarch = get_objfile_arch (objfile);
   struct symbol onesymbol;
   struct symbol *sym = &onesymbol;
   struct symbol *sym2 = NULL;
@@ -1447,48 +1543,46 @@ process_xcoff_symbol (struct coff_symbol *cs, struct objfile *objfile)
   if (cs->c_secnum < 0)
     {
       /* The value is a register number, offset within a frame, etc.,
-         and does not get relocated.  */
+        and does not get relocated.  */
       off = 0;
       sec = -1;
     }
   else
     {
       sec = secnum_to_section (cs->c_secnum, objfile);
-      off = ANOFFSET (objfile->section_offsets, sec);
+      off = objfile->section_offsets[sec];
     }
 
   name = cs->c_name;
   if (name[0] == '.')
     ++name;
 
-  memset (sym, '\0', sizeof (struct symbol));
-
   /* default assumptions */
-  SYMBOL_VALUE_ADDRESS (sym) = cs->c_value + off;
+  SET_SYMBOL_VALUE_ADDRESS (sym, cs->c_value + off);
   SYMBOL_DOMAIN (sym) = VAR_DOMAIN;
-  SYMBOL_SECTION (sym) = secnum_to_section (cs->c_secnum, objfile);
+  sym->set_section_index (secnum_to_section (cs->c_secnum, objfile));
 
   if (ISFCN (cs->c_type))
     {
       /* At this point, we don't know the type of the function.  This
-         will be patched with the type from its stab entry later on in
-         patch_block_stabs (), unless the file was compiled without -g.  */
+        will be patched with the type from its stab entry later on in
+        patch_block_stabs (), unless the file was compiled without -g.  */
 
-      SYMBOL_SET_LINKAGE_NAME (sym, SYMNAME_ALLOC (name, symname_alloced));
-      SYMBOL_TYPE (sym) = builtin_type (gdbarch)->nodebug_text_symbol;
+      sym->set_linkage_name (SYMNAME_ALLOC (name, symname_alloced));
+      SYMBOL_TYPE (sym) = objfile_type (objfile)->nodebug_text_symbol;
 
-      SYMBOL_CLASS (sym) = LOC_BLOCK;
-      SYMBOL_DUP (sym, sym2);
+      SYMBOL_ACLASS_INDEX (sym) = LOC_BLOCK;
+      sym2 = new (&objfile->objfile_obstack) symbol (*sym);
 
-      if (cs->c_sclass == C_EXT)
-       add_symbol_to_list (sym2, &global_symbols);
+      if (cs->c_sclass == C_EXT || C_WEAKEXT)
+       add_symbol_to_list (sym2, get_global_symbols ());
       else if (cs->c_sclass == C_HIDEXT || cs->c_sclass == C_STAT)
-       add_symbol_to_list (sym2, &file_symbols);
+       add_symbol_to_list (sym2, get_file_symbols ());
     }
   else
     {
-      /* In case we can't figure out the type, provide default. */
-      SYMBOL_TYPE (sym) = builtin_type (gdbarch)->nodebug_data_symbol;
+      /* In case we can't figure out the type, provide default.  */
+      SYMBOL_TYPE (sym) = objfile_type (objfile)->nodebug_data_symbol;
 
       switch (cs->c_sclass)
        {
@@ -1516,7 +1610,7 @@ process_xcoff_symbol (struct coff_symbol *cs, struct objfile *objfile)
          break;
 
        default:
-         complaint (&symfile_complaints, _("Unexpected storage class: %d"),
+         complaint (_("Unexpected storage class: %d"),
                     cs->c_sclass);
          /* FALLTHROUGH */
 
@@ -1532,7 +1626,7 @@ process_xcoff_symbol (struct coff_symbol *cs, struct objfile *objfile)
            sym = define_symbol (cs->c_value + off, cs->c_name, 0, 0, objfile);
            if (sym != NULL)
              {
-               SYMBOL_SECTION (sym) = sec;
+               sym->set_section_index (sec);
              }
            return sym;
          }
@@ -1546,20 +1640,25 @@ process_xcoff_symbol (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;
          if (*pp == 'V' && !within_function)
            *pp = 'S';
          sym = define_symbol ((cs->c_value
-                               + ANOFFSET (objfile->section_offsets,
-                                           static_block_section)),
+                               + objfile->section_offsets[static_block_section]),
                               cs->c_name, 0, 0, objfile);
          if (sym != NULL)
            {
-             SYMBOL_VALUE_ADDRESS (sym) += static_block_base;
-             SYMBOL_SECTION (sym) = static_block_section;
+             SET_SYMBOL_VALUE_ADDRESS (sym,
+                                       SYMBOL_VALUE_ADDRESS (sym)
+                                       + static_block_base);
+             sym->set_section_index (static_block_section);
            }
          return sym;
 
@@ -1577,9 +1676,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->deprecated_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);
@@ -1592,18 +1690,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->deprecated_sym_private)->symtbl_num_syms;
-  char *stbl =
-  ((struct coff_symfile_info *) this_symtab_psymtab->objfile->deprecated_sym_private)->symtbl;
+  struct xcoff_symfile_info *xcoff = XCOFF_DATA (this_symtab_objfile);
+  int nsyms = xcoff->symtbl_num_syms;
+  char *stbl = xcoff->symtbl;
+
   if (symno < 0 || symno >= nsyms)
     {
-      complaint (&symfile_complaints, _("Invalid symbol offset"));
+      complaint (_("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);
 }
@@ -1626,11 +1724,10 @@ read_symbol_nvalue (int symno)
 static int
 read_symbol_lineno (int symno)
 {
-  struct objfile *objfile = this_symtab_psymtab->objfile;
+  struct objfile *objfile = this_symtab_objfile;
   int xcoff64 = bfd_xcoff_is_xcoff64 (objfile->obfd);
 
-  struct coff_symfile_info *info =
-    (struct coff_symfile_info *)objfile->deprecated_sym_private;
+  struct xcoff_symfile_info *info = XCOFF_DATA (objfile);
   int nsyms = info->symtbl_num_syms;
   char *stbl = info->symtbl;
   char *strtbl = info->strtbl;
@@ -1667,6 +1764,7 @@ read_symbol_lineno (int symno)
       if (symbol->n_sclass == C_FCN)
        {
          char *name = xcoff64 ? strtbl + symbol->n_offset : symbol->n_name;
+
          if (strcmp (name, ".bf") == 0)
            goto gotit;
        }
@@ -1677,7 +1775,7 @@ read_symbol_lineno (int symno)
   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,
@@ -1686,7 +1784,7 @@ 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
@@ -1695,7 +1793,7 @@ gotit:
 static void
 find_linenos (struct bfd *abfd, struct bfd_section *asect, void *vpinfo)
 {
-  struct coff_symfile_info *info;
+  struct xcoff_symfile_info *info;
   int size, count;
   file_ptr offset, maxoff;
 
@@ -1705,7 +1803,7 @@ find_linenos (struct bfd *abfd, struct bfd_section *asect, void *vpinfo)
     return;
 
   size = count * coff_data (abfd)->local_linesz;
-  info = (struct coff_symfile_info *) vpinfo;
+  info = (struct xcoff_symfile_info *) vpinfo;
   offset = asect->line_filepos;
   maxoff = offset + size;
 
@@ -1716,103 +1814,44 @@ find_linenos (struct bfd *abfd, struct bfd_section *asect, void *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_expand_psymtab (legacy_psymtab *pst, struct objfile *objfile)
 {
-  struct cleanup *old_chain;
-  int i;
-
-  if (!pst)
-    return;
-
-  if (pst->readin)
-    {
-      fprintf_unfiltered
-       (gdb_stderr, "Psymtab for %s already read in.  Shouldn't happen.\n",
-        pst->filename);
-      return;
-    }
+  gdb_assert (!pst->readin);
 
-  /* 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)
-      {
-       /* Inform about additional files that need to be read in.  */
-       if (info_verbose)
-         {
-           fputs_filtered (" ", gdb_stdout);
-           wrap_here ("");
-           fputs_filtered ("and ", gdb_stdout);
-           wrap_here ("");
-           printf_filtered ("%s...", pst->dependencies[i]->filename);
-           wrap_here ("");     /* Flush output */
-           gdb_flush (gdb_stdout);
-         }
-       xcoff_psymtab_to_symtab_1 (pst->dependencies[i]);
-      }
+  /* Read in all partial symtabs on which this one is dependent.  */
+  pst->expand_dependencies (objfile);
 
   if (((struct symloc *) pst->read_symtab_private)->numsyms != 0)
     {
       /* Init stuff necessary for reading in symbols.  */
       stabsread_init ();
-      buildsym_init ();
-      old_chain = make_cleanup (really_free_pendings, 0);
 
-      read_xcoff_symtab (pst);
-
-      do_cleanups (old_chain);
+      scoped_free_pendings free_pending;
+      read_xcoff_symtab (objfile, pst);
     }
 
-  pst->readin = 1;
+  pst->readin = true;
 }
 
-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.  */
+   Be verbose about it if the user wants that.  SELF is not NULL.  */
 
 static void
-xcoff_psymtab_to_symtab (struct partial_symtab *pst)
+xcoff_read_symtab (legacy_psymtab *self, struct objfile *objfile)
 {
-  bfd *sym_bfd;
-
-  if (!pst)
-    return;
-
-  if (pst->readin)
-    {
-      fprintf_unfiltered
-       (gdb_stderr, "Psymtab for %s already read in.  Shouldn't happen.\n",
-        pst->filename);
-      return;
-    }
+  gdb_assert (!self->readin);
 
-  if (((struct symloc *) pst->read_symtab_private)->numsyms != 0
-      || pst->number_of_dependencies)
+  if (((struct symloc *) self->read_symtab_private)->numsyms != 0
+      || self->number_of_dependencies)
     {
-      /* Print the message now, before reading the string table,
-         to avoid disconcerting pauses.  */
-      if (info_verbose)
-       {
-         printf_filtered ("Reading in symbols for %s...", pst->filename);
-         gdb_flush (gdb_stdout);
-       }
-
-      sym_bfd = pst->objfile->obfd;
-
       next_symbol_text_func = xcoff_next_symbol_text;
 
-      xcoff_psymtab_to_symtab_1 (pst);
+      self->expand_psymtab (objfile);
 
       /* 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);
-
-      /* Finish up the debug error message.  */
-      if (info_verbose)
-       printf_filtered ("done.\n");
+        after all of the symtabs and dependencies have been read in.   */
+      scan_file_globals (objfile);
     }
 }
 \f
@@ -1820,7 +1859,6 @@ static void
 xcoff_new_init (struct objfile *objfile)
 {
   stabsread_new_init ();
-  buildsym_new_init ();
 }
 
 /* Do initialization in preparation for reading symbols from OBJFILE.
@@ -1832,30 +1870,23 @@ xcoff_new_init (struct objfile *objfile)
 static void
 xcoff_symfile_init (struct objfile *objfile)
 {
-  /* Allocate struct to keep track of the symfile */
-  objfile->deprecated_sym_private = xmalloc (sizeof (struct coff_symfile_info));
+  /* Allocate struct to keep track of the symfile */
+  xcoff_objfile_data_key.emplace (objfile);
 
   /* 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->deprecated_sym_private != NULL)
-    {
-      xfree (objfile->deprecated_sym_private);
-    }
-
   /* Start with a fresh include table for the next objfile.  */
   if (inclTable)
     {
@@ -1873,8 +1904,9 @@ init_stringtab (bfd *abfd, file_ptr offset, struct objfile *objfile)
   int val;
   unsigned char lengthbuf[4];
   char *strtbl;
+  struct xcoff_symfile_info *xcoff = XCOFF_DATA (objfile);
 
-  ((struct coff_symfile_info *) objfile->deprecated_sym_private)->strtbl = NULL;
+  xcoff->strtbl = NULL;
 
   if (bfd_seek (abfd, offset, SEEK_SET) < 0)
     error (_("cannot seek to string table in %s: %s"),
@@ -1889,11 +1921,11 @@ init_stringtab (bfd *abfd, file_ptr offset, struct objfile *objfile)
   if (val != sizeof lengthbuf || length < sizeof lengthbuf)
     return;
 
-  /* Allocate string table from objfile_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->objfile_obstack, length);
-  ((struct coff_symfile_info *) objfile->deprecated_sym_private)->strtbl = strtbl;
+  xcoff->strtbl = strtbl;
 
   /* Copy length buffer, the first byte is usually zero and is
      used for stabs with a name length of zero.  */
@@ -1907,7 +1939,8 @@ init_stringtab (bfd *abfd, file_ptr offset, struct objfile *objfile)
     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;
 }
@@ -1917,45 +1950,33 @@ 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,
-                    struct partial_symbol **global_syms,
-                    struct partial_symbol **static_syms)
+static legacy_psymtab *
+xcoff_start_psymtab (struct objfile *objfile,
+                    const char *filename, int first_symnum)
 {
-  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->objfile_obstack, sizeof (struct symloc));
+  /* We fill in textlow later.  */
+  legacy_psymtab *result = new legacy_psymtab (filename, objfile, 0);
+
+  result->read_symtab_private =
+    XOBNEW (&objfile->objfile_obstack, struct symloc);
   ((struct symloc *) result->read_symtab_private)->first_symnum = first_symnum;
-  result->read_symtab = xcoff_psymtab_to_symtab;
+  result->legacy_read_symtab = xcoff_read_symtab;
+  result->legacy_expand_psymtab = xcoff_expand_psymtab;
 
-  /* 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).
@@ -1963,14 +1984,14 @@ static struct partial_symtab *xcoff_end_psymtab
    INCLUDE_LIST, NUM_INCLUDES, DEPENDENCY_LIST, and NUMBER_DEPENDENCIES
    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,
-                  struct partial_symtab **dependency_list,
+static legacy_psymtab *
+xcoff_end_psymtab (struct objfile *objfile, legacy_psymtab *pst,
+                  const char **include_list, int num_includes,
+                  int capping_symbol_number,
+                  legacy_psymtab **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 =
@@ -1979,104 +2000,78 @@ 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->end ();
 
   pst->number_of_dependencies = number_dependencies;
   if (number_dependencies)
     {
-      pst->dependencies = (struct partial_symtab **)
-       obstack_alloc (&objfile->objfile_obstack,
-                   number_dependencies * sizeof (struct partial_symtab *));
+      pst->dependencies
+       = objfile->partial_symtabs->allocate_dependencies (number_dependencies);
       memcpy (pst->dependencies, dependency_list,
-             number_dependencies * sizeof (struct partial_symtab *));
+             number_dependencies * sizeof (legacy_psymtab *));
     }
   else
     pst->dependencies = 0;
 
   for (i = 0; i < num_includes; i++)
     {
-      struct partial_symtab *subpst =
-      allocate_psymtab (include_list[i], objfile);
+      legacy_psymtab *subpst =
+       new legacy_psymtab (include_list[i], objfile);
 
-      subpst->section_offsets = pst->section_offsets;
-      subpst->read_symtab_private =
-       (char *) obstack_alloc (&objfile->objfile_obstack,
-                               sizeof (struct symloc));
+      subpst->read_symtab_private = XOBNEW (&objfile->objfile_obstack, symloc);
       ((struct symloc *) subpst->read_symtab_private)->first_symnum = 0;
       ((struct symloc *) subpst->read_symtab_private)->numsyms = 0;
-      subpst->textlow = 0;
-      subpst->texthigh = 0;
 
       /* 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->objfile_obstack,
-                      sizeof (struct partial_symtab *));
+        shared by the entire set of include files.  FIXME-someday.  */
+      subpst->dependencies =
+       objfile->partial_symtabs->allocate_dependencies (1);
       subpst->dependencies[0] = pst;
       subpst->number_of_dependencies = 1;
 
-      subpst->globals_offset =
-       subpst->n_global_syms =
-       subpst->statics_offset =
-       subpst->n_static_syms = 0;
-
-      subpst->readin = 0;
-      subpst->symtab = 0;
-      subpst->read_symtab = pst->read_symtab;
+      subpst->legacy_read_symtab = pst->legacy_read_symtab;
+      subpst->legacy_expand_psymtab = pst->legacy_expand_psymtab;
     }
 
-  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);
-
   if (num_includes == 0
       && number_dependencies == 0
-      && pst->n_global_syms == 0
-      && pst->n_static_syms == 0)
+      && pst->empty ())
     {
-      /* Throw away this psymtab, it's empty.  We can't deallocate it, since
-         it is on the obstack, but we can forget to chain it on the list.  */
+      /* Throw away this psymtab, it's empty.  */
       /* Empty psymtabs happen as a result of header files which don't have
-         any symbols in them.  There can be a lot of them.  */
+        any symbols in them.  There can be a lot of them.  */
 
-      discard_psymtab (pst);
+      objfile->partial_symtabs->discard_psymtab (pst);
 
       /* Indicate that psymtab was thrown away.  */
-      pst = (struct partial_symtab *) NULL;
+      pst = NULL;
     }
   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);
   if (symbol->n_zeroes)
     {
       /* If it's exactly E_SYMNMLEN characters long it isn't
-         '\0'-terminated.  */
+        '\0'-terminated.  */
       if (symbol->n_name[E_SYMNMLEN - 1] != '\0')
        {
          /* FIXME: wastes memory for symbols which we don't end up putting
             into the minimal symbols.  */
          char *p;
-         p = obstack_alloc (&objfile->objfile_obstack, E_SYMNMLEN + 1);
+
+         p = (char *) obstack_alloc (&objfile->objfile_obstack,
+                                     E_SYMNMLEN + 1);
          strncpy (p, symbol->n_name, E_SYMNMLEN);
          p[E_SYMNMLEN] = '\0';
          *name = p;
@@ -2088,13 +2083,11 @@ swap_sym (struct internal_syment *symbol, union internal_auxent *aux,
     }
   else if (symbol->n_sclass & 0x80)
     {
-      *name = ((struct coff_symfile_info *) objfile->deprecated_sym_private)->debugsec
-       + symbol->n_offset;
+      *name = XCOFF_DATA (objfile)->debugsec + symbol->n_offset;
     }
   else
     {
-      *name = ((struct coff_symfile_info *) objfile->deprecated_sym_private)->strtbl
-       + symbol->n_offset;
+      *name = XCOFF_DATA (objfile)->strtbl + symbol->n_offset;
     }
   ++*symnump;
   *raw += coff_data (objfile->obfd)->local_symesz;
@@ -2111,34 +2104,33 @@ 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"),
+  complaint (_("function `%s' appears to be defined "
+              "outside of all compilation units"),
             arg1);
 }
 
 static void
-scan_xcoff_symtab (struct objfile *objfile)
+scan_xcoff_symtab (minimal_symbol_reader &reader,
+                  struct objfile *objfile)
 {
-  struct gdbarch *gdbarch = get_objfile_arch (objfile);
-  CORE_ADDR toc_offset = 0;    /* toc offset value in data section. */
-  char *filestring = NULL;
+  CORE_ADDR toc_offset = 0;    /* toc offset value in data section.  */
+  const char *filestring = NULL;
 
-  char *namestring;
-  int past_first_source_file = 0;
+  const char *namestring;
   bfd *abfd;
   asection *bfd_sect;
   unsigned int nsyms;
 
   /* Current partial symtab */
-  struct partial_symtab *pst;
+  legacy_psymtab *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 */
-  struct partial_symtab **dependency_list;
+  /* Index within current psymtab dependency list */
+  legacy_psymtab **dependency_list;
   int dependencies_used, dependencies_allocated;
 
   char *sraw_symbol;
@@ -2146,32 +2138,32 @@ 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;
+  pst = (legacy_psymtab *) 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;
   dependency_list =
-    (struct partial_symtab **) alloca (dependencies_allocated *
-                                      sizeof (struct partial_symtab *));
+    (legacy_psymtab **) alloca (dependencies_allocated *
+                                      sizeof (legacy_psymtab *));
 
-  last_source_file = NULL;
+  set_last_source_file (NULL);
 
   abfd = objfile->obfd;
   next_symbol_text_func = xcoff_next_symbol_text;
 
-  sraw_symbol = ((struct coff_symfile_info *) objfile->deprecated_sym_private)->symtbl;
-  nsyms = ((struct coff_symfile_info *) objfile->deprecated_sym_private)->symtbl_num_syms;
+  sraw_symbol = XCOFF_DATA (objfile)->symtbl;
+  nsyms = XCOFF_DATA (objfile)->symtbl_num_syms;
   ssymnum = 0;
   while (ssymnum < nsyms)
     {
@@ -2186,6 +2178,7 @@ scan_xcoff_symtab (struct objfile *objfile)
        {
        case C_EXT:
        case C_HIDEXT:
+       case C_WEAKEXT:
          {
            /* The CSECT auxent--always the last auxent.  */
            union internal_auxent csect_aux;
@@ -2221,16 +2214,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
-                             (last_csect_name, last_csect_val,
-                              mst_text, last_csect_sec,
-                              objfile);
+                           record_minimal_symbol
+                             (reader, last_csect_name, last_csect_val,
+                              mst_text, last_csect_sec, objfile);
+                           misc_func_recorded = 1;
                          }
 
                        if (pst != NULL)
@@ -2239,8 +2232,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;
@@ -2249,9 +2242,7 @@ scan_xcoff_symtab (struct objfile *objfile)
                            pst = xcoff_start_psymtab
                              (objfile,
                               filestring,
-                              symnum_before,
-                              objfile->global_psymbols.next,
-                              objfile->static_psymbols.next);
+                              symnum_before);
                          }
                      }
                    /* Activate the misc_func_recorded mechanism for
@@ -2262,17 +2253,18 @@ 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;
-                       if (highval > pst->texthigh)
-                         pst->texthigh = highval;
-                       if (pst->textlow == 0 || symbol.n_value < pst->textlow)
-                         pst->textlow = symbol.n_value;
+                         symbol.n_value + csect_aux.x_csect.x_scnlen.l;
+
+                       if (highval > pst->raw_text_high ())
+                         pst->set_text_high (highval);
+                       if (!pst->text_low_valid
+                           || symbol.n_value < pst->raw_text_low ())
+                         pst->set_text_low (symbol.n_value);
                      }
                    misc_func_recorded = 0;
                    break;
@@ -2282,11 +2274,10 @@ scan_xcoff_symtab (struct objfile *objfile)
                    /* Data variables are recorded in the minimal symbol
                       table, except for section symbols.  */
                    if (*namestring != '.')
-                     prim_record_minimal_symbol_and_info
-                       (namestring, symbol.n_value,
+                     record_minimal_symbol
+                       (reader, namestring, symbol.n_value,
                         sclass == C_HIDEXT ? mst_file_data : mst_data,
-                        secnum_to_section (symbol.n_scnum, objfile),
-                        NULL, objfile);
+                        symbol.n_scnum, objfile);
                    break;
 
                  case XMC_TC0:
@@ -2294,10 +2285,11 @@ scan_xcoff_symtab (struct objfile *objfile)
                      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);
+                     toc_offset -= bfd_section_vma (bfd_sect);
                    break;
 
                  case XMC_TC:
@@ -2319,26 +2311,26 @@ 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,
+
+                   record_minimal_symbol
+                     (reader, namestring, symbol.n_value,
                       sclass == C_HIDEXT ? mst_file_text : mst_text,
-                      secnum_to_section (symbol.n_scnum, objfile),
-                      objfile);
+                      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
-                     (namestring, symbol.n_value,
-                      mst_solib_trampoline,
-                      secnum_to_section (symbol.n_scnum, objfile),
-                      objfile);
+                      mst_solib_trampoline.  */
+                   record_minimal_symbol
+                     (reader, namestring, symbol.n_value,
+                      mst_solib_trampoline, symbol.n_scnum, objfile);
+                   misc_func_recorded = 1;
                    break;
 
                  case XMC_DS:
@@ -2357,11 +2349,10 @@ scan_xcoff_symtab (struct objfile *objfile)
                       typically be XMC_RW; I suspect XMC_RO and
                       XMC_BS might be possible too.  */
                    if (*namestring != '.')
-                     prim_record_minimal_symbol_and_info
-                       (namestring, symbol.n_value,
+                     record_minimal_symbol
+                       (reader, namestring, symbol.n_value,
                         sclass == C_HIDEXT ? mst_file_data : mst_data,
-                        secnum_to_section (symbol.n_scnum, objfile),
-                        NULL, objfile);
+                        symbol.n_scnum, objfile);
                    break;
                  }
                break;
@@ -2374,11 +2365,10 @@ scan_xcoff_symtab (struct objfile *objfile)
                    /* Common variables are recorded in the minimal symbol
                       table, except for section symbols.  */
                    if (*namestring != '.')
-                     prim_record_minimal_symbol_and_info
-                       (namestring, symbol.n_value,
+                     record_minimal_symbol
+                       (reader, namestring, symbol.n_value,
                         sclass == C_HIDEXT ? mst_file_bss : mst_bss,
-                        secnum_to_section (symbol.n_scnum, objfile),
-                        NULL, objfile);
+                        symbol.n_scnum, objfile);
                    break;
                  }
                break;
@@ -2400,21 +2390,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 (reader, 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;
              }
@@ -2434,23 +2424,21 @@ scan_xcoff_symtab (struct objfile *objfile)
 
            pst = xcoff_start_psymtab (objfile,
                                       filestring,
-                                      symnum_before,
-                                      objfile->global_psymbols.next,
-                                      objfile->static_psymbols.next);
+                                      symnum_before);
            last_csect_name = NULL;
          }
          break;
 
        default:
          {
-           complaint (&symfile_complaints,
-                      _("Storage class %d not recognized during scan"), sclass);
+           complaint (_("Storage class %d not recognized during scan"),
+                      sclass);
          }
          /* FALLTHROUGH */
 
+       case C_FCN:
          /* C_FCN is .bf and .ef symbols.  I think it is sufficient
             to handle only the C_FUN and C_EXT.  */
-       case C_FCN:
 
        case C_BSTAT:
        case C_ESTAT:
@@ -2494,6 +2482,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 +2490,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 +2509,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.
 
@@ -2529,8 +2519,10 @@ scan_xcoff_symtab (struct objfile *objfile)
               in a binary tree, if profiling shows this is a major hog).  */
            if (pst && strcmp (namestring, pst->filename) == 0)
              continue;
+
            {
              int i;
+
              for (i = 0; i < includes_used; i++)
                if (strcmp (namestring, psymtab_include_list[i]) == 0)
                  {
@@ -2543,13 +2535,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 *));
+                         sizeof (const char *));
                memcpy (psymtab_include_list, orig,
-                       includes_used * sizeof (char *));
+                       includes_used * sizeof (const char *));
              }
            continue;
          }
@@ -2564,11 +2556,12 @@ scan_xcoff_symtab (struct objfile *objfile)
        case C_DECL:
        case C_STSYM:
          {
-           char *p;
+           const 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.   */
 
@@ -2582,28 +2575,25 @@ scan_xcoff_symtab (struct objfile *objfile)
            switch (p[1])
              {
              case 'S':
-               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,
-                                    VAR_DOMAIN, LOC_STATIC,
-                                    &objfile->static_psymbols,
-                                    0, symbol.n_value,
-                                    psymtab_language, objfile);
+               pst->add_psymbol (gdb::string_view (namestring,
+                                                   p - namestring),
+                                 true, VAR_DOMAIN, LOC_STATIC,
+                                 SECT_OFF_DATA (objfile),
+                                 psymbol_placement::STATIC,
+                                 symbol.n_value,
+                                 psymtab_language, objfile);
                continue;
 
              case 'G':
-               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_DOMAIN, LOC_STATIC,
-                                    &objfile->global_psymbols,
-                                    0, symbol.n_value,
-                                    psymtab_language, objfile);
+                  wrong.  See the code that reads 'G's for symtabs.  */
+               pst->add_psymbol (gdb::string_view (namestring,
+                                                   p - namestring),
+                                 true, VAR_DOMAIN, LOC_STATIC,
+                                 SECT_OFF_DATA (objfile),
+                                 psymbol_placement::GLOBAL,
+                                 symbol.n_value,
+                                 psymtab_language, objfile);
                continue;
 
              case 'T':
@@ -2617,32 +2607,32 @@ scan_xcoff_symtab (struct objfile *objfile)
                    || (p == namestring + 1
                        && namestring[0] != ' '))
                  {
-                   add_psymbol_to_list (namestring, p - namestring,
-                                        STRUCT_DOMAIN, LOC_TYPEDEF,
-                                        &objfile->static_psymbols,
-                                        symbol.n_value, 0,
-                                        psymtab_language, objfile);
+                   pst->add_psymbol (gdb::string_view (namestring,
+                                                       p - namestring),
+                                     true, STRUCT_DOMAIN, LOC_TYPEDEF, -1,
+                                     psymbol_placement::STATIC,
+                                     0, psymtab_language, objfile);
                    if (p[2] == 't')
                      {
                        /* Also a typedef with the same name.  */
-                       add_psymbol_to_list (namestring, p - namestring,
-                                            VAR_DOMAIN, LOC_TYPEDEF,
-                                            &objfile->static_psymbols,
-                                            symbol.n_value, 0,
-                                            psymtab_language, objfile);
+                       pst->add_psymbol (gdb::string_view (namestring,
+                                                           p - namestring),
+                                         true, VAR_DOMAIN, LOC_TYPEDEF, -1,
+                                         psymbol_placement::STATIC,
+                                         0, psymtab_language, objfile);
                        p += 1;
                      }
                  }
                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_DOMAIN, LOC_TYPEDEF,
-                                        &objfile->static_psymbols,
-                                        symbol.n_value, 0,
-                                        psymtab_language, objfile);
+                   pst->add_psymbol (gdb::string_view (namestring,
+                                                       p - namestring),
+                                     true, VAR_DOMAIN, LOC_TYPEDEF, -1,
+                                     psymbol_placement::STATIC,
+                                     0, psymtab_language, objfile);
                  }
              check_enum:
                /* If this is an enumerated type, we need to
@@ -2669,7 +2659,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 (?).  */
@@ -2687,7 +2678,7 @@ scan_xcoff_symtab (struct objfile *objfile)
                       Accept either.  */
                    while (*p && *p != ';' && *p != ',')
                      {
-                       char *q;
+                       const char *q;
 
                        /* Check for and handle cretinous dbx symbol name
                           continuation!  */
@@ -2700,10 +2691,10 @@ 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_DOMAIN, LOC_CONST,
-                                            &objfile->static_psymbols, 0,
-                                            0, psymtab_language, objfile);
+                       pst->add_psymbol (gdb::string_view (p, q - p), true,
+                                         VAR_DOMAIN, LOC_CONST, -1,
+                                         psymbol_placement::STATIC,
+                                         0, psymtab_language, objfile);
                        /* Point past the name.  */
                        p = q;
                        /* Skip over the value.  */
@@ -2718,28 +2709,31 @@ scan_xcoff_symtab (struct objfile *objfile)
 
              case 'c':
                /* Constant, e.g. from "const" in Pascal.  */
-               add_psymbol_to_list (namestring, p - namestring,
-                                    VAR_DOMAIN, LOC_CONST,
-                                    &objfile->static_psymbols, symbol.n_value,
-                                    0, psymtab_language, objfile);
+               pst->add_psymbol (gdb::string_view (namestring,
+                                                   p - namestring),
+                                 true, VAR_DOMAIN, LOC_CONST, -1,
+                                 psymbol_placement::STATIC,
+                                 0, psymtab_language, objfile);
                continue;
 
              case 'f':
                if (! pst)
                  {
                    int name_len = p - namestring;
-                   char *name = xmalloc (name_len + 1);
+                   char *name = (char *) xmalloc (name_len + 1);
+
                    memcpy (name, namestring, name_len);
                    name[name_len] = '\0';
                    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_DOMAIN, LOC_BLOCK,
-                                    &objfile->static_psymbols,
-                                    0, symbol.n_value,
-                                    psymtab_language, objfile);
+               pst->add_psymbol (gdb::string_view (namestring,
+                                                   p - namestring),
+                                 true, VAR_DOMAIN, LOC_BLOCK,
+                                 SECT_OFF_TEXT (objfile),
+                                 psymbol_placement::STATIC,
+                                 symbol.n_value,
+                                 psymtab_language, objfile);
                continue;
 
                /* Global functions were ignored here, but now they
@@ -2749,7 +2743,8 @@ scan_xcoff_symtab (struct objfile *objfile)
                if (! pst)
                  {
                    int name_len = p - namestring;
-                   char *name = xmalloc (name_len + 1);
+                   char *name = (char *) xmalloc (name_len + 1);
+
                    memcpy (name, namestring, name_len);
                    name[name_len] = '\0';
                    function_outside_compilation_unit_complaint (name);
@@ -2757,18 +2752,19 @@ scan_xcoff_symtab (struct objfile *objfile)
                  }
 
                /* We need only the minimal symbols for these
-                  loader-generated definitions.   Keeping the global
+                  loader-generated definitions.  Keeping the global
                   symbols leads to "in psymbols but not in symbols"
-                  errors. */
-               if (strncmp (namestring, "@FIX", 4) == 0)
+                  errors.  */
+               if (startswith (namestring, "@FIX"))
                  continue;
 
-               symbol.n_value += ANOFFSET (objfile->section_offsets, SECT_OFF_TEXT (objfile));
-               add_psymbol_to_list (namestring, p - namestring,
-                                    VAR_DOMAIN, LOC_BLOCK,
-                                    &objfile->global_psymbols,
-                                    0, symbol.n_value,
-                                    psymtab_language, objfile);
+               pst->add_psymbol (gdb::string_view (namestring,
+                                                   p - namestring),
+                                 true, VAR_DOMAIN, LOC_BLOCK,
+                                 SECT_OFF_TEXT (objfile),
+                                 psymbol_placement::GLOBAL,
+                                 symbol.n_value,
+                                 psymtab_language, objfile);
                continue;
 
                /* Two things show up here (hopefully); static symbols of
@@ -2787,7 +2783,8 @@ scan_xcoff_symtab (struct objfile *objfile)
              case '8':
              case '9':
              case '-':
-             case '#':         /* for symbol identification (used in live ranges) */
+             case '#':         /* For symbol identification (used in
+                                  live ranges).  */
                continue;
 
              case ':':
@@ -2801,15 +2798,15 @@ 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.  */
 
-               complaint (&symfile_complaints,
-                          _("unknown symbol descriptor `%c'"), p[1]);
+               complaint (_("unknown symbol descriptor `%c'"), p[1]);
 
                /* Ignore it; perhaps it is an extension that we don't
                   know about.  */
@@ -2821,25 +2818,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->deprecated_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->deprecated_sym_private)->toc_offset;
+    return XCOFF_DATA (objfile)->toc_offset;
   return 0;
 }
 
@@ -2849,26 +2847,24 @@ 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, symfile_add_flags symfile_flags)
 {
   bfd *abfd;
   int val;
-  struct cleanup *back_to;
   int num_symbols;             /* # of symbols */
   file_ptr symtab_offset;      /* symbol table and */
   file_ptr stringtab_offset;   /* string table file offsets */
-  struct coff_symfile_info *info;
-  char *name;
+  struct xcoff_symfile_info *info;
+  const char *name;
   unsigned int size;
 
-  info = (struct coff_symfile_info *) objfile->deprecated_sym_private;
+  info = XCOFF_DATA (objfile);
   symfile_bfd = abfd = objfile->obfd;
-  name = objfile->name;
+  name = objfile_name (objfile);
 
   num_symbols = bfd_get_symcount (abfd);       /* # of symbols */
   symtab_offset = obj_sym_filepos (abfd);      /* symbol table file offset */
@@ -2884,32 +2880,33 @@ xcoff_initial_scan (struct objfile *objfile, int mainline)
       /* Read the string table.  */
       init_stringtab (abfd, stringtab_offset, objfile);
 
-      /* Read the .debug section, if present.  */
-      {
-       struct bfd_section *secp;
-       bfd_size_type length;
-       char *debugsec = NULL;
+      /* Read the .debug section, if present and if we're not ignoring
+        it.  */
+      if (!(objfile->flags & OBJF_READNEVER))
+       {
+         struct bfd_section *secp;
+         bfd_size_type length;
+         bfd_byte *debugsec = NULL;
 
-       secp = bfd_get_section_by_name (abfd, ".debug");
-       if (secp)
-         {
-           length = bfd_section_size (abfd, secp);
-           if (length)
-             {
-               debugsec =
-                 (char *) obstack_alloc (&objfile->objfile_obstack, length);
+         secp = bfd_get_section_by_name (abfd, ".debug");
+         if (secp)
+           {
+             length = bfd_section_size (secp);
+             if (length)
+               {
+                 debugsec
+                   = (bfd_byte *) obstack_alloc (&objfile->objfile_obstack,
+                                                 length);
 
-               if (!bfd_get_section_contents (abfd, secp, debugsec,
-                                              (file_ptr) 0, length))
-                 {
-                   error (_("Error reading .debug section of `%s': %s"),
-                          name, bfd_errmsg (bfd_get_error ()));
-                 }
-             }
-         }
-       ((struct coff_symfile_info *) objfile->deprecated_sym_private)->debugsec =
-         debugsec;
-      }
+                 if (!bfd_get_full_section_contents (abfd, secp, &debugsec))
+                   {
+                     error (_("Error reading .debug section of `%s': %s"),
+                            name, bfd_errmsg (bfd_get_error ()));
+                   }
+               }
+           }
+         info->debugsec = (char *) debugsec;
+       }
     }
 
   /* Read the symbols.  We keep them in core because we will want to
@@ -2919,91 +2916,74 @@ xcoff_initial_scan (struct objfile *objfile, int mainline)
     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->deprecated_sym_private)->symtbl =
-    obstack_alloc (&objfile->objfile_obstack, size);
-  ((struct coff_symfile_info *) objfile->deprecated_sym_private)->symtbl_num_syms =
-    num_symbols;
+  info->symtbl = (char *) obstack_alloc (&objfile->objfile_obstack, size);
+  info->symtbl_num_syms = num_symbols;
 
-  val = bfd_bread (((struct coff_symfile_info *) objfile->deprecated_sym_private)->symtbl,
-                  size, abfd);
+  val = bfd_bread (info->symtbl, size, abfd);
   if (val != size)
     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))
-    /* 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
-       include N_SLINE.  */
-    init_psymbol_list (objfile, num_symbols);
-
-  free_pending_blocks ();
-  back_to = make_cleanup (really_free_pendings, 0);
-
-  init_minimal_symbol_collection ();
-  make_cleanup_discard_minimal_symbols ();
+  scoped_free_pendings free_pending;
+  minimal_symbol_reader reader (objfile);
 
   /* Now that the symbol table data of the executable file are all in core,
      process them and define symbols accordingly.  */
 
-  scan_xcoff_symtab (objfile);
+  scan_xcoff_symtab (reader, 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);
+  reader.install ();
 
-  do_cleanups (back_to);
+  /* DWARF2 sections.  */
+
+  if (dwarf2_has_info (objfile, &dwarf2_xcoff_names))
+    dwarf2_build_psymtabs (objfile);
+
+  dwarf2_build_frame_info (objfile);
 }
 \f
 static void
-xcoff_symfile_offsets (struct objfile *objfile, struct section_addr_info *addrs)
+xcoff_symfile_offsets (struct objfile *objfile,
+                      const section_addr_info &addrs)
 {
-  asection *sect = NULL;
-  int i;
+  const char *first_section_name;
 
-  objfile->num_sections = bfd_count_sections (objfile->obfd);
-  objfile->section_offsets = (struct section_offsets *)
-    obstack_alloc (&objfile->objfile_obstack, 
-                  SIZEOF_N_SECTION_OFFSETS (objfile->num_sections));
+  default_symfile_offsets (objfile, addrs);
 
-  /* Initialize the section indexes for future use. */
-  sect = bfd_get_section_by_name (objfile->obfd, ".text");
-  if (sect) 
-    objfile->sect_index_text = sect->index;
+  /* Oneof the weird side-effects of default_symfile_offsets is that
+     it sometimes sets some section indices to zero for sections that,
+     in fact do not exist. See the body of default_symfile_offsets
+     for more info on when that happens. Undo that, as this then allows
+     us to test whether the associated section exists or not, and then
+     access it quickly (without searching it again).  */
 
-  sect = bfd_get_section_by_name (objfile->obfd, ".data");
-  if (sect) 
-    objfile->sect_index_data = sect->index;
+  if (objfile->section_offsets.empty ())
+    return; /* Is that even possible?  Better safe than sorry.  */
 
-  sect = bfd_get_section_by_name (objfile->obfd, ".bss");
-  if (sect) 
-    objfile->sect_index_bss = sect->index;
+  first_section_name = bfd_section_name (objfile->sections[0].the_bfd_section);
 
-  sect = bfd_get_section_by_name (objfile->obfd, ".rodata");
-  if (sect) 
-    objfile->sect_index_rodata = sect->index;
+  if (objfile->sect_index_text == 0
+      && strcmp (first_section_name, ".text") != 0)
+    objfile->sect_index_text = -1;
 
-  for (i = 0; i < objfile->num_sections; ++i)
-    {
-      /* syms_from_objfile kindly subtracts from addr the
-        bfd_section_vma of the .text section.  This strikes me as
-        wrong--whether the offset to be applied to symbol reading is
-        relative to the start address of the section depends on the
-        symbol format.  In any event, this whole "addr" concept is
-        pretty broken (it doesn't handle any section but .text
-        sensibly), so just ignore the addr parameter and use 0.
-        rs6000-nat.c will set the correct section offsets via
-        objfile_relocate.  */
-       (objfile->section_offsets)->offsets[i] = 0;
-    }
+  if (objfile->sect_index_data == 0
+      && strcmp (first_section_name, ".data") != 0)
+    objfile->sect_index_data = -1;
+
+  if (objfile->sect_index_bss == 0
+      && strcmp (first_section_name, ".bss") != 0)
+    objfile->sect_index_bss = -1;
+
+  if (objfile->sect_index_rodata == 0
+      && strcmp (first_section_name, ".rodata") != 0)
+    objfile->sect_index_rodata = -1;
 }
 
 /* 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
@@ -3015,21 +2995,83 @@ static struct sym_fns xcoff_sym_fns =
      xcoffread.c reads all the symbols and does in fact randomly access them
      (in C_BSTAT and line number processing).  */
 
-  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 */
-  default_symfile_segments,    /* sym_segments: Get segment information from
-                                  a file.  */
-  aix_process_linenos,          /* sym_read_linetable */
-  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_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;
+}
+
+void _initialize_xcoffread ();
 void
-_initialize_xcoffread (void)
+_initialize_xcoffread ()
 {
-  add_symtab_fns (&xcoff_sym_fns);
+  add_symtab_fns (bfd_target_xcoff_flavour, &xcoff_sym_fns);
 }
This page took 0.086275 seconds and 4 git commands to generate.