* fork-child.c (escape_bang_in_quoted_argument): New function.
[deliverable/binutils-gdb.git] / gdb / stabsread.c
index a03fc1ce32fad2fd2a1e477b84246ab5091ed797..cf7a21bf6de2d06b93cee0cc4c3c4fe7abb7797c 100644 (file)
@@ -1,22 +1,24 @@
 /* Support routines for decoding "stabs" debugging information format.
-   Copyright 1986, 87, 88, 89, 90, 91, 92, 93, 94, 95, 96, 1997
-             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.
 
-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.  */
+   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.  */
 
 /* Support routines for reading and decoding debugging information in
    the "stabs" format.  This format is used with many systems that use
@@ -27,7 +29,7 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include "defs.h"
 #include "gdb_string.h"
 #include "bfd.h"
-#include "obstack.h"
+#include "gdb_obstack.h"
 #include "symtab.h"
 #include "gdbtypes.h"
 #include "expression.h"
@@ -41,14 +43,20 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include "complaints.h"
 #include "demangle.h"
 #include "language.h"
+#include "doublest.h"
+#include "cp-abi.h"
+#include "cp-support.h"
 
 #include <ctype.h>
 
 /* Ask stabsread.h to define the vars it normally declares `extern'.  */
-#define        EXTERN  /**/
+#define        EXTERN
+/**/
 #include "stabsread.h"         /* Our own declarations */
 #undef EXTERN
 
+extern void _initialize_stabsread (void);
+
 /* The routines that read and process a complete stabs for a C struct or 
    C++ class pass lists of data member fields and lists of member function
    fields in an instance of a field_info structure, as defined below.
@@ -56,189 +64,167 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
    expected to eventually go away... (FIXME) */
 
 struct field_info
-{
-  struct nextfield
-    {
-      struct nextfield *next;
+  {
+    struct nextfield
+      {
+       struct nextfield *next;
 
-      /* This is the raw visibility from the stab.  It is not checked
-        for being one of the visibilities we recognize, so code which
-        examines this field better be able to deal.  */
-      int visibility;
+       /* This is the raw visibility from the stab.  It is not checked
+          for being one of the visibilities we recognize, so code which
+          examines this field better be able to deal.  */
+       int visibility;
 
-      struct field field;
-    } *list;
-  struct next_fnfieldlist
-    {
-      struct next_fnfieldlist *next;
-      struct fn_fieldlist fn_fieldlist;
-    } *fnlist;
-};
+       struct field field;
+      }
+     *list;
+    struct next_fnfieldlist
+      {
+       struct next_fnfieldlist *next;
+       struct fn_fieldlist fn_fieldlist;
+      }
+     *fnlist;
+  };
 
 static void
-read_one_struct_field PARAMS ((struct field_info *, char **, char *,
-                              struct type *, struct objfile *));
+read_one_struct_field (struct field_info *, char **, char *,
+                      struct type *, struct objfile *);
 
-static char * 
-get_substring PARAMS ((char **, int));
+static char *get_substring (char **, int);
 
-static struct type *
-dbx_alloc_type PARAMS ((int [2], struct objfile *));
+static struct type *dbx_alloc_type (int[2], struct objfile *);
 
-static long read_huge_number PARAMS ((char **, int, int *));
+static long read_huge_number (char **, int, int *);
 
-static struct type *error_type PARAMS ((char **, struct objfile *));
+static struct type *error_type (char **, struct objfile *);
 
 static void
-patch_block_stabs PARAMS ((struct pending *, struct pending_stabs *,
-                          struct objfile *));
+patch_block_stabs (struct pending *, struct pending_stabs *,
+                  struct objfile *);
 
-static void
-fix_common_block PARAMS ((struct symbol *, int));
+static void fix_common_block (struct symbol *, int);
 
-static int
-read_type_number PARAMS ((char **, int *));
+static int read_type_number (char **, int *);
 
-static struct type *
-read_range_type PARAMS ((char **, int [2], struct objfile *));
+static struct type *read_range_type (char **, int[2], struct objfile *);
 
-static struct type *
-read_sun_builtin_type PARAMS ((char **, int [2], struct objfile *));
+static struct type *read_sun_builtin_type (char **, int[2], struct objfile *);
 
-static struct type *
-read_sun_floating_type PARAMS ((char **, int [2], struct objfile *));
+static struct type *read_sun_floating_type (char **, int[2],
+                                           struct objfile *);
 
-static struct type *
-read_enum_type PARAMS ((char **, struct type *, struct objfile *));
+static struct type *read_enum_type (char **, struct type *, struct objfile *);
 
-static struct type *
-rs6000_builtin_type PARAMS ((int));
+static struct type *rs6000_builtin_type (int);
 
 static int
-read_member_functions PARAMS ((struct field_info *, char **, struct type *,
-                              struct objfile *));
+read_member_functions (struct field_info *, char **, struct type *,
+                      struct objfile *);
 
 static int
-read_struct_fields PARAMS ((struct field_info *, char **, struct type *,
-                           struct objfile *));
+read_struct_fields (struct field_info *, char **, struct type *,
+                   struct objfile *);
 
 static int
-read_baseclasses PARAMS ((struct field_info *, char **, struct type *,
-                         struct objfile *));
+read_baseclasses (struct field_info *, char **, struct type *,
+                 struct objfile *);
 
 static int
-read_tilde_fields PARAMS ((struct field_info *, char **, struct type *,
-                          struct objfile *));
+read_tilde_fields (struct field_info *, char **, struct type *,
+                  struct objfile *);
 
-static int
-attach_fn_fields_to_type PARAMS ((struct field_info *, struct type *));
+static int attach_fn_fields_to_type (struct field_info *, struct type *);
 
-static int
-attach_fields_to_type PARAMS ((struct field_info *, struct type *,
-                              struct objfile *));
+static int attach_fields_to_type (struct field_info *, struct type *,
+                                 struct objfile *);
 
-static struct type *
-read_struct_type PARAMS ((char **, struct type *, struct objfile *));
+static struct type *read_struct_type (char **, struct type *,
+                                      enum type_code,
+                                     struct objfile *);
 
-static struct type *
-read_array_type PARAMS ((char **, struct type *, struct objfile *));
+static struct type *read_array_type (char **, struct type *,
+                                    struct objfile *);
 
-static struct type **
-read_args PARAMS ((char **, int, struct objfile *));
+static struct field *read_args (char **, int, struct objfile *, int *, int *);
 
 static int
-read_cpp_abbrev PARAMS ((struct field_info *, char **, struct type *,
-                        struct objfile *));
+read_cpp_abbrev (struct field_info *, char **, struct type *,
+                struct objfile *);
+#if 0 /* OBSOLETE CFront */
+// OBSOLETE  /* new functions added for cfront support */
 
-/* new functions added for cfront support */
+// OBSOLETE  static int
+// OBSOLETE  copy_cfront_struct_fields (struct field_info *, struct type *,
+// OBSOLETE                       struct objfile *);
 
-static int
-copy_cfront_struct_fields PARAMS ((struct field_info *, struct type *,
-                        struct objfile *));
+// OBSOLETE  static char *get_cfront_method_physname (char *);
 
-static char *
-get_cfront_method_physname PARAMS ((char *));
+// OBSOLETE  static int
+// OBSOLETE  read_cfront_baseclasses (struct field_info *, char **,
+// OBSOLETE                     struct type *, struct objfile *);
 
-static int
-read_cfront_baseclasses PARAMS ((struct field_info *, char **, 
-                       struct type *, struct objfile *));
+// OBSOLETE  static int
+// OBSOLETE  read_cfront_static_fields (struct field_info *, char **,
+// OBSOLETE                       struct type *, struct objfile *);
+// OBSOLETE  static int
+// OBSOLETE  read_cfront_member_functions (struct field_info *, char **,
+// OBSOLETE                          struct type *, struct objfile *);
 
-static int
-read_cfront_static_fields PARAMS ((struct field_info *, char**,
-                       struct type *, struct objfile *));
-static int
-read_cfront_member_functions PARAMS ((struct field_info *, char **, 
-                       struct type *, struct objfile *));
-/* end new functions added for cfront support */
+// OBSOLETE  /* end new functions added for cfront support */
+#endif /* OBSOLETE CFront */
 
-static void 
-add_live_range PARAMS ((struct objfile *, struct symbol *, 
-                       CORE_ADDR, CORE_ADDR));
+static char *find_name_end (char *name);
 
-static int 
-resolve_live_range PARAMS ((struct objfile *, struct symbol *, char *));
+static void add_live_range (struct objfile *, struct symbol *, CORE_ADDR,
+                           CORE_ADDR);
 
-static int 
-process_reference PARAMS ((char **string));
+static int resolve_live_range (struct objfile *, struct symbol *, char *);
 
-static CORE_ADDR 
-ref_search_value PARAMS ((int refnum));
+static int process_reference (char **string);
 
-struct symbol * 
-ref_search PARAMS ((int refnum));
+static CORE_ADDR ref_search_value (int refnum);
 
+static int resolve_symbol_reference (struct objfile *, struct symbol *,
+                                    char *);
 
+void stabsread_clear_cache (void);
 
-static const char vptr_name[] = { '_','v','p','t','r',CPLUS_MARKER,'\0' };
-static const char vb_name[] =   { '_','v','b',CPLUS_MARKER,'\0' };
+static const char vptr_name[] = "_vptr$";
+static const char vb_name[] = "_vb$";
 
 /* Define this as 1 if a pcc declaration of a char or short argument
    gives the correct address.  Otherwise assume pcc gives the
    address of the corresponding int, which is not the same on a
    big-endian machine.  */
 
-#ifndef BELIEVE_PCC_PROMOTION
+#if !defined (BELIEVE_PCC_PROMOTION)
 #define BELIEVE_PCC_PROMOTION 0
 #endif
 
-struct complaint invalid_cpp_abbrev_complaint =
-  {"invalid C++ abbreviation `%s'", 0, 0};
-
-struct complaint invalid_cpp_type_complaint =
-  {"C++ abbreviated type name unknown at symtab pos %d", 0, 0};
-
-struct complaint member_fn_complaint =
-  {"member function type missing, got '%c'", 0, 0};
-
-struct complaint const_vol_complaint =
-  {"const/volatile indicator missing, got '%c'", 0, 0};
-
-struct complaint error_type_complaint =
-  {"debug info mismatch between compiler and debugger", 0, 0};
-
-struct complaint invalid_member_complaint =
-  {"invalid (minimal) member type data format at symtab pos %d.", 0, 0};
-
-struct complaint range_type_base_complaint =
-  {"base type %d of range type is not defined", 0, 0};
-
-struct complaint reg_value_complaint =
-  {"register number %d too large (max %d) in symbol %s", 0, 0};
-
-struct complaint vtbl_notfound_complaint =
-  {"virtual function table pointer not found when defining class `%s'", 0, 0};
-
-struct complaint unrecognized_cplus_name_complaint =
-  {"Unknown C++ symbol name `%s'", 0, 0};
+static void
+invalid_cpp_abbrev_complaint (const char *arg1)
+{
+  complaint (&symfile_complaints, "invalid C++ abbreviation `%s'", arg1);
+}
 
-struct complaint rs6000_builtin_complaint =
-  {"Unknown builtin type %d", 0, 0};
+static void
+reg_value_complaint (int arg1, int arg2, const char *arg3)
+{
+  complaint (&symfile_complaints,
+            "register number %d too large (max %d) in symbol %s", arg1, arg2,
+            arg3);
+}
 
-struct complaint unresolved_sym_chain_complaint =
-  {"%s: common block `%s' from global_sym_chain unresolved", 0, 0};
+static void
+stabs_general_complaint (const char *arg1)
+{
+  complaint (&symfile_complaints, "%s", arg1);
+}
 
-struct complaint stabs_general_complaint =
-  {"%s", 0, 0};
+static void
+lrs_general_complaint (const char *arg1)
+{
+  complaint (&symfile_complaints, "%s", arg1);
+}
 
 /* Make a list of forward references which haven't been defined.  */
 
@@ -254,34 +240,6 @@ static struct symbol *current_symbol = NULL;
       *(pp) = next_symbol_text (objfile);      \
   } while (0)
 \f
-/* FIXME: These probably should be our own types (like rs6000_builtin_type
-   has its own types) rather than builtin_type_*.  */
-static struct type **os9k_type_vector[] = {
-       0,
-       &builtin_type_int,
-       &builtin_type_char,
-       &builtin_type_long,
-       &builtin_type_short,
-       &builtin_type_unsigned_char,
-       &builtin_type_unsigned_short,
-       &builtin_type_unsigned_long,
-       &builtin_type_unsigned_int,
-       &builtin_type_float,
-       &builtin_type_double,
-       &builtin_type_void,
-       &builtin_type_long_double
-};
-
-static void os9k_init_type_vector PARAMS ((struct type **));
-
-static void
-os9k_init_type_vector(tv)
-    struct type **tv;
-{
-  int i;
-  for (i=0; i<sizeof(os9k_type_vector)/sizeof(struct type **); i++)
-    tv[i] = (os9k_type_vector[i] == 0 ? 0 : *(os9k_type_vector[i]));
-}
 
 /* Look up a dbx type-number pair.  Return the address of the slot
    where the type for that number-pair is stored.
@@ -291,8 +249,7 @@ os9k_init_type_vector(tv)
    or for associating a new type with the pair.  */
 
 struct type **
-dbx_lookup_type (typenums)
-     int typenums[2];
+dbx_lookup_type (int typenums[2])
 {
   register int filenum = typenums[0];
   register int index = typenums[1];
@@ -306,10 +263,9 @@ dbx_lookup_type (typenums)
 
   if (filenum < 0 || filenum >= n_this_object_header_files)
     {
-      static struct complaint msg = {"\
-Invalid symbol data: type number (%d,%d) out of range at symtab pos %d.",
-                               0, 0};
-      complain (&msg, filenum, index, symnum);
+      complaint (&symfile_complaints,
+                "Invalid symbol data: type number (%d,%d) out of range at symtab pos %d.",
+                filenum, index, symnum);
       goto error_return;
     }
 
@@ -324,12 +280,12 @@ Invalid symbol data: type number (%d,%d) out of range at symtab pos %d.",
             this will do the right thing.  */
          static struct type *temp_type;
 
-         temp_type = rs6000_builtin_type(index);
+         temp_type = rs6000_builtin_type (index);
          return &temp_type;
        }
 
       /* Type is defined outside of header files.
-        Find it in this object file's type vector.  */
+         Find it in this object file's type vector.  */
       if (index >= type_vector_length)
        {
          old_len = type_vector_length;
@@ -348,10 +304,6 @@ Invalid symbol data: type number (%d,%d) out of range at symtab pos %d.",
                      (type_vector_length * sizeof (struct type *)));
          memset (&type_vector[old_len], 0,
                  (type_vector_length - old_len) * sizeof (struct type *));
-
-         if (os9k_stabs)
-           /* Deal with OS9000 fundamental types.  */
-           os9k_init_type_vector (type_vector);
        }
       return (&type_vector[index]);
     }
@@ -398,9 +350,7 @@ Invalid symbol data: type number (%d,%d) out of range at symtab pos %d.",
    put into the type vector, and so may not be referred to by number. */
 
 static struct type *
