Rewrite the Rust expression parser
[binutils-gdb.git] / gdb / cp-valprint.c
index 567e0abed358d0a0b5a2c37d04d1331f69611cd6..d303c83694da6e031377f049a121617e4686f1c2 100644 (file)
@@ -1,13 +1,12 @@
 /* Support for printing C++ values for GDB, the GNU debugger.
-   Copyright 1986, 1988, 1989, 1991, 1992, 1993, 1994, 1995, 1996, 1997,
-   2000, 2001
-   Free Software Foundation, Inc.
+
+   Copyright (C) 1986-2021 Free Software Foundation, Inc.
 
    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
+   the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
 
    This program is distributed in the hope that it will be useful,
    GNU General Public License for more details.
 
    You should have received a copy of the GNU General Public License
-   along with this program; if not, write to the Free Software
-   Foundation, Inc., 59 Temple Place - Suite 330,
-   Boston, MA 02111-1307, USA.  */
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 #include "defs.h"
-#include "obstack.h"
+#include "gdb_obstack.h"
 #include "symtab.h"
 #include "gdbtypes.h"
 #include "expression.h"
 #include "gdbcmd.h"
 #include "demangle.h"
 #include "annotate.h"
-#include "gdb_string.h"
 #include "c-lang.h"
 #include "target.h"
 #include "cp-abi.h"
-
-/* Indication of presence of HP-compiled object files */
-extern int hp_som_som_object_present;  /* defined in symtab.c */
-
-
-int vtblprint;                 /* Controls printing of vtbl's */
-int objectprint;               /* Controls looking up an object's derived type
-                                  using what we find in its vtables.  */
-int static_field_print;                /* Controls printing of static fields. */
+#include "valprint.h"
+#include "cp-support.h"
+#include "language.h"
+#include "extension.h"
+#include "typeprint.h"
+#include "gdbsupport/byte-vector.h"
+#include "gdbarch.h"
+#include "cli/cli-style.h"
 
 static struct obstack dont_print_vb_obstack;
 static struct obstack dont_print_statmem_obstack;
-
-extern void _initialize_cp_valprint (void);
+static struct obstack dont_print_stat_array_obstack;
 
 static void cp_print_static_field (struct type *, struct value *,
-                                  struct ui_file *, int, int,
-                                  enum val_prettyprint);
-
-static void cp_print_value (struct type *, struct type *, char *, int,
-                           CORE_ADDR, struct ui_file *, int, int,
-                           enum val_prettyprint, struct type **);
-
-static void cp_print_hpacc_virtual_table_entries (struct type *, int *,
-                                                 struct value *,
-                                                 struct ui_file *, int,
-                                                 int,
-                                                 enum val_prettyprint);
-
-
-void
-cp_print_class_method (char *valaddr,
-                      struct type *type,
-                      struct ui_file *stream)
-{
-  struct type *domain;
-  struct fn_field *f = NULL;
-  int j = 0;
-  int len2;
-  int offset;
-  char *kind = "";
-  CORE_ADDR addr;
-  struct symbol *sym;
-  unsigned len;
-  unsigned int i;
-  struct type *target_type = check_typedef (TYPE_TARGET_TYPE (type));
-
-  domain = TYPE_DOMAIN_TYPE (target_type);
-  if (domain == (struct type *) NULL)
-    {
-      fprintf_filtered (stream, "<unknown>");
-      return;
-    }
-  addr = unpack_pointer (lookup_pointer_type (builtin_type_void), valaddr);
-  if (METHOD_PTR_IS_VIRTUAL (addr))
-    {
-      offset = METHOD_PTR_TO_VOFFSET (addr);
-      len = TYPE_NFN_FIELDS (domain);
-      for (i = 0; i < len; i++)
-       {
-         f = TYPE_FN_FIELDLIST1 (domain, i);
-         len2 = TYPE_FN_FIELDLIST_LENGTH (domain, i);
-
-         for (j = 0; j < len2; j++)
-           {
-             QUIT;
-             if (TYPE_FN_FIELD_VOFFSET (f, j) == offset)
-               {
-                 if (TYPE_FN_FIELD_STUB (f, j))
-                   check_stub_method (domain, i, j);
-                 kind = "virtual ";
-                 goto common;
-               }
-           }
-       }
-    }
-  else
-    {
-      sym = find_pc_function (addr);
-      if (sym == 0)
-       {
-         /* 1997-08-01 Currently unsupported with HP aCC */
-         if (hp_som_som_object_present)
-           {
-             fputs_filtered ("?? <not supported with HP aCC>", stream);
-             return;
-           }
-         error ("invalid pointer to member function");
-       }
-      len = TYPE_NFN_FIELDS (domain);
-      for (i = 0; i < len; i++)
-       {
-         f = TYPE_FN_FIELDLIST1 (domain, i);
-         len2 = TYPE_FN_FIELDLIST_LENGTH (domain, i);
+                                  struct ui_file *, int,
+                                  const struct value_print_options *);
 
-         for (j = 0; j < len2; j++)
-           {
-             QUIT;
-             if (TYPE_FN_FIELD_STUB (f, j))
-               check_stub_method (domain, i, j);
-             if (STREQ (SYMBOL_NAME (sym), TYPE_FN_FIELD_PHYSNAME (f, j)))
-               {
-                 goto common;
-               }
-           }
-       }
-    }
- common:
-  if (i < len)
-    {
-      char *demangled_name;
-
-      fprintf_filtered (stream, "&");
-      fprintf_filtered (stream, kind);
-      demangled_name = cplus_demangle (TYPE_FN_FIELD_PHYSNAME (f, j),
-                                      DMGL_ANSI | DMGL_PARAMS);
-      if (demangled_name == NULL)
-       fprintf_filtered (stream, "<badly mangled name %s>",
-                         TYPE_FN_FIELD_PHYSNAME (f, j));
-      else
-       {
-         fputs_filtered (demangled_name, stream);
-         xfree (demangled_name);
-       }
-    }
-  else
-    {
-      fprintf_filtered (stream, "(");
-      type_print (type, "", stream, -1);
-      fprintf_filtered (stream, ") %d", (int) addr >> 3);
-    }
-}
+static void cp_print_value (struct value *, struct ui_file *,
+                           int, const struct value_print_options *,
+                           struct type **);
 
