langhooks.h (struct lang_hooks_for_tree_inlining): Add start_inlining, end_inlining.
[gcc.git] / gcc / cp / init.c
index cf480a49f42a6bd224bd4ef4bb0d0d895375918c..098697fa5a7c309206f1d7f40c2559d1b8f9f4f3 100644 (file)
@@ -1,6 +1,6 @@
 /* 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 Free Software Foundation, Inc.
    Contributed by Michael Tiemann (tiemann@cygnus.com)
 
 This file is part of GNU CC.
@@ -26,11 +26,11 @@ Boston, MA 02111-1307, USA.  */
 #include "system.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"
 
@@ -42,13 +42,14 @@ static tree build_vec_delete_1 PARAMS ((tree, tree, tree, special_function_kind,
 static void perform_member_init PARAMS ((tree, tree, int));
 static void sort_base_init PARAMS ((tree, tree, 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 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));
 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));
@@ -58,29 +59,21 @@ 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, BI_header_size;
+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);
 
-  /* Use the biggest alignment supported by the target to prevent operator
-     new from returning misaligned pointers. */
-  TYPE_ALIGN (BI_header_type) = BIGGEST_ALIGNMENT;
-  TYPE_USER_ALIGN (BI_header_type) = 0;
   finish_builtin_type (BI_header_type, "__new_cookie", fields,
-                      0, BI_header_type);
-  BI_header_size = size_in_bytes (BI_header_type);
+                      0, double_type_node);
 
   ggc_add_tree_root (&BI_header_type, 1);
-  ggc_add_tree_root (&BI_header_size, 1);
 }
 
 /* We are about to generate some complex initialization code.
@@ -100,12 +93,14 @@ begin_init_stmts (stmt_expr_p, compound_stmt_p)
   if (building_stmt_tree ())
     *stmt_expr_p = begin_stmt_expr ();
   else
-    *stmt_expr_p = genrtl_begin_stmt_expr ();
+    *stmt_expr_p = begin_global_stmt_expr ();
   
   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);
+  /*
+  else 
+    *compound_stmt_p = genrtl_begin_compound_stmt (has_no_scope=1);
+  */
 }
 
 /* Finish out the statement-expression begun by the previous call to
@@ -119,13 +114,11 @@ finish_init_stmts (stmt_expr, compound_stmt)
 {  
   if (building_stmt_tree ())
     finish_compound_stmt (/*has_no_scope=*/1, compound_stmt);
-  else
-    genrtl_finish_compound_stmt (/*has_no_scope=*/1);
   
   if (building_stmt_tree ())
     stmt_expr = finish_stmt_expr (stmt_expr);
   else
-    stmt_expr = genrtl_finish_stmt_expr (stmt_expr);
+    stmt_expr = finish_global_stmt_expr (stmt_expr);
   
   /* To avoid spurious warnings about unused values, we set 
      TREE_USED.  */
@@ -137,28 +130,21 @@ 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);
     }
@@ -182,21 +168,63 @@ 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.  */
+     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, 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);
+/* [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.  */
+
+static tree
+build_default_init (type)
+     tree type;
+{
+  tree init = NULL_TREE;
+
+  if (TYPE_NEEDS_CONSTRUCTING (type))
+    /* Other code will handle running the default constructor.  We can't do
+       anything with a CONSTRUCTOR for arrays here, as that would imply
+       copy-initialization.  */
+    return NULL_TREE;
+  else if (AGGREGATE_TYPE_P (type) && !TYPE_PTRMEMFUNC_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.  */
+      init = build (CONSTRUCTOR, NULL_TREE, NULL_TREE, NULL_TREE);
+    }
+  else if (TREE_CODE (type) == REFERENCE_TYPE)
+    /*   --if T is a reference type, no initialization is performed.  */
+    return NULL_TREE;
+  else
+    {
+      init = integer_zero_node;
+      
+      if (TREE_CODE (type) == ENUMERAL_TYPE)
+        /* We must make enumeral types the right type. */
+        init = fold (build1 (NOP_EXPR, type, init));
+    }
+
+  init = digest_init (type, init, 0);
+  return init;
 }
 
 /* Subroutine of emit_base_init.  */
@@ -240,9 +268,7 @@ 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, TREE_VALUE (init), 1));
        }
       else
        finish_expr_stmt (build_aggr_init (decl, init, 0));
