2003-01-18 Andrew Cagney <ac131313@redhat.com>
[deliverable/binutils-gdb.git] / gdb / xcoffread.c
index bde1c5d16c6afebd282d8db6baacc581e9ecf82e..34c506f56e42499f513c9037c432935c52eb0cfa 100644 (file)
@@ -1,30 +1,26 @@
 /* Read AIX xcoff symbol tables and convert to internal format, for GDB.
-   Copyright 1986, 1987, 1988, 1989, 1990, 1991, 1992, 1993, 1994, 1995
-            Free Software Foundation, Inc.
+   Copyright 1986, 1987, 1988, 1989, 1990, 1991, 1992, 1993, 1994, 1995,
+   1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003
+   Free Software Foundation, Inc.
    Derived from coffread.c, dbxread.c, and a lot of hacking.
    Contributed by IBM Corporation.
 
-This file is part of GDB.
+   This file is part of GDB.
 
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
 
-You should have received a copy of the GNU General Public License
-along with this program; if not, write to the Free Software
-Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
-
-/* Native only:  Need struct tbtable in <sys/debug.h> from host, and 
-                need xcoff_add_toc_to_loadinfo in rs6000-tdep.c from target.
-                need xcoff_init_loadinfo ditto.  
-   However, if you grab <sys/debug.h> and make it available on your
-   host, and define FAKING_RS6000, then this code will compile.  */
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 59 Temple Place - Suite 330,
+   Boston, MA 02111-1307, USA.  */
 
 #include "defs.h"
 #include "bfd.h"
@@ -34,16 +30,16 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include <ctype.h>
 #include "gdb_string.h"
 
-#include "obstack.h"
 #include <sys/param.h>
 #ifndef        NO_SYS_FILE
 #include <sys/file.h>
 #endif
 #include "gdb_stat.h"
-#include <sys/debug.h>
 
 #include "coff/internal.h"
 #include "libcoff.h"           /* FIXME, internal data from BFD */
+#include "coff/xcoff.h"
+#include "libxcoff.h"
 #include "coff/rs6000.h"
 
 #include "symtab.h"
@@ -52,6 +48,7 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include "objfiles.h"
 #include "buildsym.h"
 #include "stabsread.h"
+#include "expression.h"
 #include "complaints.h"
 
 #include "gdb-stabs.h"
@@ -59,84 +56,46 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 /* For interface with stabsread.c.  */
 #include "aout/stab_gnu.h"
 
-/* For interface with partial-stab.h.  */
-#define N_UNDF 0       /* Undefined symbol */
-#undef N_ABS
-#define N_ABS 2
-#define N_TEXT         4       /* Text sym -- defined at offset in text seg */
-#define N_DATA         6       /* Data sym -- defined at offset in data seg */
-#define N_BSS  8       /* BSS  sym -- defined at offset in zero'd seg */
-#define        N_COMM  0x12    /* Common symbol (visible after shared lib dynlink) */
-#define N_FN   0x1f    /* File name of .o file */
-#define        N_FN_SEQ 0x0C   /* N_FN from Sequent compilers (sigh) */
-/* Note: N_EXT can only be usefully OR-ed with N_UNDF, N_ABS, N_TEXT,
-   N_DATA, or N_BSS.  When the low-order bit of other types is set,
-   (e.g. N_WARNING versus N_FN), they are two different types.  */
-#define N_EXT  1       /* External symbol (as opposed to local-to-this-file) */
-#define N_INDR 0x0a
-
-/* The following symbols refer to set elements.
-   All the N_SET[ATDB] symbols with the same name form one set.
-   Space is allocated for the set in the text section, and each set
-   elements value is stored into one word of the space.
-   The first word of the space is the length of the set (number of elements).
-
-   The address of the set is made into an N_SETV symbol
-   whose name is the same as the name of the set.
-   This symbol acts like a N_DATA global symbol
-   in that it can satisfy undefined external references.  */
-
-/* These appear as input to LD, in a .o file.  */
-#define        N_SETA  0x14            /* Absolute set element symbol */
-#define        N_SETT  0x16            /* Text set element symbol */
-#define        N_SETD  0x18            /* Data set element symbol */
-#define        N_SETB  0x1A            /* Bss set element symbol */
-
-/* This is output from LD.  */
-#define N_SETV 0x1C            /* Pointer to set vector in data area.  */
 \f
 /* We put a pointer to this structure in the read_symtab_private field
    of the psymtab.  */
 
-struct symloc {
+struct symloc
+  {
 
-  /* First symbol number for this file.  */
+    /* First symbol number for this file.  */
 
-  int first_symnum;
+    int first_symnum;
 
-  /* Number of symbols in the section of the symbol table devoted to
-     this file's symbols (actually, the section bracketed may contain
-     more than just this file's symbols).  If numsyms is 0, the only
-     reason for this thing's existence is the dependency list.  Nothing
-     else will happen when it is read in.  */
+    /* Number of symbols in the section of the symbol table devoted to
+       this file's symbols (actually, the section bracketed may contain
+       more than just this file's symbols).  If numsyms is 0, the only
+       reason for this thing's existence is the dependency list.  Nothing
+       else will happen when it is read in.  */
 
-  int numsyms;
+    int numsyms;
 
-  /* Position of the start of the line number information for this psymtab.  */
-  unsigned int lineno_off;
-};
+    /* 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. */
 
 static enum language psymtab_language = language_unknown;
-
 \f
-/* 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;
-  unsigned char c_sclass;
-  int c_secnum;
-  unsigned int c_type;
-};
+/* Simplified internal version of coff symbol table information */
 
-/* The COFF line table, in raw form.  */
-static char *linetab = NULL;           /* Its actual contents */
-static long linetab_offset;            /* Its offset in the file */
-static unsigned long linetab_size;     /* Its size */
+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;
+    unsigned char c_sclass;
+    int c_secnum;
+    unsigned int c_type;
+  };
 
 /* last function's saved coff symbol `cs' */
 
@@ -168,281 +127,331 @@ int within_function;
    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 {
-  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;
+static unsigned local_symesz;
 
-  /* Pointer to debug section.  */
-  char *debugsec;
-
-  /* Pointer to the a.out symbol table.  */
-  char *symtbl;
+struct coff_symfile_info
+  {
+    file_ptr min_lineno_offset;        /* Where in file lowest line#s are */
+    file_ptr max_lineno_offset;        /* 1+last byte of line#s in file */
 
-  /* Number of symbols in symtbl.  */
-  int symtbl_num_syms;
-};
+    /* Pointer to the string table.  */
+    char *strtbl;
 
-static struct complaint rsym_complaint = 
-  {"Non-stab C_RSYM `%s' needs special handling", 0, 0};
+    /* Pointer to debug section.  */
+    char *debugsec;
 
-static struct complaint storclass_complaint =
-  {"Unexpected storage class: %d", 0, 0};
+    /* Pointer to the a.out symbol table.  */
+    char *symtbl;
 
-static struct complaint bf_notfound_complaint =
-  {"line numbers off, `.bf' symbol not found", 0, 0};
+    /* Number of symbols in symtbl.  */
+    int symtbl_num_syms;
 
-extern struct complaint ef_complaint;
-extern struct complaint eb_complaint;
+    /* Offset in data section to TOC anchor.  */
+    CORE_ADDR toc_offset;
+  };
 
 static void
-enter_line_range PARAMS ((struct subfile *, unsigned, unsigned,
-                         CORE_ADDR, CORE_ADDR, unsigned *));
+bf_notfound_complaint (void)
+{
+  complaint (&symfile_complaints, "line numbers off, `.bf' symbol not found");
+}
 
 static void
-init_stringtab PARAMS ((bfd *, file_ptr, struct objfile *));
+ef_complaint (int arg1)
+{
+  complaint (&symfile_complaints,
+            "Mismatched .ef symbol ignored starting at symnum %d", arg1);
+}
 
 static void
-xcoff_symfile_init PARAMS ((struct objfile *));
+eb_complaint (int arg1)
+{
+  complaint (&symfile_complaints,
+            "Mismatched .eb symbol ignored starting at symnum %d", arg1);
+}
 
-static void
-xcoff_new_init PARAMS ((struct objfile *));
+static void xcoff_initial_scan (struct objfile *, int);
 
-static void
-xcoff_symfile_finish PARAMS ((struct objfile *));
+static void scan_xcoff_symtab (struct objfile *);
+
+static char *xcoff_next_symbol_text (struct objfile *);
 
-static struct section_offsets *
-xcoff_symfile_offsets PARAMS ((struct objfile *, CORE_ADDR));
+static void record_include_begin (struct coff_symbol *);
 
 static void
-find_linenos PARAMS ((bfd *, sec_ptr, PTR));
+enter_line_range (struct subfile *, unsigned, unsigned,
+                 CORE_ADDR, CORE_ADDR, unsigned *);
 
-static char *
-coff_getfilename PARAMS ((union internal_auxent *, struct objfile *));
+static void init_stringtab (bfd *, file_ptr, struct objfile *);
+
+static void xcoff_symfile_init (struct objfile *);
+
+static void xcoff_new_init (struct objfile *);
+
+static void xcoff_symfile_finish (struct objfile *);
 
 static void
-read_symbol PARAMS ((struct internal_syment *, int));
+xcoff_symfile_offsets (struct objfile *, struct section_addr_info *addrs);
 
-static int
-read_symbol_lineno PARAMS ((int));
+static void find_linenos (bfd *, sec_ptr, void *);
 
-static int
-read_symbol_nvalue PARAMS ((int));
+static char *coff_getfilename (union internal_auxent *, struct objfile *);
 
-static struct symbol *
-process_xcoff_symbol PARAMS ((struct coff_symbol *, struct objfile *));
+static void read_symbol (struct internal_syment *, int);
 
-static void
-read_xcoff_symtab PARAMS ((struct partial_symtab *));
+static int read_symbol_lineno (int);
 
-static void
-add_stab_to_list PARAMS ((char *, struct pending_stabs **));
+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 *);
+
+#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 *);
+
+static void process_linenos (CORE_ADDR, CORE_ADDR);
 \f
+
 /* Translate from a COFF section number (target_index) to a SECT_OFF_*
    code.  */
-static int secnum_to_section PARAMS ((int, struct objfile *));
+static int secnum_to_section (int, struct objfile *);
+static asection *secnum_to_bfd_section (int, struct objfile *);
 
-struct find_targ_sec_arg {
-  int targ_index;
-  int *resultp;
-};
+struct find_targ_sec_arg
+  {
+    int targ_index;
+    int *resultp;
+    asection **bfd_sect;
+    struct objfile *objfile;
+  };
 
-static void find_targ_sec PARAMS ((bfd *, asection *, void *));
+static void find_targ_sec (bfd *, asection *, void *);
 
-static void find_targ_sec (abfd, sect, obj)
-     bfd *abfd;
-     asection *sect;
-     PTR obj;
+static void
+find_targ_sec (bfd *abfd, asection *sect, void *obj)
 {
-  struct find_targ_sec_arg *args = (struct find_targ_sec_arg *)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)
-       *args->resultp = SECT_OFF_TEXT;
+       *args->resultp = SECT_OFF_TEXT (objfile);
       else if (bfd_get_section_flags (abfd, sect) & SEC_LOAD)
-       *args->resultp = SECT_OFF_DATA;
+       *args->resultp = SECT_OFF_DATA (objfile);
       else
-       *args->resultp = SECT_OFF_BSS;
+       *args->resultp = sect->index;
+      *args->bfd_sect = sect;
     }
 }
 
 /* Return the section number (SECT_OFF_*) that CS points to.  */
 static int
-secnum_to_section (secnum, objfile)
-     int secnum;
-     struct objfile *objfile;
+secnum_to_section (int secnum, struct objfile *objfile)
 {
-  int off = SECT_OFF_TEXT;
+  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 off;
 }
+
+/* Return the BFD section that CS points to.  */
+static asection *
+secnum_to_bfd_section (int secnum, 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;
+}
 \f
 /* add a given stab string into given stab vector. */
 
+#if 0
+
 static void
-add_stab_to_list (stabname, stabvector)
-char *stabname;
-struct pending_stabs **stabvector;
+add_stab_to_list (char *stabname, struct pending_stabs **stabvector)
 {
-  if ( *stabvector == NULL) {
-    *stabvector = (struct pending_stabs *)
-       xmalloc (sizeof (struct pending_stabs) + 
-                       INITIAL_STABVECTOR_LENGTH * sizeof (char*));
-    (*stabvector)->count = 0;
-    (*stabvector)->length = INITIAL_STABVECTOR_LENGTH;
-  }
-  else if ((*stabvector)->count >= (*stabvector)->length) {
-    (*stabvector)->length += INITIAL_STABVECTOR_LENGTH;
-    *stabvector = (struct pending_stabs *)
-       xrealloc ((char *) *stabvector, sizeof (struct pending_stabs) + 
-       (*stabvector)->length * sizeof (char*));
-  }
-  (*stabvector)->stab [(*stabvector)->count++] = stabname;
+  if (*stabvector == NULL)
+    {
+      *stabvector = (struct pending_stabs *)
+       xmalloc (sizeof (struct pending_stabs) +
+                INITIAL_STABVECTOR_LENGTH * sizeof (char *));
+      (*stabvector)->count = 0;
+      (*stabvector)->length = INITIAL_STABVECTOR_LENGTH;
+    }
+  else if ((*stabvector)->count >= (*stabvector)->length)
+    {
+      (*stabvector)->length += INITIAL_STABVECTOR_LENGTH;
+      *stabvector = (struct pending_stabs *)
+       xrealloc ((char *) *stabvector, sizeof (struct pending_stabs) +
+                   (*stabvector)->length * sizeof (char *));
+    }
+  (*stabvector)->stab[(*stabvector)->count++] = stabname;
 }