-/* This was what it was for gcc 2.4.5 and earlier.  */
-static const char vtbl_ptr_name_old[] =
-{
-  CPLUS_MARKER, 'v', 't', 'b', 'l', '_', 'p', 't', 'r', '_', 
-  't', 'y', 'p', 'e', 0
-};
 
-/* It was changed to this after 2.4.5.  */
+/* GCC versions after 2.4.5 use this.  */
 const char vtbl_ptr_name[] = "__vtbl_ptr_type";
 
-/* HP aCC uses different names */
-const char hpacc_vtbl_ptr_name[] = "__vfp";
-const char hpacc_vtbl_ptr_type_name[] = "__vftyp";
-
-
 /* Return truth value for assertion that TYPE is of the type
    "pointer to virtual function".  */
 
 int
 cp_is_vtbl_ptr_type (struct type *type)
 {
-  char *typename = type_name_no_tag (type);
+  const char *type_name = type->name ();
 
-  return (typename != NULL
-         && (STREQ (typename, vtbl_ptr_name)
-             || STREQ (typename, vtbl_ptr_name_old)));
+  return (type_name != NULL && !strcmp (type_name, vtbl_ptr_name));
 }
 
 /* Return truth value for the assertion that TYPE is of the type
@@ -201,111 +72,148 @@ cp_is_vtbl_ptr_type (struct type *type)
 int
 cp_is_vtbl_member (struct type *type)
 {
-  if (TYPE_CODE (type) == TYPE_CODE_PTR)
+  /* With older versions of g++, the vtbl field pointed to an array of
+     structures.  Nowadays it points directly to the structure.  */
+  if (type->code () == TYPE_CODE_PTR)
     {
       type = TYPE_TARGET_TYPE (type);
-      if (TYPE_CODE (type) == TYPE_CODE_ARRAY)
+      if (type->code () == TYPE_CODE_ARRAY)
        {
          type = TYPE_TARGET_TYPE (type);
-         if (TYPE_CODE (type) == TYPE_CODE_STRUCT      /* if not using thunks */
-             || TYPE_CODE (type) == TYPE_CODE_PTR)     /* if using thunks */
+         if (type->code () == TYPE_CODE_STRUCT    /* if not using thunks */
+             || type->code () == TYPE_CODE_PTR)   /* if using thunks */
            {
              /* Virtual functions tables are full of pointers
-                to virtual functions. */
+                to virtual functions.  */
              return cp_is_vtbl_ptr_type (type);
            }
        }
+      else if (type->code () == TYPE_CODE_STRUCT)  /* if not using thunks */
+       {
+         return cp_is_vtbl_ptr_type (type);
+       }
+      else if (type->code () == TYPE_CODE_PTR)     /* if using thunks */
+       {
+         /* The type name of the thunk pointer is NULL when using
+            dwarf2.  We could test for a pointer to a function, but
+            there is no type info for the virtual table either, so it
+            wont help.  */
+         return cp_is_vtbl_ptr_type (type);
+       }
     }
   return 0;
 }
 
 /* Mutually recursive subroutines of cp_print_value and c_val_print to
-   print out a structure's fields: cp_print_value_fields and cp_print_value.
+   print out a structure's fields: cp_print_value_fields and
+   cp_print_value.
 
-   TYPE, VALADDR, ADDRESS, STREAM, RECURSE, and PRETTY have the
-   same meanings as in cp_print_value and c_val_print.
+   TYPE, VALADDR, ADDRESS, STREAM, RECURSE, and OPTIONS have the same
+   meanings as in cp_print_value and c_val_print.
 
-   2nd argument REAL_TYPE is used to carry over the type of the derived
-   class across the recursion to base classes. 
+   2nd argument REAL_TYPE is used to carry over the type of the
+   derived class across the recursion to base classes.
 
-   DONT_PRINT is an array of baseclass types that we
-   should not print, or zero if called from top level.  */
+   DONT_PRINT is an array of baseclass types that we should not print,
+   or zero if called from top level.  */
 
 void