@@ -253,28 +279,14 @@ 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);
+             if (TREE_CODE (type) == REFERENCE_TYPE)
+               cp_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)
+         else if (TREE_CODE (type) == REFERENCE_TYPE)
            cp_pedwarn ("uninitialized reference member `%D'", member);
        }
       else if (TREE_CODE (init) == TREE_LIST)
@@ -538,68 +550,40 @@ sort_base_init (t, base_init_list, rbase_ptr, vbase_ptr)
   tree vbases = NULL_TREE;
 
   /* First walk through and splice out vbase and invalid initializers.
-     Also replace names with binfos.  */
+     Also replace types 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;
+      tree binfo = (TREE_CODE (basetype) == TREE_VEC
+                   ? basetype : binfo_or_else (basetype, t));
+      
+      if (binfo == NULL_TREE)
+       /* BASETYPE might be an inaccessible direct base (because it
+          is also an indirect base).  */
+       continue;
 
-      if (basetype == NULL_TREE)
+      if (TREE_VIA_VIRTUAL (binfo))
        {
-         /* 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);
+         /* 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.  */
+         tree v = binfo_for_vbase (BINFO_TYPE (binfo), t);
+         vbases = tree_cons (v, TREE_VALUE (x), vbases);
        }
-      else if (is_aggr_type (basetype, 1))
+      else
        {
-         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;
-               }
-           }
+         /* Otherwise, it must be an immediate base class.  */
+         my_friendly_assert
+           (same_type_p (BINFO_TYPE (BINFO_INHERITANCE_CHAIN (binfo)),
+                         t), 20011113);
+
+         TREE_PURPOSE (x) = binfo;
+         TREE_CHAIN (last) = x;
+         last = x;
        }
-      else
-       my_friendly_abort (365);
-
-      TREE_PURPOSE (x) = binfo;
-      TREE_CHAIN (last) = x;
-      last = x;
     }
   TREE_CHAIN (last) = NULL_TREE;
 
@@ -669,10 +653,7 @@ sort_base_init (t, base_init_list, rbase_ptr, vbase_ptr)
 
    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.  */
+   at the PARM level of scope.  */
 
 void
 emit_base_init (mem_init_list, base_init_list)
@@ -724,9 +705,10 @@ emit_base_init (mem_init_list, base_init_list)
 
       if (init != void_list_node)
        {
-         member = convert_pointer_to_real (base_binfo, current_class_ptr);
+         member = build_base_path (PLUS_EXPR, current_class_ptr,
+                                   base_binfo, 1);
          expand_aggr_init_1 (base_binfo, NULL_TREE,
-                             build_indirect_ref (member, NULL_PTR), init,
+                             build_indirect_ref (member, NULL), init,
                              LOOKUP_NORMAL);
        }
 
@@ -775,16 +757,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);
@@ -804,20 +797,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)
     {
@@ -825,7 +812,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,
@@ -837,16 +824,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);
@@ -889,7 +876,7 @@ expand_aggr_vbase_init_1 (binfo, exp, addr, init_list)
      tree binfo, exp, addr, init_list;
 {
   tree init = purpose_member (binfo, init_list);
-  tree ref = build_indirect_ref (addr, NULL_PTR);
+  tree ref = build_indirect_ref (addr, NULL);
 
   if (init)
     init = TREE_VALUE (init);
@@ -915,22 +902,6 @@ construct_virtual_bases (type, this_ref, this_ptr, init_list, flag)
   /* 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))
@@ -1007,19 +978,19 @@ 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)
     {
-      cp_error ("class `%T' does not have any field named `%s'", type,
+      cp_error ("class `%T' does not have any field named `%D'", type,
                member_name);
       return 0;
     }
   if (TREE_STATIC (field))
     {
-      cp_error ("field `%#D' is static; only point of initialization is its declaration",
+      cp_error ("field `%#D' is static; the only point of initialization is its definition",
                field);
       return 0;
     }
@@ -1027,17 +998,16 @@ member_init_ok_or_else (field, type, member_name)
   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.
+/* EXP is an expression of aggregate type. NAME is 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. EXP is used
+   only to get its type.  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)
@@ -1050,67 +1020,56 @@ expand_member_init (exp, name, init)
     return NULL_TREE;
 
   type = TYPE_MAIN_VARIANT (TREE_TYPE (exp));
+  my_friendly_assert (IS_AGGR_TYPE (type), 20011113);
 
-  if (name && TREE_CODE (name) == TYPE_DECL)
+  if (!name)
     {
-      basetype = TYPE_MAIN_VARIANT (TREE_TYPE (name));
-      name = DECL_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;
+      /* This is an obsolete unnamed base class initializer.  The
+        parser will already have warned about its use.  */
+      switch (CLASSTYPE_N_BASECLASSES (type))
+       {
+       case 0:
+         cp_error ("unnamed initializer for `%T', which has no base classes",
+                   type);
+         return NULL_TREE;
+       case 1:
+         basetype = TYPE_BINFO_BASETYPE (type, 0);
+         break;
+       default:
+         cp_error ("unnamed initializer for `%T', which uses multiple inheritance",
+                   type);
+         return NULL_TREE;
       }
+    }
+  else if (TYPE_P (name))
+    {
+      basetype = name;
+      name = TYPE_NAME (name);
+    }
+  else if (TREE_CODE (name) == TYPE_DECL)
+    basetype = TYPE_MAIN_VARIANT (TREE_TYPE (name));
 
   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 (init == void_type_node)
     init = NULL_TREE;
 
