OSDN Git Service

* g++.old-deja/g++.benjamin/16077.C: Adjust warnings.
[pf3gnuchains/gcc-fork.git] / gcc / cp / init.c
index 68f92b6..99812c6 100644 (file)
@@ -1,22 +1,22 @@
 /* Handle initialization things in C++.
    Copyright (C) 1987, 1989, 1992, 1993, 1994, 1995, 1996, 1997, 1998,
-   1999, 2000 Free Software Foundation, Inc.
+   1999, 2000, 2001, 2002, 2003 Free Software Foundation, Inc.
    Contributed by Michael Tiemann (tiemann@cygnus.com)
 
-This file is part of GNU CC.
+This file is part of GCC.
 
-GNU CC is free software; you can redistribute it and/or modify
+GCC 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, or (at your option)
 any later version.
 
-GNU CC is distributed in the hope that it will be useful,
+GCC 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 GNU CC; see the file COPYING.  If not, write to
+along with GCC; see the file COPYING.  If not, write to
 the Free Software Foundation, 59 Temple Place - Suite 330,
 Boston, MA 02111-1307, USA.  */
 
@@ -24,59 +24,37 @@ Boston, MA 02111-1307, USA.  */
 
 #include "config.h"
 #include "system.h"
+#include "coretypes.h"
+#include "tm.h"
 #include "tree.h"
 #include "rtl.h"
+#include "expr.h"
 #include "cp-tree.h"
 #include "flags.h"
 #include "output.h"
 #include "except.h"
-#include "expr.h"
 #include "toplev.h"
-#include "ggc.h"
 
-static void expand_aggr_vbase_init_1 PARAMS ((tree, tree, tree, tree));
-static void construct_virtual_bases PARAMS ((tree, tree, tree, tree, tree));
+static void construct_virtual_base (tree, tree);
 static void expand_aggr_init_1 PARAMS ((tree, tree, tree, tree, int));
 static void expand_default_init PARAMS ((tree, tree, tree, tree, int));
 static tree build_vec_delete_1 PARAMS ((tree, tree, tree, special_function_kind, int));
-static void perform_member_init PARAMS ((tree, tree, int));
-static void sort_base_init PARAMS ((tree, tree, tree *, tree *));
+static void perform_member_init (tree, tree);
 static tree build_builtin_delete_call PARAMS ((tree));
-static int member_init_ok_or_else PARAMS ((tree, tree, const char *));
+static int member_init_ok_or_else PARAMS ((tree, tree, tree));
 static void expand_virtual_init PARAMS ((tree, tree));
-static tree sort_member_init PARAMS ((tree, tree));
+static tree sort_mem_initializers (tree, tree);
 static tree initializing_context PARAMS ((tree));
 static void expand_cleanup_for_base PARAMS ((tree, tree));
 static tree get_temp_regvar PARAMS ((tree, tree));
 static tree dfs_initialize_vtbl_ptrs PARAMS ((tree, void *));
+static tree build_default_init PARAMS ((tree, tree));
 static tree build_new_1        PARAMS ((tree));
 static tree get_cookie_size PARAMS ((tree));
 static tree build_dtor_call PARAMS ((tree, special_function_kind, int));
 static tree build_field_list PARAMS ((tree, tree, int *));
 static tree build_vtbl_address PARAMS ((tree));
 
-/* Set up local variable for this file.  MUST BE CALLED AFTER
-   INIT_DECL_PROCESSING.  */
-
-static tree BI_header_type;
-
-void init_init_processing ()
-{
-  tree fields[1];
-
-  minus_one_node = build_int_2 (-1, -1);
-
-  /* Define the structure that holds header information for
-     arrays allocated via operator new.  */
-  BI_header_type = make_aggr_type (RECORD_TYPE);
-  fields[0] = build_decl (FIELD_DECL, nelts_identifier, sizetype);
-
-  finish_builtin_type (BI_header_type, "__new_cookie", fields,
-                      0, double_type_node);
-
-  ggc_add_tree_root (&BI_header_type, 1);
-}
-
 /* We are about to generate some complex initialization code.
    Conceptually, it is all a single expression.  However, we may want
    to include conditionals, loops, and other such statement-level
@@ -98,8 +76,6 @@ begin_init_stmts (stmt_expr_p, compound_stmt_p)
   
   if (building_stmt_tree ())
     *compound_stmt_p = begin_compound_stmt (/*has_no_scope=*/1);
-  //  else
-  //    *compound_stmt_p = genrtl_begin_compound_stmt (has_no_scope=1);
 }
 
 /* Finish out the statement-expression begun by the previous call to
@@ -115,7 +91,10 @@ finish_init_stmts (stmt_expr, compound_stmt)
     finish_compound_stmt (/*has_no_scope=*/1, compound_stmt);
   
   if (building_stmt_tree ())
-    stmt_expr = finish_stmt_expr (stmt_expr);
+    {
+      stmt_expr = finish_stmt_expr (stmt_expr);
+      STMT_EXPR_NO_SCOPE (stmt_expr) = true;
+    }
   else
     stmt_expr = finish_global_stmt_expr (stmt_expr);
   
@@ -129,33 +108,26 @@ finish_init_stmts (stmt_expr, compound_stmt)
 
 /* Constructors */
 
-/* Called from initialize_vtbl_ptrs via dfs_walk.  */
+/* Called from initialize_vtbl_ptrs via dfs_walk.  BINFO is the base
+   which we want to initialize the vtable pointer for, DATA is
+   TREE_LIST whose TREE_VALUE is the this ptr expression.  */
 
 static tree
 dfs_initialize_vtbl_ptrs (binfo, data)
      tree binfo;
      void *data;
 {
-  if (!BINFO_PRIMARY_MARKED_P (binfo) 
+  if ((!BINFO_PRIMARY_P (binfo) || TREE_VIA_VIRTUAL (binfo))
       && CLASSTYPE_VFIELDS (BINFO_TYPE (binfo)))
     {
       tree base_ptr = TREE_VALUE ((tree) data);
 
-      if (TREE_VIA_VIRTUAL (binfo))
-       base_ptr = convert_pointer_to_vbase (BINFO_TYPE (binfo),
-                                            base_ptr);
-      else
-       base_ptr 
-         = build_vbase_path (PLUS_EXPR, 
-                             build_pointer_type (BINFO_TYPE (binfo)),
-                             base_ptr,
-                             binfo,
-                             /*nonnull=*/1);
+      base_ptr = build_base_path (PLUS_EXPR, base_ptr, binfo, /*nonnull=*/1);
 
       expand_virtual_init (binfo, base_ptr);
     }
 
-  SET_BINFO_MARKED (binfo);
+  BINFO_MARKED (binfo) = 1;
 
   return NULL_TREE;
 }
@@ -174,35 +146,196 @@ initialize_vtbl_ptrs (addr)
   list = build_tree_list (type, addr);
 
   /* Walk through the hierarchy, initializing the vptr in each base
-     class.  We do these in pre-order because under the new ABI we
-     can't find the virtual bases for a class until we've initialized
-     the vtbl for that class.  */
-  dfs_walk_real (TYPE_BINFO (type), dfs_initialize_vtbl_ptrs, 
-                NULL, dfs_unmarked_real_bases_queue_p, list);
-  dfs_walk (TYPE_BINFO (type), dfs_unmark,
-           dfs_marked_real_bases_queue_p, type);
-
-  /* If we're not using thunks, we may need to adjust the deltas in
-     the vtable to handle virtual base classes correctly.  When we are
-     using thunks, we either use construction vtables (which are
-     preloaded with the right answers) or nothing (in which case
-     vitual function calls sometimes don't work right.)  */
-  if (TYPE_USES_VIRTUAL_BASECLASSES (type) && !flag_vtable_thunks)
-    fixup_all_virtual_upcast_offsets (addr);
+     class.  We do these in pre-order because can't find the virtual
+     bases for a class until we've initialized the vtbl for that
+     class.  */
+  dfs_walk_real (TYPE_BINFO (type), dfs_initialize_vtbl_ptrs,
+                NULL, unmarkedp, list);
+  dfs_walk (TYPE_BINFO (type), dfs_unmark, markedp, type);
+}
+
+/* Return an expression for the zero-initialization of an object with
+   type T.  This expression will either be a constant (in the case
+   that T is a scalar), or a CONSTRUCTOR (in the case that T is an
+   aggregate).  In either case, the value can be used as DECL_INITIAL
+   for a decl of the indicated TYPE; it is a valid static initializer.
+   If NELTS is non-NULL, and TYPE is an ARRAY_TYPE, NELTS is the
+   number of elements in the array.  If STATIC_STORAGE_P is TRUE,
+   initializers are only generated for entities for which
+   zero-initialization does not simply mean filling the storage with
+   zero bytes.  */
+
+tree
+build_zero_init (tree type, tree nelts, bool static_storage_p)
+{
+  tree init = NULL_TREE;
+
+  /* [dcl.init]
+
+     To zero-initialization storage for an object of type T means:
+
+     -- if T is a scalar type, the storage is set to the value of zero
+        converted to T.
+
+     -- if T is a non-union class type, the storage for each nonstatic
+        data member and each base-class subobject is zero-initialized.
+
+     -- if T is a union type, the storage for its first data member is
+        zero-initialized.
+
+     -- if T is an array type, the storage for each element is
+        zero-initialized.
+
+     -- if T is a reference type, no initialization is performed.  */
+
+  if (type == error_mark_node)
+    ;
+  else if (static_storage_p && zero_init_p (type))
+    /* In order to save space, we do not explicitly build initializers
+       for items that do not need them.  GCC's semantics are that
+       items with static storage duration that are not otherwise
+       initialized are initialized to zero.  */
+    ;
+  else if (SCALAR_TYPE_P (type))
+    init = convert (type, integer_zero_node);
+  else if (CLASS_TYPE_P (type))
+    {
+      tree field;
+      tree inits;
+
+      /* Build a constructor to contain the initializations.  */
+      init = build (CONSTRUCTOR, type, NULL_TREE, NULL_TREE);
+      /* Iterate over the fields, building initializations.  */
+      inits = NULL_TREE;
+      for (field = TYPE_FIELDS (type); field; field = TREE_CHAIN (field))
+       {
+         if (TREE_CODE (field) != FIELD_DECL)
+           continue;
+
+         /* Note that for class types there will be FIELD_DECLs
+            corresponding to base classes as well.  Thus, iterating
+            over TYPE_FIELDs will result in correct initialization of
+            all of the subobjects.  */
+         if (static_storage_p && !zero_init_p (TREE_TYPE (field)))
+           inits = tree_cons (field, 
+                              build_zero_init (TREE_TYPE (field),
+                                               /*nelts=*/NULL_TREE,
+                                               static_storage_p),
+                              inits);
+
+         /* For unions, only the first field is initialized.  */
+         if (TREE_CODE (type) == UNION_TYPE)
+           break;
+       }
+      CONSTRUCTOR_ELTS (init) = nreverse (inits);
+    }
+  else if (TREE_CODE (type) == ARRAY_TYPE)
+    {
+      tree index;
+      tree max_index;
+      tree inits;
+
+      /* Build a constructor to contain the initializations.  */
+      init = build (CONSTRUCTOR, type, NULL_TREE, NULL_TREE);
+      /* Iterate over the array elements, building initializations.  */
+      inits = NULL_TREE;
+      max_index = nelts ? nelts : array_type_nelts (type);
+      for (index = size_zero_node;
+          !tree_int_cst_lt (max_index, index);
+          index = size_binop (PLUS_EXPR, index, size_one_node))
+       inits = tree_cons (index,
+                          build_zero_init (TREE_TYPE (type),
+                                           /*nelts=*/NULL_TREE,
+                                           static_storage_p),
+                          inits);
+      CONSTRUCTOR_ELTS (init) = nreverse (inits);
+    }
+  else if (TREE_CODE (type) == REFERENCE_TYPE)
+    ;
+  else
+    abort ();
+
+  /* In all cases, the initializer is a constant.  */
+  if (init)
+    TREE_CONSTANT (init) = 1;
+
+  return init;
 }
 
-/* Subroutine of emit_base_init.  */
+/* Build an expression for the default-initialization of an object of
+   the indicated TYPE.  If NELTS is non-NULL, and TYPE is an
+   ARRAY_TYPE, NELTS is the number of elements in the array.  If
+   initialization of TYPE requires calling constructors, this function
+   returns NULL_TREE; the caller is responsible for arranging for the
+   constructors to be called.  */
+
+static tree
+build_default_init (type, nelts)
+     tree type;
+     tree nelts;
+{
+  /* [dcl.init]:
+
+    To default-initialize an object of type T means:
+
+    --if T is a non-POD class type (clause _class_), the default construc-
+      tor  for  T is called (and the initialization is ill-formed if T has
+      no accessible default constructor);
+
+    --if T is an array type, each element is default-initialized;
+
+    --otherwise, the storage for the object is zero-initialized.
+
+    A program that calls for default-initialization of an entity of refer-
+    ence type is ill-formed.  */
+
+  /* If TYPE_NEEDS_CONSTRUCTING is true, the caller is responsible for
+     performing the initialization.  This is confusing in that some
+     non-PODs do not have TYPE_NEEDS_CONSTRUCTING set.  (For example,
+     a class with a pointer-to-data member as a non-static data member
+     does not have TYPE_NEEDS_CONSTRUCTING set.)  Therefore, we end up
+     passing non-PODs to build_zero_init below, which is contrary to
+     the semantics quoted above from [dcl.init].  
+
+     It happens, however, that the behavior of the constructor the
+     standard says we should have generated would be precisely the
+     same as that obtained by calling build_zero_init below, so things
+     work out OK.  */
+  if (TYPE_NEEDS_CONSTRUCTING (type))
+    return NULL_TREE;
+      
+  /* At this point, TYPE is either a POD class type, an array of POD
+     classes, or something even more inoccuous.  */
+  return build_zero_init (type, nelts, /*static_storage_p=*/false);
+}
+
+/* Initialize MEMBER, a FIELD_DECL, with INIT, a TREE_LIST of
+   arguments.  If TREE_LIST is void_type_node, an empty initializer
+   list was given; if NULL_TREE no initializer was given.  */
 
 static void