-\f
+
+#endif
+\f/* *INDENT-OFF* */
 /* Linenos are processed on a file-by-file basis.
 
    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
-       determine if those functions (statics) are needed or not, and
-       can do some garbage collection (I think). This makes line
-       numbers and corresponding addresses unordered, and we end up
-       with a line table like:
-       
-
-               lineno  addr
-        foo()    10    0x100
-                 20    0x200
-                 30    0x300
-
-       foo3()    70    0x400
-                 80    0x500
-                 90    0x600
-
-       static foo2()
-                 40    0x700
-                 50    0x800
-                 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
-       should be on function based, since gcc can emit line numbers
-       like:
-       
-               10      0x100   - for the init/test part of a for stmt.
-               20      0x200
-               30      0x300
-               10      0x400   - for the increment part of a for stmt.
-
-       arrange_linetable() will do this sorting.               
-
-     2)        aix symbol table might look like:
-
-               c_file          // beginning of a new file
-               .bi             // beginning of include file
-               .ei             // end of include file
-               .bi
-               .ei
-
-       basically, .bi/.ei pairs do not necessarily encapsulate
-       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.  */
+   1) xlc (IBM's native c compiler) postpones static function code
+   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
+   numbers and corresponding addresses unordered, and we end up
+   with a line table like:
+
+
+   lineno       addr
+   foo()          10    0x100
+   20   0x200
+   30   0x300
+
+   foo3()         70    0x400
+   80   0x500
+   90   0x600
+
+   static foo2()
+   40   0x700
+   50   0x800
+   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
+   should be on function based, since gcc can emit line numbers
+   like:
+
+   10   0x100   - for the init/test part of a for stmt.
+   20   0x200
+   30   0x300
+   10   0x400   - for the increment part of a for stmt.
+
+   arrange_linetable() will do this sorting.            
+
+   2)   aix symbol table might look like:
+
+   c_file               // beginning of a new file
+   .bi          // beginning of include file
+   .ei          // end of include file
+   .bi
+   .ei
+
+   basically, .bi/.ei pairs do not necessarily encapsulate
+   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 (lte1, lte2)
-     struct linetable_entry *lte1, *lte2;
+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;
 }
 
-/* Give a line table with function entries are marked, arrange its functions
-   in assending order and strip off function entry markers and return it in
+/* 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.  */
 
 static struct linetable *
-arrange_linetable (oldLineTb)
-     struct linetable *oldLineTb;                      /* old linetable */
+arrange_linetable (struct linetable *oldLineTb)
 {
-  int ii, jj, 
-      newline,                                         /* new line count */
-      function_count;                          /* # of functions */
+  int ii, jj, newline,         /* new line count */
+    function_count;            /* # of functions */
 
-  struct linetable_entry *fentry;              /* function entry vector */
-  int fentry_size;                             /* # of function entries */
-  struct linetable *newLineTb;                 /* new line table */
+  struct linetable_entry *fentry;      /* function entry vector */
+  int fentry_size;             /* # of function entries */
+  struct linetable *newLineTb; /* new line table */
 
 #define NUM_OF_FUNCTIONS 20
 
   fentry_size = NUM_OF_FUNCTIONS;
-  fentry = (struct linetable_entry*)
+  fentry = (struct linetable_entry *)
     xmalloc (fentry_size * sizeof (struct linetable_entry));
 
-  for (function_count=0, ii=0; ii <oldLineTb->nitems; ++ii) {
+  for (function_count = 0, ii = 0; ii < oldLineTb->nitems; ++ii)
+    {
 
-    if (oldLineTb->item[ii].line == 0) {       /* function entry found. */
+      if (oldLineTb->item[ii].line == 0)
+       {                       /* function entry found. */
 
-      if (function_count >= fentry_size) {     /* make sure you have room. */
-       fentry_size *= 2;
-       fentry = (struct linetable_entry*) 
-         xrealloc (fentry, fentry_size * sizeof (struct linetable_entry));
-      }
-      fentry[function_count].line = ii;
-      fentry[function_count].pc = oldLineTb->item[ii].pc;
-      ++function_count;
+         if (function_count >= fentry_size)
+           {                   /* make sure you have room. */
+             fentry_size *= 2;
+             fentry = (struct linetable_entry *)
+               xrealloc (fentry, fentry_size * sizeof (struct linetable_entry));
+           }
+         fentry[function_count].line = ii;
+         fentry[function_count].pc = oldLineTb->item[ii].pc;
+         ++function_count;
+       }
     }
-  }
 
-  if (function_count == 0) {
-    free (fentry);
-    return oldLineTb;
-  }
+  if (function_count == 0)
+    {
+      xfree (fentry);
+      return oldLineTb;
+    }
   else if (function_count > 1)
-    qsort (fentry, function_count, sizeof(struct linetable_entry), compare_lte);
+    qsort (fentry, function_count, sizeof (struct linetable_entry), compare_lte);
 
   /* allocate a new line table. */
   newLineTb = (struct linetable *)
     xmalloc
-      (sizeof (struct linetable) + 
-       (oldLineTb->nitems - function_count) * sizeof (struct linetable_entry));
+    (sizeof (struct linetable) +
+    (oldLineTb->nitems - function_count) * sizeof (struct linetable_entry));
 
   /* if line table does not start with a function beginning, copy up until
      a function begin. */
 
   newline = 0;
   if (oldLineTb->item[0].line != 0)
-    for (newline=0; 
-       newline < oldLineTb->nitems && oldLineTb->item[newline].line; ++newline)
+    for (newline = 0;
+    newline < oldLineTb->nitems && oldLineTb->item[newline].line; ++newline)
       newLineTb->item[newline] = oldLineTb->item[newline];
 
   /* Now copy function lines one by one. */
 
-  for (ii=0; ii < function_count; ++ii) {
-    for (jj = fentry[ii].line + 1;
-                jj < oldLineTb->nitems && oldLineTb->item[jj].line != 0; 
-                                                        ++jj, ++newline)
-      newLineTb->item[newline] = oldLineTb->item[jj];
-  }
-  free (fentry);
+  for (ii = 0; ii < function_count; ++ii)
+    {
+      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;
-  return newLineTb;  
-}     
+  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),
@@ -450,63 +459,61 @@ arrange_linetable (oldLineTb)
    represent if (the include files. */
 
 
-typedef struct _inclTable {
-  char         *name;                          /* include filename */
+typedef struct _inclTable
+{
+  char *name;                  /* include filename */
 
   /* Offsets to the line table.  end points to the last entry which is
      part of this include file.  */
-  int          begin, end;
-  
+  int begin, end;
+
   struct subfile *subfile;
-  unsigned     funStartLine;                   /* start line # of its function */
-} InclTable;
+  unsigned funStartLine;       /* start line # of its function */
+}
+InclTable;
 
 #define        INITIAL_INCLUDE_TABLE_LENGTH    20
-static InclTable  *inclTable;                  /* global include table */
-static int       inclIndx;                     /* last entry to table */
-static int       inclLength;                   /* table length */
-static int       inclDepth;                    /* nested include depth */
+static InclTable *inclTable;   /* global include table */
+static int inclIndx;           /* last entry to table */
+static int inclLength;         /* table length */
+static int inclDepth;          /* nested include depth */
 
-static void allocate_include_entry PARAMS ((void));
+static void allocate_include_entry (void);
 
 static void
-record_include_begin (cs)
-struct coff_symbol *cs;
+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.  */
-      static struct complaint msg = {"Nested C_BINCL symbols", 0, 0};
-      complain (&msg);
+         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");
     }
   ++inclDepth;
 
   allocate_include_entry ();
 
-  inclTable [inclIndx].name  = cs->c_name;
-  inclTable [inclIndx].begin = cs->c_value;
+  inclTable[inclIndx].name = cs->c_name;
+  inclTable[inclIndx].begin = cs->c_value;
 }
 
 static void
-record_include_end (cs)
-struct coff_symbol *cs;
+record_include_end (struct coff_symbol *cs)
 {
-  InclTable *pTbl;  
+  InclTable *pTbl;
 
   if (inclDepth == 0)
     {
-      static struct complaint msg = {"Mismatched C_BINCL/C_EINCL pair", 0, 0};
-      complain (&msg);
+      complaint (&symfile_complaints, "Mismatched C_BINCL/C_EINCL pair");
     }
 
   allocate_include_entry ();
 
-  pTbl = &inclTable [inclIndx];
+  pTbl = &inclTable[inclIndx];
   pTbl->end = cs->c_value;
 
   --inclDepth;
@@ -514,11 +521,11 @@ struct coff_symbol *cs;
 }
 
 static void
-allocate_include_entry ()
+allocate_include_entry (void)
 {
   if (inclTable == NULL)
     {
-      inclTable = (InclTable *) 
+      inclTable = (InclTable *)
        xmalloc (sizeof (InclTable) * INITIAL_INCLUDE_TABLE_LENGTH);
       memset (inclTable,
              '\0', sizeof (InclTable) * INITIAL_INCLUDE_TABLE_LENGTH);
@@ -528,10 +535,10 @@ allocate_include_entry ()
   else if (inclIndx >= inclLength)
     {
       inclLength += INITIAL_INCLUDE_TABLE_LENGTH;
-      inclTable = (InclTable *) 
+      inclTable = (InclTable *)
        xrealloc (inclTable, sizeof (InclTable) * inclLength);
-      memset (inclTable + inclLength - INITIAL_INCLUDE_TABLE_LENGTH, 
-             '\0', sizeof (InclTable)*INITIAL_INCLUDE_TABLE_LENGTH);
+      memset (inclTable + inclLength - INITIAL_INCLUDE_TABLE_LENGTH,
+             '\0', sizeof (InclTable) * INITIAL_INCLUDE_TABLE_LENGTH);
     }
 }
 
@@ -543,14 +550,12 @@ static struct partial_symtab *this_symtab_psymtab;
    at times) process its lines and create appropriate line vectors. */
 
 static void
-process_linenos (start, end)
-     CORE_ADDR start, end;
+process_linenos (CORE_ADDR start, CORE_ADDR end)
 {
-  char *pp;
   int offset, ii;
   file_ptr max_offset =
-    ((struct coff_symfile_info *)this_symtab_psymtab->objfile->sym_private)
-      ->max_lineno_offset;
+  ((struct coff_symfile_info *) this_symtab_psymtab->objfile->sym_private)
+  ->max_lineno_offset;
 
   /* subfile structure for the main compilation unit.  */
   struct subfile main_subfile;
@@ -560,13 +565,12 @@ process_linenos (start, end)
      All the following line numbers in the function are relative to
      this, and we record absolute line numbers in record_line().  */
 
-  int main_source_baseline = 0;
+  unsigned int main_source_baseline = 0;
 
   unsigned *firstLine;
-  CORE_ADDR addr;
 
   offset =
-    ((struct symloc *)this_symtab_psymtab->read_symtab_private)->lineno_off;
+    ((struct symloc *) this_symtab_psymtab->read_symtab_private)->lineno_off;
   if (offset == 0)
     goto return_after_cleanup;
 
@@ -575,14 +579,18 @@ process_linenos (start, end)
   if (inclIndx == 0)
     /* 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);
+    enter_line_range (&main_subfile, offset, 0, start, end,
+                     &main_source_baseline);
 
   else
     {
       /* There was source with line numbers in include files.  */
+
+      int linesz =
+       coff_data (this_symtab_psymtab->objfile->obfd)->local_linesz;
       main_source_baseline = 0;
-      for (ii=0; ii < inclIndx; ++ii)
+
+      for (ii = 0; ii < inclIndx; ++ii)
        {
          struct subfile *tmpSubfile;
 
@@ -590,7 +598,7 @@ process_linenos (start, end)
          if (offset < inclTable[ii].begin)
            {
              enter_line_range
-               (&main_subfile, offset, inclTable[ii].begin - LINESZ,
+               (&main_subfile, offset, inclTable[ii].begin - linesz,
                 start, 0, &main_source_baseline);
            }
 
@@ -603,18 +611,18 @@ process_linenos (start, end)
          firstLine = &(inclTable[ii].funStartLine);
 
          /* Enter include file's lines now.  */
-         enter_line_range (tmpSubfile, inclTable[ii].begin, 
+         enter_line_range (tmpSubfile, inclTable[ii].begin,
                            inclTable[ii].end, start, 0, firstLine);
 
          if (offset <= inclTable[ii].end)
-           offset = inclTable[ii].end + LINESZ;
+           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.  */
-      if (offset < max_offset + 1 - LINESZ)
+         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, 
+         enter_line_range (&main_subfile, offset, 0, start, end,
                            &main_source_baseline);
        }
     }
@@ -627,7 +635,7 @@ process_linenos (start, end)
       lv = main_subfile.line_vector;
 
       /* Line numbers are not necessarily ordered. xlc compilation will
-        put static function to the end. */
+         put static function to the end. */
 
       lineTb = arrange_linetable (lv);
       if (lv == lineTb)
@@ -638,19 +646,19 @@ process_linenos (start, end)
        }
       else
        {
-         free (lv);
+         xfree (lv);
          current_subfile->line_vector = lineTb;
        }
 
-      current_subfile->line_vector_length = 
+      current_subfile->line_vector_length =
        current_subfile->line_vector->nitems;
     }
 
   /* Now, process included files' line numbers.  */
 