-  if (name == NULL_TREE || basetype)
+  if (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))
+      if (current_template_parms)
+       ;
+      else if (vec_binfo_member (basetype, TYPE_BINFO_BASETYPES (type)))
+       /* A direct base.  */;
+      else if (binfo_for_vbase (basetype, type))
+       /* A virtual base.  */;
+      else
        {
-         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);
+           cp_error ("type `%D' is not a direct or virtual base of `%T'",
+                     name, type);
          else
-           cp_error ("type `%T' is not an immediate basetype for `%T'",
-                     basetype, type);
+           cp_error ("type `%D' is not a direct base of `%T'",
+                     name, type);
          return NULL_TREE;
        }
 
@@ -1118,10 +1077,9 @@ expand_member_init (exp, name, init)
     }
   else
     {
-    try_member:
       field = lookup_field (type, name, 1, 0);
 
-      if (! member_init_ok_or_else (field, type, IDENTIFIER_POINTER (name)))
+      if (! member_init_ok_or_else (field, type, name))
        return NULL_TREE;
 
       init = build_tree_list (field, init);
@@ -1189,13 +1147,8 @@ build_aggr_init (exp, init, flags)
       /* Must arrange to initialize each element of EXP
         from elements of INIT.  */
       tree itype = init ? TREE_TYPE (init) : NULL_TREE;
-      if (CP_TYPE_QUALS (type) != TYPE_UNQUALIFIED)
-       {
-         TREE_TYPE (exp) = TYPE_MAIN_VARIANT (type);
-         if (init)
-           TREE_TYPE (init) = TYPE_MAIN_VARIANT (itype);
-       }
-      if (init && TREE_TYPE (init) == NULL_TREE)
+      
+      if (init && !itype)
        {
          /* Handle bad initializers like:
             class COMPLEX {
@@ -1209,10 +1162,16 @@ build_aggr_init (exp, init, flags)
               COMPLEX zees(1.0, 0.0)[10];
             }
          */
-         error ("bad array initializer");
+         cp_error ("bad array initializer");
          return error_mark_node;
        }
-      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 (init)
+           TREE_TYPE (init) = TYPE_MAIN_VARIANT (itype);
+       }
+      stmt_expr = build_vec_init (exp, init,
                                  init && same_type_p (TREE_TYPE (init),
                                                       TREE_TYPE (exp)));
       TREE_READONLY (exp) = was_const;
@@ -1229,12 +1188,12 @@ 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;
-  stmts_are_full_exprs_p = 0;
+  destroy_temps = stmts_are_full_exprs_p ();
+  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);
-  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;
@@ -1392,7 +1351,7 @@ is_aggr_type (type, or_else)
 
   if (! IS_AGGR_TYPE (type)
       && TREE_CODE (type) != TEMPLATE_TYPE_PARM
-      && TREE_CODE (type) != TEMPLATE_TEMPLATE_PARM)
+      && TREE_CODE (type) != BOUND_TEMPLATE_TEMPLATE_PARM)
     {
       if (or_else)
        cp_error ("`%T' is not an aggregate type", type);
@@ -1424,7 +1383,7 @@ get_aggr_from_typedef (name, or_else)
 
   if (! IS_AGGR_TYPE (type)
       && TREE_CODE (type) != TEMPLATE_TYPE_PARM
-      && TREE_CODE (type) != TEMPLATE_TEMPLATE_PARM)
+      && TREE_CODE (type) != BOUND_TEMPLATE_TEMPLATE_PARM)
     {
       if (or_else)
        cp_error ("type `%T' is of non-aggregate type", type);
@@ -1485,9 +1444,6 @@ build_member_call (type, name, parmlist)
   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);