-dbx_alloc_type (typenums, objfile)
-     int typenums[2];
-     struct objfile *objfile;
+dbx_alloc_type (int typenums[2], struct objfile *objfile)
 {
   register struct type **type_addr;
 
@@ -426,10 +376,8 @@ dbx_alloc_type (typenums, objfile)
    and fix their symbols in given symbol vector. */
 
 static void
-patch_block_stabs (symbols, stabs, objfile)
-     struct pending *symbols;
-     struct pending_stabs *stabs;
-     struct objfile *objfile;
+patch_block_stabs (struct pending *symbols, struct pending_stabs *stabs,
+                  struct objfile *objfile)
 {
   int ii;
   char *name;
@@ -438,33 +386,33 @@ patch_block_stabs (symbols, stabs, objfile)
 
   if (stabs)
     {
-      
+
       /* for all the stab entries, find their corresponding symbols and 
-        patch their types! */
-      
+         patch their types! */
+
       for (ii = 0; ii < stabs->count; ++ii)
        {
          name = stabs->stab[ii];
-         pp = (char*) strchr (name, ':');
+         pp = (char *) strchr (name, ':');
          while (pp[1] == ':')
            {
-              pp += 2;
-              pp = (char *)strchr(pp, ':');
+             pp += 2;
+             pp = (char *) strchr (pp, ':');
            }
-         sym = find_symbol_in_list (symbols, name, pp-name);
+         sym = find_symbol_in_list (symbols, name, pp - name);
          if (!sym)
            {
              /* FIXME-maybe: it would be nice if we noticed whether
-                the variable was defined *anywhere*, not just whether
-                it is defined in this compilation unit.  But neither
-                xlc or GCC seem to need such a definition, and until
-                we do psymtabs (so that the minimal symbols from all
-                compilation units are available now), I'm not sure
-                how to get the information.  */
+                the variable was defined *anywhere*, not just whether
+                it is defined in this compilation unit.  But neither
+                xlc or GCC seem to need such a definition, and until
+                we do psymtabs (so that the minimal symbols from all
+                compilation units are available now), I'm not sure
+                how to get the information.  */
 
              /* On xcoff, if a global is defined and never referenced,
-                ld will remove it from the executable.  There is then
-                a N_GSYM stab for it, but no regular (C_EXT) symbol.  */
+                ld will remove it from the executable.  There is then
+                a N_GSYM stab for it, but no regular (C_EXT) symbol.  */
              sym = (struct symbol *)
                obstack_alloc (&objfile->symbol_obstack,
                               sizeof (struct symbol));
@@ -472,10 +420,10 @@ patch_block_stabs (symbols, stabs, objfile)
              memset (sym, 0, sizeof (struct symbol));
              SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
              SYMBOL_CLASS (sym) = LOC_OPTIMIZED_OUT;
-             SYMBOL_NAME (sym) =
+             DEPRECATED_SYMBOL_NAME (sym) =
                obsavestring (name, pp - name, &objfile->symbol_obstack);
              pp += 2;
-             if (*(pp-1) == 'F' || *(pp-1) == 'f')
+             if (*(pp - 1) == 'F' || *(pp - 1) == 'f')
                {
                  /* I don't think the linker does this with functions,
                     so as far as I know this is never executed.
@@ -492,7 +440,7 @@ patch_block_stabs (symbols, stabs, objfile)
          else
            {
              pp += 2;
-             if (*(pp-1) == 'F' || *(pp-1) == 'f')
+             if (*(pp - 1) == 'F' || *(pp - 1) == 'f')
                {
                  SYMBOL_TYPE (sym) =
                    lookup_function_type (read_type (&pp, objfile));
@@ -505,8 +453,8 @@ patch_block_stabs (symbols, stabs, objfile)
        }
     }
 }
-
 \f
+
 /* Read a number by which a type is referred to in dbx data,
    or perhaps read a pair (FILENUM, TYPENUM) in parentheses.
    Just a single number N is equivalent to (0,N).
@@ -516,567 +464,555 @@ patch_block_stabs (symbols, stabs, objfile)
    Returns 0 for success, -1 for error.  */
 
 static int
-read_type_number (pp, typenums)
-     register char **pp;
-     register int *typenums;
+read_type_number (register char **pp, register int *typenums)
 {
   int nbits;
   if (**pp == '(')
     {
       (*pp)++;
       typenums[0] = read_huge_number (pp, ',', &nbits);
-      if (nbits != 0) return -1;
+      if (nbits != 0)
+       return -1;
       typenums[1] = read_huge_number (pp, ')', &nbits);
-      if (nbits != 0) return -1;
+      if (nbits != 0)
+       return -1;
     }
   else
     {
       typenums[0] = 0;
       typenums[1] = read_huge_number (pp, 0, &nbits);
-      if (nbits != 0) return -1;
+      if (nbits != 0)
+       return -1;
     }
   return 0;
 }
-
 \f
-#if !defined (REG_STRUCT_HAS_ADDR)
-#define REG_STRUCT_HAS_ADDR(gcc_p,type) 0
-#endif
 
 #define VISIBILITY_PRIVATE     '0'     /* Stabs character for private field */
 #define VISIBILITY_PROTECTED   '1'     /* Stabs character for protected fld */
 #define VISIBILITY_PUBLIC      '2'     /* Stabs character for public field */
 #define VISIBILITY_IGNORE      '9'     /* Optimized out or zero length */
 
-#define CFRONT_VISIBILITY_PRIVATE      '2'     /* Stabs character for private field */
-#define CFRONT_VISIBILITY_PUBLIC       '1'     /* Stabs character for public field */
-
-/* This code added to support parsing of ARM/Cfront stabs strings */
-
-/* Get substring from string up to char c, advance string pointer past
-   suibstring. */
-
-static char * 
-get_substring (p, c)
-  char ** p;
-  int c;
-{
-  char *str;
-  str = *p;
-  *p = strchr (*p, c);
-  if (*p) 
-    {
-      **p = 0;
-      (*p)++;
-    }
-  else 
-    str = 0;
-  return str;
-}
-
-/* Physname gets strcat'd onto sname in order to recreate the mangled
-   name (see funtion gdb_mangle_name in gdbtypes.c).  For cfront, make
-   the physname look like that of g++ - take out the initial mangling
-   eg: for sname="a" and fname="foo__1aFPFs_i" return "FPFs_i" */
-
-static char * 
-get_cfront_method_physname (fname)
-  char *fname;
-{
-  int len = 0;
-  /* FIXME would like to make this generic for g++ too, but 
-     that is already handled in read_member_funcctions */
-  char * p = fname;
-
-  /* search ahead to find the start of the mangled suffix */
-  if (*p == '_' && *(p+1)=='_') /* compiler generated; probably a ctor/dtor */
-    p += 2;            
-  while (p && (unsigned) ((p+1) - fname) < strlen (fname) && *(p+1) != '_')
-    p = strchr (p, '_');
-  if (!(p && *p == '_' && *(p+1) == '_')) 
-    error ("Invalid mangled function name %s",fname);
-  p += 2; /* advance past '__' */
-
-  /* struct name length and name of type should come next; advance past it */
-  while (isdigit (*p))
-    {
-      len = len * 10 + (*p - '0');
-      p++;
-    }
-  p += len;
-
-  return p;
-}
-
-/* Read base classes within cfront class definition.
-   eg: A:ZcA;1@Bpub v2@Bvirpri;__ct__1AFv func__1AFv *sfunc__1AFv ;as__1A ;;
-             ^^^^^^^^^^^^^^^^^^
-
-       A:ZcA;;foopri__1AFv foopro__1AFv __ct__1AFv __ct__1AFRC1A foopub__1AFv ;;;
-             ^
-   */
+#if 0 /* OBSOLETE CFront */
+// OBSOLETE  #define CFRONT_VISIBILITY_PRIVATE '2'     /* Stabs character for private field */
+// OBSOLETE  #define CFRONT_VISIBILITY_PUBLIC  '1'     /* Stabs character for public field */
+
+// OBSOLETE  /* This code added to support parsing of ARM/Cfront stabs strings */
+
+// OBSOLETE  /* Get substring from string up to char c, advance string pointer past
+// OBSOLETE     suibstring. */
+
+// OBSOLETE  static char *
+// OBSOLETE  get_substring (char **p, int c)
+// OBSOLETE  {
+// OBSOLETE    char *str;
+// OBSOLETE    str = *p;
+// OBSOLETE    *p = strchr (*p, c);
+// OBSOLETE    if (*p)
+// OBSOLETE      {
+// OBSOLETE        **p = 0;
+// OBSOLETE        (*p)++;
+// OBSOLETE      }
+// OBSOLETE    else
+// OBSOLETE      str = 0;
+// OBSOLETE    return str;
+// OBSOLETE  }
+
+// OBSOLETE  /* Physname gets strcat'd onto sname in order to recreate the mangled
+// OBSOLETE     name (see funtion gdb_mangle_name in gdbtypes.c).  For cfront, make
+// OBSOLETE     the physname look like that of g++ - take out the initial mangling
+// OBSOLETE     eg: for sname="a" and fname="foo__1aFPFs_i" return "FPFs_i" */
+
+// OBSOLETE  static char *
+// OBSOLETE  get_cfront_method_physname (char *fname)
+// OBSOLETE  {
+// OBSOLETE    int len = 0;
+// OBSOLETE    /* FIXME would like to make this generic for g++ too, but 
+// OBSOLETE       that is already handled in read_member_funcctions */
+// OBSOLETE    char *p = fname;
+
+// OBSOLETE    /* search ahead to find the start of the mangled suffix */
+// OBSOLETE    if (*p == '_' && *(p + 1) == '_')       /* compiler generated; probably a ctor/dtor */
+// OBSOLETE      p += 2;
+// OBSOLETE    while (p && (unsigned) ((p + 1) - fname) < strlen (fname) && *(p + 1) != '_')
+// OBSOLETE      p = strchr (p, '_');
+// OBSOLETE    if (!(p && *p == '_' && *(p + 1) == '_'))
+// OBSOLETE      error ("Invalid mangled function name %s", fname);
+// OBSOLETE    p += 2;                 /* advance past '__' */
+
+// OBSOLETE    /* struct name length and name of type should come next; advance past it */
+// OBSOLETE    while (isdigit (*p))
+// OBSOLETE      {
+// OBSOLETE        len = len * 10 + (*p - '0');
+// OBSOLETE        p++;
+// OBSOLETE      }
+// OBSOLETE    p += len;
+
+// OBSOLETE    return p;
+// OBSOLETE  }
+
+// OBSOLETE  static void
+// OBSOLETE  msg_unknown_complaint (const char *arg1)
+// OBSOLETE  {
+// OBSOLETE    complaint (&symfile_complaints, "Unsupported token in stabs string %s", arg1);
+// OBSOLETE  }
+
+// OBSOLETE  /* Read base classes within cfront class definition.
+// OBSOLETE     eg: A:ZcA;1@Bpub v2@Bvirpri;__ct__1AFv func__1AFv *sfunc__1AFv ;as__1A ;;
+// OBSOLETE     ^^^^^^^^^^^^^^^^^^
+
+// OBSOLETE     A:ZcA;;foopri__1AFv foopro__1AFv __ct__1AFv __ct__1AFRC1A foopub__1AFv ;;;
+// OBSOLETE     ^
+// OBSOLETE   */
+
+// OBSOLETE  static int
+// OBSOLETE  read_cfront_baseclasses (struct field_info *fip, char **pp, struct type *type,
+// OBSOLETE                     struct objfile *objfile)
+// OBSOLETE  {
+// OBSOLETE    int bnum = 0;
+// OBSOLETE    char *p;
+// OBSOLETE    int i;
+// OBSOLETE    struct nextfield *new;
+
+// OBSOLETE    if (**pp == ';')                /* no base classes; return */
+// OBSOLETE      {
+// OBSOLETE        ++(*pp);
+// OBSOLETE        return 1;
+// OBSOLETE      }
+
+// OBSOLETE    /* first count base classes so we can allocate space before parsing */
+// OBSOLETE    for (p = *pp; p && *p && *p != ';'; p++)
+// OBSOLETE      {
+// OBSOLETE        if (*p == ' ')
+// OBSOLETE    bnum++;
+// OBSOLETE      }
+// OBSOLETE    bnum++;                 /* add one more for last one */
+
+// OBSOLETE    /* now parse the base classes until we get to the start of the methods 
+// OBSOLETE       (code extracted and munged from read_baseclasses) */
+// OBSOLETE    ALLOCATE_CPLUS_STRUCT_TYPE (type);
+// OBSOLETE    TYPE_N_BASECLASSES (type) = bnum;
+
+// OBSOLETE    /* allocate space */
+// OBSOLETE    {
+// OBSOLETE      int num_bytes = B_BYTES (TYPE_N_BASECLASSES (type));
+// OBSOLETE      char *pointer;
+
+// OBSOLETE      pointer = (char *) TYPE_ALLOC (type, num_bytes);
+// OBSOLETE      TYPE_FIELD_VIRTUAL_BITS (type) = (B_TYPE *) pointer;
+// OBSOLETE    }
+// OBSOLETE    B_CLRALL (TYPE_FIELD_VIRTUAL_BITS (type), TYPE_N_BASECLASSES (type));
+
+// OBSOLETE    for (i = 0; i < TYPE_N_BASECLASSES (type); i++)
+// OBSOLETE      {
+// OBSOLETE        new = (struct nextfield *) xmalloc (sizeof (struct nextfield));
+// OBSOLETE        make_cleanup (xfree, new);
+// OBSOLETE        memset (new, 0, sizeof (struct nextfield));
+// OBSOLETE        new->next = fip->list;
+// OBSOLETE        fip->list = new;
+// OBSOLETE        FIELD_BITSIZE (new->field) = 0;     /* this should be an unpacked field! */
+
+// OBSOLETE        STABS_CONTINUE (pp, objfile);
+
+// OBSOLETE        /* virtual?  eg: v2@Bvir */
+// OBSOLETE        if (**pp == 'v')
+// OBSOLETE    {
+// OBSOLETE      SET_TYPE_FIELD_VIRTUAL (type, i);
+// OBSOLETE      ++(*pp);
+// OBSOLETE    }
+
+// OBSOLETE        /* access?  eg: 2@Bvir */
+// OBSOLETE        /* Note: protected inheritance not supported in cfront */
+// OBSOLETE        switch (*(*pp)++)
+// OBSOLETE    {
+// OBSOLETE    case CFRONT_VISIBILITY_PRIVATE:
+// OBSOLETE      new->visibility = VISIBILITY_PRIVATE;
+// OBSOLETE      break;
+// OBSOLETE    case CFRONT_VISIBILITY_PUBLIC:
+// OBSOLETE      new->visibility = VISIBILITY_PUBLIC;
+// OBSOLETE      break;
+// OBSOLETE    default:
+// OBSOLETE      /* Bad visibility format.  Complain and treat it as
+// OBSOLETE         public.  */
+// OBSOLETE      {
+// OBSOLETE        complaint (&symfile_complaints,
+// OBSOLETE                   "Unknown visibility `%c' for baseclass",
+// OBSOLETE                   new->visibility);
+// OBSOLETE        new->visibility = VISIBILITY_PUBLIC;
+// OBSOLETE      }
+// OBSOLETE    }
+
+// OBSOLETE        /* "@" comes next - eg: @Bvir */
+// OBSOLETE        if (**pp != '@')
+// OBSOLETE    {
+// OBSOLETE      msg_unknown_complaint (*pp);
+// OBSOLETE      return 1;
+// OBSOLETE    }
+// OBSOLETE        ++(*pp);
+
+
+// OBSOLETE        /* Set the bit offset of the portion of the object corresponding 
+// OBSOLETE           to this baseclass.  Always zero in the absence of
+// OBSOLETE           multiple inheritance.  */
+// OBSOLETE        /* Unable to read bit position from stabs;
+// OBSOLETE           Assuming no multiple inheritance for now FIXME! */
+// OBSOLETE        /* We may have read this in the structure definition;
+// OBSOLETE           now we should fixup the members to be the actual base classes */
+// OBSOLETE        FIELD_BITPOS (new->field) = 0;
+
+// OBSOLETE        /* Get the base class name and type */
+// OBSOLETE        {
+// OBSOLETE    char *bname;            /* base class name */
+// OBSOLETE    struct symbol *bsym;    /* base class */
+// OBSOLETE    char *p1, *p2;
+// OBSOLETE    p1 = strchr (*pp, ' ');
+// OBSOLETE    p2 = strchr (*pp, ';');
+// OBSOLETE    if (p1 < p2)
+// OBSOLETE      bname = get_substring (pp, ' ');
+// OBSOLETE    else
+// OBSOLETE      bname = get_substring (pp, ';');
+// OBSOLETE    if (!bname || !*bname)
+// OBSOLETE      {
+// OBSOLETE        msg_unknown_complaint (*pp);
+// OBSOLETE        return 1;
+// OBSOLETE      }
+// OBSOLETE    /* FIXME! attach base info to type */
+// OBSOLETE    bsym = lookup_symbol (bname, 0, STRUCT_NAMESPACE, 0, 0);        /*demangled_name */
+// OBSOLETE    if (bsym)
+// OBSOLETE      {
+// OBSOLETE        new->field.type = SYMBOL_TYPE (bsym);
+// OBSOLETE        new->field.name = type_name_no_tag (new->field.type);
+// OBSOLETE      }
+// OBSOLETE    else
+// OBSOLETE      {
+// OBSOLETE        complaint (&symfile_complaints, "Unable to find base type for %s",
+// OBSOLETE                   *pp);
+// OBSOLETE        return 1;
+// OBSOLETE      }
+// OBSOLETE        }
+
+// OBSOLETE        /* If more base classes to parse, loop again.
+// OBSOLETE           We ate the last ' ' or ';' in get_substring,
+// OBSOLETE           so on exit we will have skipped the trailing ';' */
+// OBSOLETE        /* if invalid, return 0; add code to detect  - FIXME! */
+// OBSOLETE      }
+// OBSOLETE    return 1;
+// OBSOLETE  }
+
+// OBSOLETE  /* read cfront member functions.
+// OBSOLETE     pp points to string starting with list of functions
+// OBSOLETE     eg: A:ZcA;1@Bpub v2@Bvirpri;__ct__1AFv func__1AFv *sfunc__1AFv ;as__1A ;;
+// OBSOLETE     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ 
+// OBSOLETE     A:ZcA;;foopri__1AFv foopro__1AFv __ct__1AFv __ct__1AFRC1A foopub__1AFv ;;;
+// OBSOLETE     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ 
+// OBSOLETE   */
+
+// OBSOLETE  static int
+// OBSOLETE  read_cfront_member_functions (struct field_info *fip, char **pp,
+// OBSOLETE                          struct type *type, struct objfile *objfile)
+// OBSOLETE  {
+// OBSOLETE    /* This code extracted from read_member_functions 
+// OBSOLETE       so as to do the similar thing for our funcs */
+
+// OBSOLETE    int nfn_fields = 0;
+// OBSOLETE    int length = 0;
+// OBSOLETE    /* Total number of member functions defined in this class.  If the class
+// OBSOLETE       defines two `f' functions, and one `g' function, then this will have
+// OBSOLETE       the value 3.  */
+// OBSOLETE    int total_length = 0;
+// OBSOLETE    int i;
+// OBSOLETE    struct next_fnfield
+// OBSOLETE      {
+// OBSOLETE        struct next_fnfield *next;
+// OBSOLETE        struct fn_field fn_field;
+// OBSOLETE      }
+// OBSOLETE     *sublist;
+// OBSOLETE    struct type *look_ahead_type;
+// OBSOLETE    struct next_fnfieldlist *new_fnlist;
+// OBSOLETE    struct next_fnfield *new_sublist;
+// OBSOLETE    char *main_fn_name;
+// OBSOLETE    char *fname;
+// OBSOLETE    struct symbol *ref_func = 0;
+
+// OBSOLETE    /* Process each list until we find the end of the member functions.
+// OBSOLETE       eg: p = "__ct__1AFv foo__1AFv ;;;" */
+
+// OBSOLETE    STABS_CONTINUE (pp, objfile);   /* handle \\ */
+
+// OBSOLETE    while (**pp != ';' && (fname = get_substring (pp, ' '), fname))
+// OBSOLETE      {
+// OBSOLETE        int is_static = 0;
+// OBSOLETE        int sublist_count = 0;
+// OBSOLETE        char *pname;
+// OBSOLETE        if (fname[0] == '*')        /* static member */
+// OBSOLETE    {
+// OBSOLETE      is_static = 1;
+// OBSOLETE      sublist_count++;
+// OBSOLETE      fname++;
+// OBSOLETE    }
+// OBSOLETE        ref_func = lookup_symbol (fname, 0, VAR_NAMESPACE, 0, 0);           /* demangled name */
+// OBSOLETE        if (!ref_func)
+// OBSOLETE    {
+// OBSOLETE      complaint (&symfile_complaints,
+// OBSOLETE                 "Unable to find function symbol for %s", fname);
+// OBSOLETE      continue;
+// OBSOLETE    }
+// OBSOLETE        sublist = NULL;
+// OBSOLETE        look_ahead_type = NULL;
+// OBSOLETE        length = 0;
+
+// OBSOLETE        new_fnlist = (struct next_fnfieldlist *)
+// OBSOLETE    xmalloc (sizeof (struct next_fnfieldlist));
+// OBSOLETE        make_cleanup (xfree, new_fnlist);
+// OBSOLETE        memset (new_fnlist, 0, sizeof (struct next_fnfieldlist));
+
+// OBSOLETE        /* The following is code to work around cfront generated stabs.
+// OBSOLETE           The stabs contains full mangled name for each field.
+// OBSOLETE           We try to demangle the name and extract the field name out of it.  */
+// OBSOLETE        {
+// OBSOLETE    char *dem, *dem_p, *dem_args;
+// OBSOLETE    int dem_len;
+// OBSOLETE    dem = cplus_demangle (fname, DMGL_ANSI | DMGL_PARAMS);
+// OBSOLETE    if (dem != NULL)
+// OBSOLETE      {
+// OBSOLETE        dem_p = strrchr (dem, ':');
+// OBSOLETE        if (dem_p != 0 && *(dem_p - 1) == ':')
+// OBSOLETE          dem_p++;
+// OBSOLETE        /* get rid of args */
+// OBSOLETE        dem_args = strchr (dem_p, '(');
+// OBSOLETE        if (dem_args == NULL)
+// OBSOLETE          dem_len = strlen (dem_p);
+// OBSOLETE        else
+// OBSOLETE          dem_len = dem_args - dem_p;
+// OBSOLETE        main_fn_name =
+// OBSOLETE          obsavestring (dem_p, dem_len, &objfile->type_obstack);
+// OBSOLETE      }
+// OBSOLETE    else
+// OBSOLETE      {
+// OBSOLETE        main_fn_name =
+// OBSOLETE          obsavestring (fname, strlen (fname), &objfile->type_obstack);
+// OBSOLETE      }
+// OBSOLETE        }                           /* end of code for cfront work around */
+
+// OBSOLETE        new_fnlist->fn_fieldlist.name = main_fn_name;
+
+// OBSOLETE  /*-------------------------------------------------*/
+// OBSOLETE        /* Set up the sublists
+// OBSOLETE           Sublists are stuff like args, static, visibility, etc.
+// OBSOLETE           so in ARM, we have to set that info some other way.
+// OBSOLETE           Multiple sublists happen if overloading
+// OBSOLETE           eg: foo::26=##1;:;2A.;
+// OBSOLETE           In g++, we'd loop here thru all the sublists...  */
+
+// OBSOLETE        new_sublist =
+// OBSOLETE    (struct next_fnfield *) xmalloc (sizeof (struct next_fnfield));
+// OBSOLETE        make_cleanup (xfree, new_sublist);
+// OBSOLETE        memset (new_sublist, 0, sizeof (struct next_fnfield));
+
+// OBSOLETE        /* eat 1; from :;2A.; */
+// OBSOLETE        new_sublist->fn_field.type = SYMBOL_TYPE (ref_func);        /* normally takes a read_type */
+// OBSOLETE        /* Make this type look like a method stub for gdb */
+// OBSOLETE        TYPE_FLAGS (new_sublist->fn_field.type) |= TYPE_FLAG_STUB;
+// OBSOLETE        TYPE_CODE (new_sublist->fn_field.type) = TYPE_CODE_METHOD;
+
+// OBSOLETE        /* If this is just a stub, then we don't have the real name here. */
+// OBSOLETE        if (TYPE_STUB (new_sublist->fn_field.type))
+// OBSOLETE    {
+// OBSOLETE      if (!TYPE_DOMAIN_TYPE (new_sublist->fn_field.type))
+// OBSOLETE        TYPE_DOMAIN_TYPE (new_sublist->fn_field.type) = type;
+// OBSOLETE      new_sublist->fn_field.is_stub = 1;
+// OBSOLETE    }
+
+// OBSOLETE        /* physname used later in mangling; eg PFs_i,5 for foo__1aFPFs_i 
+// OBSOLETE           physname gets strcat'd in order to recreate the onto mangled name */
+// OBSOLETE        pname = get_cfront_method_physname (fname);
+// OBSOLETE        new_sublist->fn_field.physname = savestring (pname, strlen (pname));
+
+
+// OBSOLETE        /* Set this member function's visibility fields. 
+// OBSOLETE           Unable to distinguish access from stabs definition!
+// OBSOLETE           Assuming public for now.  FIXME!
+// OBSOLETE           (for private, set new_sublist->fn_field.is_private = 1,
+// OBSOLETE           for public, set new_sublist->fn_field.is_protected = 1) */
+
+// OBSOLETE        /* Unable to distinguish const/volatile from stabs definition!
+// OBSOLETE           Assuming normal for now.  FIXME! */
+
+// OBSOLETE        new_sublist->fn_field.is_const = 0;
+// OBSOLETE        new_sublist->fn_field.is_volatile = 0;      /* volatile not implemented in cfront */
+
+// OBSOLETE        /* Set virtual/static function info
+// OBSOLETE           How to get vtable offsets ? 
+// OBSOLETE           Assuming normal for now FIXME!! 
+// OBSOLETE           For vtables, figure out from whence this virtual function came.
+// OBSOLETE           It may belong to virtual function table of
+// OBSOLETE           one of its baseclasses.
+// OBSOLETE           set:
+// OBSOLETE           new_sublist -> fn_field.voffset = vtable offset,
+// OBSOLETE           new_sublist -> fn_field.fcontext = look_ahead_type;
+// OBSOLETE           where look_ahead_type is type of baseclass */
+// OBSOLETE        if (is_static)
+// OBSOLETE    new_sublist->fn_field.voffset = VOFFSET_STATIC;
+// OBSOLETE        else                        /* normal member function.  */
+// OBSOLETE    new_sublist->fn_field.voffset = 0;
+// OBSOLETE        new_sublist->fn_field.fcontext = 0;
+
+
+// OBSOLETE        /* Prepare new sublist */
+// OBSOLETE        new_sublist->next = sublist;
+// OBSOLETE        sublist = new_sublist;
+// OBSOLETE        length++;
+
+// OBSOLETE        /* In g++, we loop thu sublists - now we set from functions. */
+// OBSOLETE        new_fnlist->fn_fieldlist.fn_fields = (struct fn_field *)
+// OBSOLETE    obstack_alloc (&objfile->type_obstack,
+// OBSOLETE                   sizeof (struct fn_field) * length);
+// OBSOLETE        memset (new_fnlist->fn_fieldlist.fn_fields, 0,
+// OBSOLETE          sizeof (struct fn_field) * length);
+// OBSOLETE        for (i = length; (i--, sublist); sublist = sublist->next)
+// OBSOLETE    {
+// OBSOLETE      new_fnlist->fn_fieldlist.fn_fields[i] = sublist->fn_field;
+// OBSOLETE    }
+
+// OBSOLETE        new_fnlist->fn_fieldlist.length = length;
+// OBSOLETE        new_fnlist->next = fip->fnlist;
+// OBSOLETE        fip->fnlist = new_fnlist;
+// OBSOLETE        nfn_fields++;
+// OBSOLETE        total_length += length;
+// OBSOLETE        STABS_CONTINUE (pp, objfile);       /* handle \\ */
+// OBSOLETE      }                             /* end of loop */
+
+// OBSOLETE    if (nfn_fields)
+// OBSOLETE      {
+// OBSOLETE        /* type should already have space */
+// OBSOLETE        TYPE_FN_FIELDLISTS (type) = (struct fn_fieldlist *)
+// OBSOLETE    TYPE_ALLOC (type, sizeof (struct fn_fieldlist) * nfn_fields);
+// OBSOLETE        memset (TYPE_FN_FIELDLISTS (type), 0,
+// OBSOLETE          sizeof (struct fn_fieldlist) * nfn_fields);
+// OBSOLETE        TYPE_NFN_FIELDS (type) = nfn_fields;
+// OBSOLETE        TYPE_NFN_FIELDS_TOTAL (type) = total_length;
+// OBSOLETE      }
+
+// OBSOLETE    /* end of scope for reading member func */
+
+// OBSOLETE    /* eg: ";;" */
+
+// OBSOLETE    /* Skip trailing ';' and bump count of number of fields seen */
+// OBSOLETE    if (**pp == ';')
+// OBSOLETE      (*pp)++;
+// OBSOLETE    else
+// OBSOLETE      return 0;
+// OBSOLETE    return 1;
+// OBSOLETE  }
+
+// OBSOLETE  /* This routine fixes up partial cfront types that were created
+// OBSOLETE     while parsing the stabs.  The main need for this function is
+// OBSOLETE     to add information such as methods to classes.
+// OBSOLETE     Examples of "p": "sA;;__ct__1AFv foo__1AFv ;;;" */
+// OBSOLETE  int
+// OBSOLETE  resolve_cfront_continuation (struct objfile *objfile, struct symbol *sym,
+// OBSOLETE                         char *p)
+// OBSOLETE  {
+// OBSOLETE    struct symbol *ref_sym = 0;
+// OBSOLETE    char *sname;
+// OBSOLETE    /* snarfed from read_struct_type */
+// OBSOLETE    struct field_info fi;
+// OBSOLETE    struct type *type;
+// OBSOLETE    struct cleanup *back_to;
+
+// OBSOLETE    /* Need to make sure that fi isn't gunna conflict with struct 
+// OBSOLETE       in case struct already had some fnfs */
+// OBSOLETE    fi.list = NULL;
+// OBSOLETE    fi.fnlist = NULL;
+// OBSOLETE    back_to = make_cleanup (null_cleanup, 0);
+
+// OBSOLETE    /* We only accept structs, classes and unions at the moment. 
+// OBSOLETE       Other continuation types include t (typedef), r (long dbl), ... 
+// OBSOLETE       We may want to add support for them as well; 
+// OBSOLETE       right now they are handled by duplicating the symbol information 
+// OBSOLETE       into the type information (see define_symbol) */
+// OBSOLETE    if (*p != 's'                   /* structs */
+// OBSOLETE        && *p != 'c'                /* class */
+// OBSOLETE        && *p != 'u')               /* union */
+// OBSOLETE      return 0;                     /* only handle C++ types */
+// OBSOLETE    p++;
+
+// OBSOLETE    /* Get symbol typs name and validate 
+// OBSOLETE       eg: p = "A;;__ct__1AFv foo__1AFv ;;;" */
+// OBSOLETE    sname = get_substring (&p, ';');
+// OBSOLETE    if (!sname || strcmp (sname, DEPRECATED_SYMBOL_NAME (sym)))
+// OBSOLETE      error ("Internal error: base symbol type name does not match\n");
+
+// OBSOLETE    /* Find symbol's internal gdb reference using demangled_name.
+// OBSOLETE       This is the real sym that we want; 
+// OBSOLETE       sym was a temp hack to make debugger happy */
+// OBSOLETE    ref_sym = lookup_symbol (DEPRECATED_SYMBOL_NAME (sym), 0, STRUCT_NAMESPACE, 0, 0);
+// OBSOLETE    type = SYMBOL_TYPE (ref_sym);
+
+
+// OBSOLETE    /* Now read the baseclasses, if any, read the regular C struct or C++
+// OBSOLETE       class member fields, attach the fields to the type, read the C++
+// OBSOLETE       member functions, attach them to the type, and then read any tilde
+// OBSOLETE       field (baseclass specifier for the class holding the main vtable). */
+
+// OBSOLETE    if (!read_cfront_baseclasses (&fi, &p, type, objfile)
+// OBSOLETE    /* g++ does this next, but cfront already did this: 
+// OBSOLETE       || !read_struct_fields (&fi, &p, type, objfile) */
+// OBSOLETE        || !copy_cfront_struct_fields (&fi, type, objfile)
+// OBSOLETE        || !read_cfront_member_functions (&fi, &p, type, objfile)
+// OBSOLETE        || !read_cfront_static_fields (&fi, &p, type, objfile)
+// OBSOLETE        || !attach_fields_to_type (&fi, type, objfile)
+// OBSOLETE        || !attach_fn_fields_to_type (&fi, type)
+// OBSOLETE    /* g++ does this next, but cfront doesn't seem to have this: 
+// OBSOLETE       || !read_tilde_fields (&fi, &p, type, objfile) */
+// OBSOLETE      )
+// OBSOLETE      {
+// OBSOLETE        type = error_type (&p, objfile);
+// OBSOLETE      }
+
+// OBSOLETE    do_cleanups (back_to);
+// OBSOLETE    return 0;
+// OBSOLETE  }
+// OBSOLETE  /* End of code added to support parsing of ARM/Cfront stabs strings */
+#endif /* OBSOLETE CFront */
+
+/* This routine fixes up symbol references/aliases to point to the original
+   symbol definition.  Returns 0 on failure, non-zero on success.  */
 
 static int
-read_cfront_baseclasses (fip, pp, type, objfile) 
-  struct field_info *fip;
-  struct objfile *objfile;
-  char ** pp;
-  struct type *type;
-{
-  static struct complaint msg_unknown = {"\
-        Unsupported token in stabs string %s.\n",
-                 0, 0};
-  static struct complaint msg_notfound = {"\
-                  Unable to find base type for %s.\n",
-                                0, 0};
-  int bnum = 0;
-  char * p;
-  int i;
-  struct nextfield *new;
-
-  if (**pp == ';')             /* no base classes; return */
-    {
-      ++(*pp);
-      return 1;
-    }
-
-  /* first count base classes so we can allocate space before parsing */
-  for (p = *pp; p && *p && *p != ';'; p++)
-    {
-      if (*p == ' ')
-       bnum++;
-    }
-  bnum++;      /* add one more for last one */
-
-  /* now parse the base classes until we get to the start of the methods 
-     (code extracted and munged from read_baseclasses) */
-  ALLOCATE_CPLUS_STRUCT_TYPE (type);
-  TYPE_N_BASECLASSES(type) = bnum;
-
-  /* allocate space */
-  {
-    int num_bytes = B_BYTES (TYPE_N_BASECLASSES (type));
-    char *pointer;
-
-    pointer = (char *) TYPE_ALLOC (type, num_bytes);
-    TYPE_FIELD_VIRTUAL_BITS (type) = (B_TYPE *) pointer;
-  }
-  B_CLRALL (TYPE_FIELD_VIRTUAL_BITS (type), TYPE_N_BASECLASSES (type));
-
-  for (i = 0; i < TYPE_N_BASECLASSES (type); i++)
-    {
-      new = (struct nextfield *) xmalloc (sizeof (struct nextfield));
-      make_cleanup (free, new);
-      memset (new, 0, sizeof (struct nextfield));
-      new -> next = fip -> list;
-      fip -> list = new;
-      FIELD_BITSIZE (new->field) = 0; /* this should be an unpacked field! */
-
-      STABS_CONTINUE (pp, objfile);
-
-      /* virtual?  eg: v2@Bvir */
-      if (**pp=='v')
-        {
-          SET_TYPE_FIELD_VIRTUAL (type, i);
-          ++(*pp);
-       }
-
-      /* access?  eg: 2@Bvir */
-       /* Note: protected inheritance not supported in cfront */
-      switch (*(*pp)++)
-        {
-          case CFRONT_VISIBILITY_PRIVATE:
-            new -> visibility = VISIBILITY_PRIVATE;
-            break;
-          case CFRONT_VISIBILITY_PUBLIC:
-            new -> visibility = VISIBILITY_PUBLIC;
-            break;
-          default:
-            /* Bad visibility format.  Complain and treat it as
-               public.  */
-            {
-              static struct complaint msg = {
-                "Unknown visibility `%c' for baseclass", 0, 0};
-              complain (&msg, new -> visibility);
-              new -> visibility = VISIBILITY_PUBLIC;
-            }
-        }
-
-      /* "@" comes next - eg: @Bvir */
-      if (**pp!='@')
-        {
-          complain (&msg_unknown, *pp);
-          return 1;
-       }
-      ++(*pp);
-
-
-        /* Set the bit offset of the portion of the object corresponding 
-          to this baseclass.  Always zero in the absence of
-           multiple inheritance.  */
-       /* Unable to read bit position from stabs;
-          Assuming no multiple inheritance for now FIXME! */
-       /* We may have read this in the structure definition;
-          now we should fixup the members to be the actual base classes */
-        FIELD_BITPOS (new->field) = 0;
-
-       /* Get the base class name and type */
-         {
-           char * bname;               /* base class name */
-           struct symbol * bsym;       /* base class */
-           char * p1, * p2;
-           p1 = strchr (*pp,' ');
-           p2 = strchr (*pp,';');
-           if (p1<p2)
-              bname = get_substring (pp,' ');
-           else
-              bname = get_substring (pp,';');
-            if (!bname || !*bname)
-             {
-               complain (&msg_unknown, *pp);
-               return 1;
-             }
-           /* FIXME! attach base info to type */
-           bsym = lookup_symbol (bname, 0, STRUCT_NAMESPACE, 0, 0); /*demangled_name*/
-           if (bsym) 
-             {
-               new -> field.type = SYMBOL_TYPE(bsym);
-               new -> field.name = type_name_no_tag (new -> field.type);
-             }
-           else
-             {
-               complain (&msg_notfound, *pp);
-               return 1;
-             }
-         }
-
-      /* If more base classes to parse, loop again.
-         We ate the last ' ' or ';' in get_substring,
-         so on exit we will have skipped the trailing ';' */
-      /* if invalid, return 0; add code to detect  - FIXME! */
-    }
-  return 1;
-}
-
-/* read cfront member functions.
-   pp points to string starting with list of functions
-   eg: A:ZcA;1@Bpub v2@Bvirpri;__ct__1AFv func__1AFv *sfunc__1AFv ;as__1A ;;
-                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ 
-       A:ZcA;;foopri__1AFv foopro__1AFv __ct__1AFv __ct__1AFRC1A foopub__1AFv ;;;
-              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ 
-*/
-
-static int
-read_cfront_member_functions (fip, pp, type, objfile)
-     struct field_info *fip;
-     char **pp;
-     struct type *type;
-     struct objfile *objfile;
-{
-  /* This code extracted from read_member_functions 
-     so as to do the similar thing for our funcs */
-
-  int nfn_fields = 0;
-  int length = 0;
-  /* Total number of member functions defined in this class.  If the class
-     defines two `f' functions, and one `g' function, then this will have
-     the value 3.  */
-  int total_length = 0;
-  int i;
-  struct next_fnfield
-    {
-      struct next_fnfield *next;
-      struct fn_field fn_field;
-    } *sublist;
-  struct type *look_ahead_type;
-  struct next_fnfieldlist *new_fnlist;
-  struct next_fnfield *new_sublist;
-  char *main_fn_name;
-  char * fname;
-  struct symbol * ref_func = 0;
-      
-  /* Process each list until we find the end of the member functions.
-     eg: p = "__ct__1AFv foo__1AFv ;;;" */
-
-  STABS_CONTINUE (pp, objfile);                /* handle \\ */
-
-  while (**pp != ';' && (fname = get_substring (pp, ' '), fname)) 
-    {
-      int is_static = 0;
-      int sublist_count = 0;
-      char * pname;
-      if (fname[0] == '*')      /* static member */
-        {
-          is_static=1;
-          sublist_count++;
-          fname++;
-        }
-      ref_func = lookup_symbol (fname, 0, VAR_NAMESPACE, 0, 0); /* demangled name */
-      if (!ref_func) 
-        {
-          static struct complaint msg = {"\
-               Unable to find function symbol for %s\n",
-                                0, 0};
-         complain (&msg, fname);
-         continue;
-       }
-      sublist = NULL;
-      look_ahead_type = NULL;
-      length = 0;
-          
-      new_fnlist = (struct next_fnfieldlist *)
-      xmalloc (sizeof (struct next_fnfieldlist));
-      make_cleanup (free, new_fnlist);
-      memset (new_fnlist, 0, sizeof (struct next_fnfieldlist));
-          
-      /* The following is code to work around cfront generated stabs.
-         The stabs contains full mangled name for each field.
-         We try to demangle the name and extract the field name out of it.  */
-      {
-        char *dem, *dem_p, *dem_args;
-        int dem_len;
-        dem = cplus_demangle (fname, DMGL_ANSI | DMGL_PARAMS);
-        if (dem != NULL)
-          {
-            dem_p = strrchr (dem, ':');
-            if (dem_p != 0 && *(dem_p-1) == ':')
-              dem_p++;
-           /* get rid of args */
-            dem_args = strchr (dem_p, '(');
-           if (dem_args == NULL)
-             dem_len = strlen (dem_p);
-           else
-             dem_len = dem_args - dem_p;
-            main_fn_name =
-                   obsavestring (dem_p, dem_len, &objfile -> type_obstack);
-          }
-        else
-          {
-            main_fn_name =
-                   obsavestring (fname, strlen (fname), &objfile -> type_obstack);
-          }
-      } /* end of code for cfront work around */
-
-    new_fnlist -> fn_fieldlist.name = main_fn_name;
-      
-    /*-------------------------------------------------*/
-    /* Set up the sublists
-       Sublists are stuff like args, static, visibility, etc.
-       so in ARM, we have to set that info some other way.
-       Multiple sublists happen if overloading
-       eg: foo::26=##1;:;2A.;
-       In g++, we'd loop here thru all the sublists...  */
-
-    new_sublist =
-       (struct next_fnfield *) xmalloc (sizeof (struct next_fnfield));
-    make_cleanup (free, new_sublist);
-    memset (new_sublist, 0, sizeof (struct next_fnfield));
-         
-    /* eat 1; from :;2A.; */
-    new_sublist -> fn_field.type = SYMBOL_TYPE(ref_func); /* normally takes a read_type */
-    /* Make this type look like a method stub for gdb */
-    TYPE_FLAGS (new_sublist -> fn_field.type) |= TYPE_FLAG_STUB;
-    TYPE_CODE (new_sublist -> fn_field.type) = TYPE_CODE_METHOD;
-
-    /* If this is just a stub, then we don't have the real name here. */
-    if (TYPE_FLAGS (new_sublist -> fn_field.type) & TYPE_FLAG_STUB)
-      {
-        if (!TYPE_DOMAIN_TYPE (new_sublist -> fn_field.type))
-        TYPE_DOMAIN_TYPE (new_sublist -> fn_field.type) = type;
-        new_sublist -> fn_field.is_stub = 1;
-      }
-
-    /* physname used later in mangling; eg PFs_i,5 for foo__1aFPFs_i 
-       physname gets strcat'd in order to recreate the onto mangled name */
-    pname = get_cfront_method_physname (fname);
-    new_sublist -> fn_field.physname = savestring (pname, strlen (pname));
-      
-
-    /* Set this member function's visibility fields. 
-       Unable to distinguish access from stabs definition!
-         Assuming public for now.  FIXME!
-        (for private, set new_sublist->fn_field.is_private = 1,
-        for public, set new_sublist->fn_field.is_protected = 1) */
-       
-    /* Unable to distinguish const/volatile from stabs definition!
-       Assuming normal for now.  FIXME! */
-
-    new_sublist -> fn_field.is_const = 0;
-    new_sublist -> fn_field.is_volatile = 0;   /* volatile not implemented in cfront */
-         
-    /* Set virtual/static function info
-       How to get vtable offsets ? 
-       Assuming normal for now FIXME!! 
-       For vtables, figure out from whence this virtual function came.
-       It may belong to virtual function table of
-       one of its baseclasses.
-        set:
-          new_sublist -> fn_field.voffset = vtable offset,
-          new_sublist -> fn_field.fcontext = look_ahead_type;
-          where look_ahead_type is type of baseclass */
-    if (is_static)
-      new_sublist -> fn_field.voffset = VOFFSET_STATIC;
-    else                               /* normal member function.  */
-      new_sublist -> fn_field.voffset = 0;
-    new_sublist -> fn_field.fcontext = 0;
-
-    /* Prepare new sublist */
-    new_sublist -> next = sublist;
-    sublist = new_sublist;
-    length++;
-
-    /* In g++, we loop thu sublists - now we set from functions. */
-    new_fnlist -> fn_fieldlist.fn_fields = (struct fn_field *)
-         obstack_alloc (&objfile -> type_obstack, 
-              sizeof (struct fn_field) * length);
-    memset (new_fnlist -> fn_fieldlist.fn_fields, 0,
-           sizeof (struct fn_field) * length);
-    for (i = length; (i--, sublist); sublist = sublist -> next)
-      {
-        new_fnlist -> fn_fieldlist.fn_fields[i] = sublist -> fn_field;
-      }
-      
-    new_fnlist -> fn_fieldlist.length = length;
-    new_fnlist -> next = fip -> fnlist;
-    fip -> fnlist = new_fnlist;
-    nfn_fields++;
-    total_length += length;
-    STABS_CONTINUE (pp, objfile); /* handle \\ */
-  } /* end of loop */
-
-  if (nfn_fields)
-    {
-      /* type should already have space */
-      TYPE_FN_FIELDLISTS (type) = (struct fn_fieldlist *)
-      TYPE_ALLOC (type, sizeof (struct fn_fieldlist) * nfn_fields);
-      memset (TYPE_FN_FIELDLISTS (type), 0,
-               sizeof (struct fn_fieldlist) * nfn_fields);
-      TYPE_NFN_FIELDS (type) = nfn_fields;
-      TYPE_NFN_FIELDS_TOTAL (type) = total_length;
-    }
-
-  /* end of scope for reading member func */
-
-  /* eg: ";;" */
-
-  /* Skip trailing ';' and bump count of number of fields seen */
-  if (**pp == ';')
-    (*pp)++;
-  else
-    return 0;
-  return 1;
-}
-
-/* This routine fixes up partial cfront types that were created
-   while parsing the stabs.  The main need for this function is
-   to add information such as methods to classes.
-   Examples of "p": "sA;;__ct__1AFv foo__1AFv ;;;" */
-int
-resolve_cfront_continuation (objfile, sym, p)
-  struct objfile * objfile;
-  struct symbol * sym;
-  char * p;
-{
-  struct symbol * ref_sym=0;
-  char * sname;
-  /* snarfed from read_struct_type */
-  struct field_info fi;
-  struct type *type;
-  struct cleanup *back_to;
-
-  /* Need to make sure that fi isn't gunna conflict with struct 
-     in case struct already had some fnfs */
-  fi.list = NULL;
-  fi.fnlist = NULL;       
-  back_to = make_cleanup (null_cleanup, 0);
-
-  /* We only accept structs, classes and unions at the moment. 
-     Other continuation types include t (typedef), r (long dbl), ... 
-     We may want to add support for them as well; 
-     right now they are handled by duplicating the symbol information 
-     into the type information (see define_symbol) */
-  if (*p != 's'       /* structs */
-    && *p != 'c'      /* class */
-    && *p != 'u')     /* union */
-    return 0;  /* only handle C++ types */
-  p++;  
-
-  /* Get symbol typs name and validate 
-     eg: p = "A;;__ct__1AFv foo__1AFv ;;;" */
-  sname = get_substring (&p, ';');
-  if (!sname || strcmp (sname, SYMBOL_NAME(sym)))
-    error ("Internal error: base symbol type name does not match\n");
-
-  /* Find symbol's internal gdb reference using demangled_name.
-     This is the real sym that we want; 
-     sym was a temp hack to make debugger happy */
-  ref_sym = lookup_symbol (SYMBOL_NAME(sym), 0, STRUCT_NAMESPACE, 0, 0); 
-  type = SYMBOL_TYPE(ref_sym);
-
-
-  /* Now read the baseclasses, if any, read the regular C struct or C++
-     class member fields, attach the fields to the type, read the C++
-     member functions, attach them to the type, and then read any tilde
-     field (baseclass specifier for the class holding the main vtable). */
-
-  if (!read_cfront_baseclasses (&fi, &p, type, objfile)
-      /* g++ does this next, but cfront already did this: 
-           || !read_struct_fields (&fi, &p, type, objfile) */
-      || !copy_cfront_struct_fields (&fi, type, objfile)
-      || !read_cfront_member_functions (&fi, &p, type, objfile)
-      || !read_cfront_static_fields (&fi, &p, type, objfile)
-      || !attach_fields_to_type (&fi, type, objfile)
-      || !attach_fn_fields_to_type (&fi, type)
-      /* g++ does this next, but cfront doesn't seem to have this: 
-               || !read_tilde_fields (&fi, &p, type, objfile) */
-      )
-    {
-      type = error_type (&p, objfile);
-    }
-
-  do_cleanups (back_to);
-  return 0;  
-}
-/* End of code added to support parsing of ARM/Cfront stabs strings */
-
-
-/* This routine fixes up symbol references to point to the original
-   symbol definition.
-   The main need for this function is to add information for supporting 
-   live range splitting.
-   eg: p : "#7=", "#2=z:r(0,1)" "#2:r(0,1);l(#5,#6),l(#7,#4)" */
-int
-resolve_symbol_reference (objfile, sym, p)
-  struct objfile * objfile;
-  struct symbol * sym;
-  char * p;
+resolve_symbol_reference (struct objfile *objfile, struct symbol *sym, char *p)
 {
   int refnum;
-  struct symbol * ref_sym=0;
-  struct cleanup *back_to;
-
-  back_to = make_cleanup (null_cleanup, 0);
+  struct symbol *ref_sym = 0;
+  struct alias_list *alias;
 
-  if (*p != '#')       /* symbol ref id */
-    return 0;  
+  /* If this is not a symbol reference return now.  */
+  if (*p != '#')
+    return 0;
 
   /* Use "#<num>" as the name; we'll fix the name later.
      We stored the original symbol name as "#<id>=<name>"
      so we can now search for "#<id>" to resolving the reference.
      We'll fix the names later by removing the "#<id>" or "#<id>=" */
 
-  /*---------------------------------------------------------*/
+/*---------------------------------------------------------*/
   /* Get the reference id number, and 
      advance p past the names so we can parse the rest. 
-       eg: id=2 for p : "2=", "2=z:r(0,1)" "2:r(0,1);l(#5,#6),l(#7,#4)" */
-  /*---------------------------------------------------------*/
+     eg: id=2 for p : "2=", "2=z:r(0,1)" "2:r(0,1);l(#5,#6),l(#7,#4)" */
+/*---------------------------------------------------------*/
 
   /* This gets reference name from string.  sym may not have a name. */
+
+  /* Get the reference number associated with the reference id in the
+     gdb stab string.  From that reference number, get the main/primary
+     symbol for this alias.  */
   refnum = process_reference (&p);
   ref_sym = ref_search (refnum);
   if (!ref_sym)
-    error ("error: symbol for reference not found.\n");
+    {
+      lrs_general_complaint ("symbol for reference not found");
+      return 0;
+    }
 
   /* Parse the stab of the referencing symbol
      now that we have the referenced symbol.
@@ -1085,85 +1021,94 @@ resolve_symbol_reference (objfile, sym, p)
 
 
   /* If the stab symbol table and string contain:
-         RSYM   0      5      00000000 868    #15=z:r(0,1)
-         LBRAC  0      0      00000000 899    #5=
-         SLINE  0      16     00000003 923    #6=
+     RSYM   0      5      00000000 868    #15=z:r(0,1)
+     LBRAC  0      0      00000000 899    #5=
+     SLINE  0      16     00000003 923    #6=
      Then the same symbols can be later referenced by:
-         RSYM   0      5      00000000 927    #15:r(0,1);l(#5,#6)
+     RSYM   0      5      00000000 927    #15:r(0,1);l(#5,#6)
      This is used in live range splitting to:
      1) specify that a symbol (#15) is actually just a new storage 
-        class for a symbol (#15=z) which was previously defined.
+     class for a symbol (#15=z) which was previously defined.
      2) specify that the beginning and ending ranges for a symbol 
-        (#15) are the values of the beginning (#5) and ending (#6) 
-        symbols. */
-       
-   /* Read number as reference id.
-      eg: p : "=", "=z:r(0,1)" ":r(0,1);l(#5,#6),l(#7,#4)" */
-   /* FIXME! Might I want to use SYMBOL_CLASS (sym) = LOC_OPTIMIZED_OUT;
-      in case of "l(0,0)"? */
-
-   /*--------------------------------------------------*/
-   /* Add this symbol to the reference list.           */
-   /*--------------------------------------------------*/
-   SYMBOL_ALIASES (sym) = SYMBOL_ALIASES (ref_sym);
-   SYMBOL_ALIASES (ref_sym) = sym;
-
-   /* Want to fix up name so that other functions (eg. valops)
-      will correctly print the name.
-      Don't add_symbol_to_list so that lookup_symbol won't find it.
-      nope... needed for fixups. */
-   SYMBOL_NAME (sym) = SYMBOL_NAME (ref_sym);
+     (#15) are the values of the beginning (#5) and ending (#6) 
+     symbols. */
 
-  /* Done!  */
-
-  do_cleanups (back_to);
-  return 0;  
-}
+  /* Read number as reference id.
+     eg: p : "=", "=z:r(0,1)" ":r(0,1);l(#5,#6),l(#7,#4)" */
+  /* FIXME! Might I want to use SYMBOL_CLASS (sym) = LOC_OPTIMIZED_OUT;
+     in case of "l(0,0)"? */
 
-/* Get range symbol reference. eg. "#2),l(#3,#5)"   
-   postpone resolve_reference until after we're done reading symbols. */
-struct symbol *
-resolve_reference (p)
-     char *p;
-{
-    char sym_refid[32];
-    struct symbol *sym = 0;
-    char *s = p;
-    int len;
+/*--------------------------------------------------*/
+  /* Add this symbol to the reference list.           */
+/*--------------------------------------------------*/
 
-    if (!s || *s != '#')
-      return 0;
-    p = strchr (s, ')');
-    if (!p || p == s)
+  alias = (struct alias_list *) obstack_alloc (&objfile->type_obstack,
+                                              sizeof (struct alias_list));
+  if (!alias)
+    {
+      lrs_general_complaint ("Unable to allocate alias list memory");
       return 0;
-    len = p - s + 1;
-    strncpy (sym_refid, s, len);
-    sym_refid[len] = '\0';
-    sym = lookup_symbol (sym_refid, 0, VAR_NAMESPACE, 0, 0); 
-    return sym;
+    }
+
+  alias->next = 0;
+  alias->sym = sym;
+
+  if (!SYMBOL_ALIASES (ref_sym))
+    {
+      SYMBOL_ALIASES (ref_sym) = alias;
+    }
+  else
+    {
+      struct alias_list *temp;
+
+      /* Get to the end of the list.  */
+      for (temp = SYMBOL_ALIASES (ref_sym);
+          temp->next;
+          temp = temp->next)
+       ;
+      temp->next = alias;
+    }
+
+  /* Want to fix up name so that other functions (eg. valops)
+     will correctly print the name.
+     Don't add_symbol_to_list so that lookup_symbol won't find it.
+     nope... needed for fixups. */
+  DEPRECATED_SYMBOL_NAME (sym) = DEPRECATED_SYMBOL_NAME (ref_sym);
+
+  /* Done!  */
+  return 1;
 }
 
 /* Structure for storing pointers to reference definitions for fast lookup 
    during "process_later". */
-#define MAX_REFS 100   /* FIXME!  Change to use heap. */
-static struct ref_map_s
+
+struct ref_map
 {
   char *stabs;
   CORE_ADDR value;
   struct symbol *sym;
-} ref_map[MAX_REFS];   
-
-/* Initialize our list of references.
-   This should be called before any symbol table is read.
-   FIXME: Will reference numbers be unique only to objects?  If so, we may 
-   need to add something to disambiguate the refids.  Or, it might be OK to 
-   leave as is, as long as we read and process an object's symbol table all 
-   at once.  */
-static int ref_count = 0;      /* Ptr to free cell in linked list. */
-static void ref_init ()
+};
+
+#define MAX_CHUNK_REFS 100
+#define REF_CHUNK_SIZE (MAX_CHUNK_REFS * sizeof (struct ref_map))
+#define REF_MAP_SIZE(ref_chunk) ((ref_chunk) * REF_CHUNK_SIZE)
+
+static struct ref_map *ref_map;
+
+/* Ptr to free cell in chunk's linked list. */
+static int ref_count = 0;
+
+/* Number of chunks malloced. */
+static int ref_chunk = 0;
+
+/* This file maintains a cache of stabs aliases found in the symbol
+   table. If the symbol table changes, this cache must be cleared
+   or we are left holding onto data in invalid obstacks. */
+void
+stabsread_clear_cache (void)
 {
   ref_count = 0;
-  memset (ref_map, 0, MAX_REFS * sizeof (struct ref_map_s));
+  ref_chunk = 0;
 }
 
 /* Create array of pointers mapping refids to symbols and stab strings.
@@ -1171,77 +1116,61 @@ static void ref_init ()
    find them, using their reference numbers as our index. 
    These will be used later when we resolve references. */
 void
-ref_add (refnum, sym, stabs, value)
-     int refnum;
-     struct symbol *sym;
-     char *stabs;
-     CORE_ADDR value;
+ref_add (int refnum, struct symbol *sym, char *stabs, CORE_ADDR value)
 {
   if (ref_count == 0)
-    ref_init ();
+    ref_chunk = 0;
   if (refnum >= ref_count)
     ref_count = refnum + 1;
-  if (ref_count > MAX_REFS)
-    error ("no more free slots in chain\n");
+  if (ref_count > ref_chunk * MAX_CHUNK_REFS)
+    {
+      int new_slots = ref_count - ref_chunk * MAX_CHUNK_REFS;
+      int new_chunks = new_slots / MAX_CHUNK_REFS + 1;
+      ref_map = (struct ref_map *)
+       xrealloc (ref_map, REF_MAP_SIZE (ref_chunk + new_chunks));
+      memset (ref_map + ref_chunk * MAX_CHUNK_REFS, 0, new_chunks * REF_CHUNK_SIZE);
+      ref_chunk += new_chunks;
+    }
   ref_map[refnum].stabs = stabs;
   ref_map[refnum].sym = sym;
   ref_map[refnum].value = value;
 }
 
-/* Remove reference at refnum. 
-   This should be called before a new symbol table is read to clear out the 
-   previous symbol's reference information. */
-/* FIXME!  not used at the moment. */
-static void
-ref_rmv (refnum)
-     int refnum;
-{
-  if (ref_count < 0)
-    error ("slots are empty\n");
-  if (refnum < 0 || refnum > ref_count)
-    error ("No reference for refnum.\n");
-  /* Copy last element over the removed element and reduce count. */
-  ref_map[refnum].stabs = ref_map[ref_count].stabs;
-  ref_map[refnum].sym = ref_map[ref_count].sym;
-  --ref_count;
-}
-
-/* Return defined sym for the reference "refnum" */
+/* Return defined sym for the reference REFNUM.  */
 struct symbol *
-ref_search (refnum)
-     int refnum;
+ref_search (int refnum)
 {
   if (refnum < 0 || refnum > ref_count)
     return 0;
   return ref_map[refnum].sym;
 }
 
-/* Return value for the reference "refnum" */
-CORE_ADDR
-ref_search_value (refnum)
-     int refnum;
+/* Return value for the reference REFNUM.  */
+
+static CORE_ADDR
+ref_search_value (int refnum)
 {
   if (refnum < 0 || refnum > ref_count)
     return 0;
   return ref_map[refnum].value;
 }
-   
-/* Parse reference id and advance string to the next character following
-   the string. 
-   Return the reference number. */
-
-static int 
-process_reference (string)
-    char **string;
+
+/* Parse a reference id in STRING and return the resulting
+   reference number.  Move STRING beyond the reference id.  */
+
+static int
+process_reference (char **string)
 {
   char *p;
   int refnum = 0;
 
-  if (**string != '#') 
-    return 0;  
-     
+  if (**string != '#')
+    return 0;
+
+  /* Advance beyond the initial '#'.  */
+  p = *string + 1;
+
   /* Read number as reference id. */
-  p = *string + 1;     /* Advance beyond '#' */
   while (*p && isdigit (*p))
     {
       refnum = refnum * 10 + *p - '0';
@@ -1251,12 +1180,11 @@ process_reference (string)
   return refnum;
 }
 
-/* If string defines a reference, store away a pointer to the reference 
-   definition for fast lookup when we "process_later",
-   and return the reference number. */
+/* If STRING defines a reference, store away a pointer to the reference 
+   definition for later use.  Return the reference number.  */
+
 int
-symbol_reference_defined (string)
-     char **string;
+symbol_reference_defined (char **string)
 {
   char *p = *string;
   int refnum = 0;
@@ -1264,10 +1192,9 @@ symbol_reference_defined (string)
   refnum = process_reference (&p);
 
   /* Defining symbols end in '=' */
-  if (*p == '=') 
+  if (*p == '=')
     {
-      /* Symbol is being defined here. */ 
-
+      /* Symbol is being defined here. */
       *string = p + 1;
       return refnum;
     }
@@ -1275,23 +1202,18 @@ symbol_reference_defined (string)
     {
       /* Must be a reference.   Either the symbol has already been defined,
          or this is a forward reference to it.  */
-
       *string = p;
-      return 0;        /* Not defined here */
+      return -1;
     }
 }
 
 /* ARGSUSED */
 struct symbol *
-define_symbol (valu, string, desc, type, objfile)
-     CORE_ADDR valu;
-     char *string;
-     int desc;
-     int type;
-     struct objfile *objfile;
+define_symbol (CORE_ADDR valu, char *string, int desc, int type,
+              struct objfile *objfile)
 {
   register struct symbol *sym;
-  char *p = (char *) strchr (string, ':');
+  char *p = (char *) find_name_end (string);
   int deftype;
   int synonym = 0;
   register int i;
@@ -1313,40 +1235,40 @@ define_symbol (valu, string, desc, type, objfile)
 
   while (p[1] == ':')
     {
-       p += 2;
-       p = strchr (p, ':');
+      p += 2;
+      p = strchr (p, ':');
     }
 
   /* If a nameless stab entry, all we need is the type, not the symbol.
      e.g. ":t10=*2" or a nameless enum like " :T16=ered:0,green:1,blue:2,;" */
   nameless = (p == string || ((string[0] == ' ') && (string[1] == ':')));
 
-  current_symbol = sym = (struct symbol *) 
-    obstack_alloc (&objfile -> symbol_obstack, sizeof (struct symbol));
+  current_symbol = sym = (struct symbol *)
+    obstack_alloc (&objfile->symbol_obstack, sizeof (struct symbol));
   memset (sym, 0, sizeof (struct symbol));
 
   switch (type & N_TYPE)
     {
     case N_TEXT:
-      SYMBOL_SECTION(sym) = SECT_OFF_TEXT;
+      SYMBOL_SECTION (sym) = SECT_OFF_TEXT (objfile);
       break;
     case N_DATA:
-      SYMBOL_SECTION(sym) = SECT_OFF_DATA;
+      SYMBOL_SECTION (sym) = SECT_OFF_DATA (objfile);
       break;
     case N_BSS:
-      SYMBOL_SECTION(sym) = SECT_OFF_BSS;
+      SYMBOL_SECTION (sym) = SECT_OFF_BSS (objfile);
       break;
     }
 
   if (processing_gcc_compilation)
     {
       /* GCC 2.x puts the line number in desc.  SunOS apparently puts in the
-        number of bytes occupied by a type or object, which we ignore.  */
-      SYMBOL_LINE(sym) = desc;
+         number of bytes occupied by a type or object, which we ignore.  */
+      SYMBOL_LINE (sym) = desc;
     }
   else
     {
-      SYMBOL_LINE(sym) = 0;                    /* unknown */
+      SYMBOL_LINE (sym) = 0;   /* unknown */
     }
 
   if (is_cplus_marker (string[0]))
@@ -1354,33 +1276,34 @@ define_symbol (valu, string, desc, type, objfile)
       /* Special GNU C++ names.  */
       switch (string[1])
        {
-         case 't':
-           SYMBOL_NAME (sym) = obsavestring ("this", strlen ("this"),
-                                             &objfile -> symbol_obstack);
-           break;
+       case 't':
+         DEPRECATED_SYMBOL_NAME (sym) = obsavestring ("this", strlen ("this"),
+                                           &objfile->symbol_obstack);
+         break;
 
-         case 'v': /* $vtbl_ptr_type */
-           /* Was: SYMBOL_NAME (sym) = "vptr"; */
-           goto normal;
+       case 'v':               /* $vtbl_ptr_type */
+         /* Was: DEPRECATED_SYMBOL_NAME (sym) = "vptr"; */
+         goto normal;
 
-         case 'e':
-           SYMBOL_NAME (sym) = obsavestring ("eh_throw", strlen ("eh_throw"),
-                                             &objfile -> symbol_obstack);
-           break;
+       case 'e':
+         DEPRECATED_SYMBOL_NAME (sym) = obsavestring ("eh_throw", strlen ("eh_throw"),
+                                           &objfile->symbol_obstack);
+         break;
 
-         case '_':
-           /* This was an anonymous type that was never fixed up.  */
-           goto normal;
+       case '_':
+         /* This was an anonymous type that was never fixed up.  */
+         goto normal;
 
 #ifdef STATIC_TRANSFORM_NAME
-         case 'X':
-           /* SunPRO (3.0 at least) static variable encoding.  */
-           goto normal;
+       case 'X':
+         /* SunPRO (3.0 at least) static variable encoding.  */
+         goto normal;
 #endif
 
-         default:
-           complain (&unrecognized_cplus_name_complaint, string);
-           goto normal;                /* Do *something* with it */
+       default:
+         complaint (&symfile_complaints, "Unknown C++ symbol name `%s'",
+                    string);
+         goto normal;          /* Do *something* with it */
        }
     }
   else if (string[0] == '#')
@@ -1389,66 +1312,40 @@ define_symbol (valu, string, desc, type, objfile)
       char *s;
       int refnum, nlen;
 
-      /* Initialize symbol references and determine if this is 
-         a definition.  If symbol reference is being defined, go 
-         ahead and add it.  Otherwise, just return sym. */
+      /* If STRING defines a new reference id, then add it to the
+         reference map.  Else it must be referring to a previously
+         defined symbol, so add it to the alias list of the previously
+         defined symbol.  */
       s = string;
-      if (refnum = symbol_reference_defined (&s), refnum)
-        ref_add (refnum, sym, string, SYMBOL_VALUE (sym));
-      else
-        process_later (sym, string, resolve_symbol_reference);
-
-      /* s is after refid... advance string there
-         so that the symbol name will not include the refid. */
+      refnum = symbol_reference_defined (&s);
+      if (refnum >= 0)
+       ref_add (refnum, sym, string, SYMBOL_VALUE (sym));
+      else if (!resolve_symbol_reference (objfile, sym, string))
+       return NULL;
+
+      /* S..P contains the name of the symbol.  We need to store
+         the correct name into DEPRECATED_SYMBOL_NAME.  */
       nlen = p - s;
-      if (nlen > 0)
-        {
-          SYMBOL_NAME (sym) = (char *)
-           obstack_alloc (&objfile -> symbol_obstack, nlen);
-          strncpy (SYMBOL_NAME (sym), s, nlen);
-          SYMBOL_NAME (sym)[nlen] = '\0';
-          SYMBOL_INIT_DEMANGLED_NAME (sym, &objfile->symbol_obstack);
-        }
-      else
-        /* FIXME! Want SYMBOL_NAME (sym) = 0;
-          Get error if leave name 0.  So give it something. */
-        {
-          nlen = p - string;
-          SYMBOL_NAME (sym)    = (char *)
-           obstack_alloc (&objfile -> symbol_obstack, nlen);
-          strncpy (SYMBOL_NAME (sym), string, nlen);
-          SYMBOL_NAME (sym)[nlen] = '\0';
-          SYMBOL_INIT_DEMANGLED_NAME (sym, &objfile->symbol_obstack);
-        }
+      if (refnum >= 0)
+       {
+         if (nlen > 0)
+           SYMBOL_SET_NAMES (sym, s, nlen, objfile);
+         else
+           /* FIXME! Want DEPRECATED_SYMBOL_NAME (sym) = 0;
+              Get error if leave name 0.  So give it something. */
+           {
+             nlen = p - string;
+             SYMBOL_SET_NAMES (sym, string, nlen, objfile);
+           }
+       }
+      /* Advance STRING beyond the reference id.  */
       string = s;
     }
   else
     {
     normal:
-      SYMBOL_LANGUAGE (sym) = current_subfile -> language;
-      SYMBOL_NAME (sym)        = (char *)
-       obstack_alloc (&objfile -> symbol_obstack, ((p - string) + 1));
-      /* Open-coded memcpy--saves function call time.  */
-      /* FIXME:  Does it really?  Try replacing with simple strcpy and
-        try it on an executable with a large symbol table. */
-      /* FIXME: considering that gcc can open code memcpy anyway, I
-        doubt it.  xoxorich. */
-      {
-       register char *p1 = string;
-       register char *p2 = SYMBOL_NAME (sym);
-       while (p1 != p)
-         {
-           *p2++ = *p1++;
-         }
-       *p2++ = '\0';
-      }
-
-      /* If this symbol is from a C++ compilation, then attempt to cache the
-        demangled form for future reference.  This is a typical time versus
-        space tradeoff, that was decided in favor of time because it sped up
-        C++ symbol lookups by a factor of about 20. */
-
-      SYMBOL_INIT_DEMANGLED_NAME (sym, &objfile->symbol_obstack);
+      SYMBOL_LANGUAGE (sym) = current_subfile->language;
+      SYMBOL_SET_NAMES (sym, string, p - string, objfile);
     }
   p++;
 
@@ -1473,11 +1370,11 @@ define_symbol (valu, string, desc, type, objfile)
     {
     case 'c':
       /* c is a special case, not followed by a type-number.
-        SYMBOL:c=iVALUE for an integer constant symbol.
-        SYMBOL:c=rVALUE for a floating constant symbol.
-        SYMBOL:c=eTYPE,INTVALUE for an enum constant symbol.
-        e.g. "b:c=e6,0" for "const b = blob1"
-        (where type 6 is defined by "blobs:t6=eblob1:0,blob2:1,;").  */
+         SYMBOL:c=iVALUE for an integer constant symbol.
+         SYMBOL:c=rVALUE for a floating constant symbol.
+         SYMBOL:c=eTYPE,INTVALUE for an enum constant symbol.
+         e.g. "b:c=e6,0" for "const b = blob1"
+         (where type 6 is defined by "blobs:t6=eblob1:0,blob2:1,;").  */
       if (*p != '=')
        {
          SYMBOL_CLASS (sym) = LOC_CONST;
@@ -1508,9 +1405,9 @@ define_symbol (valu, string, desc, type, objfile)
            SYMBOL_TYPE (sym) = lookup_fundamental_type (objfile,
                                                         FT_DBL_PREC_FLOAT);
            dbl_valu = (char *)
-             obstack_alloc (&objfile -> symbol_obstack,
+             obstack_alloc (&objfile->symbol_obstack,
                             TYPE_LENGTH (SYMBOL_TYPE (sym)));
-           store_floating (dbl_valu, TYPE_LENGTH (SYMBOL_TYPE (sym)), d);
+           store_typed_floating (dbl_valu, SYMBOL_TYPE (sym), d);
            SYMBOL_VALUE_BYTES (sym) = dbl_valu;
            SYMBOL_CLASS (sym) = LOC_CONST_BYTES;
          }
@@ -1538,7 +1435,7 @@ define_symbol (valu, string, desc, type, objfile)
                init_type (TYPE_CODE_INT,
                           sizeof (int) * HOST_CHAR_BIT / TARGET_CHAR_BIT, 0,
                           "integer constant",
-                          (struct objfile *)NULL);
+                            (struct objfile *) NULL);
            SYMBOL_TYPE (sym) = int_const_type;
            SYMBOL_VALUE (sym) = atoi (p);
            SYMBOL_CLASS (sym) = LOC_CONST;
@@ -1598,21 +1495,65 @@ define_symbol (valu, string, desc, type, objfile)
 
     process_function_types:
       /* Function result types are described as the result type in stabs.
-        We need to convert this to the function-returning-type-X type
-        in GDB.  E.g. "int" is converted to "function returning int".  */
+         We need to convert this to the function-returning-type-X type
+         in GDB.  E.g. "int" is converted to "function returning int".  */
       if (TYPE_CODE (SYMBOL_TYPE (sym)) != TYPE_CODE_FUNC)
        SYMBOL_TYPE (sym) = lookup_function_type (SYMBOL_TYPE (sym));
+
+      /* All functions in C++ have prototypes.  Stabs does not offer an
+         explicit way to identify prototyped or unprototyped functions,
+         but both GCC and Sun CC emit stabs for the "call-as" type rather
+         than the "declared-as" type for unprototyped functions, so
+         we treat all functions as if they were prototyped.  This is used
+         primarily for promotion when calling the function from GDB.  */
+      TYPE_FLAGS (SYMBOL_TYPE (sym)) |= TYPE_FLAG_PROTOTYPED;
+
       /* fall into process_prototype_types */
 
     process_prototype_types:
-      /* Sun acc puts declared types of arguments here.  We don't care
-        about their actual types (FIXME -- we should remember the whole
-        function prototype), but the list may define some new types
-        that we have to remember, so we must scan it now.  */
-      while (*p == ';') {
-       p++;
-       read_type (&p, objfile);
-      }
+      /* Sun acc puts declared types of arguments here.  */
+      if (*p == ';')
+       {
+         struct type *ftype = SYMBOL_TYPE (sym);
+         int nsemi = 0;
+         int nparams = 0;
+         char *p1 = p;
+
+         /* Obtain a worst case guess for the number of arguments
+            by counting the semicolons.  */
+         while (*p1)
+           {
+             if (*p1++ == ';')
+               nsemi++;
+           }
+
+         /* Allocate parameter information fields and fill them in. */
+         TYPE_FIELDS (ftype) = (struct field *)
+           TYPE_ALLOC (ftype, nsemi * sizeof (struct field));
+         while (*p++ == ';')
+           {
+             struct type *ptype;
+
+             /* A type number of zero indicates the start of varargs.
+                FIXME: GDB currently ignores vararg functions.  */
+             if (p[0] == '0' && p[1] == '\0')
+               break;
+             ptype = read_type (&p, objfile);
+
+             /* The Sun compilers mark integer arguments, which should
+                be promoted to the width of the calling conventions, with
+                a type which references itself. This type is turned into
+                a TYPE_CODE_VOID type by read_type, and we have to turn
+                it back into builtin_type_int here.
+                FIXME: Do we need a new builtin_type_promoted_int_arg ?  */
+             if (TYPE_CODE (ptype) == TYPE_CODE_VOID)
+               ptype = builtin_type_int;
+             TYPE_FIELD_TYPE (ftype, nparams) = ptype;
+             TYPE_FIELD_ARTIFICIAL (ftype, nparams++) = 0;
+           }
+         TYPE_NFIELDS (ftype) = nparams;
+         TYPE_FLAGS (ftype) |= TYPE_FLAG_PROTOTYPED;
+       }
       break;
 
     case 'F':
@@ -1625,28 +1566,28 @@ define_symbol (valu, string, desc, type, objfile)
 
     case 'G':
       /* For a class G (global) symbol, it appears that the
-        value is not correct.  It is necessary to search for the
-        corresponding linker definition to find the value.
-        These definitions appear at the end of the namelist.  */
+         value is not correct.  It is necessary to search for the
+         corresponding linker definition to find the value.
+         These definitions appear at the end of the namelist.  */
       SYMBOL_TYPE (sym) = read_type (&p, objfile);
       SYMBOL_CLASS (sym) = LOC_STATIC;
       SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
       /* Don't add symbol references to global_sym_chain.
-        Symbol references don't have valid names and wont't match up with
-        minimal symbols when the global_sym_chain is relocated.
-        We'll fixup symbol references when we fixup the defining symbol.  */
-      if (SYMBOL_NAME (sym) && SYMBOL_NAME (sym)[0] != '#')
+         Symbol references don't have valid names and wont't match up with
+         minimal symbols when the global_sym_chain is relocated.
+         We'll fixup symbol references when we fixup the defining symbol.  */
+      if (DEPRECATED_SYMBOL_NAME (sym) && DEPRECATED_SYMBOL_NAME (sym)[0] != '#')
        {
-          i = hashname (SYMBOL_NAME (sym));
-          SYMBOL_VALUE_CHAIN (sym) = global_sym_chain[i];
-          global_sym_chain[i] = sym;
+         i = hashname (DEPRECATED_SYMBOL_NAME (sym));
+         SYMBOL_VALUE_CHAIN (sym) = global_sym_chain[i];
+         global_sym_chain[i] = sym;
        }
       add_symbol_to_list (sym, &global_symbols);
       break;
 
       /* This case is faked by a conditional above,
-        when there is no code letter in the dbx data.
-        Dbx data never actually contains 'l'.  */
+         when there is no code letter in the dbx data.
+         Dbx data never actually contains 'l'.  */
     case 's':
     case 'l':
       SYMBOL_TYPE (sym) = read_type (&p, objfile);
@@ -1665,23 +1606,17 @@ define_symbol (valu, string, desc, type, objfile)
          p++;
          SYMBOL_TYPE (sym)
            = lookup_pointer_type
-             (lookup_function_type (read_type (&p, objfile)));
+           (lookup_function_type (read_type (&p, objfile)));
        }
       else
        SYMBOL_TYPE (sym) = read_type (&p, objfile);
 
-      /* Normally this is a parameter, a LOC_ARG.  On the i960, it
-        can also be a LOC_LOCAL_ARG depending on symbol type.  */
-#ifndef DBX_PARM_SYMBOL_CLASS
-#define        DBX_PARM_SYMBOL_CLASS(type)     LOC_ARG
-#endif
-
-      SYMBOL_CLASS (sym) = DBX_PARM_SYMBOL_CLASS (type);
+      SYMBOL_CLASS (sym) = LOC_ARG;
       SYMBOL_VALUE (sym) = valu;
       SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
       add_symbol_to_list (sym, &local_symbols);
 
-      if (TARGET_BYTE_ORDER != BIG_ENDIAN)
+      if (TARGET_BYTE_ORDER != BFD_ENDIAN_BIG)
        {
          /* On little-endian machines, this crud is never necessary,
             and, if the extra bytes contain garbage, is harmful.  */
@@ -1692,80 +1627,80 @@ define_symbol (valu, string, desc, type, objfile)
       if (processing_gcc_compilation || BELIEVE_PCC_PROMOTION)
        break;
 
-#if !BELIEVE_PCC_PROMOTION
-      {
-       /* This is the signed type which arguments get promoted to.  */
-       static struct type *pcc_promotion_type;
-       /* This is the unsigned type which arguments get promoted to.  */
-       static struct type *pcc_unsigned_promotion_type;
-
-       /* Call it "int" because this is mainly C lossage.  */
-       if (pcc_promotion_type == NULL)
-         pcc_promotion_type =
-           init_type (TYPE_CODE_INT, TARGET_INT_BIT / TARGET_CHAR_BIT,
-                      0, "int", NULL);
-
-       if (pcc_unsigned_promotion_type == NULL)
-         pcc_unsigned_promotion_type =
-           init_type (TYPE_CODE_INT, TARGET_INT_BIT / TARGET_CHAR_BIT,
-                      TYPE_FLAG_UNSIGNED, "unsigned int", NULL);
-
-#if defined(BELIEVE_PCC_PROMOTION_TYPE)
-       /* This macro is defined on machines (e.g. sparc) where
-          we should believe the type of a PCC 'short' argument,
-          but shouldn't believe the address (the address is
-          the address of the corresponding int).
-          
-          My guess is that this correction, as opposed to changing
-          the parameter to an 'int' (as done below, for PCC
-          on most machines), is the right thing to do
-          on all machines, but I don't want to risk breaking
-          something that already works.  On most PCC machines,
-          the sparc problem doesn't come up because the calling
-          function has to zero the top bytes (not knowing whether
-          the called function wants an int or a short), so there
-          is little practical difference between an int and a short
-          (except perhaps what happens when the GDB user types
-          "print short_arg = 0x10000;"). 
-          
-          Hacked for SunOS 4.1 by gnu@cygnus.com.  In 4.1, the compiler
-          actually produces the correct address (we don't need to fix it
-          up).  I made this code adapt so that it will offset the symbol
-          if it was pointing at an int-aligned location and not
-          otherwise.  This way you can use the same gdb for 4.0.x and
-          4.1 systems.
-          
-          If the parameter is shorter than an int, and is integral
-          (e.g. char, short, or unsigned equivalent), and is claimed to
-          be passed on an integer boundary, don't believe it!  Offset the
-          parameter's address to the tail-end of that integer.  */
-       
-       if (TYPE_LENGTH (SYMBOL_TYPE (sym)) < TYPE_LENGTH (pcc_promotion_type)
-           && TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_INT
-           && 0 == SYMBOL_VALUE (sym) % TYPE_LENGTH (pcc_promotion_type))
-         {
-           SYMBOL_VALUE (sym) += TYPE_LENGTH (pcc_promotion_type)
-             - TYPE_LENGTH (SYMBOL_TYPE (sym));
-         }
-       break;
-       
-#else /* no BELIEVE_PCC_PROMOTION_TYPE.  */
-
-       /* If PCC says a parameter is a short or a char,
-          it is really an int.  */
-       if (TYPE_LENGTH (SYMBOL_TYPE (sym)) < TYPE_LENGTH (pcc_promotion_type)
-           && TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_INT)
-         {
-           SYMBOL_TYPE (sym) =
-             TYPE_UNSIGNED (SYMBOL_TYPE (sym))
-               ? pcc_unsigned_promotion_type
-               : pcc_promotion_type;
-         }
-       break;
-
-#endif /* no BELIEVE_PCC_PROMOTION_TYPE.  */
-      }
-#endif /* !BELIEVE_PCC_PROMOTION.  */
+      if (!BELIEVE_PCC_PROMOTION)
+       {
+         /* This is the signed type which arguments get promoted to.  */
+         static struct type *pcc_promotion_type;
+         /* This is the unsigned type which arguments get promoted to.  */
+         static struct type *pcc_unsigned_promotion_type;
+
+         /* Call it "int" because this is mainly C lossage.  */
+         if (pcc_promotion_type == NULL)
+           pcc_promotion_type =
+             init_type (TYPE_CODE_INT, TARGET_INT_BIT / TARGET_CHAR_BIT,
+                        0, "int", NULL);
+
+         if (pcc_unsigned_promotion_type == NULL)
+           pcc_unsigned_promotion_type =
+             init_type (TYPE_CODE_INT, TARGET_INT_BIT / TARGET_CHAR_BIT,
+                        TYPE_FLAG_UNSIGNED, "unsigned int", NULL);
+
+         if (BELIEVE_PCC_PROMOTION_TYPE)
+           {
+             /* This is defined on machines (e.g. sparc) where we
+                should believe the type of a PCC 'short' argument,
+                but shouldn't believe the address (the address is the
+                address of the corresponding int).
+
+                My guess is that this correction, as opposed to
+                changing the parameter to an 'int' (as done below,
+                for PCC on most machines), is the right thing to do
+                on all machines, but I don't want to risk breaking
+                something that already works.  On most PCC machines,
+                the sparc problem doesn't come up because the calling
+                function has to zero the top bytes (not knowing
+                whether the called function wants an int or a short),
+                so there is little practical difference between an
+                int and a short (except perhaps what happens when the
+                GDB user types "print short_arg = 0x10000;").
+
+                Hacked for SunOS 4.1 by gnu@cygnus.com.  In 4.1, the
+                compiler actually produces the correct address (we
+                don't need to fix it up).  I made this code adapt so
+                that it will offset the symbol if it was pointing at
+                an int-aligned location and not otherwise.  This way
+                you can use the same gdb for 4.0.x and 4.1 systems.
+
+                If the parameter is shorter than an int, and is
+                integral (e.g. char, short, or unsigned equivalent),
+                and is claimed to be passed on an integer boundary,
+                don't believe it!  Offset the parameter's address to
+                the tail-end of that integer.  */
+
+             if (TYPE_LENGTH (SYMBOL_TYPE (sym)) < TYPE_LENGTH (pcc_promotion_type)
+                 && TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_INT
+             && 0 == SYMBOL_VALUE (sym) % TYPE_LENGTH (pcc_promotion_type))
+               {
+                 SYMBOL_VALUE (sym) += TYPE_LENGTH (pcc_promotion_type)
+                   - TYPE_LENGTH (SYMBOL_TYPE (sym));
+               }
+             break;
+           }
+         else
+           {
+             /* If PCC says a parameter is a short or a char,
+                it is really an int.  */
+             if (TYPE_LENGTH (SYMBOL_TYPE (sym)) < TYPE_LENGTH (pcc_promotion_type)
+                 && TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_INT)
+               {
+                 SYMBOL_TYPE (sym) =
+                   TYPE_UNSIGNED (SYMBOL_TYPE (sym))
+                   ? pcc_unsigned_promotion_type
+                   : pcc_promotion_type;
+               }
+             break;
+           }
+       }
 
     case 'P':
       /* acc seems to use P to declare the prototypes of functions that
@@ -1776,18 +1711,19 @@ define_symbol (valu, string, desc, type, objfile)
          SYMBOL_TYPE (sym) = read_type (&p, objfile);
          goto process_prototype_types;
        }
-      /*FALLTHROUGH*/
+      /*FALLTHROUGH */
 
     case 'R':
       /* Parameter which is in a register.  */
       SYMBOL_TYPE (sym) = read_type (&p, objfile);
       SYMBOL_CLASS (sym) = LOC_REGPARM;
       SYMBOL_VALUE (sym) = STAB_REG_TO_REGNUM (valu);
-      if (SYMBOL_VALUE (sym) >= NUM_REGS)
+      if (SYMBOL_VALUE (sym) >= NUM_REGS + NUM_PSEUDO_REGS)
        {
-         complain (&reg_value_complaint, SYMBOL_VALUE (sym), NUM_REGS,
-                   SYMBOL_SOURCE_NAME (sym));
-         SYMBOL_VALUE (sym) = SP_REGNUM;  /* Known safe, though useless */
+         reg_value_complaint (SYMBOL_VALUE (sym),
+                              NUM_REGS + NUM_PSEUDO_REGS,
+                              SYMBOL_PRINT_NAME (sym));
+         SYMBOL_VALUE (sym) = SP_REGNUM;       /* Known safe, though useless */
        }
       SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
       add_symbol_to_list (sym, &local_symbols);
@@ -1798,11 +1734,12 @@ define_symbol (valu, string, desc, type, objfile)
       SYMBOL_TYPE (sym) = read_type (&p, objfile);
       SYMBOL_CLASS (sym) = LOC_REGISTER;
       SYMBOL_VALUE (sym) = STAB_REG_TO_REGNUM (valu);
-      if (SYMBOL_VALUE (sym) >= NUM_REGS)
+      if (SYMBOL_VALUE (sym) >= NUM_REGS + NUM_PSEUDO_REGS)
        {
-         complain (&reg_value_complaint, SYMBOL_VALUE (sym), NUM_REGS,
-                   SYMBOL_SOURCE_NAME (sym));
-         SYMBOL_VALUE (sym) = SP_REGNUM;  /* Known safe, though useless */
+         reg_value_complaint (SYMBOL_VALUE (sym),
+                              NUM_REGS + NUM_PSEUDO_REGS,
+                              SYMBOL_PRINT_NAME (sym));
+         SYMBOL_VALUE (sym) = SP_REGNUM;       /* Known safe, though useless */
        }
       SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
       if (within_function)
@@ -1820,13 +1757,14 @@ define_symbol (valu, string, desc, type, objfile)
             of saved registers in backtraces, etc.).
 
             Note that this code illegally combines
-              main(argc) struct foo argc; { register struct foo argc; }
+            main(argc) struct foo argc; { register struct foo argc; }
             but this case is considered pathological and causes a warning
             from a decent compiler.  */
 
          if (local_symbols
              && local_symbols->nsyms > 0
 #ifndef USE_REGISTER_NOT_ARG
+             && REG_STRUCT_HAS_ADDR_P ()
              && REG_STRUCT_HAS_ADDR (processing_gcc_compilation,
                                      SYMBOL_TYPE (sym))
              && (TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_STRUCT
@@ -1834,13 +1772,13 @@ define_symbol (valu, string, desc, type, objfile)
                  || TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_SET
                  || TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_BITSTRING)
 #endif
-             )
+           )
            {
              struct symbol *prev_sym;
              prev_sym = local_symbols->symbol[local_symbols->nsyms - 1];
              if ((SYMBOL_CLASS (prev_sym) == LOC_REF_ARG
                   || SYMBOL_CLASS (prev_sym) == LOC_ARG)
-                 && STREQ (SYMBOL_NAME (prev_sym), SYMBOL_NAME(sym)))
+                 && STREQ (DEPRECATED_SYMBOL_NAME (prev_sym), DEPRECATED_SYMBOL_NAME (sym)))
                {
                  SYMBOL_CLASS (prev_sym) = LOC_REGPARM;
                  /* Use the type from the LOC_REGISTER; that is the type
@@ -1851,10 +1789,10 @@ define_symbol (valu, string, desc, type, objfile)
                  break;
                }
            }
-          add_symbol_to_list (sym, &local_symbols);
+         add_symbol_to_list (sym, &local_symbols);
        }
       else
-        add_symbol_to_list (sym, &file_symbols);
+       add_symbol_to_list (sym, &file_symbols);
       break;
 
     case 'S':
@@ -1863,35 +1801,37 @@ define_symbol (valu, string, desc, type, objfile)
       SYMBOL_CLASS (sym) = LOC_STATIC;
       SYMBOL_VALUE_ADDRESS (sym) = valu;
 #ifdef STATIC_TRANSFORM_NAME
-      if (SYMBOL_NAME (sym)[0] == '$')
-      {
-       struct minimal_symbol *msym;
-       msym = lookup_minimal_symbol (SYMBOL_NAME (sym), NULL, objfile);
-       if (msym != NULL)
-         {
-           SYMBOL_NAME (sym) = STATIC_TRANSFORM_NAME (SYMBOL_NAME (sym));
-           SYMBOL_VALUE_ADDRESS (sym) = SYMBOL_VALUE_ADDRESS (msym);
-         }
-      }
+      if (IS_STATIC_TRANSFORM_NAME (DEPRECATED_SYMBOL_NAME (sym)))
+       {
+         struct minimal_symbol *msym;
+         msym = lookup_minimal_symbol (DEPRECATED_SYMBOL_NAME (sym), NULL, objfile);
+         if (msym != NULL)
+           {
+             DEPRECATED_SYMBOL_NAME (sym) = STATIC_TRANSFORM_NAME (DEPRECATED_SYMBOL_NAME (sym));
+             SYMBOL_VALUE_ADDRESS (sym) = SYMBOL_VALUE_ADDRESS (msym);
+           }
+       }
 #endif
       SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
       add_symbol_to_list (sym, &file_symbols);
       break;
 
     case 't':
+      /* Typedef */
       SYMBOL_TYPE (sym) = read_type (&p, objfile);
 
       /* For a nameless type, we don't want a create a symbol, thus we
-        did not use `sym'. Return without further processing. */
-      if (nameless) return NULL;
+         did not use `sym'. Return without further processing. */
+      if (nameless)
+       return NULL;
 
       SYMBOL_CLASS (sym) = LOC_TYPEDEF;
       SYMBOL_VALUE (sym) = valu;
       SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
       /* C++ vagaries: we may have a type which is derived from
-        a base type which did not have its name defined when the
-        derived class was output.  We fill in the derived class's
-        base part member's name here in that case.  */
+         a base type which did not have its name defined when the
+         derived class was output.  We fill in the derived class's
+         base part member's name here in that case.  */
       if (TYPE_NAME (SYMBOL_TYPE (sym)) != NULL)
        if ((TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_STRUCT
             || TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_UNION)
@@ -1912,30 +1852,51 @@ define_symbol (valu, string, desc, type, objfile)
          extern const char vtbl_ptr_name[];
 
          if ((TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_PTR
-              && strcmp (SYMBOL_NAME (sym), vtbl_ptr_name))
+              && strcmp (DEPRECATED_SYMBOL_NAME (sym), vtbl_ptr_name))
              || TYPE_CODE (SYMBOL_TYPE (sym)) == TYPE_CODE_FUNC)
            {
              /* If we are giving a name to a type such as "pointer to
-                foo" or "function returning foo", we better not set
-                the TYPE_NAME.  If the program contains "typedef char
-                *caddr_t;", we don't want all variables of type char
-                * to print as caddr_t.  This is not just a
-                consequence of GDB's type management; PCC and GCC (at
-                least through version 2.4) both output variables of
-                either type char * or caddr_t with the type number
-                defined in the 't' symbol for caddr_t.  If a future
-                compiler cleans this up it GDB is not ready for it
-                yet, but if it becomes ready we somehow need to
-                disable this check (without breaking the PCC/GCC2.4
-                case).
-
-                Sigh.
-
-                Fortunately, this check seems not to be necessary
-                for anything except pointers or functions.  */
+                foo" or "function returning foo", we better not set
+                the TYPE_NAME.  If the program contains "typedef char
+                *caddr_t;", we don't want all variables of type char
+                * to print as caddr_t.  This is not just a
+                consequence of GDB's type management; PCC and GCC (at
+                least through version 2.4) both output variables of
+                either type char * or caddr_t with the type number
+                defined in the 't' symbol for caddr_t.  If a future
+                compiler cleans this up it GDB is not ready for it
+                yet, but if it becomes ready we somehow need to
+                disable this check (without breaking the PCC/GCC2.4
+                case).
+
+                Sigh.
+
+                Fortunately, this check seems not to be necessary
+                for anything except pointers or functions.  */
+              /* ezannoni: 2000-10-26. This seems to apply for
+                versions of gcc older than 2.8. This was the original
+                problem: with the following code gdb would tell that
+                the type for name1 is caddr_t, and func is char()
+                typedef char *caddr_t;
+                char *name2;
+                struct x
+                {
+                char *name1;
+                } xx;
+                char *func()
+                {
+                }
+                main () {}
+                */
+
+             /* Pascal accepts names for pointer types. */
+             if (current_subfile->language == language_pascal)
+               {
+                 TYPE_NAME (SYMBOL_TYPE (sym)) = DEPRECATED_SYMBOL_NAME (sym);
+               }
            }
          else
-           TYPE_NAME (SYMBOL_TYPE (sym)) = SYMBOL_NAME (sym);
+           TYPE_NAME (SYMBOL_TYPE (sym)) = DEPRECATED_SYMBOL_NAME (sym);
        }
 
       add_symbol_to_list (sym, &file_symbols);
@@ -1943,44 +1904,48 @@ define_symbol (valu, string, desc, type, objfile)
 
     case 'T':
       /* Struct, union, or enum tag.  For GNU C++, this can be be followed
-        by 't' which means we are typedef'ing it as well.  */
+         by 't' which means we are typedef'ing it as well.  */
       synonym = *p == 't';
 
       if (synonym)
        p++;
-      /* The semantics of C++ state that "struct foo { ... }" also defines 
-        a typedef for "foo".  Unfortunately, cfront never makes the typedef
-        when translating C++ into C.  We make the typedef here so that
-        "ptype foo" works as expected for cfront translated code.  */
-      else if (current_subfile->language == language_cplus)
-       synonym = 1;
+#if 0 /* OBSOLETE CFront */
+// OBSOLETE      /* The semantics of C++ state that "struct foo { ... }" also defines 
+// OBSOLETE         a typedef for "foo".  Unfortunately, cfront never makes the typedef
+// OBSOLETE         when translating C++ into C.  We make the typedef here so that
+// OBSOLETE         "ptype foo" works as expected for cfront translated code.  */
+// OBSOLETE       else if ((current_subfile->language == language_cplus)
+// OBSOLETE           || (current_subfile->language == language_objc))
+// OBSOLETE    synonym = 1;
+#endif /* OBSOLETE CFront */
 
       SYMBOL_TYPE (sym) = read_type (&p, objfile);
-
       /* For a nameless type, we don't want a create a symbol, thus we
-        did not use `sym'. Return without further processing. */
-      if (nameless) return NULL;
+         did not use `sym'. Return without further processing. */
+      if (nameless)
+       return NULL;
 
       SYMBOL_CLASS (sym) = LOC_TYPEDEF;
       SYMBOL_VALUE (sym) = valu;
       SYMBOL_NAMESPACE (sym) = STRUCT_NAMESPACE;
       if (TYPE_TAG_NAME (SYMBOL_TYPE (sym)) == 0)
        TYPE_TAG_NAME (SYMBOL_TYPE (sym))
-         = obconcat (&objfile -> type_obstack, "", "", SYMBOL_NAME (sym));
+         = obconcat (&objfile->type_obstack, "", "", DEPRECATED_SYMBOL_NAME (sym));
       add_symbol_to_list (sym, &file_symbols);
 
       if (synonym)
        {
          /* Clone the sym and then modify it. */
          register struct symbol *typedef_sym = (struct symbol *)
-           obstack_alloc (&objfile -> symbol_obstack, sizeof (struct symbol));
+         obstack_alloc (&objfile->symbol_obstack, sizeof (struct symbol));
          *typedef_sym = *sym;
          SYMBOL_CLASS (typedef_sym) = LOC_TYPEDEF;
          SYMBOL_VALUE (typedef_sym) = valu;
          SYMBOL_NAMESPACE (typedef_sym) = VAR_NAMESPACE;
          if (TYPE_NAME (SYMBOL_TYPE (sym)) == 0)
            TYPE_NAME (SYMBOL_TYPE (sym))
-             = obconcat (&objfile -> type_obstack, "", "", SYMBOL_NAME (sym));
+             = obconcat (&objfile->type_obstack, "", "", DEPRECATED_SYMBOL_NAME (sym));
          add_symbol_to_list (typedef_sym, &file_symbols);
        }
       break;
@@ -1991,21 +1956,18 @@ define_symbol (valu, string, desc, type, objfile)
       SYMBOL_CLASS (sym) = LOC_STATIC;
       SYMBOL_VALUE_ADDRESS (sym) = valu;
 #ifdef STATIC_TRANSFORM_NAME
-      if (SYMBOL_NAME (sym)[0] == '$')
-      {
-       struct minimal_symbol *msym;
-       msym = lookup_minimal_symbol (SYMBOL_NAME (sym), NULL, objfile);
-       if (msym != NULL)
-         {
-           SYMBOL_NAME (sym) = STATIC_TRANSFORM_NAME (SYMBOL_NAME (sym));
-           SYMBOL_VALUE_ADDRESS (sym) = SYMBOL_VALUE_ADDRESS (msym);
-         }
-      }
+      if (IS_STATIC_TRANSFORM_NAME (DEPRECATED_SYMBOL_NAME (sym)))
+       {
+         struct minimal_symbol *msym;
+         msym = lookup_minimal_symbol (DEPRECATED_SYMBOL_NAME (sym), NULL, objfile);
+         if (msym != NULL)
+           {
+             DEPRECATED_SYMBOL_NAME (sym) = STATIC_TRANSFORM_NAME (DEPRECATED_SYMBOL_NAME (sym));
+             SYMBOL_VALUE_ADDRESS (sym) = SYMBOL_VALUE_ADDRESS (msym);
+           }
+       }
 #endif
       SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
-      if (os9k_stabs)
-       add_symbol_to_list (sym, &global_symbols);
-      else
        add_symbol_to_list (sym, &local_symbols);
       break;
 
@@ -2023,11 +1985,12 @@ define_symbol (valu, string, desc, type, objfile)
       SYMBOL_TYPE (sym) = read_type (&p, objfile);
       SYMBOL_CLASS (sym) = LOC_REGPARM_ADDR;
       SYMBOL_VALUE (sym) = STAB_REG_TO_REGNUM (valu);
-      if (SYMBOL_VALUE (sym) >= NUM_REGS)
+      if (SYMBOL_VALUE (sym) >= NUM_REGS + NUM_PSEUDO_REGS)
        {
-         complain (&reg_value_complaint, SYMBOL_VALUE (sym), NUM_REGS,
-                   SYMBOL_SOURCE_NAME (sym));
-         SYMBOL_VALUE (sym) = SP_REGNUM;  /* Known safe, though useless */
+         reg_value_complaint (SYMBOL_VALUE (sym),
+                              NUM_REGS + NUM_PSEUDO_REGS,
+                              SYMBOL_PRINT_NAME (sym));
+         SYMBOL_VALUE (sym) = SP_REGNUM;       /* Known safe, though useless */
        }
       SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
       add_symbol_to_list (sym, &local_symbols);
@@ -2035,40 +1998,41 @@ define_symbol (valu, string, desc, type, objfile)
 
     case 'X':
       /* This is used by Sun FORTRAN for "function result value".
-        Sun claims ("dbx and dbxtool interfaces", 2nd ed)
-        that Pascal uses it too, but when I tried it Pascal used
-        "x:3" (local symbol) instead.  */
+         Sun claims ("dbx and dbxtool interfaces", 2nd ed)
+         that Pascal uses it too, but when I tried it Pascal used
+         "x:3" (local symbol) instead.  */
       SYMBOL_TYPE (sym) = read_type (&p, objfile);
       SYMBOL_CLASS (sym) = LOC_LOCAL;
       SYMBOL_VALUE (sym) = valu;
       SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
       add_symbol_to_list (sym, &local_symbols);
       break;
-
-    /* New code added to support cfront stabs strings.
-       Note: case 'P' already handled above */
-    case 'Z':
-      /* Cfront type continuation coming up!
-        Find the original definition and add to it.
-        We'll have to do this for the typedef too,
-        since we cloned the symbol to define a type in read_type.
-        Stabs info examples:
-                __1C :Ztl 
-                foo__1CFv :ZtF (first def foo__1CFv:F(0,3);(0,24))
-                C:ZsC;;__ct__1CFv func1__1CFv func2__1CFv ... ;;;
-                where C is the name of the class.
-        Unfortunately, we can't lookup the original symbol yet 'cuz 
-        we haven't finished reading all the symbols.
-        Instead, we save it for processing later */
-      process_later (sym, p, resolve_cfront_continuation);
-      SYMBOL_TYPE (sym) = error_type (&p, objfile); /* FIXME! change later */ 
-      SYMBOL_CLASS (sym) = LOC_CONST; 
-      SYMBOL_VALUE (sym) = 0; 
-      SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
-      /* Don't add to list - we'll delete it later when 
-         we add the continuation to the real sym */
-      return sym;
-      /* End of new code added to support cfront stabs strings */
+#if 0 /* OBSOLETE CFront */
+// OBSOLETE        /* New code added to support cfront stabs strings.
+// OBSOLETE           Note: case 'P' already handled above */
+// OBSOLETE      case 'Z':
+// OBSOLETE        /* Cfront type continuation coming up!
+// OBSOLETE           Find the original definition and add to it.
+// OBSOLETE           We'll have to do this for the typedef too,
+// OBSOLETE           since we cloned the symbol to define a type in read_type.
+// OBSOLETE           Stabs info examples:
+// OBSOLETE           __1C :Ztl 
+// OBSOLETE           foo__1CFv :ZtF (first def foo__1CFv:F(0,3);(0,24))
+// OBSOLETE           C:ZsC;;__ct__1CFv func1__1CFv func2__1CFv ... ;;;
+// OBSOLETE           where C is the name of the class.
+// OBSOLETE           Unfortunately, we can't lookup the original symbol yet 'cuz 
+// OBSOLETE           we haven't finished reading all the symbols.
+// OBSOLETE           Instead, we save it for processing later */
+// OBSOLETE        process_later (sym, p, resolve_cfront_continuation);
+// OBSOLETE        SYMBOL_TYPE (sym) = error_type (&p, objfile);       /* FIXME! change later */
+// OBSOLETE        SYMBOL_CLASS (sym) = LOC_CONST;
+// OBSOLETE        SYMBOL_VALUE (sym) = 0;
+// OBSOLETE        SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
+// OBSOLETE        /* Don't add to list - we'll delete it later when 
+// OBSOLETE           we add the continuation to the real sym */
+// OBSOLETE        return sym;
+// OBSOLETE        /* End of new code added to support cfront stabs strings */
+#endif /* OBSOLETE CFront */
 
     default:
       SYMBOL_TYPE (sym) = error_type (&p, objfile);
@@ -2082,7 +2046,8 @@ define_symbol (valu, string, desc, type, objfile)
   /* When passing structures to a function, some systems sometimes pass
      the address in a register, not the structure itself. */
 
-  if (REG_STRUCT_HAS_ADDR (processing_gcc_compilation, SYMBOL_TYPE (sym))
+  if (REG_STRUCT_HAS_ADDR_P ()
+      && REG_STRUCT_HAS_ADDR (processing_gcc_compilation, SYMBOL_TYPE (sym))
       && (SYMBOL_CLASS (sym) == LOC_REGPARM || SYMBOL_CLASS (sym) == LOC_ARG))
     {
       struct type *symbol_type = check_typedef (SYMBOL_TYPE (sym));
@@ -2103,130 +2068,136 @@ define_symbol (valu, string, desc, type, objfile)
        }
     }
 
-  /* Is there more to parse?  eg. ";l(#1,#2);l(#3,#5)" */
-  while (*p && (*p == ';' || *p == ','))
+  /* Is there more to parse?  For example LRS/alias information?  */
+  while (*p && *p == ';')
     {
       p++;
-      if (*p && *p == 'l')
-        {
-          /* GNU extensions for live range splitting may be appended to 
-             the end of the stab string.  eg. "l(#1,#2);l(#3,#5)" */
+      if (*p && p[0] == 'l' && p[1] == '(')
+       {
+         /* GNU extensions for live range splitting may be appended to 
+            the end of the stab string.  eg. "l(#1,#2);l(#3,#5)" */
 
-          /* Fix up ranges later. */
-          process_later (sym, p, resolve_live_range);
+         /* Resolve the live range and add it to SYM's live range list.  */
+         if (!resolve_live_range (objfile, sym, p))
+           return NULL;
 
          /* Find end of live range info. */
          p = strchr (p, ')');
-          if (!*p || *p != ')')
-            error ("Internal error: live range format not recognized.\n");
-          p++;
-       }
+         if (!*p || *p != ')')
+           {
+             lrs_general_complaint ("live range format not recognized");
+             return NULL;
+           }
+         p++;
+       }
     }
   return sym;
 }
 
+/* Add the live range found in P to the symbol SYM in objfile OBJFILE.  Returns
+   non-zero on success, zero otherwise.  */
 
-/* Add live range information to symbol. (eg. p is l(#1,#2)...) */
-static int 
-resolve_live_range (objfile, sym, p)
-  struct objfile * objfile;
-  struct symbol *sym;
-  char *p;
+static int
+resolve_live_range (struct objfile *objfile, struct symbol *sym, char *p)
 {
-  char *s=p;
   int refnum;
   CORE_ADDR start, end;
-  
+
+  /* Sanity check the beginning of the stabs string.  */
   if (!*p || *p != 'l')
-    error ("Internal error: live range string.\n");
+    {
+      lrs_general_complaint ("live range string 1");
+      return 0;
+    }
   p++;
 
   if (!*p || *p != '(')
-    error ("Internal error: live range string.\n");
+    {
+      lrs_general_complaint ("live range string 2");
+      return 0;
+    }
   p++;
-       
-  /* Get starting value of range symbol reference. eg. "#1,#2),l(#3,#5)"
-     and advance p past the refid. */
+
+  /* Get starting value of range and advance P past the reference id.
+
+     ?!? In theory, the process_reference should never fail, but we should
+     catch that case just in case the compiler scrogged the stabs.  */
   refnum = process_reference (&p);
   start = ref_search_value (refnum);
   if (!start)
-    error ("Internal error: live range symbol not found.\n");
+    {
+      lrs_general_complaint ("Live range symbol not found 1");
+      return 0;
+    }
 
   if (!*p || *p != ',')
-    error ("Internal error: live range string.\n");
+    {
+      lrs_general_complaint ("live range string 3");
+      return 0;
+    }
   p++;
 
-  /* Get ending value of range symbol reference. eg. "#2),l(#3,#5)" */
+  /* Get ending value of range and advance P past the reference id.
+
+     ?!? In theory, the process_reference should never fail, but we should
+     catch that case just in case the compiler scrogged the stabs.  */
   refnum = process_reference (&p);
   end = ref_search_value (refnum);
   if (!end)
-    error ("Internal error: live range symbol not found.\n");
+    {
+      lrs_general_complaint ("Live range symbol not found 2");
+      return 0;
+    }
+
+  if (!*p || *p != ')')
+    {
+      lrs_general_complaint ("live range string 4");
+      return 0;
+    }
 
+  /* Now that we know the bounds of the range, add it to the
+     symbol.  */
   add_live_range (objfile, sym, start, end);
 
-  if (!*p || *p != ')')
-    error ("Internal error: live range string.\n");
-  p++;
-  return 0; 
+  return 1;
 }
 
-void
-add_live_range (objfile, sym, start, end)
-  struct objfile *objfile;
-  struct symbol *sym;
-  CORE_ADDR start, end;
+/* Add a new live range defined by START and END to the symbol SYM
+   in objfile OBJFILE.  */
+
+static void
+add_live_range (struct objfile *objfile, struct symbol *sym, CORE_ADDR start,
+               CORE_ADDR end)
 {
-  struct live_range *r, *rs;
+  struct range_list *r, *rs;
 
   if (start >= end)
-    error ("Internal error: end of live range follows start.\n");
+    {
+      lrs_general_complaint ("end of live range follows start");
+      return;
+    }
 
   /* Alloc new live range structure. */
-  r = (struct live_range *)
-               obstack_alloc (&objfile->type_obstack, 
-                               sizeof (struct live_range));
+  r = (struct range_list *)
+    obstack_alloc (&objfile->type_obstack,
+                  sizeof (struct range_list));
   r->start = start;
   r->end = end;
   r->next = 0;
 
   /* Append this range to the symbol's range list. */
-  if (!SYMBOL_RANGE (sym))
-    {
-      SYMBOL_RANGE (sym) = r;
-    }
+  if (!SYMBOL_RANGES (sym))
+    SYMBOL_RANGES (sym) = r;
   else
     {
       /* Get the last range for the symbol. */
-      for (rs = SYMBOL_RANGE (sym); rs->next; rs = rs->next)
+      for (rs = SYMBOL_RANGES (sym); rs->next; rs = rs->next)
        ;
       rs->next = r;
     }
 }
-
-/* Given addr, Search thu alias list to find the one active. */
-struct symbol *
-ref_search_val (sym, addr)
-  struct symbol *sym;
-  CORE_ADDR addr;
-{
-  struct live_range *r;
-
-  while (sym)
-    {
-      if (!SYMBOL_RANGE (sym))
-        return sym;
-      for (r = SYMBOL_RANGE (sym); r; r = r->next)
-       {
-         if (r->start <= addr
-                   && r->end > addr)
-           return sym;
-       }
-      sym = SYMBOL_ALIASES (sym);
-    }
-  return 0;
-}
-
 \f
+
 /* Skip rest of this symbol and return an error type.
 
    General notes on error recovery:  error_type always skips to the
@@ -2234,7 +2205,7 @@ ref_search_val (sym, addr)
    Thus code like this:
 
    if (*(*pp)++ != ';')
-     return error_type (pp, objfile);
+   return error_type (pp, objfile);
 
    is wrong because if *pp starts out pointing at '\0' (typically as the
    result of an earlier error), it will be incremented to point to the
@@ -2242,26 +2213,24 @@ ref_search_val (sym, addr)
    if you run off the end of the string table.  Instead use
 
    if (**pp != ';')
-     return error_type (pp, objfile);
+   return error_type (pp, objfile);
    ++*pp;
 
    or
 
    if (**pp != ';')
-     foo = error_type (pp, objfile);
+   foo = error_type (pp, objfile);
    else
-     ++*pp;
+   ++*pp;
 
    And in case it isn't obvious, the point of all this hair is so the compiler
    can define new types and new syntaxes, and old versions of the
    debugger will be able to read the new symbol tables.  */
 
 static struct type *
-error_type (pp, objfile)
-     char **pp;
-     struct objfile *objfile;
+error_type (char **pp, struct objfile *objfile)
 {
-  complain (&error_type_complaint);
+  complaint (&symfile_complaints, "couldn't parse type; debugger out of date?");
   while (1)
     {
       /* Skip to end of symbol.  */
@@ -2282,8 +2251,8 @@ error_type (pp, objfile)
     }
   return (builtin_type_error);
 }
-
 \f
+
 /* Read type information or a type definition; return the type.  Even
    though this routine accepts either type information or a type
    definition, the distinction is relevant--some parts of stabsread.c
@@ -2291,9 +2260,7 @@ error_type (pp, objfile)
    deciding whether to call read_type.  */
 
 struct type *
-read_type (pp, objfile)
-     register char **pp;
-     struct objfile *objfile;
+read_type (register char **pp, struct objfile *objfile)
 {
   register struct type *type = 0;
   struct type *type1;
@@ -2307,6 +2274,9 @@ read_type (pp, objfile)
   /* Used to distinguish string and bitstring from char-array and set. */
   int is_string = 0;
 
+  /* Used to distinguish vector from array. */
+  int is_vector = 0;
+
   /* Read type number if present.  The type number may be omitted.
      for instance in a two-dimensional array declared with type
      "ar1;1;10;ar1;1;10;4".  */
@@ -2316,27 +2286,27 @@ read_type (pp, objfile)
     {
       if (read_type_number (pp, typenums) != 0)
        return error_type (pp, objfile);
-      
+
       /* Type is not being defined here.  Either it already exists,
-        or this is a forward reference to it.  dbx_alloc_type handles
-        both cases.  */
+         or this is a forward reference to it.  dbx_alloc_type handles
+         both cases.  */
       if (**pp != '=')
        return dbx_alloc_type (typenums, objfile);
 
       /* Type is being defined here.  */
       /* Skip the '='.
-        Also skip the type descriptor - we get it below with (*pp)[-1].  */
-      (*pp)+=2;
+         Also skip the type descriptor - we get it below with (*pp)[-1].  */
+      (*pp) += 2;
     }
   else
     {
       /* 'typenums=' not present, type is anonymous.  Read and return
-        the definition, but don't put it in the type vector.  */
+         the definition, but don't put it in the type vector.  */
       typenums[0] = typenums[1] = -1;
       (*pp)++;
     }
 
- again:
+again:
   type_descriptor = (*pp)[-1];
   switch (type_descriptor)
     {
@@ -2347,13 +2317,13 @@ read_type (pp, objfile)
        /* Used to index through file_symbols.  */
        struct pending *ppt;
        int i;
-       
+
        /* Name including "struct", etc.  */
        char *type_name;
-       
+
        {
          char *from, *to, *p, *q1, *q2;
-         
+
          /* Set the type code according to the following letter.  */
          switch ((*pp)[0])
            {
@@ -2370,14 +2340,13 @@ read_type (pp, objfile)
              {
                /* Complain and keep going, so compilers can invent new
                   cross-reference types.  */
-               static struct complaint msg =
-                 {"Unrecognized cross-reference type `%c'", 0, 0};
-               complain (&msg, (*pp)[0]);
+               complaint (&symfile_complaints,
+                          "Unrecognized cross-reference type `%c'", (*pp)[0]);
                code = TYPE_CODE_STRUCT;
                break;
              }
            }
-          
+
          q1 = strchr (*pp, '<');
          p = strchr (*pp, ':');
          if (p == NULL)
@@ -2398,15 +2367,15 @@ read_type (pp, objfile)
              if (*p != ':')
                return error_type (pp, objfile);
            }
-         to = type_name = 
-               (char *)obstack_alloc (&objfile->type_obstack, p - *pp + 1);
-       
+         to = type_name =
+           (char *) obstack_alloc (&objfile->type_obstack, p - *pp + 1);
+
          /* Copy the name.  */
          from = *pp + 1;
-         while (from < p) 
+         while (from < p)
            *to++ = *from++;
          *to = '\0';
-         
+
          /* Set the pointer ahead of the name which we just read, and
             the colon.  */
          *pp = from + 1;
@@ -2426,9 +2395,9 @@ read_type (pp, objfile)
              if (SYMBOL_CLASS (sym) == LOC_TYPEDEF
                  && SYMBOL_NAMESPACE (sym) == STRUCT_NAMESPACE
                  && (TYPE_CODE (SYMBOL_TYPE (sym)) == code)
-                 && STREQ (SYMBOL_NAME (sym), type_name))
+                 && STREQ (DEPRECATED_SYMBOL_NAME (sym), type_name))
                {
-                 obstack_free (&objfile -> type_obstack, type_name);
+                 obstack_free (&objfile->type_obstack, type_name);
                  type = SYMBOL_TYPE (sym);
                  return type;
                }
@@ -2442,14 +2411,14 @@ read_type (pp, objfile)
        type = dbx_alloc_type (typenums, objfile);
        TYPE_CODE (type) = code;
        TYPE_TAG_NAME (type) = type_name;
-       INIT_CPLUS_SPECIFIC(type);
+       INIT_CPLUS_SPECIFIC (type);
        TYPE_FLAGS (type) |= TYPE_FLAG_STUB;
 
        add_undefined_type (type);
        return type;
       }
 
-    case '-':                          /* RS/6000 built-in type */
+    case '-':                  /* RS/6000 built-in type */
     case '0':
     case '1':
     case '2':
@@ -2464,13 +2433,14 @@ read_type (pp, objfile)
       (*pp)--;
 
       /* We deal with something like t(1,2)=(3,4)=... which
-        the Lucid compiler and recent gcc versions (post 2.7.3) use. */
+         the Lucid compiler and recent gcc versions (post 2.7.3) use. */
 
       /* Allocate and enter the typedef type first.
-        This handles recursive types. */
+         This handles recursive types. */
       type = dbx_alloc_type (typenums, objfile);
       TYPE_CODE (type) = TYPE_CODE_TYPEDEF;
-      { struct type *xtype = read_type (pp, objfile);
+      {
+       struct type *xtype = read_type (pp, objfile);
        if (type == xtype)
          {
            /* It's being defined as itself.  That means it is "void".  */
@@ -2479,7 +2449,30 @@ read_type (pp, objfile)
          }
        else if (type_size >= 0 || is_string)
          {
-           *type = *xtype;
+           /* This is the absolute wrong way to construct types.  Every
+              other debug format has found a way around this problem and
+              the related problems with unnecessarily stubbed types;
+              someone motivated should attempt to clean up the issue
+              here as well.  Once a type pointed to has been created it
+              should not be modified.
+
+               Well, it's not *absolutely* wrong.  Constructing recursive
+               types (trees, linked lists) necessarily entails modifying
+               types after creating them.  Constructing any loop structure
+               entails side effects.  The Dwarf 2 reader does handle this
+               more gracefully (it never constructs more than once
+               instance of a type object, so it doesn't have to copy type
+               objects wholesale), but it still mutates type objects after
+               other folks have references to them.
+
+               Keep in mind that this circularity/mutation issue shows up
+               at the source language level, too: C's "incomplete types",
+               for example.  So the proper cleanup, I think, would be to
+               limit GDB's type smashing to match exactly those required
+               by the source language.  So GDB could have a
+               "complete_this_type" function, but never create unnecessary
+               copies of a type otherwise.  */
+           replace_type (type, xtype);
            TYPE_NAME (type) = NULL;
            TYPE_TAG_NAME (type) = NULL;
          }
@@ -2491,65 +2484,116 @@ read_type (pp, objfile)
       }
       break;
 
-    /* In the following types, we must be sure to overwrite any existing
-       type that the typenums refer to, rather than allocating a new one
-       and making the typenums point to the new one.  This is because there
-       may already be pointers to the existing type (if it had been
-       forward-referenced), and we must change it to a pointer, function,
-       reference, or whatever, *in-place*.  */
+      /* In the following types, we must be sure to overwrite any existing
+         type that the typenums refer to, rather than allocating a new one
+         and making the typenums point to the new one.  This is because there
+         may already be pointers to the existing type (if it had been
+         forward-referenced), and we must change it to a pointer, function,
+         reference, or whatever, *in-place*.  */
 
-    case '*':
+    case '*':                  /* Pointer to another type */
       type1 = read_type (pp, objfile);
       type = make_pointer_type (type1, dbx_lookup_type (typenums));
       break;
 
-    case '&':                          /* Reference to another type */
+    case '&':                  /* Reference to another type */
       type1 = read_type (pp, objfile);
       type = make_reference_type (type1, dbx_lookup_type (typenums));
       break;
 
-    case 'f':                          /* Function returning another type */
-      if (os9k_stabs && **pp == '(')
-       {
-         /* Function prototype; parse it.
-            We must conditionalize this on os9k_stabs because otherwise
-            it could be confused with a Sun-style (1,3) typenumber
-            (I think).  */
-         struct type *t;
-         ++*pp;
-         while (**pp != ')')
-            {
-              t = read_type (pp, objfile);
-              if (**pp == ',') ++*pp;
-            }
-       }
+    case 'f':                  /* Function returning another type */
       type1 = read_type (pp, objfile);
       type = make_function_type (type1, dbx_lookup_type (typenums));
       break;
 
-    case 'k':                     /* Const qualifier on some type (Sun) */
-    case 'c':                     /* Const qualifier on some type (OS9000) */
-      /* Because 'c' means other things to AIX and 'k' is perfectly good,
-        only accept 'c' in the os9k_stabs case.  */
-      if (type_descriptor == 'c' && !os9k_stabs)
-       return error_type (pp, objfile);
+    case 'g':                   /* Prototyped function.  (Sun)  */
+      {
+        /* Unresolved questions:
+
+           - According to Sun's ``STABS Interface Manual'', for 'f'
+           and 'F' symbol descriptors, a `0' in the argument type list
+           indicates a varargs function.  But it doesn't say how 'g'
+           type descriptors represent that info.  Someone with access
+           to Sun's toolchain should try it out.
+
+           - According to the comment in define_symbol (search for
+           `process_prototype_types:'), Sun emits integer arguments as
+           types which ref themselves --- like `void' types.  Do we
+           have to deal with that here, too?  Again, someone with
+           access to Sun's toolchain should try it out and let us
+           know.  */
+
+        const char *type_start = (*pp) - 1;
+        struct type *return_type = read_type (pp, objfile);
+        struct type *func_type
+          = make_function_type (return_type, dbx_lookup_type (typenums));
+        struct type_list {
+          struct type *type;
+          struct type_list *next;
+        } *arg_types = 0;
+        int num_args = 0;
+
+        while (**pp && **pp != '#')
+          {
+            struct type *arg_type = read_type (pp, objfile);
+            struct type_list *new = alloca (sizeof (*new));
+            new->type = arg_type;
+            new->next = arg_types;
+            arg_types = new;
+            num_args++;
+          }
+        if (**pp == '#')
+          ++*pp;
+        else
+          {
+           complaint (&symfile_complaints,
+                      "Prototyped function type didn't end arguments with `#':\n%s",
+                      type_start);
+          }
+
+        /* If there is just one argument whose type is `void', then
+           that's just an empty argument list.  */
+        if (arg_types
+            && ! arg_types->next
+            && TYPE_CODE (arg_types->type) == TYPE_CODE_VOID)
+          num_args = 0;
+
+        TYPE_FIELDS (func_type)
+          = (struct field *) TYPE_ALLOC (func_type,
+                                         num_args * sizeof (struct field));
+        memset (TYPE_FIELDS (func_type), 0, num_args * sizeof (struct field));
+        {
+          int i;
+          struct type_list *t;
+
+          /* We stuck each argument type onto the front of the list
+             when we read it, so the list is reversed.  Build the
+             fields array right-to-left.  */
+          for (t = arg_types, i = num_args - 1; t; t = t->next, i--)
+            TYPE_FIELD_TYPE (func_type, i) = t->type;
+        }
+        TYPE_NFIELDS (func_type) = num_args;
+        TYPE_FLAGS (func_type) |= TYPE_FLAG_PROTOTYPED;
+
+        type = func_type;
+        break;
+      }
+
+    case 'k':                  /* Const qualifier on some type (Sun) */
       type = read_type (pp, objfile);
-      /* FIXME! For now, we ignore const and volatile qualifiers.  */
+      type = make_cv_type (1, TYPE_VOLATILE (type), type,
+                          dbx_lookup_type (typenums));
       break;
 
-    case 'B':                       /* Volatile qual on some type (Sun) */
-    case 'i':                       /* Volatile qual on some type (OS9000) */
-      /* Because 'i' means other things to AIX and 'B' is perfectly good,
-        only accept 'i' in the os9k_stabs case.  */
-      if (type_descriptor == 'i' && !os9k_stabs)
-       return error_type (pp, objfile);
+    case 'B':                  /* Volatile qual on some type (Sun) */
       type = read_type (pp, objfile);
-      /* FIXME! For now, we ignore const and volatile qualifiers.  */
+      type = make_cv_type (TYPE_CONST (type), 1, type,
+                          dbx_lookup_type (typenums));
       break;
 
     case '@':
-      if (isdigit (**pp) || **pp ==  '(' || **pp == '-')
-       { /* Member (class & variable) type */
+      if (isdigit (**pp) || **pp == '(' || **pp == '-')
+       {                       /* Member (class & variable) type */
          /* FIXME -- we should be doing smash_to_XXX types here.  */
 
          struct type *domain = read_type (pp, objfile);
@@ -2564,7 +2608,8 @@ read_type (pp, objfile)
          type = dbx_alloc_type (typenums, objfile);
          smash_to_member_type (type, domain, memtype);
        }
-      else /* type attribute */
+      else
+       /* type attribute */
        {
          char *attr = *pp;
          /* Skip to the semicolon.  */
@@ -2573,23 +2618,29 @@ read_type (pp, objfile)
          if (**pp == '\0')
            return error_type (pp, objfile);
          else
-           ++*pp;  /* Skip the semicolon.  */
+           ++ * pp;            /* Skip the semicolon.  */
 
          switch (*attr)
            {
-           case 's':
+           case 's':           /* Size attribute */
              type_size = atoi (attr + 1);
              if (type_size <= 0)
                type_size = -1;
              break;
 
-           case 'S':
+           case 'S':           /* String attribute */
+             /* FIXME: check to see if following type is array? */
              is_string = 1;
              break;
 
+           case 'V':           /* Vector attribute */
+             /* FIXME: check to see if following type is array? */
+             is_vector = 1;
+             break;
+
            default:
              /* Ignore unrecognized type attributes, so future compilers
-                can invent new ones.  */
+                can invent new ones.  */
              break;
            }
          ++*pp;
@@ -2597,7 +2648,7 @@ read_type (pp, objfile)
        }
       break;
 
-    case '#':                          /* Method (class & fn) type */
+    case '#':                  /* Method (class & fn) type */
       if ((*pp)[0] == '#')
        {
          /* We'll get the parameter types from the name.  */
@@ -2606,7 +2657,9 @@ read_type (pp, objfile)
          (*pp)++;
          return_type = read_type (pp, objfile);
          if (*(*pp)++ != ';')
-           complain (&invalid_member_complaint, symnum);
+           complaint (&symfile_complaints,
+                      "invalid (minimal) member type data format at symtab pos %d.",
+                      symnum);
          type = allocate_stub_method (return_type);
          if (typenums[0] != -1)
            *dbx_lookup_type (typenums) = type;
@@ -2615,7 +2668,8 @@ read_type (pp, objfile)
        {
          struct type *domain = read_type (pp, objfile);
          struct type *return_type;
-         struct type **args;
+         struct field *args;
+         int nargs, varargs;
 
          if (**pp != ',')
            /* Invalid member type data format.  */
@@ -2624,23 +2678,20 @@ read_type (pp, objfile)
            ++(*pp);
 
          return_type = read_type (pp, objfile);
-         args = read_args (pp, ';', objfile);
+         args = read_args (pp, ';', objfile, &nargs, &varargs);
          type = dbx_alloc_type (typenums, objfile);
-         smash_to_method_type (type, domain, return_type, args);
+         smash_to_method_type (type, domain, return_type, args,
+                               nargs, varargs);
        }
       break;
 
-    case 'r':                          /* Range type */
+    case 'r':                  /* Range type */
       type = read_range_type (pp, typenums, objfile);
       if (typenums[0] != -1)
        *dbx_lookup_type (typenums) = type;
       break;
 
     case 'b':
-      if (os9k_stabs)
-       /* Const and volatile qualified type.  */
-       type = read_type (pp, objfile);
-      else
        {
          /* Sun ACC builtin int type */
          type = read_sun_builtin_type (pp, typenums, objfile);
@@ -2649,48 +2700,53 @@ read_type (pp, objfile)
        }
       break;
 
-    case 'R':                          /* Sun ACC builtin float type */
+    case 'R':                  /* Sun ACC builtin float type */
       type = read_sun_floating_type (pp, typenums, objfile);
       if (typenums[0] != -1)
        *dbx_lookup_type (typenums) = type;
       break;
-    
-    case 'e':                          /* Enumeration type */
+
+    case 'e':                  /* Enumeration type */
       type = dbx_alloc_type (typenums, objfile);
       type = read_enum_type (pp, type, objfile);
       if (typenums[0] != -1)
        *dbx_lookup_type (typenums) = type;
       break;
 
-    case 's':                          /* Struct type */
-    case 'u':                          /* Union type */
-      type = dbx_alloc_type (typenums, objfile);
-      switch (type_descriptor)
-       {
-         case 's':
-           TYPE_CODE (type) = TYPE_CODE_STRUCT;
-           break;
-         case 'u':
-           TYPE_CODE (type) = TYPE_CODE_UNION;
-           break;
-       }
-      type = read_struct_type (pp, type, objfile);
-      break;
+    case 's':                  /* Struct type */
+    case 'u':                  /* Union type */
+      {
+        enum type_code type_code = TYPE_CODE_UNDEF;
+        type = dbx_alloc_type (typenums, objfile);
+        switch (type_descriptor)
+          {
+          case 's':
+            type_code = TYPE_CODE_STRUCT;
+            break;
+          case 'u':
+            type_code = TYPE_CODE_UNION;
+            break;
+          }
+        type = read_struct_type (pp, type, type_code, objfile);
+        break;
+      }
 
-    case 'a':                          /* Array type */
+    case 'a':                  /* Array type */
       if (**pp != 'r')
        return error_type (pp, objfile);
       ++*pp;
-      
+
       type = dbx_alloc_type (typenums, objfile);
       type = read_array_type (pp, type, objfile);
       if (is_string)
        TYPE_CODE (type) = TYPE_CODE_STRING;
+      if (is_vector)
+       TYPE_FLAGS (type) |= TYPE_FLAG_VECTOR;
       break;
 
-    case 'S':
+    case 'S':                  /* Set or bitstring  type */
       type1 = read_type (pp, objfile);
-      type = create_set_type ((struct type*) NULL, type1);
+      type = create_set_type ((struct type *) NULL, type1);
       if (is_string)
        TYPE_CODE (type) = TYPE_CODE_BITSTRING;
       if (typenums[0] != -1)
@@ -2699,7 +2755,7 @@ read_type (pp, objfile)
 
     default:
       --*pp;                   /* Go back to the symbol in error */
-                               /* Particularly important if it was \0! */
+      /* Particularly important if it was \0! */
       return error_type (pp, objfile);
     }
 
@@ -2720,8 +2776,7 @@ read_type (pp, objfile)
    Return the proper type node for a given builtin type number. */
 
 static struct type *
-rs6000_builtin_type (typenum)
-     int typenum;
+rs6000_builtin_type (int typenum)
 {
   /* We recognize types numbered from -NUMBER_RECOGNIZED to -1.  */
 #define NUMBER_RECOGNIZED 34
@@ -2731,14 +2786,14 @@ rs6000_builtin_type (typenum)
 
   if (typenum >= 0 || typenum < -NUMBER_RECOGNIZED)
     {
-      complain (&rs6000_builtin_complaint, typenum);
+      complaint (&symfile_complaints, "Unknown builtin type %d", typenum);
       return builtin_type_error;
     }
   if (negative_types[-typenum] != NULL)
     return negative_types[-typenum];
 
 #if TARGET_CHAR_BIT != 8
-  #error This code wrong for TARGET_CHAR_BIT not 8
+#error This code wrong for TARGET_CHAR_BIT not 8
   /* These definitions all assume that TARGET_CHAR_BIT is 8.  I think
      that if that ever becomes not true, the correct fix will be to
      make the size in the struct type to be in bits, not in units of
@@ -2749,10 +2804,10 @@ rs6000_builtin_type (typenum)
     {
     case 1:
       /* The size of this and all the other types are fixed, defined
-        by the debugging format.  If there is a type called "int" which
-        is other than 32 bits, then it should use a new negative type
-        number (or avoid negative type numbers for that case).
-        See stabs.texinfo.  */
+         by the debugging format.  If there is a type called "int" which
+         is other than 32 bits, then it should use a new negative type
+         number (or avoid negative type numbers for that case).
+         See stabs.texinfo.  */
       rettype = init_type (TYPE_CODE_INT, 4, 0, "int", NULL);
       break;
     case 2:
@@ -2799,8 +2854,8 @@ rs6000_builtin_type (typenum)
       break;
     case 14:
       /* This is an IEEE double on the RS/6000, and different machines with
-        different sizes for "long double" should use different negative
-        type numbers.  See stabs.texinfo.  */
+         different sizes for "long double" should use different negative
+         type numbers.  See stabs.texinfo.  */
       rettype = init_type (TYPE_CODE_FLT, 8, 0, "long double", NULL);
       break;
     case 15:
@@ -2841,11 +2896,15 @@ rs6000_builtin_type (typenum)
       break;
     case 25:
       /* Complex type consisting of two IEEE single precision values.  */
-      rettype = init_type (TYPE_CODE_ERROR, 8, 0, "complex", NULL);
+      rettype = init_type (TYPE_CODE_COMPLEX, 8, 0, "complex", NULL);
+      TYPE_TARGET_TYPE (rettype) = init_type (TYPE_CODE_FLT, 4, 0, "float",
+                                             NULL);
       break;
     case 26:
       /* Complex type consisting of two IEEE double precision values.  */
-      rettype = init_type (TYPE_CODE_ERROR, 16, 0, "double complex", NULL);
+      rettype = init_type (TYPE_CODE_COMPLEX, 16, 0, "double complex", NULL);
+      TYPE_TARGET_TYPE (rettype) = init_type (TYPE_CODE_FLT, 8, 0, "double",
+                                             NULL);
       break;
     case 27:
       rettype = init_type (TYPE_CODE_INT, 1, 0, "integer*1", NULL);
@@ -2880,14 +2939,39 @@ rs6000_builtin_type (typenum)
 \f
 /* This page contains subroutines of read_type.  */
 
+/* Replace *OLD_NAME with the method name portion of PHYSNAME.  */
+
+static void
+update_method_name_from_physname (char **old_name, char *physname)
+{
+  char *method_name;
+
+  method_name = method_name_from_physname (physname);
+
+  if (method_name == NULL)
+    {
+      complaint (&symfile_complaints,
+                "Method has bad physname %s\n", physname);
+      return;
+    }
+
+  if (strcmp (*old_name, method_name) != 0)
+    {
+      xfree (*old_name);
+      *old_name = method_name;
+    }
+  else
+    xfree (method_name);
+}
+
 /* Read member function stabs info for C++ classes.  The form of each member
    function data is:
 
-       NAME :: TYPENUM[=type definition] ARGS : PHYSNAME ;
+   NAME :: TYPENUM[=type definition] ARGS : PHYSNAME ;
 
    An example with two member functions is:
 
-       afunc1::20=##15;:i;2A.;afunc2::20:i;2A.;
+   afunc1::20=##15;:i;2A.;afunc2::20:i;2A.;
 
    For the case of overloaded operators, the format is op$::*.funcs, where
    $ is the CPLUS_MARKER (usually '$'), `*' holds the place for an operator
@@ -2896,11 +2980,8 @@ rs6000_builtin_type (typenum)
    Returns 1 for success, 0 for failure.  */
 
 static int
-read_member_functions (fip, pp, type, objfile)
-     struct field_info *fip;
-     char **pp;
-     struct type *type;
-     struct objfile *objfile;
+read_member_functions (struct field_info *fip, char **pp, struct type *type,
+                      struct objfile *objfile)
 {
   int nfn_fields = 0;
   int length = 0;
@@ -2913,21 +2994,22 @@ read_member_functions (fip, pp, type, objfile)
     {
       struct next_fnfield *next;
       struct fn_field fn_field;
-    } *sublist;
+    }
+   *sublist;
   struct type *look_ahead_type;
   struct next_fnfieldlist *new_fnlist;
   struct next_fnfield *new_sublist;
   char *main_fn_name;
   register char *p;
-      
+
   /* Process each list until we find something that is not a member function
      or find the end of the functions. */
 
   while (**pp != ';')
     {
       /* We should be positioned at the start of the function name.
-        Scan forward to find the first ':' and if it is not the
-        first of a "::" delimiter, then this is not a member function. */
+         Scan forward to find the first ':' and if it is not the
+         first of a "::" delimiter, then this is not a member function. */
       p = *pp;
       while (*p != ':')
        {
@@ -2941,12 +3023,12 @@ read_member_functions (fip, pp, type, objfile)
       sublist = NULL;
       look_ahead_type = NULL;
       length = 0;
-      
+
       new_fnlist = (struct next_fnfieldlist *)
        xmalloc (sizeof (struct next_fnfieldlist));
-      make_cleanup (free, new_fnlist);
+      make_cleanup (xfree, new_fnlist);
       memset (new_fnlist, 0, sizeof (struct next_fnfieldlist));
-      
+
       if ((*pp)[0] == 'o' && (*pp)[1] == 'p' && is_cplus_marker ((*pp)[2]))
        {
          /* This is a completely wierd case.  In order to stuff in the
@@ -2958,9 +3040,9 @@ read_member_functions (fip, pp, type, objfile)
          /* This lets the user type "break operator+".
             We could just put in "+" as the name, but that wouldn't
             work for "*".  */
-         static char opname[32] = {'o', 'p', CPLUS_MARKER};
+         static char opname[32] = "op$";
          char *o = opname + 3;
-         
+
          /* Skip past '::'.  */
          *pp = p + 2;
 
@@ -2980,22 +3062,22 @@ read_member_functions (fip, pp, type, objfile)
          /* Skip past '::'.  */
          *pp = p + 2;
        }
-      new_fnlist -> fn_fieldlist.name = main_fn_name;
-      
+      new_fnlist->fn_fieldlist.name = main_fn_name;
+
       do
        {
          new_sublist =
            (struct next_fnfield *) xmalloc (sizeof (struct next_fnfield));
-         make_cleanup (free, new_sublist);
+         make_cleanup (xfree, new_sublist);
          memset (new_sublist, 0, sizeof (struct next_fnfield));
-         
+
          /* Check for and handle cretinous dbx symbol name continuation!  */
          if (look_ahead_type == NULL)
            {
              /* Normal case. */
              STABS_CONTINUE (pp, objfile);
-             
-             new_sublist -> fn_field.type = read_type (pp, objfile);
+
+             new_sublist->fn_field.type = read_type (pp, objfile);
              if (**pp != ':')
                {
                  /* Invalid symtab info for member function.  */
@@ -3005,93 +3087,94 @@ read_member_functions (fip, pp, type, objfile)
          else
            {
              /* g++ version 1 kludge */
-             new_sublist -> fn_field.type = look_ahead_type;
+             new_sublist->fn_field.type = look_ahead_type;
              look_ahead_type = NULL;
            }
-         
+
          (*pp)++;
          p = *pp;
          while (*p != ';')
            {
              p++;
            }
-         
+
          /* If this is just a stub, then we don't have the real name here. */
 
-         if (TYPE_FLAGS (new_sublist -> fn_field.type) & TYPE_FLAG_STUB)
+         if (TYPE_STUB (new_sublist->fn_field.type))
            {
-             if (!TYPE_DOMAIN_TYPE (new_sublist -> fn_field.type))
-               TYPE_DOMAIN_TYPE (new_sublist -> fn_field.type) = type;
-             new_sublist -> fn_field.is_stub = 1;
+             if (!TYPE_DOMAIN_TYPE (new_sublist->fn_field.type))
+               TYPE_DOMAIN_TYPE (new_sublist->fn_field.type) = type;
+             new_sublist->fn_field.is_stub = 1;
            }
-         new_sublist -> fn_field.physname = savestring (*pp, p - *pp);
+         new_sublist->fn_field.physname = savestring (*pp, p - *pp);
          *pp = p + 1;
-         
+
          /* Set this member function's visibility fields.  */
          switch (*(*pp)++)
            {
-             case VISIBILITY_PRIVATE:
-               new_sublist -> fn_field.is_private = 1;
-               break;
-             case VISIBILITY_PROTECTED:
-               new_sublist -> fn_field.is_protected = 1;
-               break;
+           case VISIBILITY_PRIVATE:
+             new_sublist->fn_field.is_private = 1;
+             break;
+           case VISIBILITY_PROTECTED:
+             new_sublist->fn_field.is_protected = 1;
+             break;
            }
-         
+
          STABS_CONTINUE (pp, objfile);
          switch (**pp)
            {
-             case 'A': /* Normal functions. */
-               new_sublist -> fn_field.is_const = 0;
-               new_sublist -> fn_field.is_volatile = 0;
-               (*pp)++;
-               break;
-             case 'B': /* `const' member functions. */
-               new_sublist -> fn_field.is_const = 1;
-               new_sublist -> fn_field.is_volatile = 0;
-               (*pp)++;
-               break;
-             case 'C': /* `volatile' member function. */
-               new_sublist -> fn_field.is_const = 0;
-               new_sublist -> fn_field.is_volatile = 1;
-               (*pp)++;
-               break;
-             case 'D': /* `const volatile' member function. */
-               new_sublist -> fn_field.is_const = 1;
-               new_sublist -> fn_field.is_volatile = 1;
-               (*pp)++;
-               break;
-             case '*': /* File compiled with g++ version 1 -- no info */
-             case '?':
-             case '.':
-               break;
-             default:
-               complain (&const_vol_complaint, **pp);
-               break;
+           case 'A':           /* Normal functions. */
+             new_sublist->fn_field.is_const = 0;
+             new_sublist->fn_field.is_volatile = 0;
+             (*pp)++;
+             break;
+           case 'B':           /* `const' member functions. */
+             new_sublist->fn_field.is_const = 1;
+             new_sublist->fn_field.is_volatile = 0;
+             (*pp)++;
+             break;
+           case 'C':           /* `volatile' member function. */
+             new_sublist->fn_field.is_const = 0;
+             new_sublist->fn_field.is_volatile = 1;
+             (*pp)++;
+             break;
+           case 'D':           /* `const volatile' member function. */
+             new_sublist->fn_field.is_const = 1;
+             new_sublist->fn_field.is_volatile = 1;
+             (*pp)++;
+             break;
+           case '*':           /* File compiled with g++ version 1 -- no info */
+           case '?':
+           case '.':
+             break;
+           default:
+             complaint (&symfile_complaints,
+                        "const/volatile indicator missing, got '%c'", **pp);
+             break;
            }
-         
+
          switch (*(*pp)++)
            {
-             case '*':
+           case '*':
              {
                int nbits;
-               /* virtual member function, followed by index.
+               /* virtual member function, followed by index.
                   The sign bit is set to distinguish pointers-to-methods
                   from virtual function indicies.  Since the array is
                   in words, the quantity must be shifted left by 1
                   on 16 bit machine, and by 2 on 32 bit machine, forcing
                   the sign bit out, and usable as a valid index into
                   the array.  Remove the sign bit here.  */
-               new_sublist -> fn_field.voffset =
+               new_sublist->fn_field.voffset =
                  (0x7fffffff & read_huge_number (pp, ';', &nbits)) + 2;
                if (nbits != 0)
                  return 0;
-             
+
                STABS_CONTINUE (pp, objfile);
                if (**pp == ';' || **pp == '\0')
                  {
                    /* Must be g++ version 1.  */
-                   new_sublist -> fn_field.fcontext = 0;
+                   new_sublist->fn_field.fcontext = 0;
                  }
                else
                  {
@@ -3105,7 +3188,7 @@ read_member_functions (fip, pp, type, objfile)
                      }
                    else
                      {
-                       new_sublist -> fn_field.fcontext = look_ahead_type;
+                       new_sublist->fn_field.fcontext = look_ahead_type;
                        if (**pp != ';')
                          {
                            return 0;
@@ -3119,53 +3202,240 @@ read_member_functions (fip, pp, type, objfile)
                  }
                break;
              }
-             case '?':
-               /* static member function.  */
-               new_sublist -> fn_field.voffset = VOFFSET_STATIC;
-               if (strncmp (new_sublist -> fn_field.physname,
-                            main_fn_name, strlen (main_fn_name)))
+           case '?':
+             /* static member function.  */
+             {
+               int slen = strlen (main_fn_name);
+
+               new_sublist->fn_field.voffset = VOFFSET_STATIC;
+
+               /* For static member functions, we can't tell if they
+                  are stubbed, as they are put out as functions, and not as
+                  methods.
+                  GCC v2 emits the fully mangled name if
+                  dbxout.c:flag_minimal_debug is not set, so we have to
+                  detect a fully mangled physname here and set is_stub
+                  accordingly.  Fully mangled physnames in v2 start with
+                  the member function name, followed by two underscores.
+                  GCC v3 currently always emits stubbed member functions,
+                  but with fully mangled physnames, which start with _Z.  */
+               if (!(strncmp (new_sublist->fn_field.physname,
+                              main_fn_name, slen) == 0
+                     && new_sublist->fn_field.physname[slen] == '_'
+                     && new_sublist->fn_field.physname[slen + 1] == '_'))
                  {
-                   new_sublist -> fn_field.is_stub = 1;
+                   new_sublist->fn_field.is_stub = 1;
                  }
                break;
-             
-             default:
-               /* error */
-               complain (&member_fn_complaint, (*pp)[-1]);
-               /* Fall through into normal member function.  */
-             
-             case '.':
-               /* normal member function.  */
-               new_sublist -> fn_field.voffset = 0;
-               new_sublist -> fn_field.fcontext = 0;
-               break;
+             }
+
+           default:
+             /* error */
+             complaint (&symfile_complaints,
+                        "member function type missing, got '%c'", (*pp)[-1]);
+             /* Fall through into normal member function.  */
+
+           case '.':
+             /* normal member function.  */
+             new_sublist->fn_field.voffset = 0;
+             new_sublist->fn_field.fcontext = 0;
+             break;
            }
-         
-         new_sublist -> next = sublist;
+
+         new_sublist->next = sublist;
          sublist = new_sublist;
          length++;
          STABS_CONTINUE (pp, objfile);
        }
       while (**pp != ';' && **pp != '\0');
-      
+
       (*pp)++;
-      
-      new_fnlist -> fn_fieldlist.fn_fields = (struct fn_field *)
-       obstack_alloc (&objfile -> type_obstack, 
-                      sizeof (struct fn_field) * length);
-      memset (new_fnlist -> fn_fieldlist.fn_fields, 0,
-             sizeof (struct fn_field) * length);
-      for (i = length; (i--, sublist); sublist = sublist -> next)
+      STABS_CONTINUE (pp, objfile);
+
+      /* Skip GCC 3.X member functions which are duplicates of the callable
+        constructor/destructor.  */
+      if (strcmp (main_fn_name, "__base_ctor") == 0
+         || strcmp (main_fn_name, "__base_dtor") == 0
+         || strcmp (main_fn_name, "__deleting_dtor") == 0)
        {
-         new_fnlist -> fn_fieldlist.fn_fields[i] = sublist -> fn_field;
+         xfree (main_fn_name);
+       }
+      else
+       {
+         int has_stub = 0;
+         int has_destructor = 0, has_other = 0;
+         int is_v3 = 0;
+         struct next_fnfield *tmp_sublist;
+
+         /* Various versions of GCC emit various mostly-useless
+            strings in the name field for special member functions.
+
+            For stub methods, we need to defer correcting the name
+            until we are ready to unstub the method, because the current
+            name string is used by gdb_mangle_name.  The only stub methods
+            of concern here are GNU v2 operators; other methods have their
+            names correct (see caveat below).
+
+            For non-stub methods, in GNU v3, we have a complete physname.
+            Therefore we can safely correct the name now.  This primarily
+            affects constructors and destructors, whose name will be
+            __comp_ctor or __comp_dtor instead of Foo or ~Foo.  Cast
+            operators will also have incorrect names; for instance,
+            "operator int" will be named "operator i" (i.e. the type is
+            mangled).
+
+            For non-stub methods in GNU v2, we have no easy way to
+            know if we have a complete physname or not.  For most
+            methods the result depends on the platform (if CPLUS_MARKER
+            can be `$' or `.', it will use minimal debug information, or
+            otherwise the full physname will be included).
+
+            Rather than dealing with this, we take a different approach.
+            For v3 mangled names, we can use the full physname; for v2,
+            we use cplus_demangle_opname (which is actually v2 specific),
+            because the only interesting names are all operators - once again
+            barring the caveat below.  Skip this process if any method in the
+            group is a stub, to prevent our fouling up the workings of
+            gdb_mangle_name.
+
+            The caveat: GCC 2.95.x (and earlier?) put constructors and
+            destructors in the same method group.  We need to split this
+            into two groups, because they should have different names.
+            So for each method group we check whether it contains both
+            routines whose physname appears to be a destructor (the physnames
+            for and destructors are always provided, due to quirks in v2
+            mangling) and routines whose physname does not appear to be a
+            destructor.  If so then we break up the list into two halves.
+            Even if the constructors and destructors aren't in the same group
+            the destructor will still lack the leading tilde, so that also
+            needs to be fixed.
+
+            So, to summarize what we expect and handle here:
+
+               Given         Given          Real         Real       Action
+            method name     physname      physname   method name
+
+            __opi            [none]     __opi__3Foo  operator int    opname
+                                                                  [now or later]
+            Foo              _._3Foo       _._3Foo      ~Foo       separate and
+                                                                      rename
+            operator i     _ZN3FoocviEv _ZN3FoocviEv operator int    demangle
+            __comp_ctor  _ZN3FooC1ERKS_ _ZN3FooC1ERKS_   Foo         demangle
+         */
+
+         tmp_sublist = sublist;
+         while (tmp_sublist != NULL)
+           {
+             if (tmp_sublist->fn_field.is_stub)
+               has_stub = 1;
+             if (tmp_sublist->fn_field.physname[0] == '_'
+                 && tmp_sublist->fn_field.physname[1] == 'Z')
+               is_v3 = 1;
+
+             if (is_destructor_name (tmp_sublist->fn_field.physname))
+               has_destructor++;
+             else
+               has_other++;
+
+             tmp_sublist = tmp_sublist->next;
+           }
+
+         if (has_destructor && has_other)
+           {
+             struct next_fnfieldlist *destr_fnlist;
+             struct next_fnfield *last_sublist;
+
+             /* Create a new fn_fieldlist for the destructors.  */
+
+             destr_fnlist = (struct next_fnfieldlist *)
+               xmalloc (sizeof (struct next_fnfieldlist));
+             make_cleanup (xfree, destr_fnlist);
+             memset (destr_fnlist, 0, sizeof (struct next_fnfieldlist));
+             destr_fnlist->fn_fieldlist.name
+               = obconcat (&objfile->type_obstack, "", "~",
+                           new_fnlist->fn_fieldlist.name);
+
+             destr_fnlist->fn_fieldlist.fn_fields = (struct fn_field *)
+               obstack_alloc (&objfile->type_obstack,
+                              sizeof (struct fn_field) * has_destructor);
+             memset (destr_fnlist->fn_fieldlist.fn_fields, 0,
+                 sizeof (struct fn_field) * has_destructor);
+             tmp_sublist = sublist;
+             last_sublist = NULL;
+             i = 0;
+             while (tmp_sublist != NULL)
+               {
+                 if (!is_destructor_name (tmp_sublist->fn_field.physname))
+                   {
+                     tmp_sublist = tmp_sublist->next;
+                     continue;
+                   }
+                 
+                 destr_fnlist->fn_fieldlist.fn_fields[i++]
+                   = tmp_sublist->fn_field;
+                 if (last_sublist)
+                   last_sublist->next = tmp_sublist->next;
+                 else
+                   sublist = tmp_sublist->next;
+                 last_sublist = tmp_sublist;
+                 tmp_sublist = tmp_sublist->next;
+               }
+
+             destr_fnlist->fn_fieldlist.length = has_destructor;
+             destr_fnlist->next = fip->fnlist;
+             fip->fnlist = destr_fnlist;
+             nfn_fields++;
+             total_length += has_destructor;
+             length -= has_destructor;
+           }
+         else if (is_v3)
+           {
+             /* v3 mangling prevents the use of abbreviated physnames,
+                so we can do this here.  There are stubbed methods in v3
+                only:
+                - in -gstabs instead of -gstabs+
+                - or for static methods, which are output as a function type
+                  instead of a method type.  */
+
+             update_method_name_from_physname (&new_fnlist->fn_fieldlist.name,
+                                               sublist->fn_field.physname);
+           }
+         else if (has_destructor && new_fnlist->fn_fieldlist.name[0] != '~')
+           {
+             new_fnlist->fn_fieldlist.name = concat ("~", main_fn_name, NULL);
+             xfree (main_fn_name);
+           }
+         else if (!has_stub)
+           {
+             char dem_opname[256];
+             int ret;
+             ret = cplus_demangle_opname (new_fnlist->fn_fieldlist.name,
+                                             dem_opname, DMGL_ANSI);
+             if (!ret)
+               ret = cplus_demangle_opname (new_fnlist->fn_fieldlist.name,
+                                            dem_opname, 0);
+             if (ret)
+               new_fnlist->fn_fieldlist.name
+                 = obsavestring (dem_opname, strlen (dem_opname),
+                                 &objfile->type_obstack);
+           }
+
+         new_fnlist->fn_fieldlist.fn_fields = (struct fn_field *)
+           obstack_alloc (&objfile->type_obstack,
+                          sizeof (struct fn_field) * length);
+         memset (new_fnlist->fn_fieldlist.fn_fields, 0,
+                 sizeof (struct fn_field) * length);
+         for (i = length; (i--, sublist); sublist = sublist->next)
+           {
+             new_fnlist->fn_fieldlist.fn_fields[i] = sublist->fn_field;
+           }
+
+         new_fnlist->fn_fieldlist.length = length;
+         new_fnlist->next = fip->fnlist;
+         fip->fnlist = new_fnlist;
+         nfn_fields++;
+         total_length += length;
        }
-      
-      new_fnlist -> fn_fieldlist.length = length;
-      new_fnlist -> next = fip -> fnlist;
-      fip -> fnlist = new_fnlist;
-      nfn_fields++;
-      total_length += length;
-      STABS_CONTINUE (pp, objfile);
     }
 
   if (nfn_fields)
@@ -3188,11 +3458,8 @@ read_member_functions (fip, pp, type, objfile)
    keep parsing and it's time for error_type().  */
 
 static int
-read_cpp_abbrev (fip, pp, type, objfile)
-     struct field_info *fip;
-     char **pp;
-     struct type *type;
-     struct objfile *objfile;
+read_cpp_abbrev (struct field_info *fip, char **pp, struct type *type,
+                struct objfile *objfile)
 {
   register char *p;
   char *name;
@@ -3208,50 +3475,57 @@ read_cpp_abbrev (fip, pp, type, objfile)
       *pp = p + 1;
 
       /* At this point, *pp points to something like "22:23=*22...",
-        where the type number before the ':' is the "context" and
-        everything after is a regular type definition.  Lookup the
-        type, find it's name, and construct the field name. */
+         where the type number before the ':' is the "context" and
+         everything after is a regular type definition.  Lookup the
+         type, find it's name, and construct the field name. */
 
       context = read_type (pp, objfile);
 
       switch (cpp_abbrev)
        {
-         case 'f':             /* $vf -- a virtual function table pointer */
-           fip->list->field.name =
-             obconcat (&objfile->type_obstack, vptr_name, "", "");
-           break;
+       case 'f':               /* $vf -- a virtual function table pointer */
+         name = type_name_no_tag (context);
+         if (name == NULL)
+         {
+                 name = "";
+         }
+         fip->list->field.name =
+           obconcat (&objfile->type_obstack, vptr_name, name, "");
+         break;
 
-         case 'b':             /* $vb -- a virtual bsomethingorother */
-           name = type_name_no_tag (context);
-           if (name == NULL)
-             {
-               complain (&invalid_cpp_type_complaint, symnum);
-               name = "FOO";
-             }
-           fip->list->field.name =
-             obconcat (&objfile->type_obstack, vb_name, name, "");
-           break;
+       case 'b':               /* $vb -- a virtual bsomethingorother */
+         name = type_name_no_tag (context);
+         if (name == NULL)
+           {
+             complaint (&symfile_complaints,
+                        "C++ abbreviated type name unknown at symtab pos %d",
+                        symnum);
+             name = "FOO";
+           }
+         fip->list->field.name =
+           obconcat (&objfile->type_obstack, vb_name, name, "");
+         break;
 
-         default:
-           complain (&invalid_cpp_abbrev_complaint, *pp);
-           fip->list->field.name =
-             obconcat (&objfile->type_obstack,
-                       "INVALID_CPLUSPLUS_ABBREV", "", "");
-           break;
+       default:
+         invalid_cpp_abbrev_complaint (*pp);
+         fip->list->field.name =
+           obconcat (&objfile->type_obstack,
+                     "INVALID_CPLUSPLUS_ABBREV", "", "");
+         break;
        }
 
       /* At this point, *pp points to the ':'.  Skip it and read the
-        field type. */
+         field type. */
 
       p = ++(*pp);
       if (p[-1] != ':')
        {
-         complain (&invalid_cpp_abbrev_complaint, *pp);
+         invalid_cpp_abbrev_complaint (*pp);
          return 0;
        }
       fip->list->field.type = read_type (pp, objfile);
       if (**pp == ',')
-       (*pp)++;                        /* Skip the comma.  */
+       (*pp)++;                /* Skip the comma.  */
       else
        return 0;
 
@@ -3267,69 +3541,67 @@ read_cpp_abbrev (fip, pp, type, objfile)
     }
   else
     {
-      complain (&invalid_cpp_abbrev_complaint, *pp);
+      invalid_cpp_abbrev_complaint (*pp);
       /* We have no idea what syntax an unrecognized abbrev would have, so
-        better return 0.  If we returned 1, we would need to at least advance
-        *pp to avoid an infinite loop.  */
+         better return 0.  If we returned 1, we would need to at least advance
+         *pp to avoid an infinite loop.  */
       return 0;
     }
   return 1;
 }
 
 static void
-read_one_struct_field (fip, pp, p, type, objfile)
-     struct field_info *fip;
-     char **pp;
-     char *p;
-     struct type *type;
-     struct objfile *objfile;
+read_one_struct_field (struct field_info *fip, char **pp, char *p,
+                      struct type *type, struct objfile *objfile)
 {
-  /* The following is code to work around cfront generated stabs.
-     The stabs contains full mangled name for each field.
-     We try to demangle the name and extract the field name out of it.
-  */
-  if (ARM_DEMANGLING && current_subfile->language == language_cplus)
-    {
-      char save_p;
-      char *dem, *dem_p;
-      save_p = *p;
-      *p = '\0';
-      dem = cplus_demangle (*pp, DMGL_ANSI | DMGL_PARAMS);
-      if (dem != NULL)
-        {
-          dem_p = strrchr (dem, ':');
-          if (dem_p != 0 && *(dem_p-1)==':')
-            dem_p++;
-          FIELD_NAME (fip->list->field) =
-            obsavestring (dem_p, strlen (dem_p), &objfile -> type_obstack);
-        }
-      else
-        {
-          FIELD_NAME (fip->list->field) =
-            obsavestring (*pp, p - *pp, &objfile -> type_obstack);
-        }
-      *p = save_p;
-    }
-  /* end of code for cfront work around */
-
-  else
-    fip -> list -> field.name =
-    obsavestring (*pp, p - *pp, &objfile -> type_obstack);
+#if 0 /* OBSOLETE CFront */
+// OBSOLETE    /* The following is code to work around cfront generated stabs.
+// OBSOLETE       The stabs contains full mangled name for each field.
+// OBSOLETE       We try to demangle the name and extract the field name out of it.
+// OBSOLETE     */
+// OBSOLETE    if (ARM_DEMANGLING && current_subfile->language == language_cplus)
+// OBSOLETE      {
+// OBSOLETE        char save_p;
+// OBSOLETE        char *dem, *dem_p;
+// OBSOLETE        save_p = *p;
+// OBSOLETE        *p = '\0';
+// OBSOLETE        dem = cplus_demangle (*pp, DMGL_ANSI | DMGL_PARAMS);
+// OBSOLETE        if (dem != NULL)
+// OBSOLETE    {
+// OBSOLETE      dem_p = strrchr (dem, ':');
+// OBSOLETE      if (dem_p != 0 && *(dem_p - 1) == ':')
+// OBSOLETE        dem_p++;
+// OBSOLETE      FIELD_NAME (fip->list->field) =
+// OBSOLETE        obsavestring (dem_p, strlen (dem_p), &objfile->type_obstack);
+// OBSOLETE    }
+// OBSOLETE        else
+// OBSOLETE    {
+// OBSOLETE      FIELD_NAME (fip->list->field) =
+// OBSOLETE        obsavestring (*pp, p - *pp, &objfile->type_obstack);
+// OBSOLETE    }
+// OBSOLETE        *p = save_p;
+// OBSOLETE      }
+// OBSOLETE    /* end of code for cfront work around */
+
+// OBSOLETE   else
+#endif /* OBSOLETE CFront */
+    fip->list->field.name =
+      obsavestring (*pp, p - *pp, &objfile->type_obstack);
   *pp = p + 1;
 
   /* This means we have a visibility for a field coming. */
   if (**pp == '/')
     {
       (*pp)++;
-      fip -> list -> visibility = *(*pp)++;
+      fip->list->visibility = *(*pp)++;
     }
   else
     {
       /* normal dbx-style format, no explicit visibility */
-      fip -> list -> visibility = VISIBILITY_PUBLIC;
+      fip->list->visibility = VISIBILITY_PUBLIC;
     }
 
-  fip -> list -> field.type = read_type (pp, objfile);
+  fip->list->field.type = read_type (pp, objfile);
   if (**pp == ':')
     {
       p = ++(*pp);
@@ -3337,12 +3609,13 @@ read_one_struct_field (fip, pp, p, type, objfile)
       /* Possible future hook for nested types. */
       if (**pp == '!')
        {
-         fip -> list -> field.bitpos = (long)-2; /* nested type */
+         fip->list->field.bitpos = (long) -2;  /* nested type */
          p = ++(*pp);
        }
-      else ...;
+      else
+       ...;
 #endif
-      while (*p != ';') 
+      while (*p != ';')
        {
          p++;
        }
@@ -3354,7 +3627,7 @@ read_one_struct_field (fip, pp, p, type, objfile)
   else if (**pp != ',')
     {
       /* Bad structure-type format.  */
-      complain (&stabs_general_complaint, "bad structure-type format");
+      stabs_general_complaint ("bad structure-type format");
       return;
     }
 
@@ -3365,13 +3638,13 @@ read_one_struct_field (fip, pp, p, type, objfile)
     FIELD_BITPOS (fip->list->field) = read_huge_number (pp, ',', &nbits);
     if (nbits != 0)
       {
-       complain (&stabs_general_complaint, "bad structure-type format");
+       stabs_general_complaint ("bad structure-type format");
        return;
       }
     FIELD_BITSIZE (fip->list->field) = read_huge_number (pp, ';', &nbits);
     if (nbits != 0)
       {
-       complain (&stabs_general_complaint, "bad structure-type format");
+       stabs_general_complaint ("bad structure-type format");
        return;
       }
   }
@@ -3380,40 +3653,43 @@ read_one_struct_field (fip, pp, p, type, objfile)
       && FIELD_BITSIZE (fip->list->field) == 0)
     {
       /* This can happen in two cases: (1) at least for gcc 2.4.5 or so,
-        it is a field which has been optimized out.  The correct stab for
-        this case is to use VISIBILITY_IGNORE, but that is a recent
-        invention.  (2) It is a 0-size array.  For example
-        union { int num; char str[0]; } foo.  Printing "<no value>" for
-        str in "p foo" is OK, since foo.str (and thus foo.str[3])
-        will continue to work, and a 0-size array as a whole doesn't
-        have any contents to print.
-
-        I suspect this probably could also happen with gcc -gstabs (not
-        -gstabs+) for static fields, and perhaps other C++ extensions.
-        Hopefully few people use -gstabs with gdb, since it is intended
-        for dbx compatibility.  */
+         it is a field which has been optimized out.  The correct stab for
+         this case is to use VISIBILITY_IGNORE, but that is a recent
+         invention.  (2) It is a 0-size array.  For example
+         union { int num; char str[0]; } foo.  Printing "<no value>" for
+         str in "p foo" is OK, since foo.str (and thus foo.str[3])
+         will continue to work, and a 0-size array as a whole doesn't
+         have any contents to print.
+
+         I suspect this probably could also happen with gcc -gstabs (not
+         -gstabs+) for static fields, and perhaps other C++ extensions.
+         Hopefully few people use -gstabs with gdb, since it is intended
+         for dbx compatibility.  */
 
       /* Ignore this field.  */
-      fip -> list-> visibility = VISIBILITY_IGNORE;
+      fip->list->visibility = VISIBILITY_IGNORE;
     }
   else
     {
       /* Detect an unpacked field and mark it as such.
-        dbx gives a bit size for all fields.
-        Note that forward refs cannot be packed,
-        and treat enums as if they had the width of ints.  */
+         dbx gives a bit size for all fields.
+         Note that forward refs cannot be packed,
+         and treat enums as if they had the width of ints.  */
+
+      struct type *field_type = check_typedef (FIELD_TYPE (fip->list->field));
 
-      if (TYPE_CODE (FIELD_TYPE (fip->list->field)) != TYPE_CODE_INT
-         && TYPE_CODE (FIELD_TYPE (fip->list->field)) != TYPE_CODE_BOOL
-         && TYPE_CODE (FIELD_TYPE (fip->list->field)) != TYPE_CODE_ENUM)
+      if (TYPE_CODE (field_type) != TYPE_CODE_INT
+         && TYPE_CODE (field_type) != TYPE_CODE_RANGE
+         && TYPE_CODE (field_type) != TYPE_CODE_BOOL
+         && TYPE_CODE (field_type) != TYPE_CODE_ENUM)
        {
          FIELD_BITSIZE (fip->list->field) = 0;
        }
-      if ((FIELD_BITSIZE (fip->list->field) 
-          == TARGET_CHAR_BIT * TYPE_LENGTH (FIELD_TYPE (fip->list->field))
-          || (TYPE_CODE (FIELD_TYPE (fip->list->field)) == TYPE_CODE_ENUM
-              && FIELD_BITSIZE (fip->list->field) == TARGET_INT_BIT )
-          )
+      if ((FIELD_BITSIZE (fip->list->field)
+          == TARGET_CHAR_BIT * TYPE_LENGTH (field_type)
+          || (TYPE_CODE (field_type) == TYPE_CODE_ENUM
+              && FIELD_BITSIZE (fip->list->field) == TARGET_INT_BIT)
+         )
          &&
          FIELD_BITPOS (fip->list->field) % 8 == 0)
        {
@@ -3425,7 +3701,7 @@ read_one_struct_field (fip, pp, p, type, objfile)
 
 /* Read struct or class data fields.  They have the form:
 
-       NAME : [VISIBILITY] TYPENUM , BITPOS , BITSIZE ;
+   NAME : [VISIBILITY] TYPENUM , BITPOS , BITSIZE ;
 
    At the end, we see a semicolon instead of a field.
 
@@ -3434,21 +3710,18 @@ read_one_struct_field (fip, pp, p, type, objfile)
 
    The optional VISIBILITY is one of:
 
-       '/0'    (VISIBILITY_PRIVATE)
-       '/1'    (VISIBILITY_PROTECTED)
-       '/2'    (VISIBILITY_PUBLIC)
-       '/9'    (VISIBILITY_IGNORE)
+   '/0' (VISIBILITY_PRIVATE)
+   '/1' (VISIBILITY_PROTECTED)
+   '/2' (VISIBILITY_PUBLIC)
+   '/9' (VISIBILITY_IGNORE)
 
    or nothing, for C style fields with public visibility.
 
    Returns 1 for success, 0 for failure.  */
 
 static int
-read_struct_fields (fip, pp, type, objfile)
-     struct field_info *fip;
-     char **pp;
-     struct type *type;
-     struct objfile *objfile;
+read_struct_fields (struct field_info *fip, char **pp, struct type *type,
+                   struct objfile *objfile)
 {
   register char *p;
   struct nextfield *new;
@@ -3460,25 +3733,26 @@ read_struct_fields (fip, pp, type, objfile)
   /* Read each data member type until we find the terminating ';' at the end of
      the data member list, or break for some other reason such as finding the
      start of the member function list. */
+  /* Stab string for structure/union does not end with two ';' in
+     SUN C compiler 5.3 i.e. F6U2, hence check for end of string. */
 
-  while (**pp != ';')
+  while (**pp != ';' && **pp != '\0')
     {
-      if (os9k_stabs && **pp == ',') break;
       STABS_CONTINUE (pp, objfile);
       /* Get space to record the next field's data.  */
       new = (struct nextfield *) xmalloc (sizeof (struct nextfield));
-      make_cleanup (free, new);
+      make_cleanup (xfree, new);
       memset (new, 0, sizeof (struct nextfield));
-      new -> next = fip -> list;
-      fip -> list = new;
+      new->next = fip->list;
+      fip->list = new;
 
       /* Get the field name.  */
       p = *pp;
 
       /* If is starts with CPLUS_MARKER it is a special abbreviation,
-        unless the CPLUS_MARKER is followed by an underscore, in
-        which case it is just the name of an anonymous type, which we
-        should handle like any other type name.  */
+         unless the CPLUS_MARKER is followed by an underscore, in
+         which case it is just the name of an anonymous type, which we
+         should handle like any other type name.  */
 
       if (is_cplus_marker (p[0]) && p[1] != '_')
        {
@@ -3488,11 +3762,11 @@ read_struct_fields (fip, pp, type, objfile)
        }
 
       /* Look for the ':' that separates the field name from the field
-        values.  Data members are delimited by a single ':', while member
-        functions are delimited by a pair of ':'s.  When we hit the member
-        functions (if any), terminate scan loop and return. */
+         values.  Data members are delimited by a single ':', while member
+         functions are delimited by a pair of ':'s.  When we hit the member
+         functions (if any), terminate scan loop and return. */
 
-      while (*p != ':' && *p != '\0') 
+      while (*p != ':' && *p != '\0')
        {
          p++;
        }
@@ -3508,13 +3782,14 @@ read_struct_fields (fip, pp, type, objfile)
     }
   if (p[0] == ':' && p[1] == ':')
     {
-      /* chill the list of fields: the last entry (at the head) is a
-        partially constructed entry which we now scrub. */
-      fip -> list = fip -> list -> next;
+      /* (the deleted) chill the list of fields: the last entry (at
+         the head) is a partially constructed entry which we now
+         scrub. */
+      fip->list = fip->list->next;
     }
   return 1;
 }
-
+/* *INDENT-OFF* */
 /* The stabs for C++ derived classes contain baseclass information which
    is marked by a '!' character after the total size.  This function is
    called when we encounter the baseclass marker, and slurps up all the
@@ -3538,13 +3813,13 @@ read_struct_fields (fip, pp, type, objfile)
        Type number of base class ____________________________________|
 
   Return 1 for success, 0 for (error-type-inducing) failure.  */
+/* *INDENT-ON* */
+
+
 
 static int
-read_baseclasses (fip, pp, type, objfile)
-     struct field_info *fip;
-     char **pp;
-     struct type *type;
-     struct objfile *objfile;
+read_baseclasses (struct field_info *fip, char **pp, struct type *type,
+                 struct objfile *objfile)
 {
   int i;
   struct nextfield *new;
@@ -3587,52 +3862,51 @@ read_baseclasses (fip, pp, type, objfile)
   for (i = 0; i < TYPE_N_BASECLASSES (type); i++)
     {
       new = (struct nextfield *) xmalloc (sizeof (struct nextfield));
-      make_cleanup (free, new);
+      make_cleanup (xfree, new);
       memset (new, 0, sizeof (struct nextfield));
-      new -> next = fip -> list;
-      fip -> list = new;
+      new->next = fip->list;
+      fip->list = new;
       FIELD_BITSIZE (new->field) = 0;  /* this should be an unpacked field! */
 
       STABS_CONTINUE (pp, objfile);
       switch (**pp)
        {
-         case '0':
-           /* Nothing to do. */
-           break;
-         case '1':
-           SET_TYPE_FIELD_VIRTUAL (type, i);
-           break;
-         default:
-           /* Unknown character.  Complain and treat it as non-virtual.  */
-           {
-             static struct complaint msg = {
-               "Unknown virtual character `%c' for baseclass", 0, 0};
-             complain (&msg, **pp);
-           }
+       case '0':
+         /* Nothing to do. */
+         break;
+       case '1':
+         SET_TYPE_FIELD_VIRTUAL (type, i);
+         break;
+       default:
+         /* Unknown character.  Complain and treat it as non-virtual.  */
+         {
+           complaint (&symfile_complaints,
+                      "Unknown virtual character `%c' for baseclass", **pp);
+         }
        }
       ++(*pp);
 
-      new -> visibility = *(*pp)++;
-      switch (new -> visibility)
+      new->visibility = *(*pp)++;
+      switch (new->visibility)
        {
-         case VISIBILITY_PRIVATE:
-         case VISIBILITY_PROTECTED:
-         case VISIBILITY_PUBLIC:
-           break;
-         default:
-           /* Bad visibility format.  Complain and treat it as
-              public.  */
-           {
-             static struct complaint msg = {
-               "Unknown visibility `%c' for baseclass", 0, 0};
-             complain (&msg, new -> visibility);
-             new -> visibility = VISIBILITY_PUBLIC;
-           }
+       case VISIBILITY_PRIVATE:
+       case VISIBILITY_PROTECTED:
+       case VISIBILITY_PUBLIC:
+         break;
+       default:
+         /* Bad visibility format.  Complain and treat it as
+            public.  */
+         {
+           complaint (&symfile_complaints,
+                      "Unknown visibility `%c' for baseclass",
+                      new->visibility);
+           new->visibility = VISIBILITY_PUBLIC;
+         }
        }
 
       {
        int nbits;
-       
+
        /* The remaining value is the bit offset of the portion of the object
           corresponding to this baseclass.  Always zero in the absence of
           multiple inheritance.  */
@@ -3643,11 +3917,11 @@ read_baseclasses (fip, pp, type, objfile)
       }
 
       /* The last piece of baseclass information is the type of the
-        base class.  Read it, and remember it's type name as this
-        field's name. */
+         base class.  Read it, and remember it's type name as this
+         field's name. */
 
-      new -> field.type = read_type (pp, objfile);
-      new -> field.name = type_name_no_tag (new -> field.type);
+      new->field.type = read_type (pp, objfile);
+      new->field.name = type_name_no_tag (new->field.type);
 
       /* skip trailing ';' and bump count of number of fields seen */
       if (**pp == ';')
@@ -3667,11 +3941,8 @@ read_baseclasses (fip, pp, type, objfile)
    so we can look for the vptr base class info.  */
 
 static int
-read_tilde_fields (fip, pp, type, objfile)
-     struct field_info *fip;
-     char **pp;
-     struct type *type;
-     struct objfile *objfile;
+read_tilde_fields (struct field_info *fip, char **pp, struct type *type,
+                  struct objfile *objfile)
 {
   register char *p;
 
@@ -3717,23 +3988,26 @@ read_tilde_fields (fip, pp, type, objfile)
              /* Premature end of symbol.  */
              return 0;
            }
-         
+
          TYPE_VPTR_BASETYPE (type) = t;
-         if (type == t)                /* Our own class provides vtbl ptr */
+         if (type == t)        /* Our own class provides vtbl ptr */
            {
              for (i = TYPE_NFIELDS (t) - 1;
                   i >= TYPE_N_BASECLASSES (t);
                   --i)
                {
-                 if (! strncmp (TYPE_FIELD_NAME (t, i), vptr_name, 
-                                sizeof (vptr_name) - 1))
+                 char *name = TYPE_FIELD_NAME (t, i);
+                 if (!strncmp (name, vptr_name, sizeof (vptr_name) - 2)
+                     && is_cplus_marker (name[sizeof (vptr_name) - 2]))
                    {
                      TYPE_VPTR_FIELDNO (type) = i;
                      goto gotit;
                    }
                }
              /* Virtual function table field not found.  */
-             complain (&vtbl_notfound_complaint, TYPE_NAME (type));
+             complaint (&symfile_complaints,
+                        "virtual function table pointer not found when defining class `%s'",
+                        TYPE_NAME (type));
              return 0;
            }
          else
@@ -3741,7 +4015,7 @@ read_tilde_fields (fip, pp, type, objfile)
              TYPE_VPTR_FIELDNO (type) = TYPE_VPTR_FIELDNO (t);
            }
 
-    gotit:
+       gotit:
          *pp = p + 1;
        }
     }
@@ -3749,168 +4023,159 @@ read_tilde_fields (fip, pp, type, objfile)
 }
 
 static int
-attach_fn_fields_to_type (fip, type)
-     struct field_info *fip;
-     register struct type *type;
+attach_fn_fields_to_type (struct field_info *fip, register struct type *type)
 {
   register int n;
 
   for (n = TYPE_NFN_FIELDS (type);
-       fip -> fnlist != NULL;
-       fip -> fnlist = fip -> fnlist -> next)
-    {
-      --n;                      /* Circumvent Sun3 compiler bug */
-      TYPE_FN_FIELDLISTS (type)[n] = fip -> fnlist -> fn_fieldlist;
-    }
-  return 1;
-}
-
-/* read cfront class static data.
-   pp points to string starting with the list of static data
-   eg: A:ZcA;1@Bpub v2@Bvirpri;__ct__1AFv func__1AFv *sfunc__1AFv ;as__1A ;;
-                                                                  ^^^^^^^^
-
-       A:ZcA;;foopri__1AFv foopro__1AFv __ct__1AFv __ct__1AFRC1A foopub__1AFv ;;;
-                                                                              ^
-   */
-
-static int
-read_cfront_static_fields (fip, pp, type, objfile)
-     struct field_info *fip;
-     char **pp;
-     struct type *type;
-     struct objfile *objfile;
-{
-  struct nextfield * new;
-  struct type *stype;
-  char * sname;
-  struct symbol * ref_static=0;
-      
-  if (**pp==';')               /* no static data; return */
+       fip->fnlist != NULL;
+       fip->fnlist = fip->fnlist->next)
     {
-      ++(*pp);
-      return 1;
+      --n;                     /* Circumvent Sun3 compiler bug */
+      TYPE_FN_FIELDLISTS (type)[n] = fip->fnlist->fn_fieldlist;
     }
-
-  /* Process each field in the list until we find the terminating ";" */
-
-  /* eg: p = "as__1A ;;;" */
-  STABS_CONTINUE (pp, objfile);                /* handle \\ */
-  while (**pp!=';' && (sname = get_substring (pp, ' '), sname)) 
-    {
-      ref_static = lookup_symbol (sname, 0, VAR_NAMESPACE, 0, 0); /*demangled_name*/
-      if (!ref_static) 
-        {
-          static struct complaint msg = {"\
-               Unable to find symbol for static data field %s\n",
-                                0, 0};
-         complain (&msg, sname);
-         continue;
-       }
-      stype = SYMBOL_TYPE(ref_static);
-
-      /* allocate a new fip */
-      new = (struct nextfield *) xmalloc (sizeof (struct nextfield));
-      make_cleanup (free, new);
-      memset (new, 0, sizeof (struct nextfield));
-      new -> next = fip -> list;
-      fip -> list = new;
-
-      /* set visibility */
-      /* FIXME! no way to tell visibility from stabs??? */
-      new -> visibility = VISIBILITY_PUBLIC;
-
-      /* set field info into fip */
-      fip -> list -> field.type = stype; 
-
-      /* set bitpos & bitsize */
-      SET_FIELD_PHYSNAME (fip->list->field, savestring (sname, strlen (sname)));
-
-      /* set name field */
-      /* The following is code to work around cfront generated stabs.
-         The stabs contains full mangled name for each field.
-         We try to demangle the name and extract the field name out of it.
-      */
-      if (ARM_DEMANGLING)
-        {
-          char *dem, *dem_p;
-          dem = cplus_demangle (sname, DMGL_ANSI | DMGL_PARAMS);
-          if (dem != NULL)
-            {
-              dem_p = strrchr (dem, ':');
-              if (dem_p != 0 && *(dem_p-1)==':')
-                dem_p++;
-              fip->list->field.name =
-                obsavestring (dem_p, strlen (dem_p), &objfile -> type_obstack);
-            }
-          else
-            {
-              fip->list->field.name =
-                obsavestring (sname, strlen (sname), &objfile -> type_obstack); 
-           }
-        } /* end of code for cfront work around */ 
-    } /* loop again for next static field */
   return 1;
 }
 
-/* Copy structure fields to fip so attach_fields_to_type will work.
-   type has already been created with the initial instance data fields.
-   Now we want to be able to add the other members to the class,
-   so we want to add them back to the fip and reattach them again
-   once we have collected all the class members. */
-
-static int
-copy_cfront_struct_fields (fip, type, objfile)
-     struct field_info *fip;
-     struct type *type;
-     struct objfile *objfile;
-{
-  int nfields = TYPE_NFIELDS(type);
-  int i;
-  struct nextfield * new;
-
-  /* Copy the fields into the list of fips and reset the types 
-     to remove the old fields */
-
-  for (i=0; i<nfields; i++)
-    {
-      /* allocate a new fip */
-      new = (struct nextfield *) xmalloc (sizeof (struct nextfield));
-      make_cleanup (free, new);
-      memset (new, 0, sizeof (struct nextfield));
-      new -> next = fip -> list;
-      fip -> list = new;
-
-      /* copy field info into fip */
-      new -> field = TYPE_FIELD (type, i);
-      /* set visibility */
-      if (TYPE_FIELD_PROTECTED (type, i))
-       new -> visibility = VISIBILITY_PROTECTED;
-      else if (TYPE_FIELD_PRIVATE (type, i))
-        new -> visibility = VISIBILITY_PRIVATE;
-      else
-        new -> visibility = VISIBILITY_PUBLIC;
-    }
-  /* Now delete the fields from the type since we will be 
-     allocing new space once we get the rest of the fields 
-     in attach_fields_to_type.
-     The pointer TYPE_FIELDS(type) is left dangling but should 
-     be freed later by objstack_free */
-  TYPE_FIELDS (type)=0;
-  TYPE_NFIELDS (type) = 0;
-
-  return 1;
-}
+#if 0 /* OBSOLETE CFront */
+// OBSOLETE  /* read cfront class static data.
+// OBSOLETE     pp points to string starting with the list of static data
+// OBSOLETE     eg: A:ZcA;1@Bpub v2@Bvirpri;__ct__1AFv func__1AFv *sfunc__1AFv ;as__1A ;;
+// OBSOLETE     ^^^^^^^^
+
+// OBSOLETE     A:ZcA;;foopri__1AFv foopro__1AFv __ct__1AFv __ct__1AFRC1A foopub__1AFv ;;;
+// OBSOLETE     ^
+// OBSOLETE   */
+
+// OBSOLETE  static int
+// OBSOLETE  read_cfront_static_fields (struct field_info *fip, char **pp, struct type *type,
+// OBSOLETE                       struct objfile *objfile)
+// OBSOLETE  {
+// OBSOLETE    struct nextfield *new;
+// OBSOLETE    struct type *stype;
+// OBSOLETE    char *sname;
+// OBSOLETE    struct symbol *ref_static = 0;
+
+// OBSOLETE    if (**pp == ';')                /* no static data; return */
+// OBSOLETE      {
+// OBSOLETE        ++(*pp);
+// OBSOLETE        return 1;
+// OBSOLETE      }
+
+// OBSOLETE    /* Process each field in the list until we find the terminating ";" */
+
+// OBSOLETE    /* eg: p = "as__1A ;;;" */
+// OBSOLETE    STABS_CONTINUE (pp, objfile);   /* handle \\ */
+// OBSOLETE    while (**pp != ';' && (sname = get_substring (pp, ' '), sname))
+// OBSOLETE      {
+// OBSOLETE        ref_static = lookup_symbol (sname, 0, VAR_NAMESPACE, 0, 0); /*demangled_name */
+// OBSOLETE        if (!ref_static)
+// OBSOLETE    {
+// OBSOLETE      complaint (&symfile_complaints,
+// OBSOLETE                 "Unable to find symbol for static data field %s", sname);
+// OBSOLETE      continue;
+// OBSOLETE    }
+// OBSOLETE        stype = SYMBOL_TYPE (ref_static);
+
+// OBSOLETE        /* allocate a new fip */
+// OBSOLETE        new = (struct nextfield *) xmalloc (sizeof (struct nextfield));
+// OBSOLETE        make_cleanup (xfree, new);
+// OBSOLETE        memset (new, 0, sizeof (struct nextfield));
+// OBSOLETE        new->next = fip->list;
+// OBSOLETE        fip->list = new;
+
+// OBSOLETE        /* set visibility */
+// OBSOLETE        /* FIXME! no way to tell visibility from stabs??? */
+// OBSOLETE        new->visibility = VISIBILITY_PUBLIC;
+
+// OBSOLETE        /* set field info into fip */
+// OBSOLETE        fip->list->field.type = stype;
+
+// OBSOLETE        /* set bitpos & bitsize */
+// OBSOLETE        SET_FIELD_PHYSNAME (fip->list->field, savestring (sname, strlen (sname)));
+
+// OBSOLETE        /* set name field */
+// OBSOLETE        /* The following is code to work around cfront generated stabs.
+// OBSOLETE           The stabs contains full mangled name for each field.
+// OBSOLETE           We try to demangle the name and extract the field name out of it.
+// OBSOLETE         */
+// OBSOLETE        if (ARM_DEMANGLING)
+// OBSOLETE    {
+// OBSOLETE      char *dem, *dem_p;
+// OBSOLETE      dem = cplus_demangle (sname, DMGL_ANSI | DMGL_PARAMS);
+// OBSOLETE      if (dem != NULL)
+// OBSOLETE        {
+// OBSOLETE          dem_p = strrchr (dem, ':');
+// OBSOLETE          if (dem_p != 0 && *(dem_p - 1) == ':')
+// OBSOLETE            dem_p++;
+// OBSOLETE          fip->list->field.name =
+// OBSOLETE            obsavestring (dem_p, strlen (dem_p), &objfile->type_obstack);
+// OBSOLETE        }
+// OBSOLETE      else
+// OBSOLETE        {
+// OBSOLETE          fip->list->field.name =
+// OBSOLETE            obsavestring (sname, strlen (sname), &objfile->type_obstack);
+// OBSOLETE        }
+// OBSOLETE    }                       /* end of code for cfront work around */
+// OBSOLETE      }                             /* loop again for next static field */
+// OBSOLETE    return 1;
+// OBSOLETE  }
+
+// OBSOLETE  /* Copy structure fields to fip so attach_fields_to_type will work.
+// OBSOLETE     type has already been created with the initial instance data fields.
+// OBSOLETE     Now we want to be able to add the other members to the class,
+// OBSOLETE     so we want to add them back to the fip and reattach them again
+// OBSOLETE     once we have collected all the class members. */
+
+// OBSOLETE  static int
+// OBSOLETE  copy_cfront_struct_fields (struct field_info *fip, struct type *type,
+// OBSOLETE                       struct objfile *objfile)
+// OBSOLETE  {
+// OBSOLETE    int nfields = TYPE_NFIELDS (type);
+// OBSOLETE    int i;
+// OBSOLETE    struct nextfield *new;
+
+// OBSOLETE    /* Copy the fields into the list of fips and reset the types 
+// OBSOLETE       to remove the old fields */
+
+// OBSOLETE    for (i = 0; i < nfields; i++)
+// OBSOLETE      {
+// OBSOLETE        /* allocate a new fip */
+// OBSOLETE        new = (struct nextfield *) xmalloc (sizeof (struct nextfield));
+// OBSOLETE        make_cleanup (xfree, new);
+// OBSOLETE        memset (new, 0, sizeof (struct nextfield));
+// OBSOLETE        new->next = fip->list;
+// OBSOLETE        fip->list = new;
+
+// OBSOLETE        /* copy field info into fip */
+// OBSOLETE        new->field = TYPE_FIELD (type, i);
+// OBSOLETE        /* set visibility */
+// OBSOLETE        if (TYPE_FIELD_PROTECTED (type, i))
+// OBSOLETE    new->visibility = VISIBILITY_PROTECTED;
+// OBSOLETE        else if (TYPE_FIELD_PRIVATE (type, i))
+// OBSOLETE    new->visibility = VISIBILITY_PRIVATE;
+// OBSOLETE        else
+// OBSOLETE    new->visibility = VISIBILITY_PUBLIC;
+// OBSOLETE      }
+// OBSOLETE    /* Now delete the fields from the type since we will be 
+// OBSOLETE       allocing new space once we get the rest of the fields 
+// OBSOLETE       in attach_fields_to_type.
+// OBSOLETE       The pointer TYPE_FIELDS(type) is left dangling but should 
+// OBSOLETE       be freed later by objstack_free */
+// OBSOLETE    TYPE_FIELDS (type) = 0;
+// OBSOLETE    TYPE_NFIELDS (type) = 0;
+
+// OBSOLETE    return 1;
+// OBSOLETE  }
+#endif /* OBSOLETE CFront */
 
 /* Create the vector of fields, and record how big it is.
    We need this info to record proper virtual function table information
    for this class's virtual functions.  */
 
 static int
-attach_fields_to_type (fip, type, objfile)
-     struct field_info *fip;
-     register struct type *type;
-     struct objfile *objfile;
+attach_fields_to_type (struct field_info *fip, register struct type *type,
+                      struct objfile *objfile)
 {
   register int nfields = 0;
   register int non_public_fields = 0;
@@ -3921,10 +4186,10 @@ attach_fields_to_type (fip, type, objfile)
      allocate and build the private_field_bits and protected_field_bits
      bitfields. */
 
-  for (scan = fip -> list; scan != NULL; scan = scan -> next)
+  for (scan = fip->list; scan != NULL; scan = scan->next)
     {
       nfields++;
-      if (scan -> visibility != VISIBILITY_PUBLIC)
+      if (scan->visibility != VISIBILITY_PUBLIC)
        {
          non_public_fields++;
        }
@@ -3962,38 +4227,73 @@ attach_fields_to_type (fip, type, objfile)
 
   while (nfields-- > 0)
     {
-      TYPE_FIELD (type, nfields) = fip -> list -> field;
-      switch (fip -> list -> visibility)
+      TYPE_FIELD (type, nfields) = fip->list->field;
+      switch (fip->list->visibility)
        {
-         case VISIBILITY_PRIVATE:
-           SET_TYPE_FIELD_PRIVATE (type, nfields);
-           break;
+       case VISIBILITY_PRIVATE:
+         SET_TYPE_FIELD_PRIVATE (type, nfields);
+         break;
 
-         case VISIBILITY_PROTECTED:
-           SET_TYPE_FIELD_PROTECTED (type, nfields);
-           break;
+       case VISIBILITY_PROTECTED:
+         SET_TYPE_FIELD_PROTECTED (type, nfields);
+         break;
 
-         case VISIBILITY_IGNORE:
-           SET_TYPE_FIELD_IGNORE (type, nfields);
-           break;
+       case VISIBILITY_IGNORE:
+         SET_TYPE_FIELD_IGNORE (type, nfields);
+         break;
 
-         case VISIBILITY_PUBLIC:
-           break;
+       case VISIBILITY_PUBLIC:
+         break;
 
-         default:
-           /* Unknown visibility.  Complain and treat it as public.  */
-           {
-             static struct complaint msg = {
-               "Unknown visibility `%c' for field", 0, 0};
-             complain (&msg, fip -> list -> visibility);
-           }
-           break;
+       default:
+         /* Unknown visibility.  Complain and treat it as public.  */
+         {
+           complaint (&symfile_complaints, "Unknown visibility `%c' for field",
+                      fip->list->visibility);
+         }
+         break;
        }
-      fip -> list = fip -> list -> next;
+      fip->list = fip->list->next;
     }
   return 1;
 }
 
+
+/* Complain that the compiler has emitted more than one definition for the
+   structure type TYPE.  */
+static void 
+complain_about_struct_wipeout (struct type *type)
+{
+  char *name = "";
+  char *kind = "";
+
+  if (TYPE_TAG_NAME (type))
+    {
+      name = TYPE_TAG_NAME (type);
+      switch (TYPE_CODE (type))
+        {
+        case TYPE_CODE_STRUCT: kind = "struct "; break;
+        case TYPE_CODE_UNION:  kind = "union ";  break;
+        case TYPE_CODE_ENUM:   kind = "enum ";   break;
+        default: kind = "";
+        }
+    }
+  else if (TYPE_NAME (type))
+    {
+      name = TYPE_NAME (type);
+      kind = "";
+    }
+  else
+    {
+      name = "<unknown>";
+      kind = "";
+    }
+
+  complaint (&symfile_complaints,
+            "struct/union type gets multiply defined: %s%s", kind, name);
+}
+
+
 /* Read the description of a structure (or union type) and return an object
    describing the type.
 
@@ -4006,13 +4306,11 @@ attach_fields_to_type (fip, type, objfile)
    OBJFILE points to the current objfile from which the stabs information is
    being read.  (Note that it is redundant in that TYPE also contains a pointer
    to this same objfile, so it might be a good idea to eliminate it.  FIXME). 
  */
+ */
 
 static struct type *
-read_struct_type (pp, type, objfile)
-     char **pp;
-     struct type *type;
-     struct objfile *objfile;
+read_struct_type (char **pp, struct type *type, enum type_code type_code,
+                  struct objfile *objfile)
 {
   struct cleanup *back_to;
   struct field_info fi;
@@ -4020,9 +4318,30 @@ read_struct_type (pp, type, objfile)
   fi.list = NULL;
   fi.fnlist = NULL;
 
+  /* When describing struct/union/class types in stabs, G++ always drops
+     all qualifications from the name.  So if you've got:
+       struct A { ... struct B { ... }; ... };
+     then G++ will emit stabs for `struct A::B' that call it simply
+     `struct B'.  Obviously, if you've got a real top-level definition for
+     `struct B', or other nested definitions, this is going to cause
+     problems.
+
+     Obviously, GDB can't fix this by itself, but it can at least avoid
+     scribbling on existing structure type objects when new definitions
+     appear.  */
+  if (! (TYPE_CODE (type) == TYPE_CODE_UNDEF
+         || TYPE_STUB (type)))
+    {
+      complain_about_struct_wipeout (type);
+
+      /* It's probably best to return the type unchanged.  */
+      return type;
+    }
+
   back_to = make_cleanup (null_cleanup, 0);
 
   INIT_CPLUS_SPECIFIC (type);
+  TYPE_CODE (type) = type_code;
   TYPE_FLAGS (type) &= ~TYPE_FLAG_STUB;
 
   /* First comes the total size in bytes.  */
@@ -4059,10 +4378,8 @@ read_struct_type (pp, type, objfile)
    array.  */
 
 static struct type *
-read_array_type (pp, type, objfile)
-     register char **pp;
-     register struct type *type;
-     struct objfile *objfile;
+read_array_type (register char **pp, register struct type *type,
+                struct objfile *objfile)
 {
   struct type *index_type, *element_type, *range_type;
   int lower, upper;
@@ -4076,9 +4393,6 @@ read_array_type (pp, type, objfile)
      Fortran adjustable arrays use Adigits or Tdigits for lower or upper;
      for these, produce a type like float[][].  */
 
-  if (os9k_stabs)
-    index_type = builtin_type_int;
-  else
     {
       index_type = read_type (pp, objfile);
       if (**pp != ';')
@@ -4092,7 +4406,8 @@ read_array_type (pp, type, objfile)
       (*pp)++;
       adjustable = 1;
     }
-  lower = read_huge_number (pp, os9k_stabs ? ',' : ';', &nbits);
+  lower = read_huge_number (pp, ';', &nbits);
+
   if (nbits != 0)
     return error_type (pp, objfile);
 
@@ -4104,7 +4419,7 @@ read_array_type (pp, type, objfile)
   upper = read_huge_number (pp, ';', &nbits);
   if (nbits != 0)
     return error_type (pp, objfile);
-  
+
   element_type = read_type (pp, objfile);
 
   if (adjustable)
@@ -4126,10 +4441,8 @@ read_array_type (pp, type, objfile)
    Also defines the symbols that represent the values of the type.  */
 
 static struct type *
-read_enum_type (pp, type, objfile)
-     register char **pp;
-     register struct type *type;
-     struct objfile *objfile;
+read_enum_type (register char **pp, register struct type *type,
+               struct objfile *objfile)
 {
   register char *p;
   char *name;
@@ -4154,23 +4467,13 @@ read_enum_type (pp, type, objfile)
   osyms = *symlist;
   o_nsyms = osyms ? osyms->nsyms : 0;
 
-  if (os9k_stabs)
-    {
-      /* Size.  Perhaps this does not have to be conditionalized on
-        os9k_stabs (assuming the name of an enum constant can't start
-        with a digit).  */
-      read_huge_number (pp, 0, &nbits);
-      if (nbits != 0)
-       return error_type (pp, objfile);
-    }
-
   /* The aix4 compiler emits an extra field before the enum members;
      my guess is it's a type of some sort.  Just ignore it.  */
   if (**pp == '-')
     {
       /* Skip over the type.  */
       while (**pp != ':')
-        (*pp)++;
+       (*pp)++;
 
       /* Skip over the colon.  */
       (*pp)++;
@@ -4183,18 +4486,19 @@ read_enum_type (pp, type, objfile)
     {
       STABS_CONTINUE (pp, objfile);
       p = *pp;
-      while (*p != ':') p++;
-      name = obsavestring (*pp, p - *pp, &objfile -> symbol_obstack);
+      while (*p != ':')
+       p++;
+      name = obsavestring (*pp, p - *pp, &objfile->symbol_obstack);
       *pp = p + 1;
       n = read_huge_number (pp, ',', &nbits);
       if (nbits != 0)
        return error_type (pp, objfile);
 
       sym = (struct symbol *)
-       obstack_alloc (&objfile -> symbol_obstack, sizeof (struct symbol));
+       obstack_alloc (&objfile->symbol_obstack, sizeof (struct symbol));
       memset (sym, 0, sizeof (struct symbol));
-      SYMBOL_NAME (sym) = name;
-      SYMBOL_LANGUAGE (sym) = current_subfile -> language;
+      DEPRECATED_SYMBOL_NAME (sym) = name;
+      SYMBOL_LANGUAGE (sym) = current_subfile->language;
       SYMBOL_CLASS (sym) = LOC_CONST;
       SYMBOL_NAMESPACE (sym) = VAR_NAMESPACE;
       SYMBOL_VALUE (sym) = n;
@@ -4235,7 +4539,7 @@ read_enum_type (pp, type, objfile)
        {
          struct symbol *xsym = syms->symbol[j];
          SYMBOL_TYPE (xsym) = type;
-         TYPE_FIELD_NAME (type, n) = SYMBOL_NAME (xsym);
+         TYPE_FIELD_NAME (type, n) = DEPRECATED_SYMBOL_NAME (xsym);
          TYPE_FIELD_BITPOS (type, n) = SYMBOL_VALUE (xsym);
          TYPE_FIELD_BITSIZE (type, n) = 0;
        }
@@ -4249,45 +4553,50 @@ read_enum_type (pp, type, objfile)
 /* Sun's ACC uses a somewhat saner method for specifying the builtin
    typedefs in every file (for int, long, etc):
 
-       type = b <signed> <width>; <offset>; <nbits>
-       signed = u or s.  Possible c in addition to u or s (for char?).
-       offset = offset from high order bit to start bit of type.
-       width is # bytes in object of this type, nbits is # bits in type.
+   type = b <signed> <width> <format type>; <offset>; <nbits>
+   signed = u or s.
+   optional format type = c or b for char or boolean.
+   offset = offset from high order bit to start bit of type.
+   width is # bytes in object of this type, nbits is # bits in type.
 
    The width/offset stuff appears to be for small objects stored in
    larger ones (e.g. `shorts' in `int' registers).  We ignore it for now,
    FIXME.  */
 
 static struct type *
-read_sun_builtin_type (pp, typenums, objfile)
-     char **pp;
-     int typenums[2];
-     struct objfile *objfile;
+read_sun_builtin_type (char **pp, int typenums[2], struct objfile *objfile)
 {
   int type_bits;
   int nbits;
   int signed_type;
+  enum type_code code = TYPE_CODE_INT;
 
   switch (**pp)
     {
-      case 's':
-        signed_type = 1;
-       break;
-      case 'u':
-       signed_type = 0;
-       break;
-      default:
-       return error_type (pp, objfile);
+    case 's':
+      signed_type = 1;
+      break;
+    case 'u':
+      signed_type = 0;
+      break;
+    default:
+      return error_type (pp, objfile);
     }
   (*pp)++;
 
   /* For some odd reason, all forms of char put a c here.  This is strange
      because no other type has this honor.  We can safely ignore this because
      we actually determine 'char'acterness by the number of bits specified in
-     the descriptor.  */
+     the descriptor.
+     Boolean forms, e.g Fortran logical*X, put a b here.  */
 
   if (**pp == 'c')
     (*pp)++;
+  else if (**pp == 'b')
+    {
+      code = TYPE_CODE_BOOL;
+      (*pp)++;
+    }
 
   /* The first number appears to be the number of bytes occupied
      by this type, except that unsigned short is 4 instead of 2.
@@ -4317,24 +4626,22 @@ read_sun_builtin_type (pp, typenums, objfile)
 
   if (type_bits == 0)
     return init_type (TYPE_CODE_VOID, 1,
-                     signed_type ? 0 : TYPE_FLAG_UNSIGNED, (char *)NULL,
+                     signed_type ? 0 : TYPE_FLAG_UNSIGNED, (char *) NULL,
                      objfile);
   else
-    return init_type (TYPE_CODE_INT,
+    return init_type (code,
                      type_bits / TARGET_CHAR_BIT,
-                     signed_type ? 0 : TYPE_FLAG_UNSIGNED, (char *)NULL,
+                     signed_type ? 0 : TYPE_FLAG_UNSIGNED, (char *) NULL,
                      objfile);
 }
 
 static struct type *
-read_sun_floating_type (pp, typenums, objfile)
-     char **pp;
-     int typenums[2];
-     struct objfile *objfile;
+read_sun_floating_type (char **pp, int typenums[2], struct objfile *objfile)
 {
   int nbits;
   int details;
   int nbytes;
+  struct type *rettype;
 
   /* The first number has more details about the type, for example
      FN_COMPLEX.  */
@@ -4349,9 +4656,12 @@ read_sun_floating_type (pp, typenums, objfile)
 
   if (details == NF_COMPLEX || details == NF_COMPLEX16
       || details == NF_COMPLEX32)
-    /* This is a type we can't handle, but we do know the size.
-       We also will be able to give it a name.  */
-    return init_type (TYPE_CODE_ERROR, nbytes, 0, NULL, objfile);
+    {
+      rettype = init_type (TYPE_CODE_COMPLEX, nbytes, 0, NULL, objfile);
+      TYPE_TARGET_TYPE (rettype)
+       = init_type (TYPE_CODE_FLT, nbytes / 2, 0, NULL, objfile);
+      return rettype;
+    }
 
   return init_type (TYPE_CODE_FLT, nbytes, 0, NULL, objfile);
 }
@@ -4369,10 +4679,7 @@ read_sun_floating_type (pp, typenums, objfile)
    If encounter garbage, set *BITS to -1 and return 0.  */
 
 static long
-read_huge_number (pp, end, bits)
-     char **pp;
-     int end;
-     int *bits;
+read_huge_number (char **pp, int end, int *bits)
 {
   char *p = *pp;
   int sign = 1;
@@ -4382,7 +4689,7 @@ read_huge_number (pp, end, bits)
   int nbits = 0;
   int c;
   long upper_limit;
-  
+
   if (*p == '-')
     {
       sign = -1;
@@ -4397,10 +4704,7 @@ read_huge_number (pp, end, bits)
       p++;
     }
 
-  if (os9k_stabs)
-    upper_limit = ULONG_MAX / radix;
-  else
-    upper_limit = LONG_MAX / radix;
+  upper_limit = LONG_MAX / radix;
 
   while ((c = *p++) >= '0' && c < ('0' + radix))
     {
@@ -4411,9 +4715,9 @@ read_huge_number (pp, end, bits)
        }
       else
        overflow = 1;
-      
+
       /* This depends on large values being output in octal, which is
-        what GCC does. */
+         what GCC does. */
       if (radix == 8)
        {
          if (nbits == 0)
@@ -4455,9 +4759,9 @@ read_huge_number (pp, end, bits)
            *bits = -1;
          return 0;
        }
-      
+
       /* -0x7f is the same as 0x80.  So deal with it by adding one to
-        the number of bits.  */
+         the number of bits.  */
       if (sign == -1)
        ++nbits;
       if (bits)
@@ -4474,10 +4778,7 @@ read_huge_number (pp, end, bits)
 }
 
 static struct type *
-read_range_type (pp, typenums, objfile)
-     char **pp;
-     int typenums[2];
-     struct objfile *objfile;
+read_range_type (char **pp, int typenums[2], struct objfile *objfile)
 {
   char *orig_pp = *pp;
   int rangenums[2];
@@ -4530,8 +4831,8 @@ read_range_type (pp, typenums, objfile)
          nbits = n3bits;
        }
       /* Range from <large number> to <large number>-1 is a large signed
-        integral type.  Take care of the case where <large number> doesn't
-        fit in a long but <large number>-1 does.  */
+         integral type.  Take care of the case where <large number> doesn't
+         fit in a long but <large number>-1 does.  */
       else if ((n2bits != 0 && n3bits != 0 && n2bits == n3bits + 1)
               || (n2bits != 0 && n3bits == 0
                   && (n2bits == sizeof (long) * HOST_CHAR_BIT)
@@ -4555,20 +4856,32 @@ read_range_type (pp, typenums, objfile)
   if (self_subrange && n2 == 0 && n3 == 0)
     return init_type (TYPE_CODE_VOID, 1, 0, NULL, objfile);
 
-  /* If n3 is zero and n2 is positive, we want a floating type,
-     and n2 is the width in bytes.
-
-     Fortran programs appear to use this for complex types also,
-     and they give no way to distinguish between double and single-complex!
+  /* If n3 is zero and n2 is positive, we want a floating type, and n2
+     is the width in bytes.
 
-     GDB does not have complex types.
+     Fortran programs appear to use this for complex types also.  To
+     distinguish between floats and complex, g77 (and others?)  seem
+     to use self-subranges for the complexes, and subranges of int for
+     the floats.
 
-     Just return the complex as a float of that size.  It won't work right
-     for the complex values, but at least it makes the file loadable.  */
+     Also note that for complexes, g77 sets n2 to the size of one of
+     the member floats, not the whole complex beast.  My guess is that
+     this was to work well with pre-COMPLEX versions of gdb. */
 
   if (n3 == 0 && n2 > 0)
     {
-      return init_type (TYPE_CODE_FLT, n2, 0, NULL, objfile);
+      struct type *float_type
+       = init_type (TYPE_CODE_FLT, n2, 0, NULL, objfile);
+
+      if (self_subrange)
+       {
+         struct type *complex_type = 
+           init_type (TYPE_CODE_COMPLEX, 2 * n2, 0, NULL, objfile);
+         TYPE_TARGET_TYPE (complex_type) = float_type;
+         return complex_type;
+       }
+      else
+       return float_type;
     }
 
   /* If the upper bound is -1, it must really be an unsigned int.  */
@@ -4577,7 +4890,7 @@ read_range_type (pp, typenums, objfile)
     {
       /* It is unsigned int or unsigned long.  */
       /* GCC 2.3.3 uses this for long long too, but that is just a GDB 3.5
-        compatibility hack.  */
+         compatibility hack.  */
       return init_type (TYPE_CODE_INT, TARGET_INT_BIT / TARGET_CHAR_BIT,
                        TYPE_FLAG_UNSIGNED, NULL, objfile);
     }
@@ -4585,36 +4898,44 @@ read_range_type (pp, typenums, objfile)
   /* Special case: char is defined (Who knows why) as a subrange of
      itself with range 0-127.  */
   else if (self_subrange && n2 == 0 && n3 == 127)
-    return init_type (TYPE_CODE_INT, 1, 0, NULL, objfile);
-
-  else if (current_symbol && SYMBOL_LANGUAGE (current_symbol) == language_chill
-          && !self_subrange)
-    goto handle_true_range;
+    return init_type (TYPE_CODE_INT, 1, TYPE_FLAG_NOSIGN, NULL, objfile);
 
   /* We used to do this only for subrange of self or subrange of int.  */
   else if (n2 == 0)
     {
+      /* -1 is used for the upper bound of (4 byte) "unsigned int" and
+         "unsigned long", and we already checked for that,
+         so don't need to test for it here.  */
+
       if (n3 < 0)
        /* n3 actually gives the size.  */
-       return init_type (TYPE_CODE_INT, - n3, TYPE_FLAG_UNSIGNED,
+       return init_type (TYPE_CODE_INT, -n3, TYPE_FLAG_UNSIGNED,
                          NULL, objfile);
-      if (n3 == 0xff)
-       return init_type (TYPE_CODE_INT, 1, TYPE_FLAG_UNSIGNED, NULL, objfile);
-      if (n3 == 0xffff)
-       return init_type (TYPE_CODE_INT, 2, TYPE_FLAG_UNSIGNED, NULL, objfile);
 
-      /* -1 is used for the upper bound of (4 byte) "unsigned int" and
-        "unsigned long", and we already checked for that,
-        so don't need to test for it here.  */
+      /* Is n3 == 2**(8n)-1 for some integer n?  Then it's an
+         unsigned n-byte integer.  But do require n to be a power of
+         two; we don't want 3- and 5-byte integers flying around.  */
+      {
+       int bytes;
+       unsigned long bits;
+
+       bits = n3;
+       for (bytes = 0; (bits & 0xff) == 0xff; bytes++)
+         bits >>= 8;
+       if (bits == 0
+           && ((bytes - 1) & bytes) == 0) /* "bytes is a power of two" */
+         return init_type (TYPE_CODE_INT, bytes, TYPE_FLAG_UNSIGNED, NULL,
+                           objfile);
+      }
     }
   /* I think this is for Convex "long long".  Since I don't know whether
      Convex sets self_subrange, I also accept that particular size regardless
      of self_subrange.  */
   else if (n3 == 0 && n2 < 0
           && (self_subrange
-              || n2 == - TARGET_LONG_LONG_BIT / TARGET_CHAR_BIT))
-    return init_type (TYPE_CODE_INT, - n2, 0, NULL, objfile);
-  else if (n2 == -n3 -1)
+              || n2 == -TARGET_LONG_LONG_BIT / TARGET_CHAR_BIT))
+    return init_type (TYPE_CODE_INT, -n2, 0, NULL, objfile);
+  else if (n2 == -n3 - 1)
     {
       if (n3 == 0x7f)
        return init_type (TYPE_CODE_INT, 1, 0, NULL, objfile);
@@ -4626,7 +4947,7 @@ read_range_type (pp, typenums, objfile)
 
   /* We have a real range type on our hands.  Allocate space and
      return a real pointer.  */
- handle_true_range:
+handle_true_range:
 
   if (self_subrange)
     index_type = builtin_type_int;
@@ -4639,7 +4960,8 @@ read_range_type (pp, typenums, objfile)
 
       static struct type *range_type_index;
 
-      complain (&range_type_base_complaint, rangenums[1]);
+      complaint (&symfile_complaints,
+                "base type %d of range type is not defined", rangenums[1]);
       if (range_type_index == NULL)
        range_type_index =
          init_type (TYPE_CODE_INT, TARGET_INT_BIT / TARGET_CHAR_BIT,
@@ -4655,41 +4977,39 @@ read_range_type (pp, typenums, objfile)
    and terminated with END.  Return the list of types read in, or (struct type
    **)-1 if there is an error.  */
 
-static struct type **
-read_args (pp, end, objfile)
-     char **pp;
-     int end;
-     struct objfile *objfile;
+static struct field *
+read_args (char **pp, int end, struct objfile *objfile, int *nargsp,
+          int *varargsp)
 {
   /* FIXME!  Remove this arbitrary limit!  */
-  struct type *types[1024], **rval; /* allow for fns of 1023 parameters */
-  int n = 0;
+  struct type *types[1024];    /* allow for fns of 1023 parameters */
+  int n = 0, i;
+  struct field *rval;
 
   while (**pp != end)
     {
       if (**pp != ',')
        /* Invalid argument list: no ','.  */
-       return (struct type **)-1;
+       return (struct field *) -1;
       (*pp)++;
       STABS_CONTINUE (pp, objfile);
       types[n++] = read_type (pp, objfile);
     }
   (*pp)++;                     /* get past `end' (the ':' character) */
 
-  if (n == 1)
-    {
-      rval = (struct type **) xmalloc (2 * sizeof (struct type *));
-    }
-  else if (TYPE_CODE (types[n-1]) != TYPE_CODE_VOID)
-    {
-      rval = (struct type **) xmalloc ((n + 1) * sizeof (struct type *));
-      memset (rval + n, 0, sizeof (struct type *));
-    }
+  if (TYPE_CODE (types[n - 1]) != TYPE_CODE_VOID)
+    *varargsp = 1;
   else
     {
-      rval = (struct type **) xmalloc (n * sizeof (struct type *));
+      n--;
+      *varargsp = 0;
     }
-  memcpy (rval, types, n * sizeof (struct type *));
+
+  rval = (struct field *) xmalloc (n * sizeof (struct field));
+  memset (rval, 0, n * sizeof (struct field));
+  for (i = 0; i < n; i++)
+    rval[i].type = types[i];
+  *nargsp = n;
   return rval;
 }
 \f
@@ -4713,28 +5033,23 @@ static char *common_block_name;
    to remain after this function returns.  */
 
 void
-common_block_start (name, objfile)
-     char *name;
-     struct objfile *objfile;
+common_block_start (char *name, struct objfile *objfile)
 {
   if (common_block_name != NULL)
     {
-      static struct complaint msg = {
-       "Invalid symbol data: common block within common block",
-       0, 0};
-      complain (&msg);
+      complaint (&symfile_complaints,
+                "Invalid symbol data: common block within common block");
     }
   common_block = local_symbols;
   common_block_i = local_symbols ? local_symbols->nsyms : 0;
   common_block_name = obsavestring (name, strlen (name),
-                                   &objfile -> symbol_obstack);
+                                   &objfile->symbol_obstack);
 }
 
 /* Process a N_ECOMM symbol.  */
 
 void
-common_block_end (objfile)
-     struct objfile *objfile;
+common_block_end (struct objfile *objfile)
 {
   /* Symbols declared since the BCOMM are to have the common block
      start address added in when we know it.  common_block and
@@ -4749,16 +5064,15 @@ common_block_end (objfile)
 
   if (common_block_name == NULL)
     {
-      static struct complaint msg = {"ECOMM symbol unmatched by BCOMM", 0, 0};
-      complain (&msg);
+      complaint (&symfile_complaints, "ECOMM symbol unmatched by BCOMM");
       return;
     }
 
-  sym = (struct symbol *) 
-    obstack_alloc (&objfile -> symbol_obstack, sizeof (struct symbol));
+  sym = (struct symbol *)
+    obstack_alloc (&objfile->symbol_obstack, sizeof (struct symbol));
   memset (sym, 0, sizeof (struct symbol));
   /* Note: common_block_name already saved on symbol_obstack */
-  SYMBOL_NAME (sym) = common_block_name;
+  DEPRECATED_SYMBOL_NAME (sym) = common_block_name;
   SYMBOL_CLASS (sym) = LOC_BLOCK;
 
   /* Now we copy all the symbols which have been defined since the BCOMM.  */
@@ -4785,7 +5099,7 @@ common_block_end (objfile)
   /* Should we be putting local_symbols back to what it was?
      Does it matter?  */
 
-  i = hashname (SYMBOL_NAME (sym));
+  i = hashname (DEPRECATED_SYMBOL_NAME (sym));
   SYMBOL_VALUE_CHAIN (sym) = global_sym_chain[i];
   global_sym_chain[i] = sym;
   common_block_name = NULL;
@@ -4796,29 +5110,26 @@ common_block_end (objfile)
    the common block name).  */
 
 static void
-fix_common_block (sym, valu)
-    struct symbol *sym;
-    int valu;
+fix_common_block (struct symbol *sym, int valu)
 {
   struct pending *next = (struct pending *) SYMBOL_TYPE (sym);
-  for ( ; next; next = next->next)
+  for (; next; next = next->next)
     {
       register int j;
       for (j = next->nsyms - 1; j >= 0; j--)
        SYMBOL_VALUE_ADDRESS (next->symbol[j]) += valu;
     }
 }
+\f
 
 
-\f
 /* What about types defined as forward references inside of a small lexical
    scope?  */
 /* Add a type to the list of undefined types to be checked through
    once this file has been read in.  */
 
 void
-add_undefined_type (type)
-     struct type *type;
+add_undefined_type (struct type *type)
 {
   if (undef_types_length == undef_types_allocated)
     {
@@ -4834,13 +5145,13 @@ add_undefined_type (type)
    up if possible.  We have two kinds of undefined types:
 
    TYPE_CODE_ARRAY:  Array whose target type wasn't defined yet.
-                       Fix:  update array length using the element bounds
-                       and the target type's length.
+   Fix:  update array length using the element bounds
+   and the target type's length.
    TYPE_CODE_STRUCT, TYPE_CODE_UNION:  Structure whose fields were not
-                       yet defined at the time a pointer to it was made.
-                       Fix:  Do a full lookup on the struct/union tag.  */
+   yet defined at the time a pointer to it was made.
+   Fix:  Do a full lookup on the struct/union tag.  */
 void
-cleanup_undefined_types ()
+cleanup_undefined_types (void)
 {
   struct type **type;
 
@@ -4849,15 +5160,15 @@ cleanup_undefined_types ()
       switch (TYPE_CODE (*type))
        {
 
-         case TYPE_CODE_STRUCT:
-         case TYPE_CODE_UNION:
-         case TYPE_CODE_ENUM:
+       case TYPE_CODE_STRUCT:
+       case TYPE_CODE_UNION:
+       case TYPE_CODE_ENUM:
          {
            /* Check if it has been defined since.  Need to do this here
               as well as in check_typedef to deal with the (legitimate in
               C though not C++) case of several types with the same name
               in different source files.  */
-           if (TYPE_FLAGS (*type) & TYPE_FLAG_STUB)
+           if (TYPE_STUB (*type))
              {
                struct pending *ppt;
                int i;
@@ -4866,8 +5177,7 @@ cleanup_undefined_types ()
 
                if (typename == NULL)
                  {
-                   static struct complaint msg = {"need a type name", 0, 0};
-                   complain (&msg);
+                   complaint (&symfile_complaints, "need a type name");
                    break;
                  }
                for (ppt = file_symbols; ppt; ppt = ppt->next)
@@ -4875,16 +5185,13 @@ cleanup_undefined_types ()
                    for (i = 0; i < ppt->nsyms; i++)
                      {
                        struct symbol *sym = ppt->symbol[i];
-                       
+
                        if (SYMBOL_CLASS (sym) == LOC_TYPEDEF
                            && SYMBOL_NAMESPACE (sym) == STRUCT_NAMESPACE
                            && (TYPE_CODE (SYMBOL_TYPE (sym)) ==
                                TYPE_CODE (*type))
-                           && STREQ (SYMBOL_NAME (sym), typename))
-                         {
-                           memcpy (*type, SYMBOL_TYPE (sym),
-                                   sizeof (struct type));
-                         }
+                           && STREQ (DEPRECATED_SYMBOL_NAME (sym), typename))
+                          replace_type (*type, SYMBOL_TYPE (sym));
                      }
                  }
              }
@@ -4893,9 +5200,9 @@ cleanup_undefined_types ()
 
        default:
          {
-           static struct complaint msg = {"\
-GDB internal error.  cleanup_undefined_types with bad type %d.", 0, 0};
-           complain (&msg, TYPE_CODE (*type));
+           complaint (&symfile_complaints,
+                      "GDB internal error.  cleanup_undefined_types with bad type %d.",
+                      TYPE_CODE (*type));
          }
          break;
        }
@@ -4909,8 +5216,7 @@ GDB internal error.  cleanup_undefined_types with bad type %d.", 0, 0};
    to.  Get these symbols from the minimal symbol table.  */
 
 void
-scan_file_globals (objfile)
-     struct objfile *objfile;
+scan_file_globals (struct objfile *objfile)
 {
   int hash;
   struct minimal_symbol *msymbol;
@@ -4930,7 +5236,7 @@ scan_file_globals (objfile)
   while (1)
     {
       /* Avoid expensive loop through all minimal symbols if there are
-        no unresolved symbols.  */
+         no unresolved symbols.  */
       for (hash = 0; hash < HASHSIZE; hash++)
        {
          if (global_sym_chain[hash])
@@ -4939,8 +5245,8 @@ scan_file_globals (objfile)
       if (hash >= HASHSIZE)
        return;
 
-      for (msymbol = resolve_objfile -> msymbols;
-          msymbol && SYMBOL_NAME (msymbol) != NULL;
+      for (msymbol = resolve_objfile->msymbols;
+          msymbol && DEPRECATED_SYMBOL_NAME (msymbol) != NULL;
           msymbol++)
        {
          QUIT;
@@ -4961,13 +5267,16 @@ scan_file_globals (objfile)
          /* Get the hash index and check all the symbols
             under that hash index. */
 
-         hash = hashname (SYMBOL_NAME (msymbol));
+         hash = hashname (DEPRECATED_SYMBOL_NAME (msymbol));
 
          for (sym = global_sym_chain[hash]; sym;)
            {
-             if (SYMBOL_NAME (msymbol)[0] == SYMBOL_NAME (sym)[0] &&
-                 STREQ(SYMBOL_NAME (msymbol) + 1, SYMBOL_NAME (sym) + 1))
+             if (DEPRECATED_SYMBOL_NAME (msymbol)[0] == DEPRECATED_SYMBOL_NAME (sym)[0] &&
+                 STREQ (DEPRECATED_SYMBOL_NAME (msymbol) + 1, DEPRECATED_SYMBOL_NAME (sym) + 1))
                {
+
+                 struct alias_list *aliases;
+
                  /* Splice this symbol out of the hash chain and
                     assign the value we have to it. */
                  if (prev)
@@ -4978,7 +5287,7 @@ scan_file_globals (objfile)
                    {
                      global_sym_chain[hash] = SYMBOL_VALUE_CHAIN (sym);
                    }
-                 
+
                  /* Check to see whether we need to fix up a common block.  */
                  /* Note: this code might be executed several times for
                     the same symbol if there are multiple references.  */
@@ -4986,23 +5295,34 @@ scan_file_globals (objfile)
                  /* If symbol has aliases, do minimal symbol fixups for each.
                     These live aliases/references weren't added to 
                     global_sym_chain hash but may also need to be fixed up. */
-                 /* FIXME: Maybe should have added aliases to the global chain,                     resolved symbol name, then treated aliases as normal 
+                 /* FIXME: Maybe should have added aliases to the global chain,                     resolved symbol name, then treated aliases as normal 
                     symbols?  Still, we wouldn't want to add_to_list. */
                  /* Now do the same for each alias of this symbol */
-                 for (rsym = sym; rsym; rsym = SYMBOL_ALIASES (rsym))
+                 rsym = sym;
+                 aliases = SYMBOL_ALIASES (sym);
+                 while (rsym)
                    {
                      if (SYMBOL_CLASS (rsym) == LOC_BLOCK)
-                       {
-                         fix_common_block (rsym, SYMBOL_VALUE_ADDRESS (msymbol));
-                       }
+                       {
+                         fix_common_block (rsym,
+                                           SYMBOL_VALUE_ADDRESS (msymbol));
+                       }
                      else
-                       {
-                         SYMBOL_VALUE_ADDRESS (rsym)
+                       {
+                         SYMBOL_VALUE_ADDRESS (rsym)
                            = SYMBOL_VALUE_ADDRESS (msymbol);
-                       }
+                       }
                      SYMBOL_SECTION (rsym) = SYMBOL_SECTION (msymbol);
+                     if (aliases)
+                       {
+                         rsym = aliases->sym;
+                         aliases = aliases->next;
+                       }
+                     else
+                       rsym = NULL;
                    }
-                 
+
+
                  if (prev)
                    {
                      sym = SYMBOL_VALUE_CHAIN (prev);
@@ -5042,8 +5362,9 @@ scan_file_globals (objfile)
          if (SYMBOL_CLASS (prev) == LOC_STATIC)
            SYMBOL_CLASS (prev) = LOC_UNRESOLVED;
          else
-           complain (&unresolved_sym_chain_complaint,
-                     objfile -> name, SYMBOL_NAME (prev));
+           complaint (&symfile_complaints,
+                      "%s: common block `%s' from global_sym_chain unresolved",
+                      objfile->name, DEPRECATED_SYMBOL_NAME (prev));
        }
     }
   memset (global_sym_chain, 0, sizeof (global_sym_chain));
@@ -5054,7 +5375,7 @@ scan_file_globals (objfile)
    to a psymtab.  */
 
 void
-stabsread_init ()
+stabsread_init (void)
 {
 }
 
@@ -5063,7 +5384,7 @@ stabsread_init ()
    file, e.g. a shared library).  */
 
 void
-stabsread_new_init ()
+stabsread_new_init (void)
 {
   /* Empty the hash table of global syms looking for values.  */
   memset (global_sym_chain, 0, sizeof (global_sym_chain));
@@ -5072,7 +5393,8 @@ stabsread_new_init ()
 /* Initialize anything that needs initializing at the same time as
    start_symtab() is called. */
 
-void start_stabs ()
+void
+start_stabs (void)
 {
   global_stabs = NULL;         /* AIX COFF */
   /* Leave FILENUM of 0 free for builtin types and this file's types.  */
@@ -5082,17 +5404,16 @@ void start_stabs ()
 
   /* FIXME: If common_block_name is not already NULL, we should complain().  */
   common_block_name = NULL;
-
-  os9k_stabs = 0;
 }
 
 /* Call after end_symtab() */
 
-void end_stabs ()
+void
+end_stabs (void)
 {
   if (type_vector)
     {
-      free ((char *) type_vector);
+      xfree (type_vector);
     }
   type_vector = 0;
   type_vector_length = 0;
@@ -5100,21 +5421,46 @@ void end_stabs ()
 }
 
 void
-finish_global_stabs (objfile)
-     struct objfile *objfile;
+finish_global_stabs (struct objfile *objfile)
 {
   if (global_stabs)
     {
       patch_block_stabs (global_symbols, global_stabs, objfile);
-      free ((PTR) global_stabs);
+      xfree (global_stabs);
       global_stabs = NULL;
     }
 }
 
+/* Find the end of the name, delimited by a ':', but don't match
+   ObjC symbols which look like -[Foo bar::]:bla.  */
+static char *
+find_name_end (char *name)
+{
+  char *s = name;
+  if (s[0] == '-' || *s == '+')
+    {
+      /* Must be an ObjC method symbol.  */
+      if (s[1] != '[')
+       {
+         error ("invalid symbol name \"%s\"", name);
+       }
+      s = strchr (s, ']');
+      if (s == NULL)
+       {
+         error ("invalid symbol name \"%s\"", name);
+       }
+      return strchr (s, ':');
+    }
+  else
+    {
+      return strchr (s, ':');
+    }
+}
+
 /* Initializer for this module */
 
 void
-_initialize_stabsread ()
+_initialize_stabsread (void)
 {
   undef_types_allocated = 20;
   undef_types_length = 0;
This page took 0.105435 seconds and 4 git commands to generate.