-  for (ii=0; ii < inclIndx; ++ii)
+  for (ii = 0; ii < inclIndx; ++ii)
     {
-      if ((inclTable[ii].subfile)->line_vector) /* Useless if!!! FIXMEmgo */
+      if ((inclTable[ii].subfile)->line_vector)                /* Useless if!!! FIXMEmgo */
        {
          struct linetable *lineTb, *lv;
 
@@ -666,11 +674,11 @@ process_linenos (start, end)
          /* For the same include file, we might want to have more than one
             subfile.  This happens if we have something like:
 
-               ......
-               #include "foo.h"
-               ......
-               #include "foo.h"
-               ......
+            ......
+            #include "foo.h"
+            ......
+            #include "foo.h"
+            ......
 
             while foo.h including code in it. (stupid but possible)
             Since start_subfile() looks at the name and uses an
@@ -678,7 +686,7 @@ process_linenos (start, end)
             fool it.  */
 
 #if 0
-         start_subfile (inclTable[ii].name, (char*)0);
+         start_subfile (inclTable[ii].name, (char *) 0);
 #else
          {
            /* Pick a fake name that will produce the same results as this
@@ -687,33 +695,33 @@ process_linenos (start, end)
            char *fakename = strrchr (inclTable[ii].name, '.');
            if (fakename == NULL)
              fakename = " ?";
-           start_subfile (fakename, (char*)0);
-           free (current_subfile->name);
+           start_subfile (fakename, (char *) 0);
+           xfree (current_subfile->name);
          }
-         current_subfile->name = strdup (inclTable[ii].name);
+         current_subfile->name = xstrdup (inclTable[ii].name);
 #endif
 
          if (lv == lineTb)
            {
              current_subfile->line_vector =
                (struct linetable *) xrealloc
-                 (lv, (sizeof (struct linetable)
-                       + lv->nitems * sizeof (struct linetable_entry)));
+               (lv, (sizeof (struct linetable)
+                     + lv->nitems * sizeof (struct linetable_entry)));
 
            }
          else
            {
-             free (lv);
+             xfree (lv);
              current_subfile->line_vector = lineTb;
            }
 
-         current_subfile->line_vector_length = 
+         current_subfile->line_vector_length =
            current_subfile->line_vector->nitems;
-         start_subfile (pop_subfile (), (char*)0);
+         start_subfile (pop_subfile (), (char *) 0);
        }
     }
 
- return_after_cleanup:
+return_after_cleanup:
 
   /* We don't want to keep alloc/free'ing the global include file table.  */
   inclIndx = 0;
@@ -723,7 +731,7 @@ process_linenos (start, end)
 }
 
 void
-aix_process_linenos ()
+aix_process_linenos (void)
 {
   /* process line numbers and enter them into line vector */
   process_linenos (last_source_start_addr, cur_src_end_addr);
@@ -732,61 +740,61 @@ aix_process_linenos ()
 
 /* 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 (subfile, beginoffset, endoffset, startaddr, endaddr,
-                 firstLine)
-     struct subfile *subfile;
-     unsigned   beginoffset, endoffset;        /* offsets to line table */
-     CORE_ADDR  startaddr, endaddr;
-     unsigned   *firstLine;
+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)
 {
   unsigned int curoffset;
   CORE_ADDR addr;
-  struct external_lineno ext_lnno;
+  void *ext_lnno;
   struct internal_lineno int_lnno;
   unsigned int limit_offset;
   bfd *abfd;
+  int linesz;
 
   if (endoffset == 0 && startaddr == 0 && endaddr == 0)
     return;
   curoffset = beginoffset;
   limit_offset =
-    ((struct coff_symfile_info *)this_symtab_psymtab->objfile->sym_private)
-      ->max_lineno_offset;
+    ((struct coff_symfile_info *) this_symtab_psymtab->objfile->sym_private)
+    ->max_lineno_offset;
 
   if (endoffset != 0)
     {
       if (endoffset >= limit_offset)
        {
-         static struct complaint msg =
-           {"Bad line table offset in C_EINCL directive", 0, 0};
-         complain (&msg);
+         complaint (&symfile_complaints,
+                    "Bad line table offset in C_EINCL directive");
          return;
        }
       limit_offset = endoffset;
     }
   else
     limit_offset -= 1;
+
   abfd = this_symtab_psymtab->objfile->obfd;
+  linesz = coff_data (abfd)->local_linesz;
+  ext_lnno = alloca (linesz);
 
   while (curoffset <= limit_offset)
     {
       bfd_seek (abfd, curoffset, SEEK_SET);
-      bfd_read (&ext_lnno, sizeof (struct external_lineno), 1, abfd);
-      bfd_coff_swap_lineno_in (abfd, &ext_lnno, &int_lnno);
+      bfd_bread (ext_lnno, linesz, abfd);
+      bfd_coff_swap_lineno_in (abfd, ext_lnno, &int_lnno);
 
       /* Find the address this line represents.  */
       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);
+                       SECT_OFF_TEXT (this_symtab_psymtab->objfile));
 
       if (addr < startaddr || (endaddr && addr >= endaddr))
        return;
@@ -799,170 +807,10 @@ enter_line_range (subfile, beginoffset, endoffset, startaddr, endaddr,
        }
       else
        record_line (subfile, *firstLine + int_lnno.l_lnno, addr);
-      curoffset += LINESZ;
+      curoffset += linesz;
     }
 }
 
-typedef struct {
-  int fsize;                           /* file size */
-  int fixedparms;                      /* number of fixed parms */
-  int floatparms;                      /* number of float parms */
-  unsigned int parminfo;               /* parameter info. 
-                                          See /usr/include/sys/debug.h
-                                          tbtable_ext.parminfo */
-  int framesize;                       /* function frame size */
-} TracebackInfo;
-
-static TracebackInfo *retrieve_tracebackinfo
-  PARAMS ((bfd *, struct coff_symbol *));
-
-/* Given a function symbol, return its traceback information. */
-
-static TracebackInfo *
-retrieve_tracebackinfo (abfd, cs)
-     bfd *abfd;
-     struct coff_symbol *cs;
-{
-#define TBTABLE_BUFSIZ  2000
-
-  static TracebackInfo tbInfo;
-  struct tbtable *ptb;
-
-  static char buffer [TBTABLE_BUFSIZ];
-
-  int  *pinsn;
-  int  bytesread=0;                    /* total # of bytes read so far */
-  int  bufferbytes;                    /* number of bytes in the buffer */
-  int functionstart;
-
-  asection *textsec;
-
-  /* FIXME: Should be looking through all sections, based on the
-     address we are considering.  Just using ".text" loses if more
-     than one section has code in it.  */
-  textsec = bfd_get_section_by_name (abfd, ".text");
-  if (!textsec)
-    {
-#if 0
-      /* If there is only data, no text, that is OK.  */
-      printf_unfiltered ("Unable to locate text section!\n");
-#endif
-      return;
-    }
-
-  functionstart = cs->c_value - textsec->vma;
-
-  memset (&tbInfo, '\0', sizeof (tbInfo));
-
-  /* keep reading blocks of data from the text section, until finding a zero
-     word and a traceback table. */
-
-  /* Note: The logical thing way to write this code would be to assign
-     to bufferbytes within the while condition.  But that triggers a
-     compiler (xlc in AIX 3.2) bug, so simplify it...  */
-  bufferbytes = 
-    (TBTABLE_BUFSIZ < (textsec->_raw_size - functionstart - bytesread) ? 
-     TBTABLE_BUFSIZ : (textsec->_raw_size - functionstart - bytesread));
-  while (bufferbytes 
-        && (bfd_get_section_contents
-            (abfd, textsec, buffer, 
-             (file_ptr)(functionstart + bytesread), bufferbytes)))
-  {
-    bytesread += bufferbytes;
-    pinsn = (int*) buffer;
-
-    /* If this is the first time we filled the buffer, retrieve function
-       framesize info.  */
-
-    if (bytesread == bufferbytes) {
-
-      /* skip over unrelated instructions */
-
-      if (*pinsn == 0x7c0802a6)                        /* mflr r0 */
-        ++pinsn;
-      if ((*pinsn & 0xfc00003e) == 0x7c000026) /* mfcr Rx */
-       ++pinsn;
-      if ((*pinsn & 0xfc000000) == 0x48000000) /* bl foo, save fprs */
-        ++pinsn;
-      if ((*pinsn  & 0xfc1f0000) == 0xbc010000)        /* stm Rx, NUM(r1) */
-        ++pinsn;
-
-      do {
-       int tmp = (*pinsn >> 16) & 0xffff;
-
-       if (tmp ==  0x9421) {                   /* stu  r1, NUM(r1) */
-         tbInfo.framesize = 0x10000 - (*pinsn & 0xffff);
-         break;
-       }
-       else if ((*pinsn == 0x93e1fffc) ||      /* st   r31,-4(r1) */
-                (tmp == 0x9001))               /* st   r0, NUM(r1) */
-       ;
-       /* else, could not find a frame size. */
-       else
-         return NULL;
-
-      } while (++pinsn && *pinsn);
-
-      if (!tbInfo.framesize)
-        return NULL;      
-
-    }
-
-    /* look for a zero word. */
-
-    while (*pinsn && (pinsn < (int*)(buffer + bufferbytes - sizeof(int))))
-      ++pinsn;
-
-    if (pinsn >= (int*)(buffer + bufferbytes))
-      continue;
-
-    if (*pinsn == 0) {
-
-      /* function size is the amount of bytes we have skipped so far. */
-      tbInfo.fsize = bytesread - (buffer + bufferbytes - (char*)pinsn);
-
-      ++pinsn;
-
-      /* if we don't have the whole traceback table in the buffer, re-read
-         the whole thing. */
-
-      /* This is how much to read to get the traceback table.
-        8 bytes of the traceback table are always present, plus we
-        look at parminfo.  */
-#define        MIN_TBTABSIZ    12
-                               
-      if ((char*)pinsn > (buffer + bufferbytes - MIN_TBTABSIZ)) {
-
-       /* In case if we are *very* close to the end of the text section
-          and cannot read properly from that point on, abort by returning
-          NULL.
-
-          This could happen if the traceback table is only 8 bytes,
-          but we try to read 12 bytes of it.
-          Handle this case more graciously -- FIXME */
-
-       if (!bfd_get_section_contents (
-               abfd, textsec, buffer, 
-               (file_ptr)(functionstart + 
-                bytesread - (buffer + bufferbytes - (char*)pinsn)),MIN_TBTABSIZ))
-         { printf_unfiltered ("Abnormal return!..\n"); return NULL; }
-
-       ptb = (struct tbtable *)buffer;
-      }
-      else
-        ptb = (struct tbtable *)pinsn;
-
-      tbInfo.fixedparms = ptb->tb.fixedparms;
-      tbInfo.floatparms = ptb->tb.floatparms;
-      tbInfo.parminfo = ptb->tb_ext.parminfo;
-      return &tbInfo;
-    }
-    bufferbytes = 
-      (TBTABLE_BUFSIZ < (textsec->_raw_size - functionstart - bytesread) ? 
-       TBTABLE_BUFSIZ : (textsec->_raw_size - functionstart - bytesread));
-  }
-  return NULL;
-}
 
 /* Save the vital information for use when closing off the current file.
    NAME is the file name the symbols came from, START_ADDR is the first
@@ -989,34 +837,15 @@ retrieve_tracebackinfo (abfd, cs)
   namestr = (NAME); \
   if (namestr[0] == '.') ++namestr; \
   prim_record_minimal_symbol_and_info (namestr, (ADDR), (TYPE), \
-                                      (char *)NULL, (SECTION), (OBJFILE)); \
+                                      (char *)NULL, (SECTION), (asection *)NULL, (OBJFILE)); \
   misc_func_recorded = 1;                                      \
 }
 
 
-/* A parameter template, used by ADD_PARM_TO_PENDING.  It is initialized
-   in our initializer function at the bottom of the file, to avoid
-   dependencies on the exact "struct symbol" format.  */
-
-static struct symbol parmsym;
-
-/* Add a parameter to a given pending symbol list. */ 
-
-#define        ADD_PARM_TO_PENDING(PARM, VALUE, PTYPE, PENDING_SYMBOLS)        \
-{                                                                      \
-  PARM = (struct symbol *)                                             \
-      obstack_alloc (&objfile->symbol_obstack, sizeof (struct symbol));        \
-  *(PARM) = parmsym;                                                   \
-  SYMBOL_TYPE (PARM) = PTYPE;                                          \
-  SYMBOL_VALUE (PARM) = VALUE;                                         \
-  add_symbol_to_list (PARM, &PENDING_SYMBOLS);                         \
-}
-
-
 /* 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;
 
 /* Section number for the current static block.  */
@@ -1033,39 +862,39 @@ static char *raw_symbol;
 
 /* This is the function which stabsread.c calls to get symbol
    continuations.  */
+
 static char *