@@ -1518,7 +1474,8 @@ 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 build_x_function_call (build_offset_ref (type, name),
+                                       parmlist, current_class_ref);
        }
     }
 
@@ -1551,7 +1508,7 @@ build_member_call (type, name, parmlist)
        {
          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);
+         decl = build_indirect_ref (decl, NULL);
        }
     }
 
@@ -1620,27 +1577,9 @@ 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);
 
-  /* Handle namespace names fully here.  */
-  if (TREE_CODE (type) == NAMESPACE_DECL)
-    {
-      t = lookup_namespace_name (type, name);
-      if (t != error_mark_node && ! type_unknown_p (t))
-       {
-         mark_used (t);
-         t = convert_from_reference (t);
-       }
-      return t;
-    }
-
-  if (type == NULL_TREE || ! is_aggr_type (type, 1))
-    return error_mark_node;
-
   if (TREE_CODE (name) == TEMPLATE_ID_EXPR)
     {
       /* If the NAME is a TEMPLATE_ID_EXPR, we are looking at
@@ -1668,6 +1607,30 @@ build_offset_ref (type, name)
       my_friendly_assert (TREE_CODE (name) == IDENTIFIER_NODE, 0);
     }
 
+  if (type == NULL_TREE)
+    return error_mark_node;
+  
+  /* Handle namespace names fully here.  */
+  if (TREE_CODE (type) == NAMESPACE_DECL)
+    {
+      t = lookup_namespace_name (type, name);
+      if (t == error_mark_node)
+        return t;
+      if (TREE_CODE (orig_name) == TEMPLATE_ID_EXPR)
+        /* Reconstruct the TEMPLATE_ID_EXPR.  */
+        t = build (TEMPLATE_ID_EXPR, TREE_TYPE (t),
+                   t, TREE_OPERAND (orig_name, 1));
+      if (! type_unknown_p (t))
+       {
+         mark_used (t);
+         t = convert_from_reference (t);
+       }
+      return t;
+    }
+
+  if (! is_aggr_type (type, 1))
+    return error_mark_node;
+
   if (TREE_CODE (name) == BIT_NOT_EXPR)
     {
       if (! check_dtor_name (type, name))
@@ -1712,13 +1675,13 @@ build_offset_ref (type, name)
               expects to encounter OVERLOADs, not raw functions.  */
            t = ovl_cons (t, NULL_TREE);
 
-         return build (OFFSET_REF, 
-                       unknown_type_node,
-                       decl,
-                       build (TEMPLATE_ID_EXPR, 
-                              TREE_TYPE (t),
-                              t,
-                              TREE_OPERAND (orig_name, 1)));
+          t = build (TEMPLATE_ID_EXPR, TREE_TYPE (t), t,
+                    TREE_OPERAND (orig_name, 1));
+         t = build (OFFSET_REF, unknown_type_node, decl, t);
+          
+          PTRMEM_OK_P (t) = 1;
+                 
+         return t;
        }
 
       if (!really_overloaded_fn (t))
@@ -1732,11 +1695,16 @@ build_offset_ref (type, name)
          mark_used (t);
          if (DECL_STATIC_FUNCTION_P (t))
            return t;
-         return build (OFFSET_REF, TREE_TYPE (t), decl, t);
+         t = build (OFFSET_REF, TREE_TYPE (t), decl, t);
+         PTRMEM_OK_P (t) = 1;
+         return t;
        }
 
       TREE_TYPE (fnfields) = unknown_type_node;
-      return build (OFFSET_REF, unknown_type_node, decl, fnfields);
+      
+      t = build (OFFSET_REF, unknown_type_node, decl, fnfields);
+      PTRMEM_OK_P (t) = 1;
+      return t;
     }
 
   t = member;
@@ -1774,7 +1742,9 @@ build_offset_ref (type, name)
   /* In member functions, the form `type::name' is no longer
      equivalent to `this->type::name', at least not until
      resolve_offset_ref.  */