-cp_print_value_fields (struct type *type, struct type *real_type, char *valaddr,
-                      int offset, CORE_ADDR address, struct ui_file *stream,
-                      int format, int recurse, enum val_prettyprint pretty,
-                      struct type **dont_print_vb, int dont_print_statmem)
+cp_print_value_fields (struct value *val, struct ui_file *stream,
+                      int recurse, const struct value_print_options *options,
+                      struct type **dont_print_vb,
+                      int dont_print_statmem)
 {
   int i, len, n_baseclasses;
-  struct obstack tmp_obstack;
-  char *last_dont_print = obstack_next_free (&dont_print_statmem_obstack);
   int fields_seen = 0;
+  static int last_set_recurse = -1;
+
+  struct type *type = check_typedef (value_type (val));
 
-  CHECK_TYPEDEF (type);
+  if (recurse == 0)
+    {
+      /* Any object can be left on obstacks only during an unexpected
+        error.  */
+
+      if (obstack_object_size (&dont_print_statmem_obstack) > 0)
+       {
+         obstack_free (&dont_print_statmem_obstack, NULL);
+         obstack_begin (&dont_print_statmem_obstack,
+                        32 * sizeof (CORE_ADDR));
+       }
+      if (obstack_object_size (&dont_print_stat_array_obstack) > 0)
+       {
+         obstack_free (&dont_print_stat_array_obstack, NULL);
+         obstack_begin (&dont_print_stat_array_obstack,
+                        32 * sizeof (struct type *));
+       }
+    }
 
   fprintf_filtered (stream, "{");
-  len = TYPE_NFIELDS (type);
+  len = type->num_fields ();
   n_baseclasses = TYPE_N_BASECLASSES (type);
 
   /* First, print out baseclasses such that we don't print
      duplicates of virtual baseclasses.  */
 
   if (n_baseclasses > 0)
-    cp_print_value (type, real_type, valaddr, offset, address, stream,
-                   format, recurse + 1, pretty, dont_print_vb);
+    cp_print_value (val, stream, recurse + 1, options, dont_print_vb);
 
   /* Second, print out data fields */
 
-  /* If there are no data fields, or if the only field is the
-   * vtbl pointer, skip this part */
-  if ((len == n_baseclasses)
-      || ((len - n_baseclasses == 1)
-         && TYPE_HAS_VTABLE (type)
-         && STREQN (TYPE_FIELD_NAME (type, n_baseclasses),
-                    hpacc_vtbl_ptr_name, 5))
-      || !len)
-    fprintf_filtered (stream, "<No data fields>");
+  /* If there are no data fields, skip this part */
+  if (len == n_baseclasses || !len)
+    fprintf_styled (stream, metadata_style.style (), "<No data fields>");
   else
     {
-      extern int inspect_it;
+      size_t statmem_obstack_initial_size = 0;
+      size_t stat_array_obstack_initial_size = 0;
+      struct type *vptr_basetype = NULL;
+      int vptr_fieldno;
 
       if (dont_print_statmem == 0)
        {
-         /* If we're at top level, carve out a completely fresh
-            chunk of the obstack and use that until this particular
-            invocation returns.  */
-         tmp_obstack = dont_print_statmem_obstack;
-         obstack_finish (&dont_print_statmem_obstack);
+         statmem_obstack_initial_size =
+           obstack_object_size (&dont_print_statmem_obstack);
+
+         if (last_set_recurse != recurse)
+           {
+             stat_array_obstack_initial_size =
+               obstack_object_size (&dont_print_stat_array_obstack);
+
+             last_set_recurse = recurse;
+           }
        }
 
+      vptr_fieldno = get_vptr_fieldno (type, &vptr_basetype);
       for (i = n_baseclasses; i < len; i++)
        {
-         /* If requested, skip printing of static fields.  */
-         if (!static_field_print && TYPE_FIELD_STATIC (type, i))
-           continue;
+         const gdb_byte *valaddr = value_contents_for_printing (val);
 
-         /* If a vtable pointer appears, we'll print it out later */
-         if (TYPE_HAS_VTABLE (type)
-             && STREQN (TYPE_FIELD_NAME (type, i), hpacc_vtbl_ptr_name, 5))
+         /* If requested, skip printing of static fields.  */
+         if (!options->static_field_print
+             && field_is_static (&type->field (i)))
            continue;
 
          if (fields_seen)
-           fprintf_filtered (stream, ", ");
+           {
+             fputs_filtered (",", stream);
+             if (!options->prettyformat)
+               fputs_filtered (" ", stream);
+           }
          else if (n_baseclasses > 0)
            {
-             if (pretty)
+             if (options->prettyformat)
                {
                  fprintf_filtered (stream, "\n");
                  print_spaces_filtered (2 + 2 * recurse, stream);
                  fputs_filtered ("members of ", stream);
-                 fputs_filtered (type_name_no_tag (type), stream);
-                 fputs_filtered (": ", stream);
+                 fputs_filtered (type->name (), stream);
+                 fputs_filtered (":", stream);
                }
            }
          fields_seen = 1;
 
-         if (pretty)
+         if (options->prettyformat)
            {
              fprintf_filtered (stream, "\n");
              print_spaces_filtered (2 + 2 * recurse, stream);
@@ -314,81 +222,118 @@ cp_print_value_fields (struct type *type, struct type *real_type, char *valaddr,
            {
              wrap_here (n_spaces (2 + 2 * recurse));
            }
-         if (inspect_it)
+
+         annotate_field_begin (type->field (i).type ());
+
+         if (field_is_static (&type->field (i)))
            {
-             if (TYPE_CODE (TYPE_FIELD_TYPE (type, i)) == TYPE_CODE_PTR)
-               fputs_filtered ("\"( ptr \"", stream);
-             else
-               fputs_filtered ("\"( nodef \"", stream);
-             if (TYPE_FIELD_STATIC (type, i))
-               fputs_filtered ("static ", stream);
-             fprintf_symbol_filtered (stream, TYPE_FIELD_NAME (type, i),
-                                      language_cplus,
-                                      DMGL_PARAMS | DMGL_ANSI);
-             fputs_filtered ("\" \"", stream);
-             fprintf_symbol_filtered (stream, TYPE_FIELD_NAME (type, i),
-                                      language_cplus,
+             fputs_filtered ("static ", stream);
+             fprintf_symbol_filtered (stream,
+                                      TYPE_FIELD_NAME (type, i),
+                                      current_language->la_language,
                                       DMGL_PARAMS | DMGL_ANSI);
-             fputs_filtered ("\") \"", stream);
            }
+         else
+           fputs_styled (TYPE_FIELD_NAME (type, i),
+                         variable_name_style.style (), stream);
+         annotate_field_name_end ();
+
+         /* We tweak various options in a few cases below.  */
+         value_print_options options_copy = *options;
+         value_print_options *opts = &options_copy;
+
+         /* Do not print leading '=' in case of anonymous
+            unions.  */
+         if (strcmp (TYPE_FIELD_NAME (type, i), ""))
+           fputs_filtered (" = ", stream);
          else
            {
-             annotate_field_begin (TYPE_FIELD_TYPE (type, i));
-
-             if (TYPE_FIELD_STATIC (type, i))
-               fputs_filtered ("static ", stream);
-             fprintf_symbol_filtered (stream, TYPE_FIELD_NAME (type, i),
-                                      language_cplus,
-                                      DMGL_PARAMS | DMGL_ANSI);
-             annotate_field_name_end ();
-             /* do not print leading '=' in case of anonymous unions */
-             if (strcmp (TYPE_FIELD_NAME (type, i), ""))
-               fputs_filtered (" = ", stream);
-             annotate_field_value ();
+             /* If this is an anonymous field then we want to consider it
+                as though it is at its parent's depth when it comes to the
+                max print depth.  */
+             if (opts->max_depth != -1 && opts->max_depth < INT_MAX)
+               ++opts->max_depth;
            }
+         annotate_field_value ();
 
-         if (!TYPE_FIELD_STATIC (type, i) && TYPE_FIELD_PACKED (type, i))
+         if (!field_is_static (&type->field (i))
+             && TYPE_FIELD_PACKED (type, i))
            {
              struct value *v;
 
-             /* Bitfields require special handling, especially due to byte
-                order problems.  */
+             /* Bitfields require special handling, especially due to
+                byte order problems.  */
              if (TYPE_FIELD_IGNORE (type, i))
                {
-                 fputs_filtered ("<optimized out or zero length>", stream);
+                 fputs_styled ("<optimized out or zero length>",
+                               metadata_style.style (), stream);
+               }
+             else if (value_bits_synthetic_pointer (val,
+                                                    TYPE_FIELD_BITPOS (type,
+                                                                       i),
+                                                    TYPE_FIELD_BITSIZE (type,
+                                                                        i)))
+               {
+                 fputs_styled (_("<synthetic pointer>"),
+                               metadata_style.style (), stream);
                }
              else
                {
-                 v = value_from_longest
-                   (TYPE_FIELD_TYPE (type, i), 
-                    unpack_field_as_long (type, valaddr + offset, i));
+                 opts->deref_ref = 0;
+
+                 v = value_field_bitfield (type, i, valaddr,
+                                           value_embedded_offset (val), val);
 
-                 val_print (TYPE_FIELD_TYPE (type, i), VALUE_CONTENTS (v),
-                            0, 0, stream, format, 0, recurse + 1, pretty);
+                 common_val_print (v, stream, recurse + 1,
+                                   opts, current_language);
                }
            }
          else
            {
              if (TYPE_FIELD_IGNORE (type, i))
                {
-                 fputs_filtered ("<optimized out or zero length>", stream);
+                 fputs_styled ("<optimized out or zero length>",
+                               metadata_style.style (), stream);
+               }
+             else if (field_is_static (&type->field (i)))
+               {
+                 try
+                   {
+                     struct value *v = value_static_field (type, i);
+
+                     cp_print_static_field (type->field (i).type (),
+                                            v, stream, recurse + 1,
+                                            opts);
+                   }
+                 catch (const gdb_exception_error &ex)
+                   {
+                     fprintf_styled (stream, metadata_style.style (),
+                                     _("<error reading variable: %s>"),
+                                     ex.what ());
+                   }
                }
-             else if (TYPE_FIELD_STATIC (type, i))
+             else if (i == vptr_fieldno && type == vptr_basetype)
                {
-                 struct value *v = value_static_field (type, i);
-                 if (v == NULL)
-                   fputs_filtered ("<optimized out>", stream);
-                 else
-                   cp_print_static_field (TYPE_FIELD_TYPE (type, i), v,
-                                          stream, format, recurse + 1,
-                                          pretty);
+                 int i_offset = TYPE_FIELD_BITPOS (type, i) / 8;
+                 struct type *i_type = type->field (i).type ();
+
+                 if (valprint_check_validity (stream, i_type, i_offset, val))
+                   {
+                     CORE_ADDR addr;
+
+                     i_offset += value_embedded_offset (val);
+                     addr = extract_typed_address (valaddr + i_offset, i_type);
+                     print_function_pointer_address (opts,
+                                                     type->arch (),
+                                                     addr, stream);
+                   }
                }
              else
                {
-                 val_print (TYPE_FIELD_TYPE (type, i),
-                            valaddr, offset + TYPE_FIELD_BITPOS (type, i) / 8,
-                            address + TYPE_FIELD_BITPOS (type, i) / 8,
-                            stream, format, 0, recurse + 1, pretty);
+                 struct value *v = value_primitive_field (val, 0, i, type);
+                 opts->deref_ref = 0;
+                 common_val_print (v, stream, recurse + 1, opts,
+                                   current_language);
                }
            }
          annotate_field_end ();
@@ -396,143 +341,86 @@ cp_print_value_fields (struct type *type, struct type *real_type, char *valaddr,
 
       if (dont_print_statmem == 0)
        {
-         /* Free the space used to deal with the printing
-            of the members from top level.  */
-         obstack_free (&dont_print_statmem_obstack, last_dont_print);
-         dont_print_statmem_obstack = tmp_obstack;
-       }
-
-      if (pretty)
-       {
-         fprintf_filtered (stream, "\n");
-         print_spaces_filtered (2 * recurse, stream);
-       }
-    }                          /* if there are data fields */
-  /* Now print out the virtual table pointer if there is one */
-  if (TYPE_HAS_VTABLE (type)
-      && STREQN (TYPE_FIELD_NAME (type, n_baseclasses),
-                hpacc_vtbl_ptr_name, 
-                5))
-    {
-      struct value *v;
-      /* First get the virtual table pointer and print it out */
-
-#if 0
-      fputs_filtered ("__vfp = ", stream);
-#endif
-
-      fputs_filtered (", Virtual table at ", stream);
-
-      /* pai: FIXME 32x64 problem? */
-      /* Not sure what the best notation is in the case where there is no
-         baseclass name.  */
-      v = value_from_pointer (lookup_pointer_type (builtin_type_unsigned_long),
-                             *(unsigned long *) (valaddr + offset));
-
-      val_print (VALUE_TYPE (v), VALUE_CONTENTS (v), 0, 0,
-                stream, format, 0, recurse + 1, pretty);
-      fields_seen = 1;
-
-      if (vtblprint)
-       {
-         /* Print out function pointers in vtable. */
+         size_t obstack_final_size =
+          obstack_object_size (&dont_print_statmem_obstack);
 
-         /* FIXME: then-clause is for non-RRBC layout of virtual
-          * table.  The RRBC case in the else-clause is yet to be
-          * implemented.  The if (1) below should be changed to a
-          * test for whether the executable we have was compiled
-          * with a version of HP aCC that doesn't have RRBC
-          * support. */
-
-         if (1)
+         if (obstack_final_size > statmem_obstack_initial_size)
            {
-             /* no RRBC support; function pointers embedded directly
-                 in vtable */
-
-             int vfuncs = count_virtual_fns (real_type);
-
-             fputs_filtered (" {", stream);
-
-             /* FIXME : doesn't work at present */
-#if 0
-             fprintf_filtered (stream, "%d entr%s: ", vfuncs,
-                               vfuncs == 1 ? "y" : "ies");
-#else
-             fputs_filtered ("not implemented", stream);
-
-
-#endif
+             /* In effect, a pop of the printed-statics stack.  */
+             size_t shrink_bytes
+               = statmem_obstack_initial_size - obstack_final_size;
+             obstack_blank_fast (&dont_print_statmem_obstack, shrink_bytes);
+           }
 
-             /* recursive function that prints all virtual function entries */
-#if 0
-             cp_print_hpacc_virtual_table_entries (real_type, &vfuncs, v,
-                                                   stream, format, recurse,
-                                                   pretty);
-#endif
-             fputs_filtered ("}", stream);
-           }                   /* non-RRBC case */
-         else
+         if (last_set_recurse != recurse)
            {
-             /* FIXME -- see comments above */
-             /* RRBC support present; function pointers are found
-              * by indirection through the class segment entries. */
+             obstack_final_size =
+               obstack_object_size (&dont_print_stat_array_obstack);
 
+             if (obstack_final_size > stat_array_obstack_initial_size)
+               {
+                 void *free_to_ptr =
+                   (char *) obstack_next_free (&dont_print_stat_array_obstack)
+                   - (obstack_final_size
+                      - stat_array_obstack_initial_size);
 
-           }                   /* RRBC case */
-       }                       /* if vtblprint */
+                 obstack_free (&dont_print_stat_array_obstack,
+                               free_to_ptr);
+               }
+             last_set_recurse = -1;
+           }
+       }
 
-      if (pretty)
+      if (options->prettyformat)
        {
          fprintf_filtered (stream, "\n");
          print_spaces_filtered (2 * recurse, stream);
        }
-
-    }                          /* if vtable exists */
+    }                          /* if there are data fields */
 
   fprintf_filtered (stream, "}");
 }
 
-/* Special val_print routine to avoid printing multiple copies of virtual
-   baseclasses.  */
+/* Special val_print routine to avoid printing multiple copies of
+   virtual baseclasses.  */
 
 static void
-cp_print_value (struct type *type, struct type *real_type, char *valaddr,
-               int offset, CORE_ADDR address, struct ui_file *stream,
-               int format, int recurse, enum val_prettyprint pretty,
+cp_print_value (struct value *val, struct ui_file *stream,
+               int recurse, const struct value_print_options *options,
                struct type **dont_print_vb)
 {
-  struct obstack tmp_obstack;
+  struct type *type = check_typedef (value_type (val));
+  CORE_ADDR address = value_address (val);
   struct type **last_dont_print
     = (struct type **) obstack_next_free (&dont_print_vb_obstack);
+  struct obstack tmp_obstack = dont_print_vb_obstack;
   int i, n_baseclasses = TYPE_N_BASECLASSES (type);
-  int thisoffset;
-  struct type *thistype;
+  const gdb_byte *valaddr = value_contents_for_printing (val);
 
   if (dont_print_vb == 0)
     {
-      /* If we're at top level, carve out a completely fresh
-         chunk of the obstack and use that until this particular
-         invocation returns.  */
-      tmp_obstack = dont_print_vb_obstack;
+      /* If we're at top level, carve out a completely fresh chunk of
+        the obstack and use that until this particular invocation
+        returns.  */
       /* Bump up the high-water mark.  Now alpha is omega.  */
       obstack_finish (&dont_print_vb_obstack);
     }
 
   for (i = 0; i < n_baseclasses; i++)
     {
-      int boffset;
-      int skip;
+      LONGEST boffset = 0;
+      int skip = 0;
       struct type *baseclass = check_typedef (TYPE_BASECLASS (type, i));
-      char *basename = TYPE_NAME (baseclass);
-      char *base_valaddr;
+      const char *basename = baseclass->name ();
+      struct value *base_val = NULL;
 
       if (BASETYPE_VIA_VIRTUAL (type, i))
        {
          struct type **first_dont_print
            = (struct type **) obstack_base (&dont_print_vb_obstack);
 
-         int j = (struct type **) obstack_next_free (&dont_print_vb_obstack)
-           - first_dont_print;
+         int j = (struct type **)
+           obstack_next_free (&dont_print_vb_obstack) - first_dont_print;
 
          while (--j >= 0)
            if (baseclass == first_dont_print[j])
@@ -541,72 +429,99 @@ cp_print_value (struct type *type, struct type *real_type, char *valaddr,
          obstack_ptr_grow (&dont_print_vb_obstack, baseclass);
        }
 
-      thisoffset = offset;
-      thistype = real_type;
-      if (TYPE_HAS_VTABLE (type) && BASETYPE_VIA_VIRTUAL (type, i))
+      try
        {
-         /* Assume HP/Taligent runtime convention */
-         find_rt_vbase_offset (type, TYPE_BASECLASS (type, i),
-                               valaddr, offset, &boffset, &skip);
-         if (skip >= 0)
-           error ("Virtual base class offset not found from vtable while"
-                  " printing");
-         base_valaddr = valaddr;
+         boffset = baseclass_offset (type, i, valaddr,
+                                     value_embedded_offset (val),
+                                     address, val);
        }
-      else
+      catch (const gdb_exception_error &ex)
        {
-         boffset = baseclass_offset (type, i,
-                                     valaddr + offset,
-                                     address + offset);
-         skip = ((boffset == -1) || (boffset + offset) < 0) ? 1 : -1;
+         if (ex.error == NOT_AVAILABLE_ERROR)
+           skip = -1;
+         else
+           skip = 1;
+       }
 
+      if (skip == 0)
+       {
          if (BASETYPE_VIA_VIRTUAL (type, i))
            {
              /* The virtual base class pointer might have been
-                clobbered by the user program. Make sure that it
-                still points to a valid memory location.  */
+                clobbered by the user program. Make sure that it
+                still points to a valid memory location.  */
 
-             if (boffset != -1
-                 && ((boffset + offset) < 0
-                     || (boffset + offset) >= TYPE_LENGTH (type)))
+             if (boffset < 0 || boffset >= TYPE_LENGTH (type))
                {
-                 base_valaddr = (char *) alloca (TYPE_LENGTH (baseclass));
-                 if (target_read_memory (address + offset + boffset, base_valaddr,
+                 gdb::byte_vector buf (TYPE_LENGTH (baseclass));
+
+                 if (target_read_memory (address + boffset, buf.data (),
                                          TYPE_LENGTH (baseclass)) != 0)
                    skip = 1;
-                 thisoffset = 0;
+                 base_val = value_from_contents_and_address (baseclass,
+                                                             buf.data (),
+                                                             address + boffset);
+                 baseclass = value_type (base_val);
                  boffset = 0;
-                 thistype = baseclass;
                }
              else
-               base_valaddr = valaddr;
+               {
+                 base_val = val;
+               }
            }
          else
-           base_valaddr = valaddr;
+           {
+             base_val = val;
+           }
        }
 
-      /* now do the printing */
-      if (pretty)
+      /* Now do the printing.  */
+      if (options->prettyformat)
        {
          fprintf_filtered (stream, "\n");
          print_spaces_filtered (2 * recurse, stream);
        }
       fputs_filtered ("<", stream);
-      /* Not sure what the best notation is in the case where there is no
-         baseclass name.  */
+      /* Not sure what the best notation is in the case where there is
+        no baseclass name.  */
       fputs_filtered (basename ? basename : "", stream);
       fputs_filtered ("> = ", stream);
 
-
-      if (skip >= 1)
-       fprintf_filtered (stream, "<invalid address>");
+      if (skip < 0)
+       val_print_unavailable (stream);
+      else if (skip > 0)
+       val_print_invalid_address (stream);
       else
-       cp_print_value_fields (baseclass, thistype, base_valaddr,
-                              thisoffset + boffset, address, stream, format,
-                              recurse, pretty,
-                              ((struct type **)
-                               obstack_base (&dont_print_vb_obstack)),
-                              0);
+       {
+         int result = 0;
+
+         if (options->max_depth > -1
+             && recurse >= options->max_depth)
+           {
+             const struct language_defn *language = current_language;
+             gdb_assert (language->struct_too_deep_ellipsis () != NULL);
+             fputs_filtered (language->struct_too_deep_ellipsis (), stream);
+           }
+         else
+           {
+             struct value *baseclass_val = value_primitive_field (val, 0,
+                                                                  i, type);
+
+             /* Attempt to run an extension language pretty-printer on the
+                baseclass if possible.  */
+             if (!options->raw)
+               result
+                 = apply_ext_lang_val_pretty_printer (baseclass_val, stream,
+                                                      recurse, options,
+                                                      current_language);
+
+             if (!result)
+               cp_print_value_fields (baseclass_val, stream, recurse, options,
+                                      ((struct type **)
+                                       obstack_base (&dont_print_vb_obstack)),
+                                      0);
+           }
+       }
       fputs_filtered (", ", stream);
 
     flush_it:
@@ -616,224 +531,205 @@ cp_print_value (struct type *type, struct type *real_type, char *valaddr,
   if (dont_print_vb == 0)
     {
       /* Free the space used to deal with the printing
-         of this type from top level.  */
+        of this type from top level.  */
       obstack_free (&dont_print_vb_obstack, last_dont_print);
       /* Reset watermark so that we can continue protecting
-         ourselves from whatever we were protecting ourselves.  */
+        ourselves from whatever we were protecting ourselves.  */
       dont_print_vb_obstack = tmp_obstack;
     }
 }
 
-/* Print value of a static member.
-   To avoid infinite recursion when printing a class that contains
-   a static instance of the class, we keep the addresses of all printed
-   static member classes in an obstack and refuse to print them more
-   than once.
+/* Print value of a static member.  To avoid infinite recursion when
+   printing a class that contains a static instance of the class, we
+   keep the addresses of all printed static member classes in an
+   obstack and refuse to print them more than once.
 
-   VAL contains the value to print, TYPE, STREAM, RECURSE, and PRETTY
+   VAL contains the value to print, TYPE, STREAM, RECURSE, and OPTIONS
    have the same meanings as in c_val_print.  */
 
 static void
 cp_print_static_field (struct type *type,
                       struct value *val,
                       struct ui_file *stream,
-                      int format,
                       int recurse,
-                      enum val_prettyprint pretty)
+                      const struct value_print_options *options)
 {
-  if (TYPE_CODE (type) == TYPE_CODE_STRUCT)
+  struct value_print_options opts;
+
+  if (value_entirely_optimized_out (val))
+    {
+      val_print_optimized_out (val, stream);
+      return;
+    }
+
+  struct type *real_type = check_typedef (type);
+  if (real_type->code () == TYPE_CODE_STRUCT)
     {
       CORE_ADDR *first_dont_print;
+      CORE_ADDR addr = value_address (val);
       int i;
 
       first_dont_print
        = (CORE_ADDR *) obstack_base (&dont_print_statmem_obstack);
-      i = (CORE_ADDR *) obstack_next_free (&dont_print_statmem_obstack)
-       - first_dont_print;
+      i = obstack_object_size (&dont_print_statmem_obstack)
+       / sizeof (CORE_ADDR);
 
       while (--i >= 0)
        {
-         if (VALUE_ADDRESS (val) == first_dont_print[i])
+         if (addr == first_dont_print[i])
            {
-             fputs_filtered ("<same as static member of an already"
-                             " seen type>",
-                             stream);
+             fputs_styled (_("<same as static member of an already"
+                             " seen type>"),
+                           metadata_style.style (), stream);
              return;
            }
        }
 
-      obstack_grow (&dont_print_statmem_obstack, (char *) &VALUE_ADDRESS (val),
+      obstack_grow (&dont_print_statmem_obstack, (char *) &addr,
                    sizeof (CORE_ADDR));
-
-      CHECK_TYPEDEF (type);
-      cp_print_value_fields (type, type, VALUE_CONTENTS_ALL (val),
-                            VALUE_EMBEDDED_OFFSET (val), VALUE_ADDRESS (val),
-                            stream, format, recurse, pretty, NULL, 1);
+      cp_print_value_fields (val, stream, recurse, options, NULL, 1);
       return;
     }
-  val_print (type, VALUE_CONTENTS_ALL (val), 
-            VALUE_EMBEDDED_OFFSET (val), VALUE_ADDRESS (val),
-            stream, format, 0, recurse, pretty);
-}
 
-void
-cp_print_class_member (char *valaddr, struct type *domain,
-                      struct ui_file *stream, char *prefix)
-{
+  if (real_type->code () == TYPE_CODE_ARRAY)
+    {
+      struct type **first_dont_print;
+      int i;
+      struct type *target_type = TYPE_TARGET_TYPE (type);
 
-  /* VAL is a byte offset into the structure type DOMAIN.
-     Find the name of the field for that offset and
-     print it.  */
-  int extra = 0;
-  int bits = 0;
-  register unsigned int i;
-  unsigned len = TYPE_NFIELDS (domain);
+      first_dont_print
+       = (struct type **) obstack_base (&dont_print_stat_array_obstack);
+      i = obstack_object_size (&dont_print_stat_array_obstack)
+       / sizeof (struct type *);
+
+      while (--i >= 0)
+       {
+         if (target_type == first_dont_print[i])
+           {
+             fputs_styled (_("<same as static member of an already"
+                             " seen type>"),
+                           metadata_style.style (), stream);
+             return;
+           }
+       }
+
+      obstack_grow (&dont_print_stat_array_obstack,
+                   (char *) &target_type,
+                   sizeof (struct type *));
+    }
+
+  opts = *options;
+  opts.deref_ref = 0;
+  common_val_print (val, stream, recurse, &opts, current_language);
+}
 
-  /* @@ Make VAL into bit offset */
+/* Find the field in *SELF, or its non-virtual base classes, with
+   bit offset OFFSET.  Set *SELF to the containing type and *FIELDNO
+   to the containing field number.  If OFFSET is not exactly at the
+   start of some field, set *SELF to NULL.  */
 
-  /* Note: HP aCC generates offsets that are the real byte offsets added
-     to a constant bias 0x20000000 (1 << 29).  This constant bias gets
-     shifted out in the code below -- joyous happenstance! */
+static void
+cp_find_class_member (struct type **self_p, int *fieldno,
+                     LONGEST offset)
+{
+  struct type *self;
+  unsigned int i;
+  unsigned len;
 
-  /* Note: HP cfront uses a constant bias of 1; if we support this
-     compiler ever, we will have to adjust the computation below */
+  *self_p = check_typedef (*self_p);
+  self = *self_p;
+  len = self->num_fields ();
 
-  LONGEST val = unpack_long (builtin_type_int, valaddr) << 3;
-  for (i = TYPE_N_BASECLASSES (domain); i < len; i++)
+  for (i = TYPE_N_BASECLASSES (self); i < len; i++)
     {
-      int bitpos = TYPE_FIELD_BITPOS (domain, i);
+      LONGEST bitpos = TYPE_FIELD_BITPOS (self, i);
+
       QUIT;
-      if (val == bitpos)
-       break;
-      if (val < bitpos && i != 0)
+      if (offset == bitpos)
        {
-         /* Somehow pointing into a field.  */
-         i -= 1;
-         extra = (val - TYPE_FIELD_BITPOS (domain, i));
-         if (extra & 0x7)
-           bits = 1;
-         else
-           extra >>= 3;
-         break;
+         *fieldno = i;
+         return;
        }
     }
-  if (i < len)
+
+  for (i = 0; i < TYPE_N_BASECLASSES (self); i++)
     {
-      char *name;
-      fprintf_filtered (stream, prefix);
-      name = type_name_no_tag (domain);
-      if (name)
-       fputs_filtered (name, stream);
-      else
-       c_type_print_base (domain, stream, 0, 0);
-      fprintf_filtered (stream, "::");
-      fputs_filtered (TYPE_FIELD_NAME (domain, i), stream);
-      if (extra)
-       fprintf_filtered (stream, " + %d bytes", extra);
-      if (bits)
-       fprintf_filtered (stream, " (offset in bits)");
+      LONGEST bitpos = TYPE_FIELD_BITPOS (self, i);
+      LONGEST bitsize = 8 * TYPE_LENGTH (self->field (i).type ());
+
+      if (offset >= bitpos && offset < bitpos + bitsize)
+       {
+         *self_p = self->field (i).type ();
+         cp_find_class_member (self_p, fieldno, offset - bitpos);
+         return;
+       }
     }
-  else
-    fprintf_filtered (stream, "%ld", (long) (val >> 3));
+
+  *self_p = NULL;
 }
 
+void
+cp_print_class_member (const gdb_byte *valaddr, struct type *type,
+                      struct ui_file *stream, const char *prefix)
+{
+  enum bfd_endian byte_order = type_byte_order (type);
 
-/* This function prints out virtual table entries for a class; it
- * recurses on the base classes to find all virtual functions
- * available in a class.
- *
- * pai/1997-05-21 Note: As the name suggests, it's currently
- * implemented for HP aCC runtime only. g++ objects are handled
- * differently and I have made no attempt to fold that logic in
- * here. The runtime layout is different for the two cases.  Also,
- * this currently has only the code for non-RRBC layouts generated by
- * the HP aCC compiler; RRBC code is stubbed out and will have to be
- * added later. */
+  /* VAL is a byte offset into the structure type SELF_TYPE.
+     Find the name of the field for that offset and
+     print it.  */
+  struct type *self_type = TYPE_SELF_TYPE (type);
+  LONGEST val;
+  int fieldno;
+
+  val = extract_signed_integer (valaddr,
+                               TYPE_LENGTH (type),
+                               byte_order);
+
+  /* Pointers to data members are usually byte offsets into an object.
+     Because a data member can have offset zero, and a NULL pointer to
+     member must be distinct from any valid non-NULL pointer to
+     member, either the value is biased or the NULL value has a
+     special representation; both are permitted by ISO C++.  HP aCC
+     used a bias of 0x20000000; HP cfront used a bias of 1; g++ 3.x
+     and other compilers which use the Itanium ABI use -1 as the NULL
+     value.  GDB only supports that last form; to add support for
+     another form, make this into a cp-abi hook.  */
+
+  if (val == -1)
+    {
+      fprintf_filtered (stream, "NULL");
+      return;
+    }
 
+  cp_find_class_member (&self_type, &fieldno, val << 3);
 
-static void
-cp_print_hpacc_virtual_table_entries (struct type *type, int *vfuncs,
-                                     struct value *v, struct ui_file *stream,
-                                     int format, int recurse,
-                                     enum val_prettyprint pretty)
-{
-  int fn, oi;
-
-  /* pai: FIXME this function doesn't work. It should handle a given
-   * virtual function only once (latest redefinition in class hierarchy)
-   */
-
-  /* Recursion on other classes that can share the same vtable */
-  struct type *pbc = primary_base_class (type);
-  if (pbc)
-    cp_print_hpacc_virtual_table_entries (pbc, vfuncs, v, stream, format,
-                                         recurse, pretty);
-
-  /* Now deal with vfuncs declared in this class */
-  for (fn = 0; fn < TYPE_NFN_FIELDS (type); fn++)
-    for (oi = 0; oi < TYPE_FN_FIELDLIST_LENGTH (type, fn); oi++)
-      if (TYPE_FN_FIELD_VIRTUAL_P (TYPE_FN_FIELDLIST1 (type, fn), oi))
-       {
-         char *vf_name;
-         const char *field_physname;
-
-         /* virtual function offset */
-         int vx = (TYPE_FN_FIELD_VOFFSET (TYPE_FN_FIELDLIST1 (type, fn), oi)
-                   - 1);
-
-         /* Get the address of the vfunction entry */
-         struct value *vf = value_copy (v);
-         if (VALUE_LAZY (vf))
-           (void) value_fetch_lazy (vf);
-         /* adjust by offset */
-         vf->aligner.contents[0] += 4 * (HP_ACC_VFUNC_START + vx);
-         vf = value_ind (vf);  /* get the entry */
-         VALUE_TYPE (vf) = VALUE_TYPE (v);     /* make it a pointer */
-
-         /* print out the entry */
-         val_print (VALUE_TYPE (vf), VALUE_CONTENTS (vf), 0, 0,
-                    stream, format, 0, recurse + 1, pretty);
-         field_physname
-           = TYPE_FN_FIELD_PHYSNAME (TYPE_FN_FIELDLIST1 (type, fn), oi);
-         /* pai: (temp) FIXME Maybe this should be DMGL_ANSI */
-         vf_name = cplus_demangle (field_physname, DMGL_ARM);
-         fprintf_filtered (stream, " %s", vf_name);
-         if (--(*vfuncs) > 0)
-           fputs_filtered (", ", stream);
-       }
-}
+  if (self_type != NULL)
+    {
+      const char *name;
 
+      fputs_filtered (prefix, stream);
+      name = self_type->name ();
+      if (name)
+       fputs_filtered (name, stream);
+      else
+       c_type_print_base (self_type, stream, 0, 0, &type_print_raw_options);
+      fprintf_filtered (stream, "::");
+      fputs_styled (TYPE_FIELD_NAME (self_type, fieldno),
+                   variable_name_style.style (), stream);
+    }
+  else
+    fprintf_filtered (stream, "%ld", (long) val);
+}
 
 
+void _initialize_cp_valprint ();
 void
-_initialize_cp_valprint (void)
+_initialize_cp_valprint ()
 {
-  add_show_from_set
-    (add_set_cmd ("static-members", class_support, var_boolean,
-                 (char *) &static_field_print,
-                 "Set printing of C++ static members.",
-                 &setprintlist),
-     &showprintlist);
-  /* Turn on printing of static fields.  */
-  static_field_print = 1;
-
-  add_show_from_set
-    (add_set_cmd ("vtbl", class_support, var_boolean, (char *) &vtblprint,
-                 "Set printing of C++ virtual function tables.",
-                 &setprintlist),
-     &showprintlist);
-
-  add_show_from_set
-    (add_set_cmd ("object", class_support, var_boolean, (char *) &objectprint,
-             "Set printing of object's derived type based on vtable info.",
-                 &setprintlist),
-     &showprintlist);
-
-  /* Give people the defaults which they are used to.  */
-  objectprint = 0;
-  vtblprint = 0;
-  obstack_begin (&dont_print_vb_obstack, 32 * sizeof (struct type *));
-  obstack_specify_allocation (&dont_print_statmem_obstack,
-                             32 * sizeof (CORE_ADDR), sizeof (CORE_ADDR),
-                             xmalloc, xfree);
+  obstack_begin (&dont_print_stat_array_obstack,
+                32 * sizeof (struct type *));
+  obstack_begin (&dont_print_statmem_obstack,
+                32 * sizeof (CORE_ADDR));
+  obstack_begin (&dont_print_vb_obstack,
+                32 * sizeof (struct type *));
 }