-xcoff_next_symbol_text ()
+xcoff_next_symbol_text (struct objfile *objfile)
 {
   struct internal_syment symbol;
-  static struct complaint msg =
-    {"Unexpected symbol continuation", 0, 0};
   char *retval;
-  struct objfile *objfile = this_symtab_psymtab->objfile;
+  /* FIXME: is this the same as the passed arg? */
+  objfile = this_symtab_psymtab->objfile;
 
   bfd_coff_swap_sym_in (objfile->obfd, raw_symbol, &symbol);
   if (symbol.n_zeroes)
     {
-      complain (&msg);
+      complaint (&symfile_complaints, "Unexpected symbol continuation");
 
       /* Return something which points to '\0' and hope the symbol reading
-        code does something reasonable.  */
+         code does something reasonable.  */
       retval = "";
     }
   else if (symbol.n_sclass & 0x80)
     {
       retval =
-       ((struct coff_symfile_info *)objfile->sym_private)->debugsec
-         + symbol.n_offset;
+       ((struct coff_symfile_info *) objfile->sym_private)->debugsec
+       + symbol.n_offset;
       raw_symbol +=
        coff_data (objfile->obfd)->local_symesz;
       ++symnum;
     }
   else
     {
-      complain (&msg);
+      complaint (&symfile_complaints, "Unexpected symbol continuation");
 
       /* Return something which points to '\0' and hope the symbol reading
-        code does something reasonable.  */
+         code does something reasonable.  */
       retval = "";
     }
   return retval;
@@ -1074,16 +903,15 @@ xcoff_next_symbol_text ()
 /* Read symbols for a given partial symbol table.  */
 
 static void
-read_xcoff_symtab (pst)
-     struct partial_symtab *pst;
+read_xcoff_symtab (struct partial_symtab *pst)
 {
   struct objfile *objfile = pst->objfile;
   bfd *abfd = objfile->obfd;
   char *raw_auxptr;            /* Pointer to first raw aux entry for sym */
-  TracebackInfo *ptb;          /* Pointer to traceback table */
-  char *strtbl = ((struct coff_symfile_info *)objfile->sym_private)->strtbl;
+  char *strtbl = ((struct coff_symfile_info *) objfile->sym_private)->strtbl;
   char *debugsec =
-    ((struct coff_symfile_info *)objfile->sym_private)->debugsec;
+  ((struct coff_symfile_info *) objfile->sym_private)->debugsec;
+  char *debugfmt = bfd_xcoff_is_xcoff64 (abfd) ? "XCOFF64" : "XCOFF";
 
   struct internal_syment symbol[1];
   union internal_auxent main_aux;
@@ -1095,23 +923,19 @@ read_xcoff_symtab (pst)
   unsigned int max_symnum;
   int just_started = 1;
   int depth = 0;
-  int val;
-  int fcn_start_addr;
-  size_t size;
+  int fcn_start_addr = 0;
 
   struct coff_symbol fcn_stab_saved;
 
   /* fcn_cs_saved is global because process_xcoff_symbol needs it. */
   union internal_auxent fcn_aux_saved;
-  struct type *fcn_type_saved = NULL;
   struct context_stack *new;
 
   char *filestring = " _start_ ";      /* Name of the current file. */
 
-  char *last_csect_name;               /* last seen csect's name and value */
+  char *last_csect_name;       /* last seen csect's name and value */
   CORE_ADDR last_csect_val;
   int last_csect_sec;
-  int  misc_func_recorded;             /* true if any misc. function */
 
   this_symtab_psymtab = pst;
 
@@ -1122,18 +946,18 @@ read_xcoff_symtab (pst)
   last_source_file = NULL;
   last_csect_name = 0;
   last_csect_val = 0;
-  misc_func_recorded = 0;
 
   start_stabs ();
-  start_symtab (filestring, (char *)NULL, file_start_addr);
-  symnum = ((struct symloc *)pst->read_symtab_private)->first_symnum;
+  start_symtab (filestring, (char *) NULL, file_start_addr);
+  record_debugformat (debugfmt);
+  symnum = ((struct symloc *) pst->read_symtab_private)->first_symnum;
   max_symnum =
-    symnum + ((struct symloc *)pst->read_symtab_private)->numsyms;
+    symnum + ((struct symloc *) pst->read_symtab_private)->numsyms;
   first_object_file_end = 0;
 
   raw_symbol =
     ((struct coff_symfile_info *) objfile->sym_private)->symtbl
-      + symnum * local_symesz;
+    + symnum * local_symesz;
 
   while (symnum < max_symnum)
     {
@@ -1142,10 +966,10 @@ read_xcoff_symtab (pst)
 
       /* 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'. */
+         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;
 
@@ -1160,7 +984,7 @@ read_xcoff_symtab (pst)
            /* We must use the original, unswapped, name here so the name field
               pointed to by cs->c_name will persist throughout xcoffread.  If
               we use the new field, it gets overwritten for each symbol.  */
-           cs->c_name = ((struct external_syment *)raw_symbol)->e.e_name;
+           cs->c_name = ((struct external_syment *) raw_symbol)->e.e_name;
            /* If it's exactly E_SYMNMLEN characters long it isn't
               '\0'-terminated.  */
            if (cs->c_name[E_SYMNMLEN - 1] != '\0')
@@ -1181,15 +1005,15 @@ read_xcoff_symtab (pst)
        else
          {
            /* in string table */
-           cs->c_name = strtbl + (int)symbol->n_offset;
+           cs->c_name = strtbl + (int) symbol->n_offset;
            symname_alloced = 1;
          }
        cs->c_value = symbol->n_value;
        cs->c_sclass = symbol->n_sclass;
        cs->c_secnum = symbol->n_scnum;
-       cs->c_type = (unsigned)symbol->n_type;
+       cs->c_type = (unsigned) symbol->n_type;
 
-       raw_symbol += coff_data (abfd)->local_symesz;
+       raw_symbol += local_symesz;
        ++symnum;
 
        /* Save addr of first aux entry.  */
@@ -1213,24 +1037,17 @@ read_xcoff_symtab (pst)
          if (last_source_file)
            {
              pst->symtab =
-               end_symtab (cur_src_end_addr, 1, 0, objfile, SECT_OFF_TEXT);
+               end_symtab (cur_src_end_addr, objfile, SECT_OFF_TEXT (objfile));
              end_stabs ();
            }
 
          start_stabs ();
-         start_symtab ("_globals_", (char *)NULL, (CORE_ADDR)0);
+         start_symtab ("_globals_", (char *) NULL, (CORE_ADDR) 0);
+         record_debugformat (debugfmt);
          cur_src_end_addr = first_object_file_end;
          /* done with all files, everything from here on is globals */
        }
 
-      /* if explicitly specified as a function, treat is as one. */
-      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;
-       }
-
       if ((cs->c_sclass == C_EXT || cs->c_sclass == C_HIDEXT)
          && cs->c_naux == 1)
        {
@@ -1263,31 +1080,31 @@ read_xcoff_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, 1, 0, objfile,
-                                     SECT_OFF_TEXT);
+                         end_symtab (file_end_addr, objfile, 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 (filestring, NULL, (CORE_ADDR) 0);
+                         record_debugformat (debugfmt);
                        }
 
                      /* If this is the very first csect seen,
-                        basically `__start'. */
+                        basically `__start'. */
                      if (just_started)
                        {
                          first_object_file_end
@@ -1297,25 +1114,24 @@ read_xcoff_symtab (pst)
 
                      file_start_addr =
                        cs->c_value + ANOFFSET (objfile->section_offsets,
-                                               SECT_OFF_TEXT);
+                                               SECT_OFF_TEXT (objfile));
                      file_end_addr = file_start_addr + CSECT_LEN (&main_aux);
 
-                     if (cs->c_name && cs->c_name[0] == '.')
+                     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);
                        }
                    }
-                   misc_func_recorded = 0;
                    continue;
 
-                 case XMC_RW :
-                   break;
+                   /* All other symbols are put into the minimal symbol
+                      table only.  */
 
-                   /* If the section is not a data description,
-                      ignore it. Note that uninitialized data will
-                      show up as XTY_CM/XMC_RW pair. */
+                 case XMC_RW:
+                   continue;
 
                  case XMC_TC0:
                    continue;
@@ -1344,101 +1160,6 @@ read_xcoff_symtab (pst)
                     when `.bf' is seen. */
                  fcn_cs_saved = *cs;
                  fcn_aux_saved = main_aux;
-
-                 ptb = NULL;
-
-                 /* If function has two auxent, then debugging information is
-                    already available for it. Process traceback table for
-                    functions with only one auxent. */
-
-                 if (cs->c_naux == 1)
-                   ptb = retrieve_tracebackinfo (abfd, cs);
-
-                 else if (cs->c_naux != 2)
-                   {
-                     static struct complaint msg =
-                       {"Expected one or two auxents for function", 0, 0};
-                     complain (&msg);
-                   }
-
-                 /* If there is traceback info, create and add parameters
-                    for it. */
-
-                 if (ptb && (ptb->fixedparms || ptb->floatparms))
-                   {
-
-                     int parmcnt = ptb->fixedparms + ptb->floatparms;
-                     char *parmcode = (char*) &ptb->parminfo;
-
-                     /* The link area is 0x18 bytes.  */
-                     int parmvalue = ptb->framesize + 0x18;
-                     unsigned int ii, mask;
-
-                     for (ii=0, mask = 0x80000000; ii <parmcnt; ++ii)
-                       {
-                         struct symbol *parm;
-
-                         if (ptb->parminfo & mask)
-                           {
-                             /* float or double */
-                             mask = mask >> 1;
-                             if (ptb->parminfo & mask)
-                               {
-                                 /* double parm */
-                                 ADD_PARM_TO_PENDING
-                                   (parm, parmvalue, builtin_type_double,
-                                    local_symbols);
-                                 parmvalue += sizeof (double);
-                               }
-                             else
-                               {
-                                 /* float parm */
-                                 ADD_PARM_TO_PENDING
-                                   (parm, parmvalue, builtin_type_float,
-                                    local_symbols);
-                                 parmvalue += sizeof (float);
-                               }
-                           }
-                         else
-                           {
-                             static struct type *intparm_type;
-                             if (intparm_type == NULL)
-                               {
-
-                                 /* Create a type, which is a pointer
-                                    type (a kludge to make it print
-                                    in hex), but which has a name
-                                    indicating we don't know the real
-                                    type.  */
-
-                                 intparm_type =
-                                   init_type
-                                     (TYPE_CODE_PTR,
-                                      TARGET_PTR_BIT / HOST_CHAR_BIT,
-                                      0,
-                                      "<non-float parameter>",
-                                      NULL);
-                                 TYPE_TARGET_TYPE (intparm_type) =
-                                   builtin_type_void;
-                               }
-                             ADD_PARM_TO_PENDING
-                               (parm, parmvalue,
-                                intparm_type,
-                                local_symbols);
-                             parmvalue += sizeof (int);
-                           }
-                         mask = mask >> 1;
-                       }
-               
-                     /* Fake this as a function.  Needed in
-                         process_xcoff_symbol().  */
-                     cs->c_type = 32;
-
-                     finish_block
-                       (process_xcoff_symbol (cs, objfile), &local_symbols, 
-                        pending_blocks, cs->c_value,
-                        cs->c_value + ptb->fsize, objfile);
-                   }
                  continue;
 
                case XMC_GL:
@@ -1454,17 +1175,33 @@ read_xcoff_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.  We still need to record them.  This will
-                    typically be XMC_RW; I suspect XMC_RO and XMC_BS might
-                    be possible too.  */
-                 break;
+                    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.  */
+                 continue;
                }
+             break;
+
+           case XTY_CM:
+             /* Common symbols are put into the minimal symbol table only.  */
+             continue;
 
            default:
              break;
            }
        }
 
+      /* 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)
        {
 
@@ -1484,7 +1221,7 @@ read_xcoff_symtab (pst)
 
          complete_symtab (filestring, file_start_addr);
          cur_src_end_addr = file_end_addr;
-         end_symtab (file_end_addr, 1, 0, objfile, SECT_OFF_TEXT);
+         end_symtab (file_end_addr, objfile, SECT_OFF_TEXT (objfile));
          end_stabs ();
 
          /* XCOFF, according to the AIX 3.2 documentation, puts the filename
@@ -1502,7 +1239,8 @@ read_xcoff_symtab (pst)
            filestring = cs->c_name;
 
          start_stabs ();
-         start_symtab (filestring, (char *)NULL, (CORE_ADDR)0);
+         start_symtab (filestring, (char *) NULL, (CORE_ADDR) 0);
+         record_debugformat (debugfmt);
          last_csect_name = 0;
 
          /* reset file start and end addresses. A compilation unit with no text
@@ -1518,7 +1256,7 @@ read_xcoff_symtab (pst)
          if (STREQ (cs->c_name, ".bf"))
            {
              CORE_ADDR off = ANOFFSET (objfile->section_offsets,
-                                       SECT_OFF_TEXT);
+                                       SECT_OFF_TEXT (objfile));
              bfd_coff_swap_aux_in (abfd, raw_auxptr, cs->c_type, cs->c_sclass,
                                    0, cs->c_naux, &main_aux);
 
@@ -1526,11 +1264,11 @@ read_xcoff_symtab (pst)
 
              new = push_context (0, fcn_start_addr + off);
 
-             new->name = define_symbol 
+             new->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;
+               SYMBOL_SECTION (new->name) = SECT_OFF_TEXT (objfile);
            }
          else if (STREQ (cs->c_name, ".ef"))
            {
@@ -1539,15 +1277,21 @@ read_xcoff_symtab (pst)
                                    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 */
+                 ef_complaint (cs->c_symnum);
+                 within_function = 0;
+                 break;
+               }
              new = pop_context ();
              /* Stack must be empty now.  */
              if (context_stack_depth > 0 || new == NULL)
                {
-                 complain (&ef_complaint, cs->c_symnum);
+                 ef_complaint (cs->c_symnum);
                  within_function = 0;
                  break;
                }
@@ -1557,7 +1301,7 @@ read_xcoff_symtab (pst)
                            (fcn_cs_saved.c_value
                             + fcn_aux_saved.x_sym.x_misc.x_fsize
                             + ANOFFSET (objfile->section_offsets,
-                                        SECT_OFF_TEXT)),
+                                        SECT_OFF_TEXT (objfile))),
                            objfile);
              within_function = 0;
            }
@@ -1589,9 +1333,8 @@ read_xcoff_symtab (pst)
        case C_UNTAG:
        case C_ENTAG:
          {
-           static struct complaint msg =
-             {"Unrecognized storage class %d.", 0, 0};
-           complain (&msg, cs->c_sclass);
+           complaint (&symfile_complaints, "Unrecognized storage class %d.",
+                      cs->c_sclass);
          }
          break;
 
@@ -1601,6 +1344,7 @@ read_xcoff_symtab (pst)
          break;
 
        case C_HIDEXT:
+       case C_STAT:
          break;
 
        case C_BINCL:
@@ -1626,14 +1370,19 @@ read_xcoff_symtab (pst)
              new = push_context (depth,
                                  (cs->c_value
                                   + ANOFFSET (objfile->section_offsets,
-                                              SECT_OFF_TEXT)));
+                                              SECT_OFF_TEXT (objfile))));
            }
          else if (STREQ (cs->c_name, ".eb"))
            {
+             if (context_stack_depth <= 0)
+               {               /* We attempted to pop an empty context stack */
+                 eb_complaint (cs->c_symnum);
+                 break;
+               }
              new = pop_context ();
              if (depth-- != new->depth)
                {
-                 complain (&eb_complaint, symnum);
+                 eb_complaint (cs->c_symnum);
                  break;
                }
              if (local_symbols && context_stack_depth > 0)
@@ -1643,7 +1392,7 @@ read_xcoff_symtab (pst)
                                new->start_addr,
                                (cs->c_value
                                 + ANOFFSET (objfile->section_offsets,
-                                            SECT_OFF_TEXT)),
+                                            SECT_OFF_TEXT (objfile))),
                                objfile);
                }
              local_symbols = new->locals;
@@ -1662,7 +1411,7 @@ read_xcoff_symtab (pst)
 
       complete_symtab (filestring, file_start_addr);
       cur_src_end_addr = file_end_addr;