-  return build (OFFSET_REF, build_offset_type (type, TREE_TYPE (t)), decl, t);
+  t = build (OFFSET_REF, build_offset_type (type, TREE_TYPE (t)), decl, t);
+  PTRMEM_OK_P (t) = 1;
+  return t;
 }
 
 /* If a OFFSET_REF made it through to here, then it did
@@ -1807,20 +1777,19 @@ resolve_offset_ref (exp)
       base = current_class_ref;
     }
 
-  if (BASELINK_P (member))
-    {
-      if (! flag_ms_extensions)
-       cp_pedwarn ("assuming & on overloaded member function");
-      return build_unary_op (ADDR_EXPR, exp, 0);
-    }
-
+  if (BASELINK_P (member) || TREE_CODE (member) == TEMPLATE_ID_EXPR)
+    return build_unary_op (ADDR_EXPR, exp, 0);
+  
   if (TREE_CODE (TREE_TYPE (member)) == METHOD_TYPE)
     {
-      if (! flag_ms_extensions)
-       cp_pedwarn ("assuming & on `%E'", member);
+      if (!flag_ms_extensions)
+        /* A single non-static member, make sure we don't allow a
+           pointer-to-member.  */
+        exp = ovl_cons (member, NULL_TREE);
+      
       return build_unary_op (ADDR_EXPR, exp, 0);
     }
-
+  
   if ((TREE_CODE (member) == VAR_DECL
        && ! TYPE_PTRMEMFUNC_P (TREE_TYPE (member))
        && ! TYPE_PTRMEM_P (TREE_TYPE (member)))
@@ -1862,14 +1831,17 @@ resolve_offset_ref (exp)
       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);
+      /* Don't check access on the conversion; we might be after a member
+        promoted by an access- or using-declaration, and we have already
+        checked access for the member itself.  */
+      basetype = lookup_base (TREE_TYPE (base), basetype, ba_ignore, NULL);
+      expr = build_base_path (PLUS_EXPR, base, basetype, 1);
 
-      if (addr == error_mark_node)
+      if (expr == error_mark_node)
        return error_mark_node;
 
       expr = build (COMPONENT_REF, TREE_TYPE (member),
-                   build_indirect_ref (addr, NULL_PTR), member);
+                   expr, member);
       return convert_from_reference (expr);
     }
 
@@ -1892,17 +1864,12 @@ resolve_offset_ref (exp)
        }
 
       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 = 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));