-perform_member_init (member, init, explicit)
-     tree member, init;
-     int explicit;
+perform_member_init (tree member, tree init)
 {
   tree decl;
   tree type = TREE_TYPE (member);
+  bool explicit;
+
+  explicit = (init != NULL_TREE);
 
-  decl = build_component_ref (current_class_ref, member, NULL_TREE, explicit);
+  /* Effective C++ rule 12 requires that all data members be
+     initialized.  */
+  if (warn_ecpp && !explicit && TREE_CODE (type) != ARRAY_TYPE)
+    warning ("`%D' should be initialized in the member initialization "
+            "list", 
+            member);
 
+  if (init == void_type_node)
+    init = NULL_TREE;
+
+  /* Get an lvalue for the data member.  */
+  decl = build_class_member_access_expr (current_class_ref, member,
+                                        /*access_path=*/NULL_TREE,
+                                        /*preserve_reference=*/true);
   if (decl == error_mark_node)
     return;
 
@@ -220,11 +353,6 @@ perform_member_init (member, init, explicit)
   else if (TYPE_NEEDS_CONSTRUCTING (type)
           || (init && TYPE_HAS_CONSTRUCTOR (type)))
     {
-      /* Since `init' is already a TREE_LIST on the member_init_list,
-        only build it into one if we aren't already a list.  */
-      if (init != NULL_TREE && TREE_CODE (init) != TREE_LIST)
-       init = build_tree_list (NULL_TREE, init);
-
       if (explicit
          && TREE_CODE (type) == ARRAY_TYPE
          && init != NULL_TREE
@@ -232,9 +360,8 @@ perform_member_init (member, init, explicit)
          && TREE_CODE (TREE_TYPE (TREE_VALUE (init))) == ARRAY_TYPE)
        {
          /* Initialization of one array from another.  */
-         finish_expr_stmt 
-           (build_vec_init (TREE_OPERAND (decl, 1), decl,
-                            array_type_nelts (type), TREE_VALUE (init), 1));
+         finish_expr_stmt (build_vec_init (decl, NULL_TREE, TREE_VALUE (init),
+                                           /* from_array=*/1));
        }
       else
        finish_expr_stmt (build_aggr_init (decl, init, 0));
@@ -245,29 +372,15 @@ perform_member_init (member, init, explicit)
        {
          if (explicit)
            {
-             /* default-initialization.  */
-             if (AGGREGATE_TYPE_P (type))
-               {
-                 /* This is a default initialization of an aggregate,
-                    but not one of non-POD class type.  We cleverly
-                    notice that the initialization rules in such a
-                    case are the same as for initialization with an
-                    empty brace-initialization list.  We don't want
-                    to call build_modify_expr as that will go looking
-                    for constructors and such.  */
-                 tree e = build (CONSTRUCTOR, type, NULL_TREE, NULL_TREE);
-                 TREE_SIDE_EFFECTS (e) = 1;
-                 finish_expr_stmt (build (INIT_EXPR, type, decl, e));
-               }
-             else if (TREE_CODE (type) == REFERENCE_TYPE)
-               cp_error ("default-initialization of `%#D', which has reference type",
-                         member);
-             else
-               init = integer_zero_node;
+             init = build_default_init (type, /*nelts=*/NULL_TREE);
+             if (TREE_CODE (type) == REFERENCE_TYPE)
+               warning
+                 ("default-initialization of `%#D', which has reference type",
+                  member);
            }
          /* member traversal: note it leaves init NULL */
-         else if (TREE_CODE (TREE_TYPE (member)) == REFERENCE_TYPE)
-           cp_pedwarn ("uninitialized reference member `%D'", member);
+         else if (TREE_CODE (type) == REFERENCE_TYPE)
+           pedwarn ("uninitialized reference member `%D'", member);
        }
       else if (TREE_CODE (init) == TREE_LIST)
        {
@@ -290,13 +403,14 @@ perform_member_init (member, init, explicit)
     {
       tree expr;
 
-      expr = build_component_ref (current_class_ref, member, NULL_TREE,
-                                 explicit);
+      expr = build_class_member_access_expr (current_class_ref, member,
+                                            /*access_path=*/NULL_TREE,
+                                            /*preserve_reference=*/false);
       expr = build_delete (type, expr, sfk_complete_destructor,
                           LOOKUP_NONVIRTUAL|LOOKUP_DESTRUCTOR, 0);
 
       if (expr != error_mark_node)
-       finish_subobject (expr);
+       finish_eh_cleanup (expr);
     }
 }
 
@@ -311,6 +425,8 @@ build_field_list (t, list, uses_unions_p)
 {
   tree fields;
 
+  *uses_unions_p = 0;
+
   /* Note whether or not T is a union.  */
   if (TREE_CODE (t) == UNION_TYPE)
     *uses_unions_p = 1;
@@ -318,7 +434,7 @@ build_field_list (t, list, uses_unions_p)
   for (fields = TYPE_FIELDS (t); fields; fields = TREE_CHAIN (fields))
     {
       /* Skip CONST_DECLs for enumeration constants and so forth.  */
-      if (TREE_CODE (fields) != FIELD_DECL)
+      if (TREE_CODE (fields) != FIELD_DECL || DECL_ARTIFICIAL (fields))
        continue;
       
       /* Keep track of whether or not any fields are unions.  */
@@ -345,79 +461,107 @@ build_field_list (t, list, uses_unions_p)
   return list;
 }
 
-/* The MEMBER_INIT_LIST is a TREE_LIST.  The TREE_PURPOSE of each list
-   gives a FIELD_DECL in T that needs initialization.  The TREE_VALUE
-   gives the initializer, or list of initializer arguments.  Sort the
-   MEMBER_INIT_LIST, returning a version that contains the same
-   information but in the order that the fields should actually be
-   initialized.  Perform error-checking in the process.  */
+/* The MEM_INITS are a TREE_LIST.  The TREE_PURPOSE of each list gives
+   a FIELD_DECL or BINFO in T that needs initialization.  The
+   TREE_VALUE gives the initializer, or list of initializer arguments.
+
+   Return a TREE_LIST containing all of the initializations required
+   for T, in the order in which they should be performed.  The output
+   list has the same format as the input.  */
 
 static tree
-sort_member_init (t, member_init_list)
-     tree t;
-     tree member_init_list;
+sort_mem_initializers (tree t, tree mem_inits)
 {
-  tree init_list;
-  tree last_field;
   tree init;
+  tree base;
+  tree sorted_inits;
+  tree next_subobject;
+  int i;
   int uses_unions_p;
 
-  /* Build up a list of the various fields, in sorted order.  */
-  init_list = nreverse (build_field_list (t, NULL_TREE, &uses_unions_p));
-
-  /* Go through the explicit initializers, adding them to the
-     INIT_LIST.  */
-  last_field = init_list;
-  for (init = member_init_list; init; init = TREE_CHAIN (init))
-    {
-      tree f;
-      tree initialized_field;
-
-      initialized_field = TREE_PURPOSE (init);
-      my_friendly_assert (TREE_CODE (initialized_field) == FIELD_DECL,
-                         20000516);
-
-      /* If the explicit initializers are in sorted order, then the
-        INITIALIZED_FIELD will be for a field following the
-        LAST_FIELD.  */
-      for (f = last_field; f; f = TREE_CHAIN (f))
-       if (TREE_PURPOSE (f) == initialized_field)
+  /* Build up a list of initializations.  The TREE_PURPOSE of entry
+     will be the subobject (a FIELD_DECL or BINFO) to initialize.  The
+     TREE_VALUE will be the constructor arguments, or NULL if no
+     explicit initialization was provided.  */
+  sorted_inits = NULL_TREE;
+  /* Process the virtual bases.  */
+  for (base = CLASSTYPE_VBASECLASSES (t); base; base = TREE_CHAIN (base))
+    sorted_inits = tree_cons (TREE_VALUE (base), NULL_TREE, sorted_inits);
+  /* Process the direct bases.  */
+  for (i = 0; i < CLASSTYPE_N_BASECLASSES (t); ++i)
+    {
+      base = BINFO_BASETYPE (TYPE_BINFO (t), i);
+      if (!TREE_VIA_VIRTUAL (base))
+       sorted_inits = tree_cons (base, NULL_TREE, sorted_inits);
+    }
+  /* Process the non-static data members.  */
+  sorted_inits = build_field_list (t, sorted_inits, &uses_unions_p);
+  /* Reverse the entire list of initializations, so that they are in
+     the order that they will actually be performed.  */
+  sorted_inits = nreverse (sorted_inits);
+
+  /* If the user presented the initializers in an order different from
+     that in which they will actually occur, we issue a warning.  Keep
+     track of the next subobject which can be explicitly initialized
+     without issuing a warning.  */
+  next_subobject = sorted_inits;
+
+  /* Go through the explicit initializers, filling in TREE_PURPOSE in
+     the SORTED_INITS.  */
+  for (init = mem_inits; init; init = TREE_CHAIN (init))
+    {
+      tree subobject;
+      tree subobject_init;
+
+      subobject = TREE_PURPOSE (init);
+
+      /* If the explicit initializers are in sorted order, then
+        SUBOBJECT will be NEXT_SUBOBJECT, or something following 
+        it.  */
+      for (subobject_init = next_subobject; 
+          subobject_init; 
+          subobject_init = TREE_CHAIN (subobject_init))
+       if (TREE_PURPOSE (subobject_init) == subobject)
          break;
 
-      /* Give a warning, if appropriate.  */
-      if (warn_reorder && !f)
+      /* Issue a warning if the explicit initializer order does not
+        match that which will actually occur.  */
+      if (warn_reorder && !subobject_init)
        {
-         cp_warning_at ("member initializers for `%#D'", 
-                        TREE_PURPOSE (last_field));
-         cp_warning_at ("  and `%#D'", initialized_field);
-         warning ("  will be re-ordered to match declaration order");
+         if (TREE_CODE (TREE_PURPOSE (next_subobject)) == FIELD_DECL)
+           cp_warning_at ("`%D' will be initialized after",
+                          TREE_PURPOSE (next_subobject));
+         else
+           warning ("base `%T' will be initialized after",
+                    TREE_PURPOSE (next_subobject));
+         if (TREE_CODE (subobject) == FIELD_DECL)
+           cp_warning_at ("  `%#D'", subobject);
+         else
+           warning ("  base `%T'", subobject);
        }
 
-      /* Look again, from the beginning of the list.  We must find the
-        field on this loop.  */
-      if (!f)
+      /* Look again, from the beginning of the list.  */
+      if (!subobject_init)
        {
-         f = init_list;
-         while (TREE_PURPOSE (f) != initialized_field)
-           f = TREE_CHAIN (f);
+         subobject_init = sorted_inits;
+         while (TREE_PURPOSE (subobject_init) != subobject)
+           subobject_init = TREE_CHAIN (subobject_init);
        }
-
-      /* If there was already an explicit initializer for this field,
-        issue an error.  */
-      if (TREE_TYPE (f))
-       cp_error ("multiple initializations given for member `%D'",
-                 initialized_field);
-      else
+       
+      /* It is invalid to initialize the same subobject more than
+        once.  */
+      if (TREE_VALUE (subobject_init))
        {
-         /* Mark the field as explicitly initialized.  */
-         TREE_TYPE (f) = error_mark_node;
-         /* And insert the initializer.  */
-         TREE_VALUE (f) = TREE_VALUE (init);
+         if (TREE_CODE (subobject) == FIELD_DECL)
+           error ("multiple initializations given for `%D'", subobject);
+         else
+           error ("multiple initializations given for base `%T'", 
+                  subobject);
        }
 
-      /* Remember the location of the last explicitly initialized
-        field.  */
-      last_field = f;
+      /* Record the initialization.  */
+      TREE_VALUE (subobject_init) = TREE_VALUE (init);
+      next_subobject = subobject_init;
     }
 
   /* [class.base.init]
@@ -427,15 +571,16 @@ sort_member_init (t, member_init_list)
      anonymous unions), the ctor-initializer is ill-formed.  */
   if (uses_unions_p)
     {
-      last_field = NULL_TREE;
-      for (init = init_list; init; init = TREE_CHAIN (init))
+      tree last_field = NULL_TREE;
+      for (init = sorted_inits; init; init = TREE_CHAIN (init))
        {
          tree field;
          tree field_type;
          int done;
 
-         /* Skip uninitialized members.  */
-         if (!TREE_TYPE (init))
+         /* Skip uninitialized members and base classes.  */
+         if (!TREE_VALUE (init) 
+             || TREE_CODE (TREE_PURPOSE (init)) != FIELD_DECL)
            continue;
          /* See if this field is a member of a union, or a member of a
             structure contained in a union, etc.  */
@@ -477,7 +622,7 @@ sort_member_init (t, member_init_list)
                  if (same_type_p (last_field_type, field_type))
                    {
                      if (TREE_CODE (field_type) == UNION_TYPE)
-                       cp_error ("initializations for multiple members of `%T'",
+                       error ("initializations for multiple members of `%T'",
                                  last_field_type);
                      done = 1;
                      break;
@@ -502,261 +647,73 @@ sort_member_init (t, member_init_list)
        }
     }
 
-  return init_list;
-}
-
-/* Like sort_member_init, but used for initializers of base classes.
-   *RBASE_PTR is filled in with the initializers for non-virtual bases;
-   vbase_ptr gets the virtual bases.  */
-
-static void
-sort_base_init (t, base_init_list, rbase_ptr, vbase_ptr)
-     tree t;
-     tree base_init_list;
-     tree *rbase_ptr, *vbase_ptr;
-{
-  tree binfos = BINFO_BASETYPES (TYPE_BINFO (t));
-  int n_baseclasses = binfos ? TREE_VEC_LENGTH (binfos) : 0;
-
-  int i;
-  tree x;
-  tree last;
-
-  /* For warn_reorder.  */
-  int last_pos = 0;
-  tree last_base = NULL_TREE;
-
-  tree rbases = NULL_TREE;
-  tree vbases = NULL_TREE;
-
-  /* First walk through and splice out vbase and invalid initializers.
-     Also replace names with binfos.  */
-
-  last = tree_cons (NULL_TREE, NULL_TREE, base_init_list);
-  for (x = TREE_CHAIN (last); x; x = TREE_CHAIN (x))
-    {
-      tree basetype = TREE_PURPOSE (x);
-      tree binfo = NULL_TREE;
-
-      if (basetype == NULL_TREE)
-       {
-         /* Initializer for single base class.  Must not
-            use multiple inheritance or this is ambiguous.  */
-         switch (n_baseclasses)
-           {
-           case 0:
-             cp_error ("`%T' does not have a base class to initialize",
-                       current_class_type);
-             return;
-           case 1:
-             break;
-           default:
-             cp_error ("unnamed initializer ambiguous for `%T' which uses multiple inheritance",
-                       current_class_type);
-             return;
-           }
-         binfo = TREE_VEC_ELT (binfos, 0);
-       }
-      else if (is_aggr_type (basetype, 1))
-       {
-         binfo = binfo_or_else (basetype, t);
-         if (binfo == NULL_TREE)
-           continue;
-
-         /* Virtual base classes are special cases.  Their initializers
-            are recorded with this constructor, and they are used when
-            this constructor is the top-level constructor called.  */
-         if (TREE_VIA_VIRTUAL (binfo))
-           {
-             tree v = binfo_for_vbase (BINFO_TYPE (binfo), t);
-             vbases = tree_cons (v, TREE_VALUE (x), vbases);
-             continue;
-           }
-         else
-           {
-             /* Otherwise, if it is not an immediate base class, complain.  */
-             for (i = n_baseclasses-1; i >= 0; i--)
-               if (BINFO_TYPE (binfo) == BINFO_TYPE (TREE_VEC_ELT (binfos, i)))
-                 break;
-             if (i < 0)
-               {
-                 cp_error ("`%T' is not an immediate base class of `%T'",
-                           basetype, current_class_type);
-                 continue;
-               }
-           }
-       }
-      else
-       my_friendly_abort (365);
-
-      TREE_PURPOSE (x) = binfo;
-      TREE_CHAIN (last) = x;
-      last = x;
-    }
-  TREE_CHAIN (last) = NULL_TREE;
-
-  /* Now walk through our regular bases and make sure they're initialized.  */
-
-  for (i = 0; i < n_baseclasses; ++i)
-    {
-      /* The base for which we're currently initializing.  */
-      tree base_binfo = TREE_VEC_ELT (binfos, i);
-      /* The initializer for BASE_BINFO.  */
-      tree init;
-      int pos;
-
-      if (TREE_VIA_VIRTUAL (base_binfo))
-       continue;
-
-      /* We haven't found the BASE_BINFO yet.  */
-      init = NULL_TREE;
-      /* Loop through all the explicitly initialized bases, looking
-        for an appropriate initializer.  */
-      for (x = base_init_list, pos = 0; x; x = TREE_CHAIN (x), ++pos)
-       {
-         tree binfo = TREE_PURPOSE (x);
-
-         if (binfo == base_binfo && !init)
-           {
-             if (warn_reorder)
-               {
-                 if (pos < last_pos)
-                   {
-                     cp_warning_at ("base initializers for `%#T'", last_base);
-                     cp_warning_at ("  and `%#T'", BINFO_TYPE (binfo));
-                     warning ("  will be re-ordered to match inheritance order");
-                   }
-                 last_pos = pos;
-                 last_base = BINFO_TYPE (binfo);
-               }
-
-             /* Make sure we won't try to work on this init again.  */
-             TREE_PURPOSE (x) = NULL_TREE;
-             init = build_tree_list (binfo, TREE_VALUE (x));
-           }
-         else if (binfo == base_binfo)
-           {
-             cp_error ("base class `%T' already initialized", 
-                       BINFO_TYPE (binfo));
-             break;
-           }
-       }
-
-      /* If we didn't find BASE_BINFO in the list, create a dummy entry
-        so the two lists (RBASES and the list of bases) will be
-        symmetrical.  */
-      if (!init)
-       init = build_tree_list (NULL_TREE, NULL_TREE);
-      rbases = chainon (rbases, init);
-    }
-
-  *rbase_ptr = rbases;
-  *vbase_ptr = vbases;
+  return sorted_inits;
 }
 
-/* Perform whatever initializations have yet to be done on the base
-   class, and non-static data members, of the CURRENT_CLASS_TYPE.
-   These actions are given by the BASE_INIT_LIST and MEM_INIT_LIST,
-   respectively.
-
-   If there is a need for a call to a constructor, we must surround
-   that call with a pushlevel/poplevel pair, since we are technically
-   at the PARM level of scope.
-
-   Note that emit_base_init does *not* initialize virtual base
-   classes.  That is done specially, elsewhere.  */
+/* Initialize all bases and members of CURRENT_CLASS_TYPE.  MEM_INITS
+   is a TREE_LIST giving the explicit mem-initializer-list for the
+   constructor.  The TREE_PURPOSE of each entry is a subobject (a
+   FIELD_DECL or a BINFO) of the CURRENT_CLASS_TYPE.  The TREE_VALUE
+   is a TREE_LIST giving the arguments to the constructor or
+   void_type_node for an empty list of arguments.  */
 
 void
-emit_base_init (mem_init_list, base_init_list)
-     tree mem_init_list;
-     tree base_init_list;
+emit_mem_initializers (tree mem_inits)
 {
-  tree member;
-  tree rbase_init_list, vbase_init_list;
-  tree t = current_class_type;
-  tree t_binfo = TYPE_BINFO (t);
-  tree binfos = BINFO_BASETYPES (t_binfo);
-  int i;
-  int n_baseclasses = BINFO_N_BASETYPES (t_binfo);
-
-  mem_init_list = sort_member_init (t, mem_init_list);
-  sort_base_init (t, base_init_list, &rbase_init_list, &vbase_init_list);
-
-  /* First, initialize the virtual base classes, if we are
-     constructing the most-derived object.  */
-  if (TYPE_USES_VIRTUAL_BASECLASSES (t))
-    {
-      tree first_arg = TREE_CHAIN (DECL_ARGUMENTS (current_function_decl));
-      construct_virtual_bases (t, current_class_ref, current_class_ptr,
-                              vbase_init_list, first_arg);
-    }
-
-  /* Now, perform initialization of non-virtual base classes.  */
-  for (i = 0; i < n_baseclasses; i++)
-    {
-      tree base_binfo = TREE_VEC_ELT (binfos, i);
-      tree init = void_list_node;
-
-      if (TREE_VIA_VIRTUAL (base_binfo))
-       continue;
-
-      my_friendly_assert (BINFO_INHERITANCE_CHAIN (base_binfo) == t_binfo,
-                         999);
-
-      if (TREE_PURPOSE (rbase_init_list))
-       init = TREE_VALUE (rbase_init_list);
-      else if (TYPE_NEEDS_CONSTRUCTING (BINFO_TYPE (base_binfo)))
-       {
-         init = NULL_TREE;
-         if (extra_warnings 
-             && DECL_COPY_CONSTRUCTOR_P (current_function_decl))
-           cp_warning ("base class `%#T' should be explicitly initialized in the copy constructor",
-                       BINFO_TYPE (base_binfo));
-       }
-
-      if (init != void_list_node)
+  /* Sort the mem-initializers into the order in which the
+     initializations should be performed.  */
+  mem_inits = sort_mem_initializers (current_class_type, mem_inits);
+
+  /* Initialize base classes.  */
+  while (mem_inits 
+        && TREE_CODE (TREE_PURPOSE (mem_inits)) != FIELD_DECL)
+    {
+      tree subobject = TREE_PURPOSE (mem_inits);
+      tree arguments = TREE_VALUE (mem_inits);
+
+      /* If these initializations are taking place in a copy
+        constructor, the base class should probably be explicitly
+        initialized.  */
+      if (extra_warnings && !arguments 
+         && DECL_COPY_CONSTRUCTOR_P (current_function_decl)
+         && TYPE_NEEDS_CONSTRUCTING (BINFO_TYPE (subobject)))
+       warning ("base class `%#T' should be explicitly initialized in the "
+                "copy constructor",
+                BINFO_TYPE (subobject));
+
+      /* If an explicit -- but empty -- initializer list was present,
+        treat it just like default initialization at this point.  */
+      if (arguments == void_type_node)
+       arguments = NULL_TREE;
+
+      /* Initialize the base.  */
+      if (TREE_VIA_VIRTUAL (subobject))
+       construct_virtual_base (subobject, arguments);
+      else
        {
-         member = convert_pointer_to_real (base_binfo, current_class_ptr);
-         expand_aggr_init_1 (base_binfo, NULL_TREE,
-                             build_indirect_ref (member, NULL_PTR), init,
+         tree base_addr;
+         
+         base_addr = build_base_path (PLUS_EXPR, current_class_ptr,
+                                      subobject, 1);
+         expand_aggr_init_1 (subobject, NULL_TREE,
+                             build_indirect_ref (base_addr, NULL), 
+                             arguments,
                              LOOKUP_NORMAL);
+         expand_cleanup_for_base (subobject, NULL_TREE);
        }
 
-      expand_cleanup_for_base (base_binfo, NULL_TREE);
-      rbase_init_list = TREE_CHAIN (rbase_init_list);
+      mem_inits = TREE_CHAIN (mem_inits);
     }
 
-  /* Initialize the vtable pointers for the class.  */
+  /* Initialize the vptrs.  */
   initialize_vtbl_ptrs (current_class_ptr);
 
-  while (mem_init_list)
+  /* Initialize the data members.  */
+  while (mem_inits)
     {
-      tree init;
-      tree member;
-      int from_init_list;
-
-      member = TREE_PURPOSE (mem_init_list);
-
-      /* See if we had a user-specified member initialization.  */
-      if (TREE_TYPE (mem_init_list))
-       {
-         init = TREE_VALUE (mem_init_list);
-         from_init_list = 1;
-       }
-      else
-       {
-         init = DECL_INITIAL (member);
-         from_init_list = 0;
-
-         /* Effective C++ rule 12.  */
-         if (warn_ecpp && init == NULL_TREE
-             && !DECL_ARTIFICIAL (member)
-             && TREE_CODE (TREE_TYPE (member)) != ARRAY_TYPE)
-           cp_warning ("`%D' should be initialized in the member initialization list", member);            
-       }
-
-      perform_member_init (member, init, from_init_list);
-      mem_init_list = TREE_CHAIN (mem_init_list);
+      perform_member_init (TREE_PURPOSE (mem_inits),
+                          TREE_VALUE (mem_inits));
+      mem_inits = TREE_CHAIN (mem_inits);
     }
 }
 
@@ -767,16 +724,27 @@ static tree
 build_vtbl_address (binfo)
      tree binfo;
 {
+  tree binfo_for = binfo;
   tree vtbl;
 
+  if (BINFO_VPTR_INDEX (binfo) && TREE_VIA_VIRTUAL (binfo)
+      && BINFO_PRIMARY_P (binfo))
+    /* If this is a virtual primary base, then the vtable we want to store
+       is that for the base this is being used as the primary base of.  We
+       can't simply skip the initialization, because we may be expanding the
+       inits of a subobject constructor where the virtual base layout
+       can be different.  */
+    while (BINFO_PRIMARY_BASE_OF (binfo_for))
+      binfo_for = BINFO_PRIMARY_BASE_OF (binfo_for);
+
   /* Figure out what vtable BINFO's vtable is based on, and mark it as
      used.  */
-  vtbl = get_vtbl_decl_for_binfo (binfo);
+  vtbl = get_vtbl_decl_for_binfo (binfo_for);
   assemble_external (vtbl);
   TREE_USED (vtbl) = 1;
 
   /* Now compute the address to use when initializing the vptr.  */
-  vtbl = BINFO_VTABLE (binfo);
+  vtbl = BINFO_VTABLE (binfo_for);
   if (TREE_CODE (vtbl) == VAR_DECL)
     {
       vtbl = build1 (ADDR_EXPR, build_pointer_type (TREE_TYPE (vtbl)), vtbl);
@@ -796,20 +764,14 @@ static void
 expand_virtual_init (binfo, decl)
      tree binfo, decl;
 {
-  tree type = BINFO_TYPE (binfo);
   tree vtbl, vtbl_ptr;
-  tree vtype, vtype_binfo;
   tree vtt_index;
 
-  /* Compute the location of the vtable.  */
-  vtype = DECL_CONTEXT (TYPE_VFIELD (type));
-  vtype_binfo = get_binfo (vtype, TREE_TYPE (TREE_TYPE (decl)), 0);
-  
   /* Compute the initializer for vptr.  */
   vtbl = build_vtbl_address (binfo);
 
-  /* Under the new ABI, we may get this vptr from a VTT, if this is a
-     subobject constructor or subobject destructor.  */
+  /* We may get this vptr from a VTT, if this is a subobject
+     constructor or subobject destructor.  */
   vtt_index = BINFO_VPTR_INDEX (binfo);
   if (vtt_index)
     {
@@ -817,7 +779,7 @@ expand_virtual_init (binfo, decl)
       tree vtt_parm;
 
       /* Compute the value to use, when there's a VTT.  */
-      vtt_parm = DECL_VTT_PARM (current_function_decl);
+      vtt_parm = current_vtt_parm;
       vtbl2 = build (PLUS_EXPR, 
                     TREE_TYPE (vtt_parm), 
                     vtt_parm,
@@ -829,16 +791,16 @@ expand_virtual_init (binfo, decl)
         the vtt_parm in the case of the non-subobject constructor.  */
       vtbl = build (COND_EXPR, 
                    TREE_TYPE (vtbl), 
-                   DECL_USE_VTT_PARM (current_function_decl),
+                   build (EQ_EXPR, boolean_type_node,
+                          current_in_charge_parm, integer_zero_node),
                    vtbl2, 
                    vtbl);
     }
 
   /* Compute the location of the vtpr.  */
-  decl = convert_pointer_to_real (vtype_binfo, decl);
-  vtbl_ptr = build_vfield_ref (build_indirect_ref (decl, NULL_PTR), vtype);
-  if (vtbl_ptr == error_mark_node)
-    return;
+  vtbl_ptr = build_vfield_ref (build_indirect_ref (decl, NULL),
+                              TREE_TYPE (binfo));
+  my_friendly_assert (vtbl_ptr != error_mark_node, 20010730);
 
   /* Assign the vtable to the vptr.  */
   vtbl = convert_force (TREE_TYPE (vtbl_ptr), vtbl, 0);
@@ -848,7 +810,7 @@ expand_virtual_init (binfo, decl)
 /* If an exception is thrown in a constructor, those base classes already
    constructed must be destroyed.  This function creates the cleanup
    for BINFO, which has just been constructed.  If FLAG is non-NULL,
-   it is a DECL which is non-zero when this base needs to be
+   it is a DECL which is nonzero when this base needs to be
    destroyed.  */
 
 static void
@@ -862,115 +824,70 @@ expand_cleanup_for_base (binfo, flag)
     return;
 
   /* Call the destructor.  */
-  expr = (build_scoped_method_call
-         (current_class_ref, binfo, base_dtor_identifier, NULL_TREE));
+  expr = build_special_member_call (current_class_ref, 
+                                   base_dtor_identifier,
+                                   NULL_TREE,
+                                   binfo,
+                                   LOOKUP_NORMAL | LOOKUP_NONVIRTUAL);
   if (flag)
     expr = fold (build (COND_EXPR, void_type_node,
-                       truthvalue_conversion (flag),
+                       c_common_truthvalue_conversion (flag),
                        expr, integer_zero_node));
 
-  finish_subobject (expr);
+  finish_eh_cleanup (expr);
 }
 
-/* Subroutine of `expand_aggr_vbase_init'.
-   BINFO is the binfo of the type that is being initialized.
-   INIT_LIST is the list of initializers for the virtual baseclass.  */
+/* Construct the virtual base-class VBASE passing the ARGUMENTS to its
+   constructor.  */
 
 static void
-expand_aggr_vbase_init_1 (binfo, exp, addr, init_list)
-     tree binfo, exp, addr, init_list;
+construct_virtual_base (tree vbase, tree arguments)
 {
-  tree init = purpose_member (binfo, init_list);
-  tree ref = build_indirect_ref (addr, NULL_PTR);
-
-  if (init)
-    init = TREE_VALUE (init);
-  /* Call constructors, but don't set up vtables.  */
-  expand_aggr_init_1 (binfo, exp, ref, init, LOOKUP_COMPLAIN);
-}
-
-/* Construct the virtual base-classes of THIS_REF (whose address is
-   THIS_PTR).  The object has the indicated TYPE.  The construction
-   actually takes place only if FLAG is non-zero.  INIT_LIST is list
-   of initializations for constructors to perform.  */
-
-static void
-construct_virtual_bases (type, this_ref, this_ptr, init_list, flag)
-     tree type;
-     tree this_ref;
-     tree this_ptr;
-     tree init_list;
-     tree flag;
-{
-  tree vbases;
-
-  /* If there are no virtual baseclasses, we shouldn't even be here.  */
-  my_friendly_assert (TYPE_USES_VIRTUAL_BASECLASSES (type), 19990621);
-
-  /* First set the pointers in our object that tell us where to find
-     our virtual baseclasses.  */
-  if (!vbase_offsets_in_vtable_p ())
-    {
-      tree if_stmt;
-      tree result;
-
-      if_stmt = begin_if_stmt ();
-      finish_if_stmt_cond (flag, if_stmt);
-      result = init_vbase_pointers (type, this_ptr);
-      if (result)
-       finish_expr_stmt (build_compound_expr (result));
-      finish_then_clause (if_stmt);
-      finish_if_stmt ();
-    }
-
-  /* Now, run through the baseclasses, initializing each.  */ 
-  for (vbases = CLASSTYPE_VBASECLASSES (type); vbases;
-       vbases = TREE_CHAIN (vbases))
-    {
-      tree inner_if_stmt;
-      tree compound_stmt;
-      tree exp;
-      tree vbase;
-
-      /* If there are virtual base classes with destructors, we need to
-        emit cleanups to destroy them if an exception is thrown during
-        the construction process.  These exception regions (i.e., the
-        period during which the cleanups must occur) begin from the time
-        the construction is complete to the end of the function.  If we
-        create a conditional block in which to initialize the
-        base-classes, then the cleanup region for the virtual base begins
-        inside a block, and ends outside of that block.  This situation
-        confuses the sjlj exception-handling code.  Therefore, we do not
-        create a single conditional block, but one for each
-        initialization.  (That way the cleanup regions always begin
-        in the outer block.)  We trust the back-end to figure out
-        that the FLAG will not change across initializations, and
-        avoid doing multiple tests.  */
-      inner_if_stmt = begin_if_stmt ();
-      finish_if_stmt_cond (flag, inner_if_stmt);
-      compound_stmt = begin_compound_stmt (/*has_no_scope=*/1);
-
-      /* Compute the location of the virtual base.  If we're
-        constructing virtual bases, then we must be the most derived
-        class.  Therefore, we don't have to look up the virtual base;
-        we already know where it is.  */
-      vbase = TREE_VALUE (vbases);
-      exp = build (PLUS_EXPR,
-                  TREE_TYPE (this_ptr),
-                  this_ptr,
-                  fold (build1 (NOP_EXPR, TREE_TYPE (this_ptr),
-                                BINFO_OFFSET (vbase))));
-      exp = build1 (NOP_EXPR, 
-                   build_pointer_type (BINFO_TYPE (vbase)), 
-                   exp);
-
-      expand_aggr_vbase_init_1 (vbase, this_ref, exp, init_list);
-      finish_compound_stmt (/*has_no_scope=*/1, compound_stmt);
-      finish_then_clause (inner_if_stmt);
-      finish_if_stmt ();
-      
-      expand_cleanup_for_base (vbase, flag);
-    }
+  tree inner_if_stmt;
+  tree compound_stmt;
+  tree exp;
+  tree flag;  
+
+  /* If there are virtual base classes with destructors, we need to
+     emit cleanups to destroy them if an exception is thrown during
+     the construction process.  These exception regions (i.e., the
+     period during which the cleanups must occur) begin from the time
+     the construction is complete to the end of the function.  If we
+     create a conditional block in which to initialize the
+     base-classes, then the cleanup region for the virtual base begins
+     inside a block, and ends outside of that block.  This situation
+     confuses the sjlj exception-handling code.  Therefore, we do not
+     create a single conditional block, but one for each
+     initialization.  (That way the cleanup regions always begin
+     in the outer block.)  We trust the back-end to figure out
+     that the FLAG will not change across initializations, and
+     avoid doing multiple tests.  */
+  flag = TREE_CHAIN (DECL_ARGUMENTS (current_function_decl));
+  inner_if_stmt = begin_if_stmt ();
+  finish_if_stmt_cond (flag, inner_if_stmt);
+  compound_stmt = begin_compound_stmt (/*has_no_scope=*/1);
+
+  /* Compute the location of the virtual base.  If we're
+     constructing virtual bases, then we must be the most derived
+     class.  Therefore, we don't have to look up the virtual base;
+     we already know where it is.  */
+  exp = build (PLUS_EXPR,
+              TREE_TYPE (current_class_ptr),
+              current_class_ptr,
+              fold (build1 (NOP_EXPR, TREE_TYPE (current_class_ptr),
+                            BINFO_OFFSET (vbase))));
+  exp = build1 (NOP_EXPR, 
+               build_pointer_type (BINFO_TYPE (vbase)), 
+               exp);
+  exp = build1 (INDIRECT_REF, BINFO_TYPE (vbase), exp);
+
+  expand_aggr_init_1 (vbase, current_class_ref, exp,
+                     arguments, LOOKUP_COMPLAIN);
+  finish_compound_stmt (/*has_no_scope=*/1, compound_stmt);
+  finish_then_clause (inner_if_stmt);
+  finish_if_stmt ();
+
+  expand_cleanup_for_base (vbase, flag);
 }
 
 /* Find the context in which this FIELD can be initialized.  */
@@ -999,132 +916,124 @@ static int
 member_init_ok_or_else (field, type, member_name)
      tree field;
      tree type;
-     const char *member_name;
+     tree member_name;
 {
   if (field == error_mark_node)
     return 0;
-  if (field == NULL_TREE || initializing_context (field) != type)
+  if (!field)
     {
-      cp_error ("class `%T' does not have any field named `%s'", type,
-               member_name);
+      error ("class `%T' does not have any field named `%D'", type,
+            member_name);
       return 0;
     }
-  if (TREE_STATIC (field))
+  if (TREE_CODE (field) == VAR_DECL)
     {
-      cp_error ("field `%#D' is static; only point of initialization is its declaration",
-               field);
+      error ("`%#D' is a static data member; it can only be "
+            "initialized at its definition",
+            field);
+      return 0;
+    }
+  if (TREE_CODE (field) != FIELD_DECL)
+    {
+      error ("`%#D' is not a non-static data member of `%T'",
+            field, type);
+      return 0;
+    }
+  if (initializing_context (field) != type)
+    {
+      error ("class `%T' does not have any field named `%D'", type,
+               member_name);
       return 0;
     }
 
   return 1;
 }
 
-/* If NAME is a viable field name for the aggregate DECL,
-   and PARMS is a viable parameter list, then expand an _EXPR
-   which describes this initialization.
+/* NAME is a FIELD_DECL, an IDENTIFIER_NODE which names a field, or it
+   is a _TYPE node or TYPE_DECL which names a base for that type.
+   INIT is a parameter list for that field's or base's constructor.
+   Check the validity of NAME, and return a TREE_LIST of the base
+   _TYPE or FIELD_DECL and the INIT.  If NAME is invalid, return
+   NULL_TREE and issue a diagnostic.
 
-   Note that we do not need to chase through the class's base classes
-   to look for NAME, because if it's in that list, it will be handled
-   by the constructor for that base class.
-
-   We do not yet have a fixed-point finder to instantiate types
-   being fed to overloaded constructors.  If there is a unique
-   constructor, then argument types can be got from that one.  */
+   An old style unnamed direct single base construction is permitted,
+   where NAME is NULL.  */
 
 tree
-expand_member_init (exp, name, init)
-     tree exp, name, init;
+expand_member_init (tree name, tree init)
 {
-  tree basetype = NULL_TREE, field;
-  tree type;
+  tree basetype;
+  tree field;
 
-  if (exp == NULL_TREE)
+  if (!current_class_ref)
     return NULL_TREE;
 
-  type = TYPE_MAIN_VARIANT (TREE_TYPE (exp));
-
-  if (name && TYPE_P (name))
+  if (!name)
     {
-      basetype = name;
-      name = TYPE_IDENTIFIER (name);
+      /* This is an obsolete unnamed base class initializer.  The
+        parser will already have warned about its use.  */
+      switch (CLASSTYPE_N_BASECLASSES (current_class_type))
+       {
+       case 0:
+         error ("unnamed initializer for `%T', which has no base classes",
+                current_class_type);
+         return NULL_TREE;
+       case 1:
+         basetype = TYPE_BINFO_BASETYPE (current_class_type, 0);
+         break;
+       default:
+         error ("unnamed initializer for `%T', which uses multiple inheritance",
+                current_class_type);
+         return NULL_TREE;
+      }
     }
-  else if (name && TREE_CODE (name) == TYPE_DECL)
+  else if (TYPE_P (name))
     {
-      basetype = TYPE_MAIN_VARIANT (TREE_TYPE (name));
-      name = DECL_NAME (name);
+      basetype = TYPE_MAIN_VARIANT (name);
+      name = TYPE_NAME (name);
     }
-
-  if (name == NULL_TREE && IS_AGGR_TYPE (type))
-    switch (CLASSTYPE_N_BASECLASSES (type))
-      {
-      case 0:
-       error ("base class initializer specified, but no base class to initialize");
-       return NULL_TREE;
-      case 1:
-       basetype = TYPE_BINFO_BASETYPE (type, 0);
-       break;
-      default:
-       error ("initializer for unnamed base class ambiguous");
-       cp_error ("(type `%T' uses multiple inheritance)", type);
-       return NULL_TREE;
-      }
+  else if (TREE_CODE (name) == TYPE_DECL)
+    basetype = TYPE_MAIN_VARIANT (TREE_TYPE (name));
+  else
+    basetype = NULL_TREE;
 
   my_friendly_assert (init != NULL_TREE, 0);
 
-  /* The grammar should not allow fields which have names that are
-     TYPENAMEs.  Therefore, if the field has a non-NULL TREE_TYPE, we
-     may assume that this is an attempt to initialize a base class
-     member of the current type.  Otherwise, it is an attempt to
-     initialize a member field.  */
+  if (basetype)
+    {
+      tree binfo;
 
-  if (init == void_type_node)
-    init = NULL_TREE;
+      if (current_template_parms)
+       return build_tree_list (basetype, init);
 
-  if (name == NULL_TREE || basetype)
-    {
-      if (name == NULL_TREE)
-       {
-#if 0
-         if (basetype)
-           name = TYPE_IDENTIFIER (basetype);
-         else
-           {
-             error ("no base class to initialize");
-             return;
-           }
-#endif
-       }
-      else if (basetype != type
-              && ! current_template_parms
-              && ! vec_binfo_member (basetype,
-                                     TYPE_BINFO_BASETYPES (type))
-              && ! binfo_for_vbase (basetype, type))
+      binfo = lookup_base (current_class_type, basetype, 
+                          ba_ignore, NULL);
+      if (!binfo || (!TREE_VIA_VIRTUAL (binfo)
+                    && (BINFO_INHERITANCE_CHAIN (binfo)
+                        != TYPE_BINFO (current_class_type))))
        {
-         if (IDENTIFIER_CLASS_VALUE (name))
-           goto try_member;
-         if (TYPE_USES_VIRTUAL_BASECLASSES (type))
-           cp_error ("type `%T' is not an immediate or virtual basetype for `%T'",
-                     basetype, type);
+         if (TYPE_USES_VIRTUAL_BASECLASSES (current_class_type))
+           error ("type `%D' is not a direct or virtual base of `%T'",
+                  name, current_class_type);
          else
-           cp_error ("type `%T' is not an immediate basetype for `%T'",
-                     basetype, type);
+           error ("type `%D' is not a direct base of `%T'",
+                  name, current_class_type);
          return NULL_TREE;
        }
-
-      init = build_tree_list (basetype, init);
+      return build_tree_list (binfo, init);
     }
   else
     {
-    try_member:
-      field = lookup_field (type, name, 1, 0);
-
-      if (! member_init_ok_or_else (field, type, IDENTIFIER_POINTER (name)))
-       return NULL_TREE;
+      if (TREE_CODE (name) == IDENTIFIER_NODE)
+       field = lookup_field (current_class_type, name, 1, false);
+      else
+       field = name;
 
-      init = build_tree_list (field, init);
+      if (member_init_ok_or_else (field, current_class_type, name))
+       return build_tree_list (field, init);
     }
 
-  return init;
+  return NULL_TREE;
 }
 
 /* This is like `expand_member_init', only it stores one aggregate
@@ -1148,8 +1057,6 @@ expand_member_init (exp, name, init)
    The virtual function table pointer cannot be set up here, because
    we do not really know its type.
 
-   Virtual baseclass pointers are also set up here.
-
    This never calls operator=().
 
    When initializing, nothing is CONST.
@@ -1201,16 +1108,14 @@ build_aggr_init (exp, init, flags)
               COMPLEX zees(1.0, 0.0)[10];
             }
          */
-         cp_error ("bad array initializer");
+         error ("bad array initializer");
          return error_mark_node;
        }
-      if (CP_TYPE_QUALS (type) != TYPE_UNQUALIFIED)
-       {
-         TREE_TYPE (exp) = TYPE_MAIN_VARIANT (type);
-         if (init)
-           TREE_TYPE (init) = TYPE_MAIN_VARIANT (itype);
-       }
-      stmt_expr = build_vec_init (exp, exp, array_type_nelts (type), init,
+      if (cp_type_quals (type) != TYPE_UNQUALIFIED)
+       TREE_TYPE (exp) = TYPE_MAIN_VARIANT (type);
+      if (itype && cp_type_quals (itype) != TYPE_UNQUALIFIED)
+       TREE_TYPE (init) = TYPE_MAIN_VARIANT (itype);
+      stmt_expr = build_vec_init (exp, NULL_TREE, init,
                                  init && same_type_p (TREE_TYPE (init),
                                                       TREE_TYPE (exp)));
       TREE_READONLY (exp) = was_const;
@@ -1228,11 +1133,11 @@ build_aggr_init (exp, init, flags)
   TREE_TYPE (exp) = TYPE_MAIN_VARIANT (type);
   begin_init_stmts (&stmt_expr, &compound_stmt);
   destroy_temps = stmts_are_full_exprs_p ();
-  current_stmt_tree->stmts_are_full_exprs_p = 0;
+  current_stmt_tree ()->stmts_are_full_exprs_p = 0;
   expand_aggr_init_1 (TYPE_BINFO (type), exp, exp,
                      init, LOOKUP_NORMAL|flags);
   stmt_expr = finish_init_stmts (stmt_expr, compound_stmt);
-  current_stmt_tree->stmts_are_full_exprs_p = destroy_temps;
+  current_stmt_tree ()->stmts_are_full_exprs_p = destroy_temps;
   TREE_TYPE (exp) = type;
   TREE_READONLY (exp) = was_const;
   TREE_THIS_VOLATILE (exp) = was_volatile;
@@ -1240,6 +1145,24 @@ build_aggr_init (exp, init, flags)
   return stmt_expr;
 }
 
+/* Like build_aggr_init, but not just for aggregates.  */
+
+tree
+build_init (decl, init, flags)
+     tree decl, init;
+     int flags;
+{
+  tree expr;
+
+  if (IS_AGGR_TYPE (TREE_TYPE (decl))
+      || TREE_CODE (TREE_TYPE (decl)) == ARRAY_TYPE)
+    expr = build_aggr_init (decl, init, flags);
+  else
+    expr = build (INIT_EXPR, TREE_TYPE (decl), decl, init);
+
+  return expr;
+}
+
 static void
 expand_default_init (binfo, true_exp, exp, init, flags)
      tree binfo;
@@ -1272,20 +1195,26 @@ expand_default_init (binfo, true_exp, exp, init, flags)
           to run a new constructor; and catching an exception, where we
           have already built up the constructor call so we could wrap it
           in an exception region.  */;
-      else if (TREE_CODE (init) == CONSTRUCTOR)
-       /* A brace-enclosed initializer has whatever type is
-          required.  There's no need to convert it.  */
-       ;
+      else if (TREE_CODE (init) == CONSTRUCTOR 
+              && TREE_HAS_CONSTRUCTOR (init))
+       {
+         /* A brace-enclosed initializer for an aggregate.  */
+         my_friendly_assert (CP_AGGREGATE_TYPE_P (type), 20021016);
+         init = digest_init (type, init, (tree *)NULL);
+       }
       else
        init = ocp_convert (type, init, CONV_IMPLICIT|CONV_FORCE_TEMP, flags);
 
-      if (TREE_CODE (init) == TRY_CATCH_EXPR)
-       /* We need to protect the initialization of a catch parm
-          with a call to terminate(), which shows up as a TRY_CATCH_EXPR
+      if (TREE_CODE (init) == MUST_NOT_THROW_EXPR)
+       /* We need to protect the initialization of a catch parm with a
+          call to terminate(), which shows up as a MUST_NOT_THROW_EXPR
           around the TARGET_EXPR for the copy constructor.  See
-          expand_start_catch_block.  */
-       TREE_OPERAND (init, 0) = build (INIT_EXPR, TREE_TYPE (exp), exp,
-                                       TREE_OPERAND (init, 0));
+          initialize_handler_parm.  */
+       {
+         TREE_OPERAND (init, 0) = build (INIT_EXPR, TREE_TYPE (exp), exp,
+                                         TREE_OPERAND (init, 0));
+         TREE_TYPE (init) = void_type_node;
+       }
       else
        init = build (INIT_EXPR, TREE_TYPE (exp), exp, init);
       TREE_SIDE_EFFECTS (init) = 1;
@@ -1308,7 +1237,7 @@ expand_default_init (binfo, true_exp, exp, init, flags)
   else
     ctor_name = base_ctor_identifier;
 
-  rval = build_method_call (exp, ctor_name, parms, binfo, flags);
+  rval = build_special_member_call (exp, ctor_name, parms, binfo, flags);
   if (TREE_SIDE_EFFECTS (rval))
     {
       if (building_stmt_tree ())
@@ -1348,6 +1277,7 @@ expand_aggr_init_1 (binfo, true_exp, exp, init, flags)
   tree type = TREE_TYPE (exp);
 
   my_friendly_assert (init != error_mark_node && type != error_mark_node, 211);
+  my_friendly_assert (building_stmt_tree (), 20021010);
 
   /* Use a function returning the desired type to initialize EXP for us.
      If the function is a constructor, and its first argument is
@@ -1362,12 +1292,7 @@ expand_aggr_init_1 (binfo, true_exp, exp, init, flags)
       /* If store_init_value returns NULL_TREE, the INIT has been
         record in the DECL_INITIAL for EXP.  That means there's
         nothing more we have to do.  */
-      if (!store_init_value (exp, init))
-       {
-         if (!building_stmt_tree ())
-           expand_decl_init (exp);
-       }
-      else
+      if (store_init_value (exp, init))
        finish_expr_stmt (build (INIT_EXPR, type, exp, init));
       return;
     }
@@ -1393,7 +1318,7 @@ is_aggr_type (type, or_else)
       && TREE_CODE (type) != BOUND_TEMPLATE_TEMPLATE_PARM)
     {
       if (or_else)
-       cp_error ("`%T' is not an aggregate type", type);
+       error ("`%T' is not an aggregate type", type);
       return 0;
     }
   return 1;
@@ -1416,7 +1341,7 @@ get_aggr_from_typedef (name, or_else)
   else
     {
       if (or_else)
-       cp_error ("`%T' fails to be an aggregate typedef", name);
+       error ("`%T' fails to be an aggregate typedef", name);
       return NULL_TREE;
     }
 
@@ -1425,7 +1350,7 @@ get_aggr_from_typedef (name, or_else)
       && TREE_CODE (type) != BOUND_TEMPLATE_TEMPLATE_PARM)
     {
       if (or_else)
-       cp_error ("type `%T' is of non-aggregate type", type);
+       error ("type `%T' is of non-aggregate type", type);
       return NULL_TREE;
     }
   return type;
@@ -1457,6 +1382,7 @@ build_member_call (type, name, parmlist)
 {
   tree t;
   tree method_name;
+  tree fns;
   int dtor = 0;
   tree basetype_path, decl;
 
@@ -1477,18 +1403,16 @@ build_member_call (type, name, parmlist)
          TREE_OPERAND (name, 0) = method_name;
        }
       my_friendly_assert (is_overloaded_fn (method_name), 980519);
-      return build_x_function_call (name, parmlist, current_class_ref);
+      return finish_call_expr (name, parmlist, /*disallow_virtual=*/true);
     }
 
   if (DECL_P (name))
     name = DECL_NAME (name);
 
-  if (type == std_node)
-    return build_x_function_call (do_scoped_id (name, 0), parmlist,
-                                 current_class_ref);
   if (TREE_CODE (type) == NAMESPACE_DECL)
-    return build_x_function_call (lookup_namespace_name (type, name),
-                                 parmlist, current_class_ref);
+    return finish_call_expr (lookup_namespace_name (type, name),
+                            parmlist,
+                            /*disallow_virtual=*/true);
 
   if (TREE_CODE (name) == TEMPLATE_ID_EXPR)
     {
@@ -1515,9 +1439,9 @@ build_member_call (type, name, parmlist)
     {
       tree ns = lookup_name (type, 0);
       if (ns && TREE_CODE (ns) == NAMESPACE_DECL)
-       {
-         return build_x_function_call (build_offset_ref (type, name), parmlist, current_class_ref);
-       }
+       return finish_call_expr (lookup_namespace_name (ns, name),
+                                parmlist,
+                                /*disallow_virtual=*/true);
     }
 
   if (type == NULL_TREE || ! is_aggr_type (type, 1))
@@ -1529,41 +1453,43 @@ build_member_call (type, name, parmlist)
 
   if (dtor)
     {
-      cp_error ("cannot call destructor `%T::~%T' without object", type,
+      error ("cannot call destructor `%T::~%T' without object", type,
                method_name);
       return error_mark_node;
     }
 
   decl = maybe_dummy_object (type, &basetype_path);
 
+  fns = lookup_fnfields (basetype_path, method_name, 0);
+  if (fns)
+    {
+      if (TREE_CODE (name) == TEMPLATE_ID_EXPR)
+       BASELINK_FUNCTIONS (fns) = build_nt (TEMPLATE_ID_EXPR,
+                                            BASELINK_FUNCTIONS (fns),
+                                            TREE_OPERAND (name, 1));
+      return build_new_method_call (decl, fns, parmlist,
+                                   /*conversion_path=*/NULL_TREE,
+                                   LOOKUP_NORMAL|LOOKUP_NONVIRTUAL);
+    }
+
   /* Convert 'this' to the specified type to disambiguate conversion
-     to the function's context.  Apparently Standard C++ says that we
-     shouldn't do this.  */
+     to the function's context.  */
   if (decl == current_class_ref
-      && ! pedantic
+      /* ??? this is wrong, but if this conversion is invalid we need to
+        defer it until we know whether we are calling a static or
+        non-static member function.  Be conservative for now.  */
       && ACCESSIBLY_UNIQUELY_DERIVED_P (type, current_class_type))
     {
-      tree olddecl = current_class_ptr;
-      tree oldtype = TREE_TYPE (TREE_TYPE (olddecl));
-      if (oldtype != type)
-       {
-         tree newtype = build_qualified_type (type, TYPE_QUALS (oldtype));
-         decl = convert_force (build_pointer_type (newtype), olddecl, 0);
-         decl = build_indirect_ref (decl, NULL_PTR);
-       }
+      basetype_path = NULL_TREE;
+      decl = build_scoped_ref (decl, type, &basetype_path);
+      if (decl == error_mark_node)
+       return error_mark_node;
     }
 
-  if (method_name == constructor_name (type)
-      || method_name == constructor_name_full (type))
+  if (constructor_name_p (method_name, type))
     return build_functional_cast (type, parmlist);
-  if (lookup_fnfields (basetype_path, method_name, 0))
-    return build_method_call (decl, 
-                             TREE_CODE (name) == TEMPLATE_ID_EXPR
-                             ? name : method_name,
-                             parmlist, basetype_path,
-                             LOOKUP_NORMAL|LOOKUP_NONVIRTUAL);
   if (TREE_CODE (name) == IDENTIFIER_NODE
-      && ((t = lookup_field (TYPE_BINFO (type), name, 1, 0))))
+      && ((t = lookup_field (TYPE_BINFO (type), name, 1, false))))
     {
       if (t == error_mark_node)
        return error_mark_node;
@@ -1571,7 +1497,7 @@ build_member_call (type, name, parmlist)
        {
          if (is_dummy_object (decl))
            {
-             cp_error ("invalid use of non-static field `%D'", t);
+             error ("invalid use of non-static field `%D'", t);
              return error_mark_node;
            }
          decl = build (COMPONENT_REF, TREE_TYPE (t), decl, t);
@@ -1580,17 +1506,17 @@ build_member_call (type, name, parmlist)
        decl = t;
       else
        {
-         cp_error ("invalid use of member `%D'", t);
+         error ("invalid use of member `%D'", t);
          return error_mark_node;
        }
       if (TYPE_LANG_SPECIFIC (TREE_TYPE (decl)))
-       return build_opfncall (CALL_EXPR, LOOKUP_NORMAL, decl,
-                              parmlist, NULL_TREE);
+       return build_new_op (CALL_EXPR, LOOKUP_NORMAL, decl,
+                            parmlist, NULL_TREE);
       return build_function_call (decl, parmlist);
     }
   else
     {
-      cp_error ("no method `%T::%D'", type, name);
+      error ("no method `%T::%D'", type, name);
       return error_mark_node;
     }
 }
@@ -1618,9 +1544,6 @@ build_offset_ref (type, name)
   if (TREE_CODE (name) == TEMPLATE_DECL)
     return name;
 
-  if (type == std_node)
-    return do_scoped_id (name, 0);
-
   if (processing_template_decl || uses_template_parms (type))
     return build_min_nt (SCOPE_REF, type, name);
 
@@ -1678,7 +1601,7 @@ build_offset_ref (type, name)
   if (TREE_CODE (name) == BIT_NOT_EXPR)
     {
       if (! check_dtor_name (type, name))
-       cp_error ("qualified type `%T' does not match destructor name `~%T'",
+       error ("qualified type `%T' does not match destructor name `~%T'",
                  type, TREE_OPERAND (name, 0));
       name = dtor_identifier;
     }
@@ -1686,24 +1609,29 @@ build_offset_ref (type, name)
   if (!COMPLETE_TYPE_P (complete_type (type))
       && !TYPE_BEING_DEFINED (type))
     {
-      cp_error ("incomplete type `%T' does not have member `%D'", type,
+      error ("incomplete type `%T' does not have member `%D'", type,
                name);
       return error_mark_node;
     }
 
   decl = maybe_dummy_object (type, &basebinfo);
 
-  member = lookup_member (basebinfo, name, 1, 0);
-
-  if (member == error_mark_node)
-    return error_mark_node;
+  if (BASELINK_P (name) || DECL_P (name))
+    member = name;
+  else
+    {
+      member = lookup_member (basebinfo, name, 1, 0);
+      
+      if (member == error_mark_node)
+       return error_mark_node;
+    }
 
   /* A lot of this logic is now handled in lookup_member.  */
   if (member && BASELINK_P (member))
     {
       /* Go from the TREE_BASELINK to the member function info.  */
       tree fnfields = member;
-      t = TREE_VALUE (fnfields);
+      t = BASELINK_FUNCTIONS (fnfields);
 
       if (TREE_CODE (orig_name) == TEMPLATE_ID_EXPR)
        {
@@ -1728,7 +1656,7 @@ build_offset_ref (type, name)
          return t;
        }
 
-      if (!really_overloaded_fn (t))
+      if (TREE_CODE (t) != TEMPLATE_ID_EXPR && !really_overloaded_fn (t))
        {
          /* Get rid of a potential OVERLOAD around it */
          t = OVL_CURRENT (t);
@@ -1755,7 +1683,7 @@ build_offset_ref (type, name)
 
   if (t == NULL_TREE)
     {
-      cp_error ("`%D' is not a member of type `%T'", name, type);
+      error ("`%D' is not a member of type `%T'", name, type);
       return error_mark_node;
     }
 
@@ -1774,14 +1702,14 @@ build_offset_ref (type, name)
 
   if (TREE_CODE (t) == FIELD_DECL && DECL_C_BIT_FIELD (t))
     {
-      cp_error ("illegal pointer to bit field `%D'", t);
+      error ("invalid pointer to bit-field `%D'", t);
       return error_mark_node;
     }
 
   /* static class functions too.  */
   if (TREE_CODE (t) == FUNCTION_DECL
       && TREE_CODE (TREE_TYPE (t)) == FUNCTION_TYPE)
-    my_friendly_abort (53);
+    abort ();
 
   /* In member functions, the form `type::name' is no longer
      equivalent to `this->type::name', at least not until
@@ -1840,7 +1768,7 @@ resolve_offset_ref (exp)
       || TREE_CODE (TREE_TYPE (member)) == FUNCTION_TYPE)
     {
       /* These were static members.  */
-      if (mark_addressable (member) == 0)
+      if (!cxx_mark_addressable (member))
        return error_mark_node;
       return member;
     }
@@ -1863,9 +1791,7 @@ resolve_offset_ref (exp)
   if (TREE_CODE (member) == FIELD_DECL
       && (base == current_class_ref || is_dummy_object (base)))
     {
-      tree expr;
-
-      basetype = DECL_CONTEXT (member);
+      tree binfo = NULL_TREE;
 
       /* Try to get to basetype from 'this'; if that doesn't work,
          nothing will.  */
@@ -1873,17 +1799,11 @@ resolve_offset_ref (exp)
 
       /* First convert to the intermediate base specified, if appropriate.  */
       if (TREE_CODE (exp) == OFFSET_REF && TREE_CODE (type) == OFFSET_TYPE)
-       base = build_scoped_ref (base, TYPE_OFFSET_BASETYPE (type));
-
-      addr = build_unary_op (ADDR_EXPR, base, 0);
-      addr = convert_pointer_to (basetype, addr);
+       base = build_scoped_ref (base, TYPE_OFFSET_BASETYPE (type), &binfo);
 
-      if (addr == error_mark_node)
-       return error_mark_node;
-
-      expr = build (COMPONENT_REF, TREE_TYPE (member),
-                   build_indirect_ref (addr, NULL_PTR), member);
-      return convert_from_reference (expr);
+      return build_class_member_access_expr (base, member,
+                                            /*access_path=*/NULL_TREE,
+                                            /*preserve_reference=*/false);
     }
 
   /* Ensure that we have an object.  */
@@ -1900,31 +1820,25 @@ resolve_offset_ref (exp)
     {
       if (addr == error_mark_node)
        {
-         cp_error ("object missing in `%E'", exp);
+         error ("object missing in `%E'", exp);
          return error_mark_node;
        }
 
       basetype = TYPE_OFFSET_BASETYPE (TREE_TYPE (TREE_TYPE (member)));
-      addr = convert_pointer_to (basetype, addr);
+      basetype = lookup_base (TREE_TYPE (TREE_TYPE (addr)),
+                             basetype, ba_check, NULL);
+      addr = build_base_path (PLUS_EXPR, addr, basetype, 1);
+      
       member = cp_convert (ptrdiff_type_node, member);
 
-      if (!flag_new_abi)
-       /* Pointer to data members are offset by one, so that a null
-          pointer with a real value of 0 is distinguishable from an
-          offset of the first member of a structure.  */
-       member = cp_build_binary_op (MINUS_EXPR, member,
-                                    cp_convert (ptrdiff_type_node, 
-                                                integer_one_node));
-
-      return build1 (INDIRECT_REF, type,
-                    build (PLUS_EXPR, build_pointer_type (type),
-                           addr, member));
+      addr = build (PLUS_EXPR, build_pointer_type (type), addr, member);
+      return build_indirect_ref (addr, 0);
     }
   else if (TYPE_PTRMEMFUNC_P (TREE_TYPE (member)))
     {
       return get_member_function_from_ptrfunc (&addr, member);
     }
-  my_friendly_abort (56);
+  abort ();
   /* NOTREACHED */
   return NULL_TREE;
 }
@@ -2013,7 +1927,7 @@ build_new (placement, decl, init, use_global_new)
       nelts = integer_one_node;
 
       if (absdcl && TREE_CODE (absdcl) == CALL_EXPR)
-       my_friendly_abort (215);
+       abort ();
       while (absdcl && TREE_CODE (absdcl) == INDIRECT_REF)
        {
          last_absdcl = absdcl;
@@ -2045,8 +1959,8 @@ build_new (placement, decl, init, use_global_new)
                }
              else
                {
-                 int flags = pedantic ? WANT_INT : (WANT_INT | WANT_ENUM);
-                 if (build_expr_type_conversion (flags, this_nelts, 0)
+                 if (build_expr_type_conversion (WANT_INT | WANT_ENUM, 
+                                                 this_nelts, false)
                      == NULL_TREE)
                    pedwarn ("size in array new must have integral type");
 
@@ -2109,7 +2023,8 @@ build_new (placement, decl, init, use_global_new)
       else
        t = type;
        
-      rval = build_min_nt (NEW_EXPR, placement, t, init);
+      rval = build_min (NEW_EXPR, build_pointer_type (type), 
+                       placement, t, init);
       NEW_EXPR_USE_GLOBAL (rval) = use_global_new;
       return rval;
     }
@@ -2158,24 +2073,39 @@ build_new (placement, decl, init, use_global_new)
   return rval;
 }
 
-/* Given a Java class, return a decl for the corresponding java.lang.Class. */
+/* Given a Java class, return a decl for the corresponding java.lang.Class.  */
 
 tree
 build_java_class_ref (type)
      tree type;
 {
-  tree name, class_decl;
-  static tree CL_prefix = NULL_TREE;
-  if (CL_prefix == NULL_TREE)
-    CL_prefix = get_identifier("_CL_");
+  tree name = NULL_TREE, class_decl;
+  static tree CL_suffix = NULL_TREE;
+  if (CL_suffix == NULL_TREE)
+    CL_suffix = get_identifier("class$");
   if (jclass_node == NULL_TREE)
     {
-      jclass_node = IDENTIFIER_GLOBAL_VALUE (get_identifier("jclass"));
+      jclass_node = IDENTIFIER_GLOBAL_VALUE (get_identifier ("jclass"));
       if (jclass_node == NULL_TREE)
-       fatal("call to Java constructor, while `jclass' undefined");
+       fatal_error ("call to Java constructor, while `jclass' undefined");
+
       jclass_node = TREE_TYPE (jclass_node);
     }
-  name = build_overload_with_type (CL_prefix, type);
+
+  /* Mangle the class$ field */
+  {
+    tree field;
+    for (field = TYPE_FIELDS (type); field; field = TREE_CHAIN (field))
+      if (DECL_NAME (field) == CL_suffix)
+       {
+         mangle_decl (field);
+         name = DECL_ASSEMBLER_NAME (field);
+         break;
+       }
+    if (!field)
+      internal_error ("can't find class$");
+    }
+
   class_decl = IDENTIFIER_GLOBAL_VALUE (name);
   if (class_decl == NULL_TREE)
     {
@@ -2186,12 +2116,12 @@ build_java_class_ref (type)
       DECL_ARTIFICIAL (class_decl) = 1;
       DECL_IGNORED_P (class_decl) = 1;
       pushdecl_top_level (class_decl);
-      make_decl_rtl (class_decl, NULL_PTR, 1);
+      make_decl_rtl (class_decl, NULL);
     }
   return class_decl;
 }
 
-/* Returns teh size of the cookie to use when allocating an array
+/* Returns the size of the cookie to use when allocating an array
    whose elements have the indicated TYPE.  Assumes that it is already
    known that a cookie is needed.  */
 
@@ -2201,27 +2131,17 @@ get_cookie_size (type)
 {
   tree cookie_size;
 
-  if (flag_new_abi)
-    {
-      /* Under the new ABI, we need to allocate an additional max
-        (sizeof (size_t), alignof (true_type)) bytes.  */
-      tree sizetype_size;
-      tree type_align;
-
-      sizetype_size = size_in_bytes (sizetype);
-      type_align = size_int (TYPE_ALIGN_UNIT (type));
-      if (INT_CST_LT_UNSIGNED (type_align, sizetype_size))
-       cookie_size = sizetype_size;
-      else
-       cookie_size = type_align;
-    }
+  /* We need to allocate an additional max (sizeof (size_t), alignof
+     (true_type)) bytes.  */
+  tree sizetype_size;
+  tree type_align;
+  
+  sizetype_size = size_in_bytes (sizetype);
+  type_align = size_int (TYPE_ALIGN_UNIT (type));
+  if (INT_CST_LT_UNSIGNED (type_align, sizetype_size))
+    cookie_size = sizetype_size;
   else
-    {
-      if (TYPE_ALIGN (type) > TYPE_ALIGN (BI_header_type))
-       return size_int (TYPE_ALIGN_UNIT (type));
-      else
-       return size_in_bytes (BI_header_type);
-    }
+    cookie_size = type_align;
 
   return cookie_size;
 }
@@ -2234,22 +2154,27 @@ build_new_1 (exp)
      tree exp;
 {
   tree placement, init;
-  tree type, true_type, size, rval;
+  tree type, true_type, size, rval, t;
+  tree full_type;
+  tree outer_nelts = NULL_TREE;
   tree nelts = NULL_TREE;
-  tree alloc_expr, alloc_node = NULL_TREE;
+  tree alloc_call, alloc_expr, alloc_node;
+  tree alloc_fn;
+  tree cookie_expr, init_expr;
   int has_array = 0;
   enum tree_code code;
-  int use_cookie, nothrow, check_new;
+  int nothrow, check_new;
   /* Nonzero if the user wrote `::new' rather than just `new'.  */
   int globally_qualified_p;
-  /* Nonzero if we're going to call a global operator new, rather than
-     a class-specific version.  */
-  int use_global_new;
   int use_java_new = 0;
   /* If non-NULL, the number of extra bytes to allocate at the
      beginning of the storage allocated for an array-new expression in
      order to store the number of elements.  */
   tree cookie_size = NULL_TREE;
+  /* True if the function we are calling is a placement allocation
+     function.  */
+  bool placement_allocation_fn_p;
+  tree args = NULL_TREE;
 
   placement = TREE_OPERAND (exp, 0);
   type = TREE_OPERAND (exp, 1);
@@ -2259,16 +2184,19 @@ build_new_1 (exp)
   if (TREE_CODE (type) == ARRAY_REF)
     {
       has_array = 1;
-      nelts = TREE_OPERAND (type, 1);
+      nelts = outer_nelts = TREE_OPERAND (type, 1);
       type = TREE_OPERAND (type, 0);
+
+      /* Use an incomplete array type to avoid VLA headaches.  */
+      full_type = build_cplus_array_type (type, NULL_TREE);
     }
+  else
+    full_type = type;
+
   true_type = type;
 
   code = has_array ? VEC_NEW_EXPR : NEW_EXPR;
 
-  if (CP_TYPE_QUALS (type))
-    type = TYPE_MAIN_VARIANT (type);
-
   /* If our base type is an array, then make sure we know how many elements
      it has.  */
   while (TREE_CODE (true_type) == ARRAY_TYPE)
@@ -2281,12 +2209,6 @@ build_new_1 (exp)
   if (!complete_type_or_else (true_type, exp))
     return error_mark_node;
 
-  if (has_array)
-    size = fold (cp_build_binary_op (MULT_EXPR, size_in_bytes (true_type),
-                                    nelts));
-  else
-    size = size_in_bytes (type);
-
   if (TREE_CODE (true_type) == VOID_TYPE)
     {
       error ("invalid type `void' for new");
@@ -2296,85 +2218,96 @@ build_new_1 (exp)
   if (abstract_virtuals_error (NULL_TREE, true_type))
     return error_mark_node;
 
-  /* Figure out whether or not we're going to use the global operator
-     new.  */
-  if (!globally_qualified_p
-      && IS_AGGR_TYPE (true_type)
-      && ((!has_array && TYPE_HAS_NEW_OPERATOR (true_type))
-         || (has_array && TYPE_HAS_ARRAY_NEW_OPERATOR (true_type))))
-    use_global_new = 0;
-  else
-    use_global_new = 1;
-
-  /* We only need cookies for arrays containing types for which we
-     need cookies.  */
-  if (!has_array || !TYPE_VEC_NEW_USES_COOKIE (true_type))
-    use_cookie = 0;
-  /* When using placement new, users may not realize that they need
-     the extra storage.  Under the old ABI, we don't allocate the
-     cookie whenever they use one placement argument of type `void
-     *'.  Under the new ABI, we require that the operator called be
-     the global placement operator delete[].  */
-  else if (placement && !TREE_CHAIN (placement) 
-          && same_type_p (TREE_TYPE (TREE_VALUE (placement)),
-                          ptr_type_node))
-    use_cookie = (!flag_new_abi || !use_global_new);
-  /* Otherwise, we need the cookie.  */
-  else
-    use_cookie = 1;
-
-  /* Compute the number of extra bytes to allocate, now that we know
-     whether or not we need the cookie.  */
-  if (use_cookie)
-    {
-      cookie_size = get_cookie_size (true_type);
-      size = size_binop (PLUS_EXPR, size, cookie_size);
-    }
-
-  if (has_array && init && pedantic)
-    cp_pedwarn ("initialization in array new");
+  size = size_in_bytes (true_type);
+  if (has_array)
+    size = size_binop (MULT_EXPR, size, convert (sizetype, nelts));
 
   /* Allocate the object.  */
-  
   if (! placement && TYPE_FOR_JAVA (true_type))
     {
       tree class_addr, alloc_decl;
       tree class_decl = build_java_class_ref (true_type);
       tree class_size = size_in_bytes (true_type);
-      static char alloc_name[] = "_Jv_AllocObject";
+      static const char alloc_name[] = "_Jv_AllocObject";
       use_java_new = 1;
       alloc_decl = IDENTIFIER_GLOBAL_VALUE (get_identifier (alloc_name));
       if (alloc_decl == NULL_TREE)
-       fatal("call to Java constructor, while `%s' undefined", alloc_name);
+       fatal_error ("call to Java constructor with `%s' undefined",
+                    alloc_name);
+
       class_addr = build1 (ADDR_EXPR, jclass_node, class_decl);
-      rval = build_function_call (alloc_decl,
-                                 tree_cons (NULL_TREE, class_addr,
-                                            build_tree_list (NULL_TREE,
-                                                             class_size)));
-      rval = cp_convert (build_pointer_type (true_type), rval);
+      alloc_call = (build_function_call
+                   (alloc_decl,
+                    tree_cons (NULL_TREE, class_addr,
+                               build_tree_list (NULL_TREE, class_size))));
     }
   else
     {
       tree fnname;
-      tree args;
 
-      args = tree_cons (NULL_TREE, size, placement);
       fnname = ansi_opname (code);
 
-      if (use_global_new)
-       rval = (build_new_function_call 
-               (lookup_function_nonclass (fnname, args),
-                args));
+      if (!globally_qualified_p 
+         && CLASS_TYPE_P (true_type)
+         && (has_array
+             ? TYPE_HAS_ARRAY_NEW_OPERATOR (true_type)
+             : TYPE_HAS_NEW_OPERATOR (true_type)))
+       {
+         /* Use a class-specific operator new.  */
+         /* If a cookie is required, add some extra space.  */
+         if (has_array && TYPE_VEC_NEW_USES_COOKIE (true_type))
+           {
+             cookie_size = get_cookie_size (true_type);
+             size = size_binop (PLUS_EXPR, size, cookie_size);
+           }
+         /* Create the argument list.  */
+         args = tree_cons (NULL_TREE, size, placement);
+         /* Call the function.  */
+         alloc_call = build_method_call (build_dummy_object (true_type),
+                                         fnname, args, 
+                                         TYPE_BINFO (true_type),
+                                         LOOKUP_NORMAL);
+       }
       else
-       rval = build_method_call (build_dummy_object (true_type),
-                                 fnname, args, NULL_TREE,
-                                 LOOKUP_NORMAL);
-      rval = cp_convert (build_pointer_type (true_type), rval);
+       {
+         /* Use a global operator new.  */
+         /* See if a cookie might be required.  */
+         if (has_array && TYPE_VEC_NEW_USES_COOKIE (true_type))
+           cookie_size = get_cookie_size (true_type);
+         else
+           cookie_size = NULL_TREE;
+
+         alloc_call = build_operator_new_call (fnname, placement, 
+                                               &size, &cookie_size);
+       }
     }
 
-  if (rval == error_mark_node)
+  if (alloc_call == error_mark_node)
     return error_mark_node;
 
+  /* The ALLOC_CALL should be a CALL_EXPR -- or a COMPOUND_EXPR whose
+     right-hand-side is ultimately a CALL_EXPR -- and the first
+     operand should be the address of a known FUNCTION_DECL.  */
+  t = alloc_call;
+  while (TREE_CODE (t) == COMPOUND_EXPR) 
+    t = TREE_OPERAND (t, 1);
+  alloc_fn = get_callee_fndecl (t);
+  my_friendly_assert (alloc_fn != NULL_TREE, 20020325);
+
+  /* Now, check to see if this function is actually a placement
+     allocation function.  This can happen even when PLACEMENT is NULL
+     because we might have something like:
+
+       struct S { void* operator new (size_t, int i = 0); };
+
+     A call to `new S' will get this allocation function, even though
+     there is no explicit placement argument.  If there is more than
+     one argument, or there are variable arguments, then this is a
+     placement allocation function.  */
+  placement_allocation_fn_p 
+    = (type_num_arguments (TREE_TYPE (alloc_fn)) > 1 
+       || varargs_function_p (alloc_fn));
+
   /*        unless an allocation function is declared with an empty  excep-
      tion-specification  (_except.spec_),  throw(), it indicates failure to
      allocate storage by throwing a bad_alloc exception  (clause  _except_,
@@ -2385,150 +2318,88 @@ build_new_1 (exp)
 
      So check for a null exception spec on the op new we just called.  */
 
-  nothrow = 0;
-  if (rval)
-    {
-      /* The CALL_EXPR.  */
-      tree t = TREE_OPERAND (rval, 0);
-      /* The function.  */
-      t = TREE_OPERAND (TREE_OPERAND (t, 0), 0);
-      nothrow = TYPE_NOTHROW_P (TREE_TYPE (t));
-    }
+  nothrow = TYPE_NOTHROW_P (TREE_TYPE (alloc_fn));
   check_new = (flag_check_new || nothrow) && ! use_java_new;
 
-  if ((check_new || flag_exceptions) && rval)
-    {
-      alloc_expr = get_target_expr (rval);
-      alloc_node = rval = TREE_OPERAND (alloc_expr, 0);
-    }
-  else
-    alloc_expr = NULL_TREE;
+  alloc_expr = alloc_call;
+
+  if (cookie_size)
+    /* Adjust so we're pointing to the start of the object.  */
+    alloc_expr = build (PLUS_EXPR, TREE_TYPE (alloc_expr),
+                       alloc_expr, cookie_size);
+
+  /* While we're working, use a pointer to the type we've actually
+     allocated.  */
+  alloc_expr = convert (build_pointer_type (full_type), alloc_expr);
 
-  /* if rval is NULL_TREE I don't have to allocate it, but are we
-     totally sure we have some extra bytes in that case for the
-     cookies? And how does that interact with the code below? (mrs) */
-  /* Finish up some magic for new'ed arrays */
-  if (use_cookie && rval != NULL_TREE)
+  /* Now save the allocation expression so we only evaluate it once.  */
+  alloc_expr = get_target_expr (alloc_expr);
+  alloc_node = TREE_OPERAND (alloc_expr, 0);
+
+  /* Now initialize the cookie.  */
+  if (cookie_size)
     {
-      tree cookie, exp1;
-      rval = convert (string_type_node, rval); /* for ptr arithmetic */
-      rval = save_expr (cp_build_binary_op (PLUS_EXPR, rval, cookie_size));
+      tree cookie;
+
       /* Store the number of bytes allocated so that we can know how
-        many elements to destroy later.  */
-      if (flag_new_abi)
-       {
-         /* Under the new ABI, we use the last sizeof (size_t) bytes
-            to store the number of elements.  */
-         cookie = build_indirect_ref (build (MINUS_EXPR,
-                                             build_pointer_type (sizetype),
-                                             rval,
-                                             size_in_bytes (sizetype)),
-                                      NULL_PTR);
-         exp1 = build (MODIFY_EXPR, void_type_node, cookie, nelts);
-       }
-      else
-       {
-         cookie 
-           = build_indirect_ref (build (MINUS_EXPR,
-                                        build_pointer_type (BI_header_type),
-                                        rval, cookie_size), NULL_PTR);
-         exp1 = build (MODIFY_EXPR, void_type_node,
-                       build_component_ref (cookie, nelts_identifier,
-                                            NULL_TREE, 0),
-                       nelts);
-       }
+        many elements to destroy later.  We use the last sizeof
+        (size_t) bytes to store the number of elements.  */
+      cookie = build (MINUS_EXPR, build_pointer_type (sizetype),
+                     alloc_node, size_in_bytes (sizetype));
+      cookie = build_indirect_ref (cookie, NULL);
 
-      /* Build `(cookie = nelts, rval)' and use that as the complete
-        expression.  */
-      rval = cp_convert (build_pointer_type (true_type), rval);
-      rval = build_compound_expr
-       (tree_cons (NULL_TREE, exp1,
-                   build_tree_list (NULL_TREE, rval)));
+      cookie_expr = build (MODIFY_EXPR, void_type_node, cookie, nelts);
+      TREE_SIDE_EFFECTS (cookie_expr) = 1;
     }
+  else
+    cookie_expr = NULL_TREE;
 
-  if (rval == error_mark_node)
-    return error_mark_node;
-
-  /* Don't call any constructors or do any initialization.  */
-  if (init == void_type_node)
-    goto done;
-
+  /* Now initialize the allocated object.  */
+  init_expr = NULL_TREE;
   if (TYPE_NEEDS_CONSTRUCTING (type) || init)
     {
-      if (! TYPE_NEEDS_CONSTRUCTING (type)
-         && ! IS_AGGR_TYPE (type) && ! has_array)
+      init_expr = build_indirect_ref (alloc_node, NULL);
+
+      if (init == void_zero_node)
+       init = build_default_init (full_type, nelts);
+      else if (init && pedantic && has_array)
+       pedwarn ("ISO C++ forbids initialization in array new");
+
+      if (has_array)
+       init_expr
+         = build_vec_init (init_expr,
+                           cp_build_binary_op (MINUS_EXPR, outer_nelts,
+                                               integer_one_node),
+                           init, /*from_array=*/0);
+      else if (TYPE_NEEDS_CONSTRUCTING (type))
+       init_expr = build_special_member_call (init_expr, 
+                                              complete_ctor_identifier,
+                                              init, TYPE_BINFO (true_type),
+                                              LOOKUP_NORMAL);
+      else
        {
          /* We are processing something like `new int (10)', which
             means allocate an int, and initialize it with 10.  */
-         tree deref;
-         tree deref_type;
-
-         /* At present RVAL is a temporary variable, created to hold
-            the value from the call to `operator new'.  We transform
-            it to (*RVAL = INIT, RVAL).  */
-         rval = save_expr (rval);
-         deref = build_indirect_ref (rval, NULL_PTR);
-
-         /* Even for something like `new const int (10)' we must
-            allow the expression to be non-const while we do the
-            initialization.  */
-         deref_type = TREE_TYPE (deref);
-         if (CP_TYPE_CONST_P (deref_type))
-           TREE_TYPE (deref) 
-             = cp_build_qualified_type (deref_type,
-                                        CP_TYPE_QUALS (deref_type) 
-                                        & ~TYPE_QUAL_CONST);
-         TREE_READONLY (deref) = 0;
-
-         if (TREE_CHAIN (init) != NULL_TREE)
-           pedwarn ("initializer list being treated as compound expression");
-         else if (TREE_CODE (init) == CONSTRUCTOR)
+
+         if (TREE_CODE (init) == TREE_LIST)
            {
-             pedwarn ("initializer list appears where operand should be used");
-             init = TREE_OPERAND (init, 1);
+             if (TREE_CHAIN (init) != NULL_TREE)
+               pedwarn
+                 ("initializer list being treated as compound expression");
+             init = build_compound_expr (init);
+           }
+         else if (TREE_CODE (init) == CONSTRUCTOR
+                  && TREE_TYPE (init) == NULL_TREE)
+           {
+             pedwarn ("ISO C++ forbids aggregate initializer to new");
+             init = digest_init (type, init, 0);
            }
-         init = build_compound_expr (init);
-
-         init = convert_for_initialization (deref, type, init, LOOKUP_NORMAL,
-                                            "new", NULL_TREE, 0);
-         rval = build (COMPOUND_EXPR, TREE_TYPE (rval),
-                       build_modify_expr (deref, NOP_EXPR, init),
-                       rval);
-         TREE_NO_UNUSED_WARNING (rval) = 1;
-         TREE_SIDE_EFFECTS (rval) = 1;
-       }
-      else if (! has_array)
-       {
-         tree newrval;
-         /* Constructors are never virtual. If it has an initialization, we
-            need to complain if we aren't allowed to use the ctor that took
-            that argument.  */
-         int flags = LOOKUP_NORMAL|LOOKUP_NONVIRTUAL|LOOKUP_COMPLAIN;
-
-         rval = save_expr (rval);
-         newrval = rval;
-
-         if (newrval && TREE_CODE (TREE_TYPE (newrval)) == POINTER_TYPE)
-           newrval = build_indirect_ref (newrval, NULL_PTR);
-
-         newrval = build_method_call (newrval, 
-                                      complete_ctor_identifier,
-                                      init, TYPE_BINFO (true_type), flags);
-
-         if (newrval == NULL_TREE || newrval == error_mark_node)
-           return error_mark_node;
 
-         newrval = build (COMPOUND_EXPR, TREE_TYPE (rval), newrval, rval);
-         rval = newrval;
-         TREE_HAS_CONSTRUCTOR (rval) = 1;
+         init_expr = build_modify_expr (init_expr, INIT_EXPR, init);
        }
-      else
-       rval = (build_vec_init
-               (NULL_TREE, 
-                save_expr (rval),
-                cp_build_binary_op (MINUS_EXPR, nelts, integer_one_node),
-                init,
-                /*from_array=*/0));
+
+      if (init_expr == error_mark_node)
+       return error_mark_node;
 
       /* If any part of the object initialization terminates by throwing an
         exception and a suitable deallocation function can be found, the
@@ -2538,82 +2409,110 @@ build_new_1 (exp)
         unambiguous matching deallocation function can be found,
         propagating the exception does not cause the object's memory to be
         freed.  */
-      if (flag_exceptions && alloc_expr && ! use_java_new)
+      if (flag_exceptions && ! use_java_new)
        {
          enum tree_code dcode = has_array ? VEC_DELETE_EXPR : DELETE_EXPR;
-         tree cleanup, fn = NULL_TREE;
+         tree cleanup;
          int flags = (LOOKUP_NORMAL 
                       | (globally_qualified_p * LOOKUP_GLOBAL));
+         tree delete_node;
+
+         if (cookie_size)
+           /* Subtract the padding back out to get to the pointer returned
+              from operator new.  */
+           delete_node = fold (build (MINUS_EXPR, TREE_TYPE (alloc_node),
+                                      alloc_node, cookie_size));
+         else
+           delete_node = alloc_node;
 
          /* The Standard is unclear here, but the right thing to do
              is to use the same method for finding deallocation
              functions that we use for finding allocation functions.  */
          flags |= LOOKUP_SPECULATIVELY;
 
-         /* We expect alloc_expr to look like a TARGET_EXPR around
-            a NOP_EXPR around the CALL_EXPR we want.  */
-         fn = TREE_OPERAND (alloc_expr, 1);
-         fn = TREE_OPERAND (fn, 0);
-
-         cleanup = build_op_delete_call (dcode, alloc_node, size, flags, fn);
+         cleanup = build_op_delete_call (dcode, delete_node, size, flags,
+                                         (placement_allocation_fn_p 
+                                          ? alloc_call : NULL_TREE));
 
          /* Ack!  First we allocate the memory.  Then we set our sentry
             variable to true, and expand a cleanup that deletes the memory
-            if sentry is true.  Then we run the constructor and store the
-            returned pointer in buf.  Then we clear sentry and return buf.  */
+            if sentry is true.  Then we run the constructor, and finally
+            clear the sentry.
+
+            It would be nice to be able to handle this without the sentry
+            variable, perhaps with a TRY_CATCH_EXPR, but this doesn't
+            work.  We allocate the space first, so if there are any
+            temporaries with cleanups in the constructor args we need this
+            EH region to extend until end of full-expression to preserve
+            nesting.
+
+            If the backend had some mechanism so that we could force the
+            allocation to be expanded after all the other args to the
+            constructor, that would fix the nesting problem and we could
+            do away with this complexity.  But that would complicate other
+            things; in particular, it would make it difficult to bail out
+            if the allocation function returns null.  Er, no, it wouldn't;
+            we just don't run the constructor.  The standard says it's
+            unspecified whether or not the args are evaluated.
+
+            FIXME FIXME FIXME inline invisible refs as refs.  That way we
+            can preevaluate value parameters.  */
 
          if (cleanup)
            {
-             tree end, sentry, begin, buf, t = TREE_TYPE (rval);
+             tree end, sentry, begin;
 
              begin = get_target_expr (boolean_true_node);
-             sentry = TREE_OPERAND (begin, 0);
+             CLEANUP_EH_ONLY (begin) = 1;
 
-             TREE_OPERAND (begin, 2)
+             sentry = TARGET_EXPR_SLOT (begin);
+
+             TARGET_EXPR_CLEANUP (begin)
                = build (COND_EXPR, void_type_node, sentry,
                         cleanup, void_zero_node);
 
-             rval = get_target_expr (rval);
-
              end = build (MODIFY_EXPR, TREE_TYPE (sentry),
                           sentry, boolean_false_node);
 
-             buf = TREE_OPERAND (rval, 0);
-
-             rval = build (COMPOUND_EXPR, t, begin,
-                           build (COMPOUND_EXPR, t, rval,
-                                  build (COMPOUND_EXPR, t, end, buf)));
+             init_expr
+               = build (COMPOUND_EXPR, void_type_node, begin,
+                        build (COMPOUND_EXPR, void_type_node, init_expr,
+                               end));
            }
        }
     }
   else if (CP_TYPE_CONST_P (true_type))
-    cp_error ("uninitialized const in `new' of `%#T'", true_type);
+    error ("uninitialized const in `new' of `%#T'", true_type);
 
- done:
-
-  if (alloc_expr && rval == alloc_node)
-    {
-      rval = TREE_OPERAND (alloc_expr, 1);
-      alloc_expr = NULL_TREE;
-    }
+  /* Now build up the return value in reverse order.  */
 
-  if (check_new && alloc_expr)
-    {
-      /* Did we modify the storage?  */
-      tree ifexp = cp_build_binary_op (NE_EXPR, alloc_node,
-                                      integer_zero_node);
-      rval = build_conditional_expr (ifexp, rval, alloc_node);
-    }
+  rval = alloc_node;
 
-  if (alloc_expr)
-    rval = build (COMPOUND_EXPR, TREE_TYPE (rval), alloc_expr, rval);
+  if (init_expr)
+    rval = build (COMPOUND_EXPR, TREE_TYPE (rval), init_expr, rval);
+  if (cookie_expr)
+    rval = build (COMPOUND_EXPR, TREE_TYPE (rval), cookie_expr, rval);
 
-  if (rval && TREE_TYPE (rval) != build_pointer_type (type))
+  if (rval == alloc_node)
+    /* If we didn't modify anything, strip the TARGET_EXPR and return the
+       (adjusted) call.  */
+    rval = TREE_OPERAND (alloc_expr, 1);
+  else
     {
-      /* The type of new int [3][3] is not int *, but int [3] * */
-      rval = build_c_cast (build_pointer_type (type), rval);
+      if (check_new)
+       {
+         tree ifexp = cp_build_binary_op (NE_EXPR, alloc_node,
+                                          integer_zero_node);
+         rval = build_conditional_expr (ifexp, rval, alloc_node);
+       }
+
+      rval = build (COMPOUND_EXPR, TREE_TYPE (rval), alloc_expr, rval);
     }
 
+  /* Now strip the outer ARRAY_TYPE, so we return a pointer to the first
+     element.  */
+  rval = convert (build_pointer_type (type), rval);
+
   return rval;
 }
 \f
@@ -2646,6 +2545,10 @@ build_vec_delete_1 (base, maxindex, type, auto_delete_vec, use_global_delete)
      This is also the containing expression returned by this function.  */
   tree controller = NULL_TREE;
 
+  /* We should only have 1-D arrays here.  */
+  if (TREE_CODE (type) == ARRAY_TYPE)
+    abort ();
+
   if (! IS_AGGR_TYPE (type) || TYPE_HAS_TRIVIAL_DESTRUCTOR (type))
     {
       loop = integer_zero_node;
@@ -2742,10 +2645,14 @@ build_vec_delete_1 (base, maxindex, type, auto_delete_vec, use_global_delete)
   if (controller)
     {
       TREE_OPERAND (controller, 1) = body;
-      return controller;
+      body = controller;
     }
-  else
-    return cp_convert (void_type_node, body);
+
+  if (TREE_CODE (base) == SAVE_EXPR)
+    /* Pre-evaluate the SAVE_EXPR outside of the BIND_EXPR.  */
+    body = build (COMPOUND_EXPR, void_type_node, base, body);
+
+  return cp_convert (void_type_node, body);
 }
 
 /* Create an unnamed variable of the indicated TYPE.  */ 
@@ -2783,8 +2690,8 @@ get_temp_regvar (type, init)
   decl = create_temporary_var (type);
   if (building_stmt_tree ())
     add_decl_stmt (decl);
-  if (!building_stmt_tree ())
-    DECL_RTL (decl) = assign_temp (type, 2, 0, 1);
+  else
+    SET_DECL_RTL (decl, assign_temp (type, 2, 0, 1));
   finish_expr_stmt (build_modify_expr (decl, INIT_EXPR, init));
 
   return decl;
@@ -2793,12 +2700,10 @@ get_temp_regvar (type, init)
 /* `build_vec_init' returns tree structure that performs
    initialization of a vector of aggregate types.
 
-   DECL is passed only for error reporting, and provides line number
-   and source file name information.
-   BASE is the space where the vector will be.  For a vector of Ts,
-     the type of BASE is `T*'.
+   BASE is a reference to the vector, of ARRAY_TYPE.
    MAXINDEX is the maximum index of the array (one less than the
-           number of elements).
+     number of elements).  It is only used if
+     TYPE_DOMAIN (TREE_TYPE (BASE)) == NULL_TREE.
    INIT is the (possibly NULL) initializer.
 
    FROM_ARRAY is 0 if we should init everything with INIT
@@ -2809,8 +2714,8 @@ get_temp_regvar (type, init)
    but use assignment instead of initialization.  */
 
 tree
-build_vec_init (decl, base, maxindex, init, from_array)
-     tree decl, base, maxindex, init;
+build_vec_init (base, maxindex, init, from_array)
+     tree base, init, maxindex;
      int from_array;
 {
   tree rval;
@@ -2818,8 +2723,10 @@ build_vec_init (decl, base, maxindex, init, from_array)
   tree size;
   tree itype = NULL_TREE;
   tree iterator;
+  /* The type of the array.  */
+  tree atype = TREE_TYPE (base);
   /* The type of an element in the array.  */
-  tree type;
+  tree type = TREE_TYPE (atype);
   /* The type of a pointer to an element in the array.  */
   tree ptype;
   tree stmt_expr;
@@ -2829,37 +2736,55 @@ build_vec_init (decl, base, maxindex, init, from_array)
   tree try_body = NULL_TREE;
   int num_initialized_elts = 0;
 
-  maxindex = cp_convert (ptrdiff_type_node, maxindex);
-  if (maxindex == error_mark_node)
+  if (TYPE_DOMAIN (atype))
+    maxindex = array_type_nelts (atype);
+
+  if (maxindex == NULL_TREE || maxindex == error_mark_node)
     return error_mark_node;
 
-  type = TREE_TYPE (TREE_TYPE (base));
+  if (init
+      && (from_array == 2
+         ? (!CLASS_TYPE_P (type) || !TYPE_HAS_COMPLEX_ASSIGN_REF (type))
+         : !TYPE_NEEDS_CONSTRUCTING (type))
+      && ((TREE_CODE (init) == CONSTRUCTOR
+          /* Don't do this if the CONSTRUCTOR might contain something
+             that might throw and require us to clean up.  */
+          && (CONSTRUCTOR_ELTS (init) == NULL_TREE
+              || ! TYPE_HAS_NONTRIVIAL_DESTRUCTOR (target_type (type))))
+         || from_array))
+    {
+      /* Do non-default initialization of POD arrays resulting from
+        brace-enclosed initializers.  In this case, digest_init and
+        store_constructor will handle the semantics for us.  */
+
+      stmt_expr = build (INIT_EXPR, atype, base, init);
+      return stmt_expr;
+    }
+
+  maxindex = cp_convert (ptrdiff_type_node, maxindex);
   ptype = build_pointer_type (type);
   size = size_in_bytes (type);
+  if (TREE_CODE (TREE_TYPE (base)) == ARRAY_TYPE)
+    base = cp_convert (ptype, default_conversion (base));
 
   /* The code we are generating looks like:
 
        T* t1 = (T*) base;
-       T* rval = base;
+       T* rval = t1;
        ptrdiff_t iterator = maxindex;
        try {
-         ... initializations from CONSTRUCTOR ...
-         if (iterator != -1) {
-          do {
-            ... initialize *base ...
-            ++base;
-          } while (--iterator != -1);
+        for (; iterator != -1; --iterator) {
+          ... initialize *t1 ...
+          ++t1;
         }
        } catch (...) {
          ... destroy elements that were constructed ...
        }
+       return rval;
        
      We can omit the try and catch blocks if we know that the
      initialization will never throw an exception, or if the array
-     elements do not have destructors.  If we have a CONSTRUCTOR to
-     give us initialization information, we emit code to initialize
-     each of the elements before the loop in the try block, and then
-     iterate over fewer elements.  We can omit the loop completely if
+     elements do not have destructors.  We can omit the loop completely if
      the elements of the array do not have constructors.  
 
      We actually wrap the entire body of the above in a STMT_EXPR, for
@@ -2873,25 +2798,25 @@ build_vec_init (decl, base, maxindex, init, from_array)
 
   begin_init_stmts (&stmt_expr, &compound_stmt);
   destroy_temps = stmts_are_full_exprs_p ();
-  current_stmt_tree->stmts_are_full_exprs_p = 0;
-  rval = get_temp_regvar (ptype, 
-                         cp_convert (ptype, default_conversion (base)));
+  current_stmt_tree ()->stmts_are_full_exprs_p = 0;
+  rval = get_temp_regvar (ptype, base);
   base = get_temp_regvar (ptype, rval);
   iterator = get_temp_regvar (ptrdiff_type_node, maxindex);
 
   /* Protect the entire array initialization so that we can destroy
-     the partially constructed array if an exception is thrown.  */
-  if (flag_exceptions && TYPE_HAS_NONTRIVIAL_DESTRUCTOR (type))
+     the partially constructed array if an exception is thrown.
+     But don't do this if we're assigning.  */
+  if (flag_exceptions && TYPE_HAS_NONTRIVIAL_DESTRUCTOR (type)
+      && from_array != 2)
     {
       try_block = begin_try_block ();
       try_body = begin_compound_stmt (/*has_no_scope=*/1);
     }
 
-  if (init != NULL_TREE && TREE_CODE (init) == CONSTRUCTOR
-      && (!decl || same_type_p (TREE_TYPE (init), TREE_TYPE (decl))))
+  if (init != NULL_TREE && TREE_CODE (init) == CONSTRUCTOR)
     {
-      /* Do non-default initialization resulting from brace-enclosed
-        initializers.  */
+      /* Do non-default initialization of non-POD arrays resulting from
+        brace-enclosed initializers.  */
 
       tree elts;
       from_array = 0;
@@ -2909,16 +2834,8 @@ build_vec_init (decl, base, maxindex, init, from_array)
            finish_expr_stmt (build_modify_expr (baseref, NOP_EXPR,
                                                 elt));
 
-         finish_expr_stmt (build_modify_expr 
-                           (base, 
-                            NOP_EXPR,
-                            build (PLUS_EXPR, build_pointer_type (type),
-                                   base, size)));
-         finish_expr_stmt (build_modify_expr
-                           (iterator,
-                            NOP_EXPR,
-                            build (MINUS_EXPR, ptrdiff_type_node,
-                                   iterator, integer_one_node)));
+         finish_expr_stmt (build_unary_op (PREINCREMENT_EXPR, base, 0));
+         finish_expr_stmt (build_unary_op (PREDECREMENT_EXPR, iterator, 0));
        }
 
       /* Clear out INIT so that we don't get confused below.  */
@@ -2929,11 +2846,6 @@ build_vec_init (decl, base, maxindex, init, from_array)
       /* If initializing one array from another, initialize element by
         element.  We rely upon the below calls the do argument
         checking.  */ 
-      if (decl == NULL_TREE)
-       {
-         sorry ("initialization of array from dissimilar array type");
-         return error_mark_node;
-       }
       if (init)
        {
          base2 = default_conversion (init);
@@ -2964,19 +2876,20 @@ build_vec_init (decl, base, maxindex, init, from_array)
     {
       /* If the ITERATOR is equal to -1, then we don't have to loop;
         we've already initialized all the elements.  */
-      tree if_stmt;
-      tree do_stmt;
-      tree do_body;
+      tree for_stmt;
+      tree for_body;
       tree elt_init;
 
-      if_stmt = begin_if_stmt ();
-      finish_if_stmt_cond (build (NE_EXPR, boolean_type_node,
-                                 iterator, minus_one_node),
-                          if_stmt);
+      for_stmt = begin_for_stmt ();
+      finish_for_init_stmt (for_stmt);
+      finish_for_cond (build (NE_EXPR, boolean_type_node,
+                             iterator, integer_minus_one_node),
+                      for_stmt);
+      finish_for_expr (build_unary_op (PREDECREMENT_EXPR, iterator, 0),
+                      for_stmt);
 
       /* Otherwise, loop through the elements.  */
-      do_stmt = begin_do_stmt ();
-      do_body = begin_compound_stmt (/*has_no_scope=*/1);
+      for_body = begin_compound_stmt (/*has_no_scope=*/1);
 
       /* When we're not building a statement-tree, things are a little
         complicated.  If, when we recursively call build_aggr_init,
@@ -3009,25 +2922,22 @@ build_vec_init (decl, base, maxindex, init, from_array)
          else if (from)
            elt_init = build_modify_expr (to, NOP_EXPR, from);
          else
-           my_friendly_abort (57);
+           abort ();
        }
       else if (TREE_CODE (type) == ARRAY_TYPE)
        {
          if (init != 0)
-           sorry ("cannot initialize multi-dimensional array with initializer");
-         elt_init = (build_vec_init 
-                     (decl, 
-                      build1 (NOP_EXPR, 
-                              build_pointer_type (TREE_TYPE (type)),
-                              base),
-                      array_type_nelts (type), 0, 0));
+           sorry
+             ("cannot initialize multi-dimensional array with initializer");
+         elt_init = build_vec_init (build1 (INDIRECT_REF, type, base),
+                                    0, 0, 0);
        }
       else
        elt_init = build_aggr_init (build1 (INDIRECT_REF, type, base), 
                                    init, 0);
       
       /* The initialization of each array element is a
-        full-expression.  */
+        full-expression, as per core issue 124.  */
       if (!building_stmt_tree ())
        {
          genrtl_expr_stmt (elt_init);
@@ -3035,47 +2945,38 @@ build_vec_init (decl, base, maxindex, init, from_array)
        }
       else
        {
-         current_stmt_tree->stmts_are_full_exprs_p = 1;
+         current_stmt_tree ()->stmts_are_full_exprs_p = 1;
          finish_expr_stmt (elt_init);
-         current_stmt_tree->stmts_are_full_exprs_p = 0;
+         current_stmt_tree ()->stmts_are_full_exprs_p = 0;
        }
 
-      finish_expr_stmt (build_modify_expr
-                       (base,
-                        NOP_EXPR,
-                        build (PLUS_EXPR, build_pointer_type (type), 
-                               base, size)));
+      finish_expr_stmt (build_unary_op (PREINCREMENT_EXPR, base, 0));
       if (base2)
-       finish_expr_stmt (build_modify_expr
-                         (base2,
-                          NOP_EXPR,
-                          build (PLUS_EXPR, build_pointer_type (type), 
-                                 base2, size)));
-
-      finish_compound_stmt (/*has_no_scope=*/1, do_body);
-      finish_do_body (do_stmt);
-      finish_do_stmt (build (NE_EXPR, boolean_type_node,
-                            build (PREDECREMENT_EXPR, 
-                                   ptrdiff_type_node, 
-                                   iterator,
-                                   integer_one_node), 
-                            minus_one_node),
-                     do_stmt);
-
-      finish_then_clause (if_stmt);
-      finish_if_stmt ();
+       finish_expr_stmt (build_unary_op (PREINCREMENT_EXPR, base2, 0));
+
+      finish_compound_stmt (/*has_no_scope=*/1, for_body);
+      finish_for_stmt (for_stmt);
     }
 
   /* Make sure to cleanup any partially constructed elements.  */
-  if (flag_exceptions && TYPE_HAS_NONTRIVIAL_DESTRUCTOR (type))
+  if (flag_exceptions && TYPE_HAS_NONTRIVIAL_DESTRUCTOR (type)
+      && from_array != 2)
     {
       tree e;
+      tree m = cp_build_binary_op (MINUS_EXPR, maxindex, iterator);
+
+      /* Flatten multi-dimensional array since build_vec_delete only
+        expects one-dimensional array.  */
+      if (TREE_CODE (type) == ARRAY_TYPE)
+       {
+         m = cp_build_binary_op (MULT_EXPR, m,
+                                 array_type_nelts_total (type));
+         type = strip_array_types (type);
+       }
 
       finish_compound_stmt (/*has_no_scope=*/1, try_body);
       finish_cleanup_try_block (try_block);
-      e = build_vec_delete_1 (rval,
-                             cp_build_binary_op (MINUS_EXPR, maxindex, 
-                                                 iterator),
+      e = build_vec_delete_1 (rval, m,
                              type,
                              sfk_base_destructor,
                              /*use_global_delete=*/0);
@@ -3087,7 +2988,7 @@ build_vec_init (decl, base, maxindex, init, from_array)
   finish_expr_stmt (rval);
 
   stmt_expr = finish_init_stmts (stmt_expr, compound_stmt);
-  current_stmt_tree->stmts_are_full_exprs_p = destroy_temps;
+  current_stmt_tree ()->stmts_are_full_exprs_p = destroy_temps;
   return stmt_expr;
 }
 
@@ -3145,9 +3046,10 @@ build_dtor_call (exp, dtor_kind, flags)
       break;
 
     default:
-      my_friendly_abort (20000524);
+      abort ();
     }
-  return build_method_call (exp, name, NULL_TREE, NULL_TREE, flags);
+  return build_method_call (exp, name, NULL_TREE, 
+                           TYPE_BINFO (TREE_TYPE (exp)), flags);
 }
 
 /* Generate a call to a destructor. TYPE is the type to cast ADDR to.
@@ -3157,9 +3059,7 @@ build_dtor_call (exp, dtor_kind, flags)
    sfk_deleting_destructor.
 
    FLAGS is the logical disjunction of zero or more LOOKUP_
-   flags.  See cp-tree.h for more info.
-
-   This function does not delete an object's virtual base classes.  */
+   flags.  See cp-tree.h for more info.  */
 
 tree
 build_delete (type, addr, auto_delete, flags, use_global_delete)
@@ -3168,9 +3068,7 @@ build_delete (type, addr, auto_delete, flags, use_global_delete)
      int flags;
      int use_global_delete;
 {
-  tree member;
   tree expr;
-  tree ref;
 
   if (addr == error_mark_node)
     return error_mark_node;
@@ -3185,11 +3083,18 @@ build_delete (type, addr, auto_delete, flags, use_global_delete)
   if (TREE_CODE (type) == POINTER_TYPE)
     {
       type = TYPE_MAIN_VARIANT (TREE_TYPE (type));
-      if (!VOID_TYPE_P (type) && !complete_type_or_else (type, addr))
-       return error_mark_node;
       if (TREE_CODE (type) == ARRAY_TYPE)
        goto handle_array;
-      if (! IS_AGGR_TYPE (type))
+
+      if (VOID_TYPE_P (type)
+         /* We don't want to warn about delete of void*, only other
+            incomplete types.  Deleting other incomplete types
+            invokes undefined behavior, but it is not ill-formed, so
+            compile to something that would even do The Right Thing
+            (TM) should the type have a trivial dtor and no delete
+            operator.  */
+         || !complete_type_or_diagnostic (type, addr, 1)
+         || !IS_AGGR_TYPE (type))
        {
          /* Call the builtin operator delete.  */
          return build_builtin_delete_call (addr);
@@ -3199,13 +3104,11 @@ build_delete (type, addr, auto_delete, flags, use_global_delete)
 
       /* throw away const and volatile on target type of addr */
       addr = convert_force (build_pointer_type (type), addr, 0);
-      ref = build_indirect_ref (addr, NULL_PTR);
     }
   else if (TREE_CODE (type) == ARRAY_TYPE)
     {
     handle_array:
-      if (TREE_SIDE_EFFECTS (addr))
-       addr = save_expr (addr);
+      
       if (TYPE_DOMAIN (type) == NULL_TREE)
        {
          error ("unknown array size in delete");
@@ -3223,45 +3126,69 @@ build_delete (type, addr, auto_delete, flags, use_global_delete)
       if (TREE_SIDE_EFFECTS (addr))
        addr = save_expr (addr);
 
-      if (TREE_CONSTANT (addr))
-       addr = convert_pointer_to (type, addr);
-      else
-       addr = convert_force (build_pointer_type (type), addr, 0);
-
-      ref = build_indirect_ref (addr, NULL_PTR);
+      addr = convert_force (build_pointer_type (type), addr, 0);
     }
 
   my_friendly_assert (IS_AGGR_TYPE (type), 220);
 
   if (TYPE_HAS_TRIVIAL_DESTRUCTOR (type))
     {
-      if (auto_delete == sfk_base_destructor)
+      if (auto_delete != sfk_deleting_destructor)
        return void_zero_node;
 
       return build_op_delete_call
-       (DELETE_EXPR, addr, c_sizeof_nowarn (type),
+       (DELETE_EXPR, addr, cxx_sizeof_nowarn (type),
         LOOKUP_NORMAL | (use_global_delete * LOOKUP_GLOBAL),
         NULL_TREE);
     }
-
-  /* Below, we will reverse the order in which these calls are made.
-     If we have a destructor, then that destructor will take care
-     of the base classes; otherwise, we must do that here.  */
-  if (TYPE_HAS_DESTRUCTOR (type))
+  else
     {
       tree do_delete = NULL_TREE;
       tree ifexp;
 
+      my_friendly_assert (TYPE_HAS_DESTRUCTOR (type), 20011213);
+
+      /* For `::delete x', we must not use the deleting destructor
+        since then we would not be sure to get the global `operator
+        delete'.  */
       if (use_global_delete && auto_delete == sfk_deleting_destructor)
        {
-         /* Delete the object. */
+         /* We will use ADDR multiple times so we must save it.  */
+         addr = save_expr (addr);
+         /* Delete the object.  */
          do_delete = build_builtin_delete_call (addr);
          /* Otherwise, treat this like a complete object destructor
             call.  */
          auto_delete = sfk_complete_destructor;
        }
+      /* If the destructor is non-virtual, there is no deleting
+        variant.  Instead, we must explicitly call the appropriate
+        `operator delete' here.  */
+      else if (!DECL_VIRTUAL_P (CLASSTYPE_DESTRUCTORS (type))
+              && auto_delete == sfk_deleting_destructor)
+       {
+         /* We will use ADDR multiple times so we must save it.  */
+         addr = save_expr (addr);
+         /* Build the call.  */
+         do_delete = build_op_delete_call (DELETE_EXPR,
+                                           addr,
+                                           cxx_sizeof_nowarn (type),
+                                           LOOKUP_NORMAL,
+                                           NULL_TREE);
+         /* Call the complete object destructor.  */
+         auto_delete = sfk_complete_destructor;
+       }
+      else if (auto_delete == sfk_deleting_destructor
+              && TYPE_GETS_REG_DELETE (type))
+       {
+         /* Make sure we have access to the member op delete, even though
+            we'll actually be calling it from the destructor.  */
+         build_op_delete_call (DELETE_EXPR, addr, cxx_sizeof_nowarn (type),
+                               LOOKUP_NORMAL, NULL_TREE);
+       }
 
-      expr = build_dtor_call (ref, auto_delete, flags);
+      expr = build_dtor_call (build_indirect_ref (addr, NULL),
+                             auto_delete, flags);
       if (do_delete)
        expr = build (COMPOUND_EXPR, void_type_node, expr, do_delete);
 
@@ -3278,55 +3205,90 @@ build_delete (type, addr, auto_delete, flags, use_global_delete)
 
       return expr;
     }
-  else
-    {
-      /* We only get here from finish_function for a destructor.  */
-      tree binfos = BINFO_BASETYPES (TYPE_BINFO (type));
-      int i, n_baseclasses = CLASSTYPE_N_BASECLASSES (type);
-      tree base_binfo = n_baseclasses > 0 ? TREE_VEC_ELT (binfos, 0) : NULL_TREE;
-      tree exprstmt = NULL_TREE;
-
-      /* Set this again before we call anything, as we might get called
-        recursively.  */
-      TYPE_HAS_DESTRUCTOR (type) = 1;
+}
 
-      /* If we have member delete or vbases, we call delete in
-        finish_function.  */
-      my_friendly_assert (auto_delete == sfk_base_destructor, 20000411);
+/* At the beginning of a destructor, push cleanups that will call the
+   destructors for our base classes and members.
 
-      /* Take care of the remaining baseclasses.  */
-      for (i = 0; i < n_baseclasses; i++)
-       {
-         base_binfo = TREE_VEC_ELT (binfos, i);
-         if (TYPE_HAS_TRIVIAL_DESTRUCTOR (BINFO_TYPE (base_binfo))
-             || TREE_VIA_VIRTUAL (base_binfo))
-           continue;
+   Called from begin_destructor_body.  */
 
-         expr = build_scoped_method_call (ref, base_binfo,
-                                          base_dtor_identifier,
-                                          NULL_TREE);
+void
+push_base_cleanups ()
+{
+  tree binfos;
+  int i, n_baseclasses;
+  tree member;
+  tree expr;
 
-         exprstmt = tree_cons (NULL_TREE, expr, exprstmt);
-       }
+  /* Run destructors for all virtual baseclasses.  */
+  if (TYPE_USES_VIRTUAL_BASECLASSES (current_class_type))
+    {
+      tree vbases;
+      tree cond = (condition_conversion
+                  (build (BIT_AND_EXPR, integer_type_node,
+                          current_in_charge_parm,
+                          integer_two_node)));
 
-      for (member = TYPE_FIELDS (type); member; member = TREE_CHAIN (member))
+      vbases = CLASSTYPE_VBASECLASSES (current_class_type);
+      /* The CLASSTYPE_VBASECLASSES list is in initialization
+        order, which is also the right order for pushing cleanups.  */
+      for (; vbases;
+          vbases = TREE_CHAIN (vbases))
        {
-         if (TREE_CODE (member) != FIELD_DECL)
-           continue;
-         if (TYPE_HAS_NONTRIVIAL_DESTRUCTOR (TREE_TYPE (member)))
+         tree vbase = TREE_VALUE (vbases);
+         tree base_type = BINFO_TYPE (vbase);
+
+         if (TYPE_HAS_NONTRIVIAL_DESTRUCTOR (base_type))
            {
-             tree this_member = build_component_ref (ref, DECL_NAME (member), NULL_TREE, 0);
-             tree this_type = TREE_TYPE (member);
-             expr = build_delete (this_type, this_member,
-                                  sfk_complete_destructor, flags, 0);
-             exprstmt = tree_cons (NULL_TREE, expr, exprstmt);
+             expr = build_special_member_call (current_class_ref, 
+                                               base_dtor_identifier,
+                                               NULL_TREE,
+                                               vbase,
+                                               (LOOKUP_NORMAL 
+                                                | LOOKUP_NONVIRTUAL));
+             expr = build (COND_EXPR, void_type_node, cond,
+                           expr, void_zero_node);
+             finish_decl_cleanup (NULL_TREE, expr);
            }
        }
+    }
+
+  binfos = BINFO_BASETYPES (TYPE_BINFO (current_class_type));
+  n_baseclasses = CLASSTYPE_N_BASECLASSES (current_class_type);
+
+  /* Take care of the remaining baseclasses.  */
+  for (i = 0; i < n_baseclasses; i++)
+    {
+      tree base_binfo = TREE_VEC_ELT (binfos, i);
+      if (TYPE_HAS_TRIVIAL_DESTRUCTOR (BINFO_TYPE (base_binfo))
+         || TREE_VIA_VIRTUAL (base_binfo))
+       continue;
 
-      if (exprstmt)
-       return build_compound_expr (exprstmt);
-      /* Virtual base classes make this function do nothing.  */
-      return void_zero_node;
+      expr = build_special_member_call (current_class_ref, 
+                                       base_dtor_identifier,
+                                       NULL_TREE, base_binfo, 
+                                       LOOKUP_NORMAL | LOOKUP_NONVIRTUAL);
+      finish_decl_cleanup (NULL_TREE, expr);
+    }
+
+  for (member = TYPE_FIELDS (current_class_type); member;
+       member = TREE_CHAIN (member))
+    {
+      if (TREE_CODE (member) != FIELD_DECL || DECL_ARTIFICIAL (member))
+       continue;
+      if (TYPE_HAS_NONTRIVIAL_DESTRUCTOR (TREE_TYPE (member)))
+       {
+         tree this_member = (build_class_member_access_expr 
+                             (current_class_ref, member, 
+                              /*access_path=*/NULL_TREE,
+                              /*preserve_reference=*/false));
+         tree this_type = TREE_TYPE (member);
+         expr = build_delete (this_type, this_member,
+                              sfk_complete_destructor,
+                              LOOKUP_NONVIRTUAL|LOOKUP_DESTRUCTOR|LOOKUP_NORMAL,
+                              0);
+         finish_decl_cleanup (NULL_TREE, expr);
+       }
     }
 }
 
@@ -3388,34 +3350,19 @@ build_vec_delete (base, maxindex, auto_delete_vec, use_global_delete)
 
   base = stabilize_reference (base);
 
-  /* Since we can use base many times, save_expr it.  */
-  if (TREE_SIDE_EFFECTS (base))
-    base = save_expr (base);
-
   if (TREE_CODE (type) == POINTER_TYPE)
     {
       /* Step back one from start of vector, and read dimension.  */
       tree cookie_addr;
 
+      if (TREE_SIDE_EFFECTS (base))
+       base = save_expr (base);
       type = strip_array_types (TREE_TYPE (type));
-      if (flag_new_abi)
-       {
-         cookie_addr = build (MINUS_EXPR,
-                              build_pointer_type (sizetype),
-                              base,
-                              TYPE_SIZE_UNIT (sizetype));
-         maxindex = build_indirect_ref (cookie_addr, NULL_PTR);
-       }
-      else
-       {
-         tree cookie;
-
-         cookie_addr = build (MINUS_EXPR, build_pointer_type (BI_header_type),
-                              base, get_cookie_size (type));
-         cookie = build_indirect_ref (cookie_addr, NULL_PTR);
-         maxindex = build_component_ref (cookie, nelts_identifier, 
-                                         NULL_TREE, 0);
-       }
+      cookie_addr = build (MINUS_EXPR,
+                          build_pointer_type (sizetype),
+                          base,
+                          TYPE_SIZE_UNIT (sizetype));
+      maxindex = build_indirect_ref (cookie_addr, NULL);
     }
   else if (TREE_CODE (type) == ARRAY_TYPE)
     {
@@ -3423,6 +3370,8 @@ build_vec_delete (base, maxindex, auto_delete_vec, use_global_delete)
       maxindex = array_type_nelts_total (type);
       type = strip_array_types (type);
       base = build_unary_op (ADDR_EXPR, base, 1);
+      if (TREE_SIDE_EFFECTS (base))
+       base = save_expr (base);
     }
   else
     {