-      s = end_symtab (file_end_addr, 1, 0, objfile, SECT_OFF_TEXT);
+      s = end_symtab (file_end_addr, objfile, 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
@@ -1677,10 +1426,10 @@ read_xcoff_symtab (pst)
   (SYMBOL2) = (struct symbol *)                \
        obstack_alloc (&objfile->symbol_obstack, sizeof (struct symbol)); \
   *(SYMBOL2) = *(SYMBOL1);
-  
+
+
 #define        SYMNAME_ALLOC(NAME, ALLOCED)    \
-  (ALLOCED) ? (NAME) : obstack_copy0 (&objfile->symbol_obstack, (NAME), strlen (NAME));
+  (ALLOCED) ? (NAME) : obsavestring ((NAME), strlen (NAME), &objfile->symbol_obstack);
 
 
 static struct type *func_symbol_type;
@@ -1689,17 +1438,12 @@ static struct type *var_symbol_type;
 /* process one xcoff symbol. */
 
 static struct symbol *
-process_xcoff_symbol (cs, objfile)
-  register struct coff_symbol *cs;
-  struct objfile *objfile;
+process_xcoff_symbol (register struct coff_symbol *cs, struct objfile *objfile)
 {
   struct symbol onesymbol;
   register struct symbol *sym = &onesymbol;
   struct symbol *sym2 = NULL;
-  struct type *ttype;
-  char *name, *pp, *qq;
-  int struct_and_type_combined;
-  int nameless;
+  char *name, *pp;
 
   int sec;
   CORE_ADDR off;
@@ -1707,7 +1451,7 @@ process_xcoff_symbol (cs, 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;
     }
@@ -1724,15 +1468,15 @@ process_xcoff_symbol (cs, objfile)
   memset (sym, '\0', sizeof (struct symbol));
 
   /* default assumptions */
-  SYMBOL_VALUE (sym) = cs->c_value + off;
+  SYMBOL_VALUE_ADDRESS (sym) = cs->c_value + off;
   SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
   SYMBOL_SECTION (sym) = 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_NAME (sym) = SYMNAME_ALLOC (name, symname_alloced);
       SYMBOL_TYPE (sym) = func_symbol_type;
@@ -1753,17 +1497,19 @@ process_xcoff_symbol (cs, objfile)
       switch (cs->c_sclass)
        {
 #if 0
+         /* The values of functions and global symbols are now resolved
+            via the global_sym_chain in stabsread.c.  */
        case C_FUN:
          if (fcn_cs_saved.c_sclass == C_EXT)
            add_stab_to_list (name, &global_stabs);
          else
            add_stab_to_list (name, &file_stabs);
          break;
-#endif
 
        case C_GSYM:
          add_stab_to_list (name, &global_stabs);
          break;
+#endif
 
        case C_BCOMM:
          common_block_start (cs->c_name, objfile);
@@ -1774,7 +1520,8 @@ process_xcoff_symbol (cs, objfile)
          break;
 
        default:
-         complain (&storclass_complaint, cs->c_sclass);
+         complaint (&symfile_complaints, "Unexpected storage class: %d",
+                    cs->c_sclass);
          /* FALLTHROUGH */
 
        case C_DECL:
@@ -1783,6 +1530,7 @@ process_xcoff_symbol (cs, objfile)
        case C_ECOML:
        case C_LSYM:
        case C_RSYM:
+       case C_GSYM:
 
          {
            sym = define_symbol (cs->c_value + off, cs->c_name, 0, 0, objfile);
@@ -1802,7 +1550,7 @@ process_xcoff_symbol (cs, objfile)
             where we need to, which is not necessarily super-clean,
             but seems workable enough.  */
 
-         if (*name == ':' || (pp = (char *) strchr(name, ':')) == NULL)
+         if (*name == ':' || (pp = (char *) strchr (name, ':')) == NULL)
            return NULL;
 
          ++pp;
@@ -1814,131 +1562,67 @@ process_xcoff_symbol (cs, objfile)
                               cs->c_name, 0, 0, objfile);
          if (sym != NULL)
            {
-             SYMBOL_VALUE (sym) += static_block_base;
+             SYMBOL_VALUE_ADDRESS (sym) += static_block_base;
              SYMBOL_SECTION (sym) = static_block_section;
            }
          return sym;
 
-#if 0
-         /* These appear to be vestigial remnants of coffread.c; I don't
-            think any of them are used for xcoff.  */
-
-       case C_AUTO:
-         SYMBOL_CLASS (sym) = LOC_LOCAL;
-         SYMBOL_NAME (sym) = SYMNAME_ALLOC (name, symname_alloced);
-         SYMBOL_SECTION (sym) = secnum_to_section (cs->c_secnum, objfile);
-         SYMBOL_DUP (sym, sym2);
-         add_symbol_to_list (sym2, &local_symbols);
-         break;
-
-       case C_STAT:
-         SYMBOL_CLASS (sym) = LOC_STATIC;
-         SYMBOL_NAME (sym) = SYMNAME_ALLOC (name, symname_alloced);
-         SYMBOL_SECTION (sym) = secnum_to_section (cs->c_secnum, objfile);
-         SYMBOL_DUP (sym, sym2);
-         add_symbol_to_list 
-           (sym2, within_function ? &local_symbols : &file_symbols);
-         break;
-
-       case C_RSYM:
-         pp = (char*) strchr (name, ':');
-         if (pp)
-           {
-             sym = define_symbol (cs->c_value, cs->c_name, 0, 0, objfile);
-             if (sym != NULL)
-               SYMBOL_SECTION (sym) = secnum_to_section (cs->c_secnum, objfile);
-             return sym;
-           }
-         else
-           {
-             complain (&rsym_complaint, name);
-             return NULL;
-           }
-#endif /* 0 */
-
-         /* I think this one is used (dubious, I think, shouldn't
-            it go into the msyms only?).  */
-       case C_EXT:
-         SYMBOL_CLASS (sym) = LOC_STATIC;
-         SYMBOL_NAME (sym) = SYMNAME_ALLOC (name, symname_alloced);
-         SYMBOL_SECTION (sym) = secnum_to_section (cs->c_secnum, objfile);
-         SYMBOL_DUP (sym, sym2);
-         add_symbol_to_list (sym2, &global_symbols);
-         break;
-
        }
     }
   return sym2;
 }
 
-/* Extract the file name from the aux entry of a C_FILE symbol.  Return
-   only the last component of the name.  Result is in static storage and
-   is only good for temporary use.  */
+/* Extract the file name from the aux entry of a C_FILE symbol.
+   Result is in static storage and is only good for temporary use.  */
 
 static char *
-coff_getfilename (aux_entry, objfile)
-     union internal_auxent *aux_entry;
-     struct objfile *objfile;
+coff_getfilename (union internal_auxent *aux_entry, struct objfile *objfile)
 {
   static char buffer[BUFSIZ];
-  register char *temp;
-  char *result;
 
   if (aux_entry->x_file.x_n.x_zeroes == 0)
     strcpy (buffer,
-           ((struct coff_symfile_info *)objfile->sym_private)->strtbl
+           ((struct coff_symfile_info *) objfile->sym_private)->strtbl
            + aux_entry->x_file.x_n.x_offset);
   else
     {
       strncpy (buffer, aux_entry->x_file.x_fname, FILNMLEN);
       buffer[FILNMLEN] = '\0';
     }
-  result = buffer;
-
-  /* FIXME: We should not be throwing away the information about what
-     directory.  It should go into dirname of the symtab, or some such
-     place.  */
-  if ((temp = strrchr (result, '/')) != NULL)
-    result = temp + 1;
-  return (result);
+  return (buffer);
 }
 
 /* Set *SYMBOL to symbol number symno in symtbl.  */
 static void
-read_symbol (symbol, symno)
-     struct internal_syment *symbol;
-     int symno;
+read_symbol (struct internal_syment *symbol, int symno)
 {
   int nsyms =
-    ((struct coff_symfile_info *)this_symtab_psymtab->objfile->sym_private)
-      ->symtbl_num_syms;
-  char *stbl = 
-    ((struct coff_symfile_info *)this_symtab_psymtab->objfile->sym_private)
-      ->symtbl;
+  ((struct coff_symfile_info *) this_symtab_psymtab->objfile->sym_private)
+  ->symtbl_num_syms;
+  char *stbl =
+  ((struct coff_symfile_info *) this_symtab_psymtab->objfile->sym_private)
+  ->symtbl;
   if (symno < 0 || symno >= nsyms)
     {
-      static struct complaint msg =
-       {"Invalid symbol offset", 0, 0};
-      complain (&msg);
+      complaint (&symfile_complaints, "Invalid symbol offset");
       symbol->n_value = 0;
       symbol->n_scnum = -1;
       return;
     }
   bfd_coff_swap_sym_in (this_symtab_psymtab->objfile->obfd,
-                       stbl + (symno*local_symesz),
+                       stbl + (symno * local_symesz),
                        symbol);
 }
-  
+
 /* Get value corresponding to symbol number symno in symtbl.  */
 
-static int
-read_symbol_nvalue (symno)
-     int symno;
+static CORE_ADDR
+read_symbol_nvalue (int symno)
 {
   struct internal_syment symbol[1];
 
   read_symbol (symbol, symno);
-  return symbol->n_value;  
+  return symbol->n_value;
 }
 
 
@@ -1946,21 +1630,23 @@ read_symbol_nvalue (symno)
    symno is the symbol pointed to by the linetable.  */
 
 static int