@@ -1975,8 +1942,6 @@ build_builtin_delete_call (addr)
 
    PLACEMENT is the `placement' list for user-defined operator new ().  */
 
-extern int flag_check_new;
-
 tree
 build_new (placement, decl, init, use_global_new)
      tree placement;
@@ -2034,8 +1999,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, 0)
                      == NULL_TREE)
                    pedwarn ("size in array new must have integral type");
 
@@ -2047,7 +2012,7 @@ build_new (placement, decl, init, use_global_new)
                      nelts = integer_zero_node;
                    }
                  else
-                   nelts = build_binary_op (MULT_EXPR, nelts, this_nelts);
+                   nelts = cp_build_binary_op (MULT_EXPR, nelts, this_nelts);
                }
            }
          else
@@ -2153,18 +2118,33 @@ 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)
     {
@@ -2175,12 +2155,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.  */
 
@@ -2190,22 +2170,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
-    cookie_size = BI_header_size;
+    cookie_size = type_align;
 
   return cookie_size;
 }
@@ -2218,9 +2193,11 @@ 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 nelts = NULL_TREE;
-  tree alloc_expr, alloc_node = NULL_TREE;
+  tree alloc_call, alloc_expr, alloc_node;
+  tree cookie_expr, init_expr;
   int has_array = 0;
   enum tree_code code;
   int use_cookie, nothrow, check_new;
@@ -2234,6 +2211,9 @@ build_new_1 (exp)
      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;
 
   placement = TREE_OPERAND (exp, 0);
   type = TREE_OPERAND (exp, 1);
@@ -2245,31 +2225,33 @@ build_new_1 (exp)
       has_array = 1;
       nelts = TREE_OPERAND (type, 1);
       type = TREE_OPERAND (type, 0);
+
+      full_type = cp_build_binary_op (MINUS_EXPR, nelts, integer_one_node);
+      full_type = build_index_type (full_type);
+      full_type = build_cplus_array_type (type, full_type);
     }
+  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)
     {
       tree this_nelts = array_type_nelts_top (true_type);
-      nelts = build_binary_op (MULT_EXPR, nelts, this_nelts);
+      nelts = cp_build_binary_op (MULT_EXPR, nelts, this_nelts);
       true_type = TREE_TYPE (true_type);
     }
 
   if (!complete_type_or_else (true_type, exp))
     return error_mark_node;
 
+  size = size_in_bytes (true_type);
   if (has_array)
-    size = fold (build_binary_op (MULT_EXPR, size_in_bytes (true_type),
-                                 nelts));
-  else
-    size = size_in_bytes (type);
+    size = fold (cp_build_binary_op (MULT_EXPR, size, nelts));
 
   if (TREE_CODE (true_type) == VOID_TYPE)
     {
@@ -2284,8 +2266,9 @@ build_new_1 (exp)
      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))))
+      && (has_array
+         ? TYPE_HAS_ARRAY_NEW_OPERATOR (true_type)
+         : TYPE_HAS_NEW_OPERATOR (true_type)))
     use_global_new = 0;
   else
     use_global_new = 1;
@@ -2295,14 +2278,12 @@ build_new_1 (exp)
   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 extra storage.  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);
+    use_cookie = !use_global_new;
   /* Otherwise, we need the cookie.  */
   else
     use_cookie = 1;
@@ -2315,9 +2296,6 @@ build_new_1 (exp)
       size = size_binop (PLUS_EXPR, size, cookie_size);
     }
 
-  if (has_array && init && pedantic)
-    cp_pedwarn ("initialization in array new");
-
   /* Allocate the object.  */
   
   if (! placement && TYPE_FOR_JAVA (true_type))
@@ -2325,17 +2303,18 @@ build_new_1 (exp)
       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
     {
@@ -2346,16 +2325,38 @@ build_new_1 (exp)
       fnname = ansi_opname (code);
 
       if (use_global_new)
-       rval = (build_new_function_call 
-               (lookup_function_nonclass (fnname, args),
-                args));
+       alloc_call = (build_new_function_call 
+                     (lookup_function_nonclass (fnname, args),
+                      args));
       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);
+       alloc_call = build_method_call (build_dummy_object (true_type),
+                                       fnname, args, NULL_TREE,
+                                       LOOKUP_NORMAL);
     }
 
+  if (alloc_call == error_mark_node)
+    return error_mark_node;
+
+  /* The ALLOC_CALL should be a CALL_EXPR, and the first operand
+     should be the address of a known FUNCTION_DECL.  */
+  my_friendly_assert (TREE_CODE (alloc_call) == CALL_EXPR, 20000521);
+  t = TREE_OPERAND (alloc_call, 0);
+  my_friendly_assert (TREE_CODE (t) == ADDR_EXPR, 20000521);
+  t = TREE_OPERAND (t, 0);
+  my_friendly_assert (TREE_CODE (t) == FUNCTION_DECL, 20000521);
+  /* 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 (t)) > 1 || varargs_function_p (t));
+
   /*        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_,
@@ -2366,150 +2367,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));
-    }
+  /* The ADDR_EXPR.  */
+  t = TREE_OPERAND (alloc_call, 0);
+  /* The function.  */
+  t = TREE_OPERAND (t, 0);
+  nothrow = TYPE_NOTHROW_P (TREE_TYPE (t));
   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 (use_cookie)
+    /* 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 BI_header_size
-     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 (use_cookie)
     {
-      tree cookie, exp1;
-      rval = convert (string_type_node, rval); /* for ptr arithmetic */
-      rval = save_expr (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);
+      else if (init && pedantic && has_array)
+       cp_pedwarn ("ISO C++ forbids initialization in array new");
+
+      if (has_array)
+       init_expr = build_vec_init (init_expr, init, 0);
+      else if (TYPE_NEEDS_CONSTRUCTING (type))
+       init_expr = build_method_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),
-                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
@@ -2519,10 +2458,10 @@ 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));
 
@@ -2531,21 +2470,32 @@ build_new_1 (exp)
              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, alloc_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.  */
 
          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);
@@ -2554,47 +2504,48 @@ build_new_1 (exp)
                = 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);
 
- done:
+  /* Now build up the return value in reverse order.  */
 
-  if (alloc_expr && rval == alloc_node)
-    {
-      rval = TREE_OPERAND (alloc_expr, 1);
-      alloc_expr = NULL_TREE;
-    }
-
-  if (check_new && alloc_expr)
-    {
-      /* Did we modify the storage?  */
-      tree ifexp = 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
@@ -2633,7 +2584,7 @@ build_vec_delete_1 (base, maxindex, type, auto_delete_vec, use_global_delete)
       goto no_destructor;
     }
 
-  /* The below is short by BI_header_size */
+  /* The below is short by the cookie size.  */
   virtual_size = size_binop (MULT_EXPR, size_exp,
                             convert (sizetype, maxindex));
 
@@ -2676,7 +2627,7 @@ build_vec_delete_1 (base, maxindex, type, auto_delete_vec, use_global_delete)
     {
       tree base_tbd;
 
-      /* The below is short by BI_header_size */
+      /* The below is short by the cookie size.  */
       virtual_size = size_binop (MULT_EXPR, size_exp,
                                 convert (sizetype, maxindex));
 
@@ -2690,9 +2641,10 @@ build_vec_delete_1 (base, maxindex, type, auto_delete_vec, use_global_delete)
          cookie_size = get_cookie_size (type);
          base_tbd 
            = cp_convert (ptype,
-                         build_binary_op (MINUS_EXPR,
-                                          cp_convert (string_type_node, base),
-                                          cookie_size));
+                         cp_build_binary_op (MINUS_EXPR,
+                                             cp_convert (string_type_node, 
+                                                         base),
+                                             cookie_size));
          /* True size with header.  */
          virtual_size = size_binop (PLUS_EXPR, virtual_size, cookie_size);
        }
@@ -2728,6 +2680,8 @@ build_vec_delete_1 (base, maxindex, type, auto_delete_vec, use_global_delete)
     return cp_convert (void_type_node, body);
 }
 
+/* Create an unnamed variable of the indicated TYPE.  */ 
+
 tree
 create_temporary_var (type)
      tree type;
@@ -2762,7 +2716,7 @@ get_temp_regvar (type, init)
   if (building_stmt_tree ())
     add_decl_stmt (decl);
   if (!building_stmt_tree ())
-    DECL_RTL (decl) = assign_temp (type, 2, 0, 1);
+    SET_DECL_RTL (decl, assign_temp (type, 2, 0, 1));
   finish_expr_stmt (build_modify_expr (decl, INIT_EXPR, init));
 
   return decl;
@@ -2771,12 +2725,7 @@ 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*'.
-   MAXINDEX is the maximum index of the array (one less than the
-           number of elements).
+   BASE is a reference to the vector, of ARRAY_TYPE.
    INIT is the (possibly NULL) initializer.
 
    FROM_ARRAY is 0 if we should init everything with INIT
@@ -2787,8 +2736,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, init, from_array)
+     tree base, init;
      int from_array;
 {
   tree rval;
@@ -2796,8 +2745,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;
@@ -2806,38 +2757,56 @@ build_vec_init (decl, base, maxindex, init, from_array)
   tree try_block = NULL_TREE;
   tree try_body = NULL_TREE;
   int num_initialized_elts = 0;
+  tree maxindex = array_type_nelts (TREE_TYPE (base));
 
-  maxindex = cp_convert (ptrdiff_type_node, maxindex);
   if (maxindex == error_mark_node)
     return error_mark_node;
 
-  type = TREE_TYPE (TREE_TYPE (base));
+  /* For g++.ext/arrnew.C.  */
+  if (init && TREE_CODE (init) == CONSTRUCTOR && TREE_TYPE (init) == NULL_TREE)
+    init = digest_init (atype, init, 0);
+      
+  if (init && !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);
+      TREE_SIDE_EFFECTS (stmt_expr) = 1;
+      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);
-        }
+        do {
+          ... initialize *t1 ...
+          ++t1;
+        } while (--iterator != -1);
        } 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
@@ -2850,26 +2819,26 @@ build_vec_init (decl, base, maxindex, init, from_array)
      of blocks of memory.  */
 
   begin_init_stmts (&stmt_expr, &compound_stmt);
-  destroy_temps = stmts_are_full_exprs_p;
-  stmts_are_full_exprs_p = 0;
-  rval = get_temp_regvar (ptype, 
-                         cp_convert (ptype, default_conversion (base)));
+  destroy_temps = stmts_are_full_exprs_p ();
+  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;
@@ -2887,16 +2856,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.  */
@@ -2907,11 +2868,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);
@@ -2949,7 +2905,7 @@ build_vec_init (decl, base, maxindex, init, from_array)
 
       if_stmt = begin_if_stmt ();
       finish_if_stmt_cond (build (NE_EXPR, boolean_type_node,
-                                 iterator, minus_one_node),
+                                 iterator, integer_minus_one_node),
                           if_stmt);
 
       /* Otherwise, loop through the elements.  */
@@ -2992,20 +2948,17 @@ build_vec_init (decl, base, maxindex, init, from_array)
       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);
        }
       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);
@@ -3013,31 +2966,20 @@ build_vec_init (decl, base, maxindex, init, from_array)
        }
       else
        {
-         stmts_are_full_exprs_p = 1;
+         current_stmt_tree ()->stmts_are_full_exprs_p = 1;
          finish_expr_stmt (elt_init);
-         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_expr_stmt (build_unary_op (PREINCREMENT_EXPR, base2, 0));
 
       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),
+                            build_unary_op (PREDECREMENT_EXPR, iterator, 0),
+                            integer_minus_one_node),
                      do_stmt);
 
       finish_then_clause (if_stmt);
@@ -3045,15 +2987,16 @@ build_vec_init (decl, base, maxindex, init, from_array)
     }
 
   /* 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;
 
       finish_compound_stmt (/*has_no_scope=*/1, try_body);
       finish_cleanup_try_block (try_block);
       e = build_vec_delete_1 (rval,
-                             build_binary_op (MINUS_EXPR, maxindex, 
-                                              iterator),
+                             cp_build_binary_op (MINUS_EXPR, maxindex, 
+                                                 iterator),
                              type,
                              sfk_base_destructor,
                              /*use_global_delete=*/0);
@@ -3065,7 +3008,7 @@ build_vec_init (decl, base, maxindex, init, from_array)
   finish_expr_stmt (rval);
 
   stmt_expr = finish_init_stmts (stmt_expr, compound_stmt);
-  stmts_are_full_exprs_p = destroy_temps;
+  current_stmt_tree ()->stmts_are_full_exprs_p = destroy_temps;
   return stmt_expr;
 }
 
@@ -3148,7 +3091,6 @@ build_delete (type, addr, auto_delete, flags, use_global_delete)
 {
   tree member;
   tree expr;
-  tree ref;
 
   if (addr == error_mark_node)
     return error_mark_node;
@@ -3177,7 +3119,6 @@ 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)
     {
@@ -3201,19 +3142,14 @@ 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
@@ -3230,16 +3166,39 @@ build_delete (type, addr, auto_delete, flags, use_global_delete)
       tree do_delete = NULL_TREE;
       tree ifexp;
 
+      /* 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)
        {
+         /* 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,
+                                           c_sizeof_nowarn (type),
+                                           LOOKUP_NORMAL,
+                                           NULL_TREE);
+         /* Call the complete object destructor.  */
+         auto_delete = sfk_complete_destructor;
+       }
 
-      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);
 
@@ -3248,7 +3207,7 @@ build_delete (type, addr, auto_delete, flags, use_global_delete)
        ifexp = integer_one_node;
       else
        /* Handle deleting a null pointer.  */
-       ifexp = fold (build_binary_op (NE_EXPR, addr, integer_zero_node));
+       ifexp = fold (cp_build_binary_op (NE_EXPR, addr, integer_zero_node));
 
       if (ifexp != integer_one_node)
        expr = build (COND_EXPR, void_type_node,
@@ -3263,6 +3222,7 @@ build_delete (type, addr, auto_delete, flags, use_global_delete)
       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;
+      tree ref = build_indirect_ref (addr, NULL);
 
       /* Set this again before we call anything, as we might get called
         recursively.  */
@@ -3375,26 +3335,12 @@ build_vec_delete (base, maxindex, auto_delete_vec, use_global_delete)
       /* Step back one from start of vector, and read dimension.  */
       tree cookie_addr;
 
-      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, BI_header_size);
-         cookie = build_indirect_ref (cookie_addr, NULL_PTR);
-         maxindex = build_component_ref (cookie, nelts_identifier, 
-                                         NULL_TREE, 0);
-       }
-
       type = strip_array_types (TREE_TYPE (type));
+      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)
     {