-read_symbol_lineno (symno)
-     int symno;
+read_symbol_lineno (int symno)
 {
-  int nsyms =
-    ((struct coff_symfile_info *)this_symtab_psymtab->objfile->sym_private)
-      ->symtbl_num_syms;
-  char *stbl = 
-    ((struct coff_symfile_info *)this_symtab_psymtab->objfile->sym_private)
-      ->symtbl;
+  struct objfile *objfile = this_symtab_psymtab->objfile;
+  int xcoff64 = bfd_xcoff_is_xcoff64 (objfile->obfd);
+
+  struct coff_symfile_info *info =
+    (struct coff_symfile_info *)objfile->sym_private;
+  int nsyms = info->symtbl_num_syms;
+  char *stbl = info->symtbl;
+  char *strtbl = info->strtbl;
+
   struct internal_syment symbol[1];
   union internal_auxent main_aux[1];
 
   if (symno < 0)
     {
-      complain (&bf_notfound_complaint);
+      bf_notfound_complaint ();
       return 0;
     }
 
@@ -1984,19 +1670,22 @@ read_symbol_lineno (symno)
     {
       bfd_coff_swap_sym_in (symfile_bfd,
                            stbl + (symno * local_symesz), symbol);
-      if (symbol->n_sclass == C_FCN && STREQ (symbol->n_name, ".bf"))
-       goto gotit;
+      if (symbol->n_sclass == C_FCN)
+       {
+         char *name = xcoff64 ? strtbl + symbol->n_offset : symbol->n_name;
+         if (STREQ (name, ".bf"))
+           goto gotit;
+       }
       symno += symbol->n_numaux + 1;
     }
 
-  complain (&bf_notfound_complaint);
+  bf_notfound_complaint ();
   return 0;
 
 gotit:
   /* take aux entry and return its lineno */
   symno++;
-  bfd_coff_swap_aux_in (this_symtab_psymtab->objfile->obfd,
-                       stbl + symno * local_symesz,
+  bfd_coff_swap_aux_in (objfile->obfd, stbl + symno * local_symesz,
                        symbol->n_type, symbol->n_sclass,
                        0, symbol->n_numaux, main_aux);
 
@@ -2010,10 +1699,7 @@ gotit:
  * mainline code can read the whole thing for efficiency.
  */
 static void
-find_linenos (abfd, asect, vpinfo)
-     bfd *abfd;
-     sec_ptr asect;
-     PTR vpinfo; 
+find_linenos (bfd *abfd, sec_ptr asect, void *vpinfo)
 {
   struct coff_symfile_info *info;
   int size, count;
@@ -2025,7 +1711,7 @@ find_linenos (abfd, asect, vpinfo)
     return;
 
   size = count * coff_data (abfd)->local_linesz;
-  info = (struct coff_symfile_info *)vpinfo;
+  info = (struct coff_symfile_info *) vpinfo;
   offset = asect->line_filepos;
   maxoff = offset + size;
 
@@ -2036,15 +1722,14 @@ find_linenos (abfd, asect, vpinfo)
     info->max_lineno_offset = maxoff;
 }
 \f
-static void xcoff_psymtab_to_symtab_1 PARAMS ((struct partial_symtab *));
+static void xcoff_psymtab_to_symtab_1 (struct partial_symtab *);
 
 static void
-xcoff_psymtab_to_symtab_1 (pst)
-     struct partial_symtab *pst;
+xcoff_psymtab_to_symtab_1 (struct partial_symtab *pst)
 {
   struct cleanup *old_chain;
   int i;
-  
+
   if (!pst)
     return;
 
@@ -2068,13 +1753,13 @@ xcoff_psymtab_to_symtab_1 (pst)
            fputs_filtered ("and ", gdb_stdout);
            wrap_here ("");
            printf_filtered ("%s...", pst->dependencies[i]->filename);
-           wrap_here ("");             /* Flush output */
+           wrap_here ("");     /* Flush output */
            gdb_flush (gdb_stdout);
          }
        xcoff_psymtab_to_symtab_1 (pst->dependencies[i]);
       }
 
-  if (((struct symloc *)pst->read_symtab_private)->numsyms != 0)
+  if (((struct symloc *) pst->read_symtab_private)->numsyms != 0)
     {
       /* Init stuff necessary for reading in symbols.  */
       stabsread_init ();
@@ -2090,14 +1775,13 @@ xcoff_psymtab_to_symtab_1 (pst)
   pst->readin = 1;
 }
 
-static void xcoff_psymtab_to_symtab PARAMS ((struct partial_symtab *));
+static void xcoff_psymtab_to_symtab (struct partial_symtab *);
 
 /* Read in all of the symbols for a given psymtab for real.
    Be verbose about it if the user wants that.  */
 
 static void
-xcoff_psymtab_to_symtab (pst)
-     struct partial_symtab *pst;
+xcoff_psymtab_to_symtab (struct partial_symtab *pst)
 {
   bfd *sym_bfd;
 
@@ -2112,11 +1796,11 @@ xcoff_psymtab_to_symtab (pst)
       return;
     }
 
-  if (((struct symloc *)pst->read_symtab_private)->numsyms != 0
+  if (((struct symloc *) pst->read_symtab_private)->numsyms != 0
       || pst->number_of_dependencies)
     {
       /* Print the message now, before reading the string table,
-        to avoid disconcerting pauses.  */
+         to avoid disconcerting pauses.  */
       if (info_verbose)
        {
          printf_filtered ("Reading in symbols for %s...", pst->filename);
@@ -2140,24 +1824,30 @@ xcoff_psymtab_to_symtab (pst)
 }
 \f
 static void
-xcoff_new_init (objfile)
-     struct objfile *objfile;
+xcoff_new_init (struct objfile *objfile)
 {
+  stabsread_new_init ();
+  buildsym_new_init ();
 }
 
 /* Do initialization in preparation for reading symbols from OBJFILE.
+
    We will only be called if this is an XCOFF or XCOFF-like file.
    BFD handles figuring out the format of the file, and code in symfile.c
    uses BFD's determination to vector to us.  */
 
 static void
-xcoff_symfile_init (objfile)
-     struct objfile *objfile;
+xcoff_symfile_init (struct objfile *objfile)
 {
   /* Allocate struct to keep track of the symfile */
-  objfile -> sym_private = xmmalloc (objfile -> md,
-                                    sizeof (struct coff_symfile_info));
+  objfile->sym_private = xmmalloc (objfile->md,
+                                  sizeof (struct coff_symfile_info));
+
+  /* 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);
 }
 
@@ -2167,18 +1857,17 @@ xcoff_symfile_init (objfile)
    objfile struct from the global list of known objfiles. */
 
 static void
-xcoff_symfile_finish (objfile)
-     struct objfile *objfile;
+xcoff_symfile_finish (struct objfile *objfile)
 {
-  if (objfile -> sym_private != NULL)
+  if (objfile->sym_private != NULL)
     {
-      mfree (objfile -> md, objfile -> sym_private);
+      xmfree (objfile->md, objfile->sym_private);
     }
 
   /* Start with a fresh include table for the next objfile.  */
   if (inclTable)
     {
-      free (inclTable);
+      xfree (inclTable);
       inclTable = NULL;
     }
   inclIndx = inclLength = inclDepth = 0;
@@ -2186,23 +1875,20 @@ xcoff_symfile_finish (objfile)
 
 
 static void
-init_stringtab (abfd, offset, objfile)
-     bfd *abfd;
-     file_ptr offset;
-     struct objfile *objfile;
+init_stringtab (bfd *abfd, file_ptr offset, struct objfile *objfile)
 {
   long length;
   int val;
   unsigned char lengthbuf[4];
   char *strtbl;
 
-  ((struct coff_symfile_info *)objfile->sym_private)->strtbl = NULL;
+  ((struct coff_symfile_info *) objfile->sym_private)->strtbl = NULL;
 
-  if (bfd_seek (abfd, offset, L_SET) < 0)
+  if (bfd_seek (abfd, offset, SEEK_SET) < 0)
     error ("cannot seek to string table in %s: %s",
           bfd_get_filename (abfd), bfd_errmsg (bfd_get_error ()));
 
-  val = bfd_read ((char *)lengthbuf, 1, sizeof lengthbuf, abfd);
+  val = bfd_bread ((char *) lengthbuf, sizeof lengthbuf, abfd);
   length = bfd_h_get_32 (abfd, lengthbuf);
 
   /* If no string table is needed, then the file may end immediately
@@ -2215,13 +1901,15 @@ init_stringtab (abfd, offset, objfile)
      as long as we have its symbol table around. */
 
   strtbl = (char *) obstack_alloc (&objfile->symbol_obstack, length);
-  ((struct coff_symfile_info *)objfile->sym_private)->strtbl = strtbl;
+  ((struct coff_symfile_info *) objfile->sym_private)->strtbl = strtbl;
 
+  /* Copy length buffer, the first byte is usually zero and is
+     used for stabs with a name length of zero.  */
+  memcpy (strtbl, lengthbuf, sizeof lengthbuf);
   if (length == sizeof lengthbuf)
     return;
 
-  val = bfd_read (strtbl + sizeof lengthbuf, 1, length - sizeof lengthbuf,
-                 abfd);
+  val = bfd_bread (strtbl + sizeof lengthbuf, length - sizeof lengthbuf, abfd);
 
   if (val != length - sizeof lengthbuf)
     error ("cannot read string table from %s: %s",
@@ -2238,8 +1926,8 @@ init_stringtab (abfd, offset, objfile)
 static unsigned int first_fun_line_offset;
 
 static struct partial_symtab *xcoff_start_psymtab
-  PARAMS ((struct objfile *, struct section_offsets *, char *, int,
-          struct partial_symbol *, struct partial_symbol *));
+  (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.
@@ -2249,25 +1937,20 @@ static struct partial_symtab *xcoff_start_psymtab
    (normal). */
 
 static struct partial_symtab *
-xcoff_start_psymtab (objfile, section_offsets,
-                    filename, first_symnum, global_syms, static_syms)
-     struct objfile *objfile;
-     struct section_offsets *section_offsets;
-     char *filename;
-     int first_symnum;
-     struct partial_symbol *global_syms;
-     struct partial_symbol *static_syms;
+xcoff_start_psymtab (struct objfile *objfile, char *filename, int first_symnum,
+                    struct partial_symbol **global_syms,
+                    struct partial_symbol **static_syms)
 {
   struct partial_symtab *result =
-    start_psymtab_common (objfile, section_offsets,
-                         filename,
-                         /* We fill in textlow later.  */
-                         0,
-                         global_syms, static_syms);
+  start_psymtab_common (objfile, objfile->section_offsets,
+                       filename,
+                       /* We fill in textlow later.  */
+                       0,
+                       global_syms, static_syms);
 
   result->read_symtab_private = (char *)
-    obstack_alloc (&objfile -> psymbol_obstack, sizeof (struct symloc));
-  ((struct symloc *)result->read_symtab_private)->first_symnum = first_symnum;
+    obstack_alloc (&objfile->psymbol_obstack, sizeof (struct symloc));
+  ((struct symloc *) result->read_symtab_private)->first_symnum = first_symnum;
   result->read_symtab = xcoff_psymtab_to_symtab;
 
   /* Deduce the source language from the filename for this psymtab. */
@@ -2277,8 +1960,8 @@ xcoff_start_psymtab (objfile, section_offsets,
 }
 
 static struct partial_symtab *xcoff_end_psymtab
-  PARAMS ((struct partial_symtab *, char **, int, int,
-          struct partial_symtab **, int));
+  (struct partial_symtab *, char **, int, int,
+   struct partial_symtab **, int, int);
 
 /* Close off the current usage of PST.  
    Returns PST, or NULL if the partial symtab was empty and thrown away.
@@ -2289,23 +1972,19 @@ static struct partial_symtab *xcoff_end_psymtab
    are the information for includes and dependencies.  */
 
 static struct partial_symtab *
-xcoff_end_psymtab (pst, include_list, num_includes, capping_symbol_number,
-                  dependency_list, number_dependencies)
-     struct partial_symtab *pst;
-     char **include_list;
-     int num_includes;
-     int capping_symbol_number;
-     struct partial_symtab **dependency_list;
-     int number_dependencies;
+xcoff_end_psymtab (struct partial_symtab *pst, char **include_list,
+                  int num_includes, int capping_symbol_number,
+                  struct partial_symtab **dependency_list,
+                  int number_dependencies, int textlow_not_set)
 {
   int i;
-  struct objfile *objfile = pst -> objfile;
+  struct objfile *objfile = pst->objfile;
 
   if (capping_symbol_number != -1)
-    ((struct symloc *)pst->read_symtab_private)->numsyms =
+    ((struct symloc *) pst->read_symtab_private)->numsyms =
       capping_symbol_number
-       - ((struct symloc *)pst->read_symtab_private)->first_symnum;
-  ((struct symloc *)pst->read_symtab_private)->lineno_off =
+      - ((struct symloc *) pst->read_symtab_private)->first_symnum;
+  ((struct symloc *) pst->read_symtab_private)->lineno_off =
     first_fun_line_offset;
   first_fun_line_offset = 0;
   pst->n_global_syms =
@@ -2318,9 +1997,9 @@ xcoff_end_psymtab (pst, include_list, num_includes, capping_symbol_number,
     {
       pst->dependencies = (struct partial_symtab **)
        obstack_alloc (&objfile->psymbol_obstack,
-                      number_dependencies * sizeof (struct partial_symtab *));
+                   number_dependencies * sizeof (struct partial_symtab *));
       memcpy (pst->dependencies, dependency_list,
-            number_dependencies * sizeof (struct partial_symtab *));
+             number_dependencies * sizeof (struct partial_symtab *));
     }
   else
     pst->dependencies = 0;
@@ -2328,19 +2007,19 @@ xcoff_end_psymtab (pst, include_list, num_includes, capping_symbol_number,
   for (i = 0; i < num_includes; i++)
     {
       struct partial_symtab *subpst =
-       allocate_psymtab (include_list[i], objfile);
+      allocate_psymtab (include_list[i], objfile);
 
       subpst->section_offsets = pst->section_offsets;
       subpst->read_symtab_private =
-         (char *) obstack_alloc (&objfile->psymbol_obstack,
-                                 sizeof (struct symloc));
-      ((struct symloc *)subpst->read_symtab_private)->first_symnum = 0;
-      ((struct symloc *)subpst->read_symtab_private)->numsyms = 0;
+       (char *) obstack_alloc (&objfile->psymbol_obstack,
+                               sizeof (struct 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.  */
+         shared by the entire set of include files.  FIXME-someday.  */
       subpst->dependencies = (struct partial_symtab **)
        obstack_alloc (&objfile->psymbol_obstack,
                       sizeof (struct partial_symtab *));
@@ -2349,8 +2028,8 @@ xcoff_end_psymtab (pst, include_list, num_includes, capping_symbol_number,
 
       subpst->globals_offset =
        subpst->n_global_syms =
-         subpst->statics_offset =
-           subpst->n_static_syms = 0;
+       subpst->statics_offset =
+       subpst->n_static_syms = 0;
 
       subpst->readin = 0;
       subpst->symtab = 0;
@@ -2370,58 +2049,36 @@ xcoff_end_psymtab (pst, include_list, num_includes, capping_symbol_number,
       && pst->n_static_syms == 0)
     {
       /* 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.  */
+         it is on the obstack, but we can forget to chain it on the list.  */
       /* Empty psymtabs happen as a result of header files which don't have
-        any symbols in them.  There can be a lot of them.  But this check
-        is wrong, in that a psymtab with N_SLINE entries but nothing else
-        is not empty, but we don't realize that.  Fixing that without slowing
-        things down might be tricky.  (FIXME: For XCOFF, it shouldn't be
-        tricky at all).  */
-      struct partial_symtab *prev_pst;
-
-      /* First, snip it out of the psymtab chain */
+         any symbols in them.  There can be a lot of them.  */
 
-      if (pst->objfile->psymtabs == pst)
-       pst->objfile->psymtabs = pst->next;
-      else
-       for (prev_pst = pst->objfile->psymtabs; prev_pst; prev_pst = pst->next)
-         if (prev_pst->next == pst)
-           prev_pst->next = pst->next;
-
-      /* Next, put it on a free list for recycling */
-
-      pst->next = pst->objfile->free_psymtabs;
-      pst->objfile->free_psymtabs = pst;
+      discard_psymtab (pst);
 
       /* Indicate that psymtab was thrown away.  */
-      pst = (struct partial_symtab *)NULL;
+      pst = (struct partial_symtab *) NULL;
     }
   return pst;
 }
 
-static void swap_sym PARAMS ((struct internal_syment *,
-                             union internal_auxent *, char **, char **,
-                             unsigned int *,
-                             struct objfile *));
+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 (symbol, aux, name, raw, symnump, objfile)
-     struct internal_syment *symbol;
-     union internal_auxent *aux;
-     char **name;
-     char **raw;
-     unsigned int *symnump;
-     struct objfile *objfile;
+swap_sym (struct internal_syment *symbol, union internal_auxent *aux,
+         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
@@ -2435,16 +2092,16 @@ swap_sym (symbol, aux, name, raw, symnump, objfile)
       else
        /* Point to the unswapped name as that persists as long as the
           objfile does.  */
-       *name = ((struct external_syment *)*raw)->e.e_name;
+       *name = ((struct external_syment *) *raw)->e.e_name;
     }
   else if (symbol->n_sclass & 0x80)
     {
-      *name = ((struct coff_symfile_info *)objfile->sym_private)->debugsec
+      *name = ((struct coff_symfile_info *) objfile->sym_private)->debugsec
        + symbol->n_offset;
     }
   else
     {
-      *name = ((struct coff_symfile_info *)objfile->sym_private)->strtbl
+      *name = ((struct coff_symfile_info *) objfile->sym_private)->strtbl
        + symbol->n_offset;
     }
   ++*symnump;
@@ -2460,19 +2117,23 @@ swap_sym (symbol, aux, name, raw, symnump, objfile)
 }
 
 static void
-scan_xcoff_symtab (section_offsets, objfile)
-     struct section_offsets *section_offsets;
-     struct objfile *objfile;
+function_outside_compilation_unit_complaint (const char *arg1)
 {
-  int toc_offset = 0;          /* toc offset value in data section. */
-  char *filestring;
+  complaint (&symfile_complaints,
+            "function `%s' appears to be defined outside of all compilation units",
+            arg1);
+}
+
+static void
+scan_xcoff_symtab (struct objfile *objfile)
+{
+  CORE_ADDR toc_offset = 0;    /* toc offset value in data section. */
+  char *filestring = NULL;
 
   char *namestring;
-  int nsl;
   int past_first_source_file = 0;
-  CORE_ADDR last_o_file_start = 0;
-  struct cleanup *back_to;
   bfd *abfd;
+  asection *bfd_sect;
   unsigned int nsyms;
 
   /* Current partial symtab */
@@ -2489,13 +2150,14 @@ scan_xcoff_symtab (section_offsets, objfile)
 
   char *sraw_symbol;
   struct internal_syment symbol;
-  union internal_auxent main_aux;
+  union internal_auxent main_aux[5];
   unsigned int ssymnum;
 
-  char *last_csect_name = NULL;                /* last seen csect's name and value */
-  CORE_ADDR last_csect_val;
-  int last_csect_sec;
-  int  misc_func_recorded;             /* true if any misc. function */
+  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 textlow_not_set = 1;
 
   pst = (struct partial_symtab *) 0;
 
@@ -2514,20 +2176,18 @@ scan_xcoff_symtab (section_offsets, objfile)
 
   abfd = objfile->obfd;
 
-  sraw_symbol = ((struct coff_symfile_info *)objfile->sym_private)->symtbl;
-  nsyms = ((struct coff_symfile_info *)objfile->sym_private)->symtbl_num_syms;
+  sraw_symbol = ((struct coff_symfile_info *) objfile->sym_private)->symtbl;
+  nsyms = ((struct coff_symfile_info *) objfile->sym_private)->symtbl_num_syms;
   ssymnum = 0;
   while (ssymnum < nsyms)
     {
-      int sclass = ((struct external_syment *)sraw_symbol)->e_sclass[0];
-      /* This is the type we pass to partial-stab.h.  A less kludgy solution
-        would be to break out partial-stab.h into its various parts--shuffle
-        off the DBXREAD_ONLY stuff to dbxread.c, and make separate
-        pstab-norm.h (for most types), pstab-sol.h (for N_SOL), etc.  */
-      int stype;
+      int sclass;
 
       QUIT;
 
+      bfd_coff_swap_sym_in (abfd, sraw_symbol, &symbol);
+      sclass = symbol.n_sclass;
+
       switch (sclass)
        {
        case C_EXT:
@@ -2537,13 +2197,13 @@ scan_xcoff_symtab (section_offsets, objfile)
            union internal_auxent csect_aux;
            unsigned int symnum_before = ssymnum;
 
-           swap_sym (&symbol, &main_aux, &namestring, &sraw_symbol,
+           swap_sym (&symbol, &main_aux[0], &namestring, &sraw_symbol,
                      &ssymnum, objfile);
            if (symbol.n_numaux > 1)
              {
                bfd_coff_swap_aux_in
                  (objfile->obfd,
-                  sraw_symbol - coff_data(abfd)->local_symesz,
+                  sraw_symbol - coff_data (abfd)->local_symesz,
                   symbol.n_type,
                   symbol.n_sclass,
                   symbol.n_numaux - 1,
@@ -2551,7 +2211,12 @@ scan_xcoff_symtab (section_offsets, objfile)
                   &csect_aux);
              }
            else
-             csect_aux = main_aux;
+             csect_aux = main_aux[0];
+
+           /* If symbol name starts with ".$" or "$", ignore it.  */
+           if (namestring[0] == '$'
+               || (namestring[0] == '.' && namestring[1] == '$'))
+             break;
 
            switch (csect_aux.x_csect.x_smtyp & 0x7)
              {
@@ -2580,23 +2245,26 @@ scan_xcoff_symtab (section_offsets, 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, dependencies_used);
+                             (pst, psymtab_include_list, includes_used,
+                              symnum_before, dependency_list,
+                              dependencies_used, textlow_not_set);
                            includes_used = 0;
                            dependencies_used = 0;
                            /* Give all psymtabs for this source file the same
                               name.  */
                            pst = xcoff_start_psymtab
-                             (objfile, section_offsets,
+                             (objfile,
                               filestring,
                               symnum_before,
                               objfile->global_psymbols.next,
                               objfile->static_psymbols.next);
                          }
                      }
-                   if (namestring && namestring[0] == '.')
+                   /* Activate the misc_func_recorded mechanism for
+                      compiler- and linker-generated CSECTs like ".strcmp"
+                      and "@FIX1".  */ 
+                   if (namestring && (namestring[0] == '.'
+                                      || namestring[0] == '@'))
                      {
                        last_csect_name = namestring;
                        last_csect_val = symbol.n_value;
@@ -2606,7 +2274,7 @@ scan_xcoff_symtab (section_offsets, objfile)
                    if (pst != NULL)
                      {
                        CORE_ADDR highval =
-                         symbol.n_value + csect_aux.x_csect.x_scnlen.l;
+                       symbol.n_value + csect_aux.x_csect.x_scnlen.l;
                        if (highval > pst->texthigh)
                          pst->texthigh = highval;
                        if (pst->textlow == 0 || symbol.n_value < pst->textlow)
@@ -2614,17 +2282,35 @@ scan_xcoff_symtab (section_offsets, objfile)
                      }
                    misc_func_recorded = 0;
                    break;
+
                  case XMC_RW:
+                 case XMC_TD:
+                   /* 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,
+                        sclass == C_HIDEXT ? mst_file_data : mst_data,
+                        NULL, secnum_to_section (symbol.n_scnum, objfile),
+                        NULL, objfile);
                    break;
+
                  case XMC_TC0:
                    if (toc_offset)
                      warning ("More than one XMC_TC0 symbol found.");
                    toc_offset = symbol.n_value;
+
+                   /* 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);
                    break;
+
                  case XMC_TC:
                    /* These symbols tell us where the TOC entry for a
                       variable is, not the variable itself.  */
                    break;
+
                  default:
                    break;
                  }
@@ -2634,36 +2320,18 @@ scan_xcoff_symtab (section_offsets, objfile)
                switch (csect_aux.x_csect.x_smclas)
                  {
                  case XMC_PR:
-                   {
-                     /* A function entry point.  */
-                     char *namestr = namestring;
-
-                     if (first_fun_line_offset == 0)
-                       first_fun_line_offset =
-                         main_aux.x_sym.x_fcnary.x_fcn.x_lnnoptr;
-                     if (namestr[0] == '.')
-                       ++namestr;
-                     prim_record_minimal_symbol_and_info
-                       (namestr, symbol.n_value, mst_text,
-                        NULL, secnum_to_section (symbol.n_scnum, objfile),
-                        objfile);
-                     misc_func_recorded = 1;
-
-                     /* We also create full symbols for these, so
-                        better make a partial symbol.  This seems bogus
-                        to me, but I'm not going to try to fix it now.
-                        (Note that allocate_space_in_inferior can't
-                        yet deal with a minimal symbol for malloc on xcoff
-                        because it doesn't understand the fact that
-                        function pointers don't just contain the address of
-                        the function).  */
-                     ADD_PSYMBOL_ADDR_TO_LIST (namestr, strlen (namestr),
-                                               VAR_NAMESPACE, LOC_BLOCK,
-                                               objfile->global_psymbols,
-                                               symbol.n_value,
-                                               psymtab_language, objfile);
-                   }
+                   /* A function entry point.  */
+
+                   if (first_fun_line_offset == 0 && symbol.n_numaux > 1)
+                     first_fun_line_offset =
+                       main_aux[0].x_sym.x_fcnary.x_fcn.x_lnnoptr;
+                   RECORD_MINIMAL_SYMBOL
+                     (namestring, symbol.n_value,
+                      sclass == C_HIDEXT ? mst_file_text : mst_text,
+                      secnum_to_section (symbol.n_scnum, objfile),
+                      objfile);
                    break;
+
                  case XMC_GL:
                    /* shared library function trampoline code entry
                       point. */
@@ -2694,17 +2362,33 @@ scan_xcoff_symtab (section_offsets, objfile)
                       still need to record them.  This will
                       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,
+                        sclass == C_HIDEXT ? mst_file_data : mst_data,
+                        NULL, secnum_to_section (symbol.n_scnum, objfile),
+                        NULL, objfile);
+                   break;
+                 }
+               break;
 
-                   /* FIXME: Shouldn't these be going into the minimal
-                      symbols instead of partial/full symbols?  */
-
-                   ADD_PSYMBOL_ADDR_TO_LIST (namestring, strlen (namestring),
-                                             VAR_NAMESPACE, LOC_STATIC,
-                                             objfile->global_psymbols,
-                                             symbol.n_value,
-                                             psymtab_language, objfile);
+             case XTY_CM:
+               switch (csect_aux.x_csect.x_smclas)
+                 {
+                 case XMC_RW:
+                 case XMC_BS:
+                   /* 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,
+                        sclass == C_HIDEXT ? mst_file_bss : mst_bss,
+                        NULL, secnum_to_section (symbol.n_scnum, objfile),
+                        NULL, objfile);
                    break;
                  }
+               break;
+
              default:
                break;
              }
@@ -2715,7 +2399,7 @@ scan_xcoff_symtab (section_offsets, objfile)
            unsigned int symnum_before;
 
            symnum_before = ssymnum;
-           swap_sym (&symbol, &main_aux, &namestring, &sraw_symbol,
+           swap_sym (&symbol, &main_aux[0], &namestring, &sraw_symbol,
                      &ssymnum, objfile);
 
            /* See if the last csect needs to be recorded.  */
@@ -2735,8 +2419,8 @@ scan_xcoff_symtab (section_offsets, objfile)
            if (pst)
              {
                xcoff_end_psymtab (pst, psymtab_include_list, includes_used,
-                                  symnum_before,
-                                  dependency_list, dependencies_used);
+                                  symnum_before, dependency_list,
+                                  dependencies_used, textlow_not_set);
                includes_used = 0;
                dependencies_used = 0;
              }
@@ -2749,12 +2433,12 @@ scan_xcoff_symtab (section_offsets, objfile)
               exists, otherwise use the symbol itself.  */
            if (!strcmp (namestring, ".file") && symbol.n_numaux > 0)
              {
-               filestring = coff_getfilename (&main_aux, objfile);
+               filestring = coff_getfilename (&main_aux[0], objfile);
              }
            else
              filestring = namestring;
 
-           pst = xcoff_start_psymtab (objfile, section_offsets,
+           pst = xcoff_start_psymtab (objfile,
                                       filestring,
                                       symnum_before,
                                       objfile->global_psymbols.next,
@@ -2765,9 +2449,8 @@ scan_xcoff_symtab (section_offsets, objfile)
 
        default:
          {
-           static struct complaint msg =
-             {"Storage class %d not recognized during scan", 0, 0};
-           complain (&msg, sclass);
+           complaint (&symfile_complaints,
+                      "Storage class %d not recognized during scan", sclass);
          }
          /* FALLTHROUGH */
 
@@ -2816,17 +2499,66 @@ scan_xcoff_symtab (section_offsets, objfile)
          {
            /* We probably could save a few instructions by assuming that
               C_LSYM, C_PSYM, etc., never have auxents.  */
-           int naux1 =
-             ((struct external_syment *)sraw_symbol)->e_numaux[0] + 1;
+           int naux1 = symbol.n_numaux + 1;
            ssymnum += naux1;
-           sraw_symbol += sizeof (struct external_syment) * naux1;
+           sraw_symbol += bfd_coff_symesz (abfd) * naux1;
          }
          break;
 
        case C_BINCL:
-         stype = N_SOL;
-         goto pstab;
+         {
+           /* Mark down an include file in the current psymtab */
+           enum language tmp_language;
+           swap_sym (&symbol, &main_aux[0], &namestring, &sraw_symbol,
+                     &ssymnum, objfile);
 
+           tmp_language = deduce_language_from_filename (namestring);
+
+           /* Only change the psymtab's language if we've learned
+              something useful (eg. tmp_language is not language_unknown).
+              In addition, to match what start_subfile does, never change
+              from C++ to C.  */
+           if (tmp_language != language_unknown
+               && (tmp_language != language_c
+                   || psymtab_language != language_cplus))
+             psymtab_language = tmp_language;
+
+           /* 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
+              if this is a file we've seen before -- either the main
+              source file, or a previously included file.
+
+              This seems to be a lot of time to be spending on N_SOL, but
+              things like "break c-exp.y:435" need to work (I
+              suppose the psymtab_include_list could be hashed or put
+              in a binary tree, if profiling shows this is a major hog).  */
+           if (pst && STREQ (namestring, pst->filename))
+             continue;
+           {
+             register int i;
+             for (i = 0; i < includes_used; i++)
+               if (STREQ (namestring, psymtab_include_list[i]))
+                 {
+                   i = -1;
+                   break;
+                 }
+             if (i == -1)
+               continue;
+           }
+           psymtab_include_list[includes_used++] = namestring;
+           if (includes_used >= includes_allocated)
+             {
+               char **orig = psymtab_include_list;
+
+               psymtab_include_list = (char **)
+                 alloca ((includes_allocated *= 2) *
+                         sizeof (char *));
+               memcpy (psymtab_include_list, orig,
+                       includes_used * sizeof (char *));
+             }
+           continue;
+         }
        case C_FUN:
          /* The value of the C_FUN is not the address of the function (it
             appears to be the address before linking), but as long as it
@@ -2837,53 +2569,295 @@ scan_xcoff_symtab (section_offsets, objfile)
        case C_ECOML:
        case C_DECL:
        case C_STSYM:
-         stype = N_LSYM;
-       pstab:;
-         swap_sym (&symbol, &main_aux, &namestring, &sraw_symbol,
-                   &ssymnum, objfile);
-#define CUR_SYMBOL_TYPE stype
-#define CUR_SYMBOL_VALUE symbol.n_value
-
-/* START_PSYMTAB and END_PSYMTAB are never used, because they are only
-   called from DBXREAD_ONLY or N_SO code.  Likewise for the symnum
-   variable.  */
-#define START_PSYMTAB(ofile,secoff,fname,low,symoff,global_syms,static_syms) 0
-#define END_PSYMTAB(pst,ilist,ninc,c_off,c_text,dep_list,n_deps)\
-  do {} while (0)
-/* We have already set the namestring.  */
-#define SET_NAMESTRING() 0
-
-#include "partial-stab.h"
+         {
+           char *p;
+           swap_sym (&symbol, &main_aux[0], &namestring, &sraw_symbol,
+                     &ssymnum, objfile);
+
+           p = (char *) strchr (namestring, ':');
+           if (!p)
+             continue;                 /* Not a debugging symbol.   */
+
+           /* Main processing section for debugging symbols which
+              the initial read through the symbol tables needs to worry
+              about.  If we reach this point, the symbol which we are
+              considering is definitely one we are interested in.
+              p must also contain the (valid) index into the namestring
+              which indicates the debugging type symbol.  */
+
+           switch (p[1])
+             {
+             case 'S':
+               symbol.n_value += ANOFFSET (objfile->section_offsets, SECT_OFF_DATA (objfile));
+#ifdef STATIC_TRANSFORM_NAME
+               namestring = STATIC_TRANSFORM_NAME (namestring);
+#endif
+               add_psymbol_to_list (namestring, p - namestring,
+                                    VAR_NAMESPACE, LOC_STATIC,
+                                    &objfile->static_psymbols,
+                                    0, 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_NAMESPACE, LOC_STATIC,
+                                    &objfile->global_psymbols,
+                                    0, symbol.n_value,
+                                    psymtab_language, objfile);
+               continue;
+
+             case 'T':
+               /* When a 'T' entry is defining an anonymous enum, it
+                  may have a name which is the empty string, or a
+                  single space.  Since they're not really defining a
+                  symbol, those shouldn't go in the partial symbol
+                  table.  We do pick up the elements of such enums at
+                  'check_enum:', below.  */
+               if (p >= namestring + 2
+                   || (p == namestring + 1
+                       && namestring[0] != ' '))
+                 {
+                   add_psymbol_to_list (namestring, p - namestring,
+                                        STRUCT_NAMESPACE, LOC_TYPEDEF,
+                                        &objfile->static_psymbols,
+                                        symbol.n_value, 0,
+                                        psymtab_language, objfile);
+                   if (p[2] == 't')
+                     {
+                       /* Also a typedef with the same name.  */
+                       add_psymbol_to_list (namestring, p - namestring,
+                                            VAR_NAMESPACE, LOC_TYPEDEF,
+                                            &objfile->static_psymbols,
+                                            symbol.n_value, 0,
+                                            psymtab_language, objfile);
+                       p += 1;
+                     }
+#if 0 /* OBSOLETE CFront */
+// OBSOLETE                /* The semantics of C++ state that "struct foo { ... }"
+// OBSOLETE                   also defines a typedef for "foo".  Unfortuantely, cfront
+// OBSOLETE                   never makes the typedef when translating from C++ to C.
+// OBSOLETE                   We make the typedef here so that "ptype foo" works as
+// OBSOLETE                   expected for cfront translated code.  */
+// OBSOLETE                else if (psymtab_language == language_cplus)
+// OBSOLETE                  {
+// OBSOLETE                    /* Also a typedef with the same name.  */
+// OBSOLETE                    add_psymbol_to_list (namestring, p - namestring,
+// OBSOLETE                                         VAR_NAMESPACE, LOC_TYPEDEF,
+// OBSOLETE                                         &objfile->static_psymbols,
+// OBSOLETE                                         symbol.n_value, 0,
+// OBSOLETE                                         psymtab_language, objfile);
+// OBSOLETE                  }
+#endif /* OBSOLETE CFront */
+                 }
+               goto check_enum;
+
+             case 't':
+               if (p != namestring)    /* a name is there, not just :T... */
+                 {
+                   add_psymbol_to_list (namestring, p - namestring,
+                                        VAR_NAMESPACE, LOC_TYPEDEF,
+                                        &objfile->static_psymbols,
+                                        symbol.n_value, 0,
+                                        psymtab_language, objfile);
+                 }
+             check_enum:
+               /* If this is an enumerated type, we need to
+                  add all the enum constants to the partial symbol
+                  table.  This does not cover enums without names, e.g.
+                  "enum {a, b} c;" in C, but fortunately those are
+                  rare.  There is no way for GDB to find those from the
+                  enum type without spending too much time on it.  Thus
+                  to solve this problem, the compiler needs to put out the
+                  enum in a nameless type.  GCC2 does this.  */
+
+               /* We are looking for something of the form
+                  <name> ":" ("t" | "T") [<number> "="] "e"
+                  {<constant> ":" <value> ","} ";".  */
+
+               /* Skip over the colon and the 't' or 'T'.  */
+               p += 2;
+               /* This type may be given a number.  Also, numbers can come
+                  in pairs like (0,26).  Skip over it.  */
+               while ((*p >= '0' && *p <= '9')
+                      || *p == '(' || *p == ',' || *p == ')'
+                      || *p == '=')
+                 p++;
+
+               if (*p++ == 'e')
+                 {
+                   /* The aix4 compiler emits extra crud before the members.  */
+                   if (*p == '-')
+                     {
+                       /* Skip over the type (?).  */
+                       while (*p != ':')
+                         p++;
+
+                       /* Skip over the colon.  */
+                       p++;
+                     }
+
+                   /* We have found an enumerated type.  */
+                   /* According to comments in read_enum_type
+                      a comma could end it instead of a semicolon.
+                      I don't know where that happens.
+                      Accept either.  */
+                   while (*p && *p != ';' && *p != ',')
+                     {
+                       char *q;
+
+                       /* Check for and handle cretinous dbx symbol name
+                          continuation!  */
+                       if (*p == '\\' || (*p == '?' && p[1] == '\0'))
+                         p = next_symbol_text (objfile);
+
+                       /* Point to the character after the name
+                          of the enum constant.  */
+                       for (q = p; *q && *q != ':'; q++)
+                         ;
+                       /* Note that the value doesn't matter for
+                          enum constants in psymtabs, just in symtabs.  */
+                       add_psymbol_to_list (p, q - p,
+                                            VAR_NAMESPACE, LOC_CONST,
+                                            &objfile->static_psymbols, 0,
+                                            0, psymtab_language, objfile);
+                       /* Point past the name.  */
+                       p = q;
+                       /* Skip over the value.  */
+                       while (*p && *p != ',')
+                         p++;
+                       /* Advance past the comma.  */
+                       if (*p)
+                         p++;
+                     }
+                 }
+               continue;
+
+             case 'c':
+               /* Constant, e.g. from "const" in Pascal.  */
+               add_psymbol_to_list (namestring, p - namestring,
+                                    VAR_NAMESPACE, LOC_CONST,
+                                    &objfile->static_psymbols, symbol.n_value,
+                                    0, psymtab_language, objfile);
+               continue;
+
+             case 'f':
+               if (! pst)
+                 {
+                   int name_len = p - namestring;
+                   char *name = 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_NAMESPACE, LOC_BLOCK,
+                                    &objfile->static_psymbols,
+                                    0, symbol.n_value,
+                                    psymtab_language, objfile);
+               continue;
+
+               /* Global functions were ignored here, but now they
+                  are put into the global psymtab like one would expect.
+                  They're also in the minimal symbol table.  */
+             case 'F':
+               if (! pst)
+                 {
+                   int name_len = p - namestring;
+                   char *name = 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_NAMESPACE, LOC_BLOCK,
+                                    &objfile->global_psymbols,
+                                    0, symbol.n_value,
+                                    psymtab_language, objfile);
+               continue;
+
+               /* Two things show up here (hopefully); static symbols of
+                  local scope (static used inside braces) or extensions
+                  of structure symbols.  We can ignore both.  */
+             case 'V':
+             case '(':
+             case '0':
+             case '1':
+             case '2':
+             case '3':
+             case '4':
+             case '5':
+             case '6':
+             case '7':
+             case '8':
+             case '9':
+             case '-':
+             case '#':         /* for symbol identification (used in live ranges) */
+#if 0 /* OBSOLETE CFront */
+// OBSOLETE            /* added to support cfront stabs strings */
+// OBSOLETE          case 'Z':         /* for definition continuations */
+// OBSOLETE          case 'P':         /* for prototypes */
+#endif /* OBSOLETE CFront */
+               continue;
+
+             case ':':
+               /* It is a C++ nested symbol.  We don't need to record it
+                  (I don't think); if we try to look up foo::bar::baz,
+                  then symbols for the symtab containing foo should get
+                  read in, I think.  */
+               /* Someone says sun cc puts out symbols like
+                  /foo/baz/maclib::/usr/local/bin/maclib,
+                  which would get here with a symbol type of ':'.  */
+               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.  */
+
+               complaint (&symfile_complaints,
+                          "unknown symbol descriptor `%c'", p[1]);
+
+               /* Ignore it; perhaps it is an extension that we don't
+                  know about.  */
+               continue;
+             }
+         }
        }
     }
-#if 0
-      /* What is this?  */
-  /* If there's stuff to be cleaned up, clean it up.  */
-  if (DBX_SYMCOUNT (objfile) > 0                       /* We have some syms */
-/*FIXME, does this have a bug at start address 0? */
-      && last_o_file_start
-      && objfile -> ei.entry_point < bufp->n_value
-      && objfile -> ei.entry_point >= last_o_file_start)
-    {
-      objfile -> ei.entry_file_lowpc = last_o_file_start;
-      objfile -> ei.entry_file_highpc = bufp->n_value;
-    }
-#endif
 
   if (pst)
     {
       xcoff_end_psymtab (pst, psymtab_include_list, includes_used,
-                        ssymnum,
-                        dependency_list, dependencies_used);
+                        ssymnum, dependency_list,
+                        dependencies_used, textlow_not_set);
     }
 
-  /* Record the toc offset value of this symbol table into ldinfo structure.
+  /* 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. */
 
-#ifndef FAKING_RS6000
-  xcoff_add_toc_to_loadinfo (toc_offset);
-#endif
+  ((struct coff_symfile_info *) objfile->sym_private)->toc_offset = toc_offset;
+}
+
+/* Return the toc offset value for a given objfile.  */
+
+CORE_ADDR
+get_toc_offset (struct objfile *objfile)
+{
+  if (objfile)
+    return ((struct coff_symfile_info *) objfile->sym_private)->toc_offset;
+  return 0;
 }
 
 /* Scan and build partial symbols for a symbol file.
@@ -2897,35 +2871,26 @@ scan_xcoff_symtab (section_offsets, objfile)
    table (as opposed to a shared lib or dynamically loaded file).  */
 
 static void
-xcoff_initial_scan (objfile, section_offsets, mainline)
-     struct objfile *objfile;
-     struct section_offsets *section_offsets;
-     int mainline;     /* FIXME comments above */
+xcoff_initial_scan (struct objfile *objfile, int mainline)
 {
   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 */
+  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;
   unsigned int size;
 
-#ifndef FAKING_RS6000
-  /* Initialize load info structure. */
-  if (mainline)
-    xcoff_init_loadinfo ();
-#endif
-
-  info = (struct coff_symfile_info *) objfile -> sym_private;
+  info = (struct coff_symfile_info *) objfile->sym_private;
   symfile_bfd = abfd = objfile->obfd;
   name = objfile->name;
 
   num_symbols = bfd_get_symcount (abfd);       /* # of symbols */
   symtab_offset = obj_sym_filepos (abfd);      /* symbol table file offset */
   stringtab_offset = symtab_offset +
-    num_symbols * coff_data(abfd)->local_symesz;
+    num_symbols * coff_data (abfd)->local_symesz;
 
   info->min_lineno_offset = 0;
   info->max_lineno_offset = 0;
@@ -2959,48 +2924,48 @@ xcoff_initial_scan (objfile, section_offsets, mainline)
                  }
              }
          }
-       ((struct coff_symfile_info *)objfile->sym_private)->debugsec =
+       ((struct coff_symfile_info *) objfile->sym_private)->debugsec =
          debugsec;
       }
     }
 
   /* Read the symbols.  We keep them in core because we will want to
      access them randomly in read_symbol*.  */
-  val = bfd_seek (abfd, symtab_offset, L_SET);
+  val = bfd_seek (abfd, symtab_offset, SEEK_SET);
   if (val < 0)
     error ("Error reading symbols from %s: %s",
           name, bfd_errmsg (bfd_get_error ()));
   size = coff_data (abfd)->local_symesz * num_symbols;
-  ((struct coff_symfile_info *)objfile->sym_private)->symtbl =
+  ((struct coff_symfile_info *) objfile->sym_private)->symtbl =
     obstack_alloc (&objfile->symbol_obstack, size);
-  ((struct coff_symfile_info *)objfile->sym_private)->symtbl_num_syms =
+  ((struct coff_symfile_info *) objfile->sym_private)->symtbl_num_syms =
     num_symbols;
 
-  val = bfd_read (((struct coff_symfile_info *)objfile->sym_private)->symtbl,
-                 size, 1, abfd);
+  val = bfd_bread (((struct coff_symfile_info *) objfile->sym_private)->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)
+      || (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);
 
-  pending_blocks = 0;
+  free_pending_blocks ();
   back_to = make_cleanup (really_free_pendings, 0);
 
   init_minimal_symbol_collection ();
-  make_cleanup (discard_minimal_symbols, 0);
+  make_cleanup_discard_minimal_symbols ();
 
   /* Now that the symbol table data of the executable file are all in core,
      process them and define symbols accordingly.  */
 
-  scan_xcoff_symtab (section_offsets, objfile);
+  scan_xcoff_symtab (objfile);
 
   /* Install any minimal symbols that have been collected as the current
      minimal symbols for this objfile. */
@@ -3010,33 +2975,46 @@ xcoff_initial_scan (objfile, section_offsets, mainline)
   do_cleanups (back_to);
 }
 \f
-static struct section_offsets *
-xcoff_symfile_offsets (objfile, addr)
-     struct objfile *objfile;
-     CORE_ADDR addr;
+static void
+xcoff_symfile_offsets (struct objfile *objfile, struct section_addr_info *addrs)
 {
-  struct section_offsets *section_offsets;
+  asection *sect = NULL;
   int i;
 
   objfile->num_sections = SECT_OFF_MAX;
-  section_offsets = (struct section_offsets *)
-    obstack_alloc
-      (&objfile -> psymbol_obstack,
-       sizeof (struct section_offsets)
-       + sizeof (section_offsets->offsets) * objfile->num_sections);
-
-  /* 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.  */
-  for (i = 0; i < objfile->num_sections; ++i)
-    ANOFFSET (section_offsets, i) = 0;
+  objfile->section_offsets = (struct section_offsets *)
+    obstack_alloc (&objfile->psymbol_obstack, SIZEOF_SECTION_OFFSETS);
+
+  /* Initialize the section indexes for future use. */
+  sect = bfd_get_section_by_name (objfile->obfd, ".text");
+  if (sect) 
+    objfile->sect_index_text = sect->index;
+
+  sect = bfd_get_section_by_name (objfile->obfd, ".data");
+  if (sect) 
+    objfile->sect_index_data = sect->index;
 
-  return section_offsets;
+  sect = bfd_get_section_by_name (objfile->obfd, ".bss");
+  if (sect) 
+    objfile->sect_index_bss = sect->index;
+
+  sect = bfd_get_section_by_name (objfile->obfd, ".rodata");
+  if (sect) 
+    objfile->sect_index_rodata = sect->index;
+
+  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;
+    }
 }
 
 /* Register our ability to parse symbols for xcoff BFD files.  */
@@ -3044,8 +3022,7 @@ xcoff_symfile_offsets (objfile, addr)
 static struct sym_fns xcoff_sym_fns =
 {
 
-  /* Because the bfd uses coff_flavour, we need to specially kludge
-     the flavour.  It is possible that coff and xcoff should be merged as
+  /* It is possible that coff and xcoff should be merged as
      they do have fundamental similarities (for example, the extra storage
      classes used for stabs could presumably be recognized in any COFF file).
      However, in addition to obvious things like all the csect hair, there are
@@ -3054,27 +3031,20 @@ 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).  */
 
-  (enum bfd_flavour)-1,
+  bfd_target_xcoff_flavour,
 
-  xcoff_new_init,      /* sym_new_init: init anything gbl to entire symtab */
-  xcoff_symfile_init,  /* sym_init: read initial info, setup for sym_read() */
-  xcoff_initial_scan,  /* sym_read: read a symbol file into symtab */
-  xcoff_symfile_finish, /* sym_finish: finished with file, cleanup */
-  xcoff_symfile_offsets, /* sym_offsets: xlate offsets ext->int form */
-  NULL                 /* next: pointer to next struct sym_fns */
+  xcoff_new_init,              /* sym_new_init: init anything gbl to entire symtab */
+  xcoff_symfile_init,          /* sym_init: read initial info, setup for sym_read() */
+  xcoff_initial_scan,          /* sym_read: read a symbol file into symtab */
+  xcoff_symfile_finish,                /* sym_finish: finished with file, cleanup */
+  xcoff_symfile_offsets,       /* sym_offsets: xlate offsets ext->int form */
+  NULL                         /* next: pointer to next struct sym_fns */
 };
 
 void
-_initialize_xcoffread ()
+_initialize_xcoffread (void)
 {
-  add_symtab_fns(&xcoff_sym_fns);
-
-  /* Initialize symbol template later used for arguments.  Its other
-     fields are zero, or are filled in later.  */
-  SYMBOL_NAME (&parmsym) = "";
-  SYMBOL_INIT_LANGUAGE_SPECIFIC (&parmsym, language_c);
-  SYMBOL_NAMESPACE (&parmsym) = VAR_NAMESPACE;
-  SYMBOL_CLASS (&parmsym) = LOC_ARG;
+  add_symtab_fns (&xcoff_sym_fns);
 
   func_symbol_type = init_type (TYPE_CODE_FUNC, 1, 0,
                                "<function, no debug info>", NULL);
This page took 0.07178 seconds and 4 git commands to generate.