OSDN Git Service

PR c++/19991
[pf3gnuchains/gcc-fork.git] / gcc / cp / init.c
index 0fa2365..1a81961 100644 (file)
@@ -1,6 +1,6 @@
 /* Handle initialization things in C++.
    Copyright (C) 1987, 1989, 1992, 1993, 1994, 1995, 1996, 1997, 1998,
-   1999, 2000, 2001, 2002, 2003, 2004 Free Software Foundation, Inc.
+   1999, 2000, 2001, 2002, 2003, 2004, 2005 Free Software Foundation, Inc.
    Contributed by Michael Tiemann (tiemann@cygnus.com)
 
 This file is part of GCC.
@@ -87,7 +87,7 @@ finish_init_stmts (bool is_global, tree stmt_expr, tree compound_stmt)
   
   stmt_expr = finish_stmt_expr (stmt_expr, true);
 
-  my_friendly_assert (!building_stmt_tree () == is_global, 20030726);
+  gcc_assert (!building_stmt_tree () == is_global);
   
   return stmt_expr;
 }
@@ -101,8 +101,10 @@ finish_init_stmts (bool is_global, tree stmt_expr, tree compound_stmt)
 static tree
 dfs_initialize_vtbl_ptrs (tree binfo, void *data)
 {
-  if ((!BINFO_PRIMARY_P (binfo) || BINFO_VIRTUAL_P (binfo))
-      && CLASSTYPE_VFIELDS (BINFO_TYPE (binfo)))
+  if (!TYPE_CONTAINS_VPTR_P (BINFO_TYPE (binfo)))
+    return dfs_skip_bases;
+  
+  if (!BINFO_PRIMARY_P (binfo) || BINFO_VIRTUAL_P (binfo))
     {
       tree base_ptr = TREE_VALUE ((tree) data);
 
@@ -111,8 +113,6 @@ dfs_initialize_vtbl_ptrs (tree binfo, void *data)
       expand_virtual_init (binfo, base_ptr);
     }
 
-  BINFO_MARKED (binfo) = 1;
-
   return NULL_TREE;
 }
 
@@ -132,9 +132,7 @@ initialize_vtbl_ptrs (tree addr)
      class.  We do these in pre-order because 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, unmarkedp, list);
-  dfs_walk (TYPE_BINFO (type), dfs_unmark, markedp, type);
+  dfs_walk_once (TYPE_BINFO (type), dfs_initialize_vtbl_ptrs, NULL, list);
 }
 
 /* Return an expression for the zero-initialization of an object with
@@ -171,8 +169,7 @@ build_zero_init (tree type, tree nelts, bool static_storage_p)
 
      -- if T is a reference type, no initialization is performed.  */
 
-  my_friendly_assert (nelts == NULL_TREE || TREE_CODE (nelts) == INTEGER_CST,
-                     20030618);
+  gcc_assert (nelts == NULL_TREE || TREE_CODE (nelts) == INTEGER_CST);
 
   if (type == error_mark_node)
     ;
@@ -217,7 +214,6 @@ build_zero_init (tree type, tree nelts, bool static_storage_p)
     }
   else if (TREE_CODE (type) == ARRAY_TYPE)
     {
-      tree index;
       tree max_index;
       tree inits;
 
@@ -225,26 +221,35 @@ build_zero_init (tree type, tree nelts, bool static_storage_p)
       init = build_constructor (type, NULL_TREE);
       /* Iterate over the array elements, building initializations.  */
       inits = NULL_TREE;
-      max_index = nelts ? nelts : array_type_nelts (type);
-      my_friendly_assert (TREE_CODE (max_index) == INTEGER_CST, 20030618);
+      if (nelts)
+       max_index = fold (build2 (MINUS_EXPR, TREE_TYPE (nelts),
+                                 nelts, integer_one_node));
+      else
+       max_index = array_type_nelts (type);
+      gcc_assert (TREE_CODE (max_index) == INTEGER_CST);
 
       /* A zero-sized array, which is accepted as an extension, will
         have an upper bound of -1.  */
       if (!tree_int_cst_equal (max_index, integer_minus_one_node))
-       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);
+       {
+         tree elt_init = build_zero_init (TREE_TYPE (type),
+                                          /*nelts=*/NULL_TREE,
+                                          static_storage_p);
+         tree range;
+         
+         /* If this is a one element array, we just use a regular init.  */
+         if (tree_int_cst_equal (size_zero_node, max_index))
+           range = size_zero_node;
+         else
+          range = build2 (RANGE_EXPR, sizetype, size_zero_node, max_index);
+         
+         inits = tree_cons (range, elt_init, inits);
+       }
+      
       CONSTRUCTOR_ELTS (init) = nreverse (inits);
     }
-  else if (TREE_CODE (type) == REFERENCE_TYPE)
-    ;
   else
-    abort ();
+    gcc_assert (TREE_CODE (type) == REFERENCE_TYPE);
 
   /* In all cases, the initializer is a constant.  */
   if (init)
@@ -318,9 +323,8 @@ perform_member_init (tree member, tree init)
   /* 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);
+    warning ("%J%qD should be initialized in the member initialization "
+            "list", current_function_decl, member);
 
   if (init == void_type_node)
     init = NULL_TREE;
@@ -339,7 +343,7 @@ perform_member_init (tree member, tree init)
     {
       if (init)
        {
-         init = build (INIT_EXPR, type, decl, TREE_VALUE (init));
+         init = build2 (INIT_EXPR, type, decl, TREE_VALUE (init));
          finish_expr_stmt (init);
        }
     }
@@ -366,16 +370,17 @@ perform_member_init (tree member, tree init)
            {
              init = build_default_init (type, /*nelts=*/NULL_TREE);
              if (TREE_CODE (type) == REFERENCE_TYPE)
-               warning
-                 ("default-initialization of `%#D', which has reference type",
-                  member);
+               warning ("%Jdefault-initialization of %q#D, "
+                        "which has reference type",
+                        current_function_decl, member);
            }
          /* member traversal: note it leaves init NULL */
          else if (TREE_CODE (type) == REFERENCE_TYPE)
-           pedwarn ("uninitialized reference member `%D'", member);
+           pedwarn ("%Juninitialized reference member %qD",
+                    current_function_decl, member);
          else if (CP_TYPE_CONST_P (type))
-           pedwarn ("uninitialized member `%D' with `const' type `%T'",
-                    member, type);
+           pedwarn ("%Juninitialized member %qD with %<const%> type %qT",
+                    current_function_decl, member, type);
        }
       else if (TREE_CODE (init) == TREE_LIST)
        /* There was an explicit member initialization.  Do some work
@@ -457,9 +462,10 @@ static tree
 sort_mem_initializers (tree t, tree mem_inits)
 {
   tree init;
-  tree base;
+  tree base, binfo, base_binfo;
   tree sorted_inits;
   tree next_subobject;
+  VEC (tree) *vbases;
   int i;
   int uses_unions_p;
 
@@ -470,17 +476,16 @@ sort_mem_initializers (tree t, tree mem_inits)
   sorted_inits = NULL_TREE;
   
   /* Process the virtual bases.  */
-  for (i = 0; (base = VEC_iterate
-              (tree, CLASSTYPE_VBASECLASSES (t), i)); i++)
+  for (vbases = CLASSTYPE_VBASECLASSES (t), i = 0;
+       VEC_iterate (tree, vbases, i, base); i++)
     sorted_inits = tree_cons (base, NULL_TREE, sorted_inits);
   
   /* Process the direct bases.  */
-  for (i = 0; i < BINFO_N_BASE_BINFOS (TYPE_BINFO (t)); ++i)
-    {
-      base = BINFO_BASE_BINFO (TYPE_BINFO (t), i);
-      if (!BINFO_VIRTUAL_P (base))
-       sorted_inits = tree_cons (base, NULL_TREE, sorted_inits);
-    }
+  for (binfo = TYPE_BINFO (t), i = 0;
+       BINFO_BASE_ITERATE (binfo, i, base_binfo); ++i)
+    if (!BINFO_VIRTUAL_P (base_binfo))
+      sorted_inits = tree_cons (base_binfo, 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
@@ -512,20 +517,21 @@ sort_mem_initializers (tree t, tree mem_inits)
          break;
 
       /* Issue a warning if the explicit initializer order does not
-        match that which will actually occur.  */
+        match that which will actually occur.
+         ??? Are all these on the correct lines?  */
       if (warn_reorder && !subobject_init)
        {
          if (TREE_CODE (TREE_PURPOSE (next_subobject)) == FIELD_DECL)
-           cp_warning_at ("`%D' will be initialized after",
+           cp_warning_at ("%qD will be initialized after",
                           TREE_PURPOSE (next_subobject));
          else
-           warning ("base `%T' will be initialized after",
+           warning ("base %qT will be initialized after",
                     TREE_PURPOSE (next_subobject));
          if (TREE_CODE (subobject) == FIELD_DECL)
-           cp_warning_at ("  `%#D'", subobject);
+           cp_warning_at ("  %q#D", subobject);
          else
-           warning ("  base `%T'", subobject);
-         warning ("  when initialized here");
+           warning ("  base %qT", subobject);
+         warning ("%J  when initialized here", current_function_decl);
        }
 
       /* Look again, from the beginning of the list.  */
@@ -541,10 +547,11 @@ sort_mem_initializers (tree t, tree mem_inits)
       if (TREE_VALUE (subobject_init))
        {
          if (TREE_CODE (subobject) == FIELD_DECL)
-           error ("multiple initializations given for `%D'", subobject);
+           error ("%Jmultiple initializations given for %qD",
+                  current_function_decl, subobject);
          else
-           error ("multiple initializations given for base `%T'", 
-                  subobject);
+           error ("%Jmultiple initializations given for base %qT", 
+                  current_function_decl, subobject);
        }
 
       /* Record the initialization.  */
@@ -610,8 +617,8 @@ sort_mem_initializers (tree t, tree mem_inits)
                  if (same_type_p (last_field_type, field_type))
                    {
                      if (TREE_CODE (field_type) == UNION_TYPE)
-                       error ("initializations for multiple members of `%T'",
-                                 last_field_type);
+                       error ("%Jinitializations for multiple members of %qT",
+                              current_function_decl, last_field_type);
                      done = 1;
                      break;
                    }
@@ -667,9 +674,9 @@ emit_mem_initializers (tree mem_inits)
       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 "
+       warning ("%Jbase class %q#T should be explicitly initialized in the "
                 "copy constructor",
-                BINFO_TYPE (subobject));
+                current_function_decl, BINFO_TYPE (subobject));
 
       /* If an explicit -- but empty -- initializer list was present,
         treat it just like default initialization at this point.  */
@@ -717,15 +724,14 @@ build_vtbl_address (tree binfo)
   tree binfo_for = binfo;
   tree vtbl;
 
-  if (BINFO_VPTR_INDEX (binfo) && BINFO_VIRTUAL_P (binfo)
-      && BINFO_PRIMARY_P (binfo))
+  if (BINFO_VPTR_INDEX (binfo) && BINFO_VIRTUAL_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);
+    while (BINFO_PRIMARY_P (binfo_for))
+      binfo_for = BINFO_INHERITANCE_CHAIN (binfo_for);
 
   /* Figure out what vtable BINFO's vtable is based on, and mark it as
      used.  */
@@ -766,28 +772,28 @@ expand_virtual_init (tree binfo, tree decl)
 
       /* Compute the value to use, when there's a VTT.  */
       vtt_parm = current_vtt_parm;
-      vtbl2 = build (PLUS_EXPR, 
-                    TREE_TYPE (vtt_parm), 
-                    vtt_parm,
-                    vtt_index);
+      vtbl2 = build2 (PLUS_EXPR, 
+                     TREE_TYPE (vtt_parm), 
+                     vtt_parm,
+                     vtt_index);
       vtbl2 = build_indirect_ref (vtbl2, NULL);
       vtbl2 = convert (TREE_TYPE (vtbl), vtbl2);
 
       /* The actual initializer is the VTT value only in the subobject
         constructor.  In maybe_clone_body we'll substitute NULL for
         the vtt_parm in the case of the non-subobject constructor.  */
-      vtbl = build (COND_EXPR, 
-                   TREE_TYPE (vtbl), 
-                   build (EQ_EXPR, boolean_type_node,
-                          current_in_charge_parm, integer_zero_node),
-                   vtbl2, 
-                   vtbl);
+      vtbl = build3 (COND_EXPR, 
+                    TREE_TYPE (vtbl), 
+                    build2 (EQ_EXPR, boolean_type_node,
+                            current_in_charge_parm, integer_zero_node),
+                    vtbl2, 
+                    vtbl);
     }
 
   /* Compute the location of the vtpr.  */
   vtbl_ptr = build_vfield_ref (build_indirect_ref (decl, NULL),
                               TREE_TYPE (binfo));
-  my_friendly_assert (vtbl_ptr != error_mark_node, 20010730);
+  gcc_assert (vtbl_ptr != error_mark_node);
 
   /* Assign the vtable to the vptr.  */
   vtbl = convert_force (TREE_TYPE (vtbl_ptr), vtbl, 0);
@@ -815,9 +821,9 @@ expand_cleanup_for_base (tree binfo, tree flag)
                                    binfo,
                                    LOOKUP_NORMAL | LOOKUP_NONVIRTUAL);
   if (flag)
-    expr = fold (build (COND_EXPR, void_type_node,
-                       c_common_truthvalue_conversion (flag),
-                       expr, integer_zero_node));
+    expr = fold (build3 (COND_EXPR, void_type_node,
+                        c_common_truthvalue_conversion (flag),
+                        expr, integer_zero_node));
 
   finish_eh_cleanup (expr);
 }
@@ -892,26 +898,26 @@ member_init_ok_or_else (tree field, tree type, tree member_name)
     return 0;
   if (!field)
     {
-      error ("class `%T' does not have any field named `%D'", type,
+      error ("class %qT does not have any field named %qD", type,
             member_name);
       return 0;
     }
   if (TREE_CODE (field) == VAR_DECL)
     {
-      error ("`%#D' is a static data member; it can only be "
+      error ("%q#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'",
+      error ("%q#D is not a non-static data member of %qT",
             field, type);
       return 0;
     }
   if (initializing_context (field) != type)
     {
-      error ("class `%T' does not have any field named `%D'", type,
+      error ("class %qT does not have any field named %qD", type,
                member_name);
       return 0;
     }
@@ -944,7 +950,7 @@ expand_member_init (tree name)
       switch (BINFO_N_BASE_BINFOS (TYPE_BINFO (current_class_type)))
        {
        case 0:
-         error ("unnamed initializer for `%T', which has no base classes",
+         error ("unnamed initializer for %qT, which has no base classes",
                 current_class_type);
          return NULL_TREE;
        case 1:
@@ -952,7 +958,7 @@ expand_member_init (tree name)
            (BINFO_BASE_BINFO (TYPE_BINFO (current_class_type), 0));
          break;
        default:
-         error ("unnamed initializer for `%T', which uses multiple inheritance",
+         error ("unnamed initializer for %qT, which uses multiple inheritance",
                 current_class_type);
          return NULL_TREE;
       }
@@ -982,14 +988,10 @@ expand_member_init (tree name)
       virtual_binfo = NULL_TREE;
 
       /* Look for a direct base.  */
-      for (i = 0; i < BINFO_N_BASE_BINFOS (class_binfo); ++i)
-       if (same_type_p
-           (basetype, BINFO_TYPE
-            (BINFO_BASE_BINFO (TYPE_BINFO (current_class_type), i))))
-         {
-           direct_binfo = BINFO_BASE_BINFO (class_binfo, i);
-           break;
-         }
+      for (i = 0; BINFO_BASE_ITERATE (class_binfo, i, direct_binfo); ++i)
+       if (SAME_BINFO_TYPE_P (BINFO_TYPE (direct_binfo), basetype))
+         break;
+
       /* Look for a virtual base -- unless the direct base is itself
         virtual.  */
       if (!direct_binfo || !BINFO_VIRTUAL_P (direct_binfo))
@@ -1002,18 +1004,18 @@ expand_member_init (tree name)
         base class, the mem-initializer is ill-formed.  */
       if (direct_binfo && virtual_binfo)
        {
-         error ("'%D' is both a direct base and an indirect virtual base",
+         error ("%qD is both a direct base and an indirect virtual base",
                 basetype);
          return NULL_TREE;
        }
 
       if (!direct_binfo && !virtual_binfo)
        {
-         if (TYPE_USES_VIRTUAL_BASECLASSES (current_class_type))
-           error ("type `%D' is not a direct or virtual base of `%T'",
+         if (CLASSTYPE_VBASECLASSES (current_class_type))
+           error ("type %qD is not a direct or virtual base of %qT",
                   name, current_class_type);
          else
-           error ("type `%D' is not a direct base of `%T'",
+           error ("type %qD is not a direct base of %qT",
                   name, current_class_type);
          return NULL_TREE;
        }
@@ -1148,7 +1150,7 @@ build_init (tree decl, tree init, int flags)
                                      TREE_TYPE (decl),
                                      LOOKUP_NORMAL|flags);
   else
-    expr = build (INIT_EXPR, TREE_TYPE (decl), decl, init);
+    expr = build2 (INIT_EXPR, TREE_TYPE (decl), decl, init);
 
   return expr;
 }
@@ -1172,8 +1174,7 @@ expand_default_init (tree binfo, tree true_exp, tree exp, tree init, int flags)
       && (flags & LOOKUP_ONLYCONVERTING))
     {
       /* Base subobjects should only get direct-initialization.  */
-      if (true_exp != exp)
-       abort ();
+      gcc_assert (true_exp == exp);
 
       if (flags & DIRECT_BIND)
        /* Do nothing.  We hit this in two cases:  Reference initialization,
@@ -1184,7 +1185,7 @@ expand_default_init (tree binfo, tree true_exp, tree exp, tree init, int flags)
       else if (BRACE_ENCLOSED_INITIALIZER_P (init))
        {
          /* A brace-enclosed initializer for an aggregate.  */
-         my_friendly_assert (CP_AGGREGATE_TYPE_P (type), 20021016);
+         gcc_assert (CP_AGGREGATE_TYPE_P (type));
          init = digest_init (type, init, (tree *)NULL);
        }
       else
@@ -1196,12 +1197,12 @@ expand_default_init (tree binfo, tree true_exp, tree exp, tree init, int flags)
           around the TARGET_EXPR for the copy constructor.  See
           initialize_handler_parm.  */
        {
-         TREE_OPERAND (init, 0) = build (INIT_EXPR, TREE_TYPE (exp), exp,
-                                         TREE_OPERAND (init, 0));
+         TREE_OPERAND (init, 0) = build2 (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);
+       init = build2 (INIT_EXPR, TREE_TYPE (exp), exp, init);
       TREE_SIDE_EFFECTS (init) = 1;
       finish_expr_stmt (init);
       return;
@@ -1252,8 +1253,8 @@ expand_aggr_init_1 (tree binfo, tree true_exp, tree exp, tree init, int 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);
+  gcc_assert (init != error_mark_node && type != error_mark_node);
+  gcc_assert (building_stmt_tree ());
 
   /* Use a function returning the desired type to initialize EXP for us.
      If the function is a constructor, and its first argument is
@@ -1293,7 +1294,7 @@ is_aggr_type (tree type, int or_else)
       && TREE_CODE (type) != BOUND_TEMPLATE_TEMPLATE_PARM)
     {
       if (or_else)
-       error ("`%T' is not an aggregate type", type);
+       error ("%qT is not an aggregate type", type);
       return 0;
     }
   return 1;
@@ -1334,7 +1335,7 @@ build_offset_ref (tree type, tree name, bool address_p)
   if (TREE_CODE (name) == TEMPLATE_DECL)
     return name;
 
-  if (processing_template_decl || uses_template_parms (type))
+  if (dependent_type_p (type) || type_dependent_expression_p (name))
     return build_min_nt (SCOPE_REF, type, name);
 
   if (TREE_CODE (name) == TEMPLATE_ID_EXPR)
@@ -1355,7 +1356,7 @@ build_offset_ref (tree type, tree name, bool address_p)
            name = DECL_NAME (OVL_CURRENT (name));
        }
 
-      my_friendly_assert (TREE_CODE (name) == IDENTIFIER_NODE, 0);
+      gcc_assert (TREE_CODE (name) == IDENTIFIER_NODE);
     }
 
   if (type == NULL_TREE)
@@ -1369,8 +1370,8 @@ build_offset_ref (tree type, tree name, bool address_p)
         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));
+       t = build2 (TEMPLATE_ID_EXPR, TREE_TYPE (t),
+                   t, TREE_OPERAND (orig_name, 1));
       if (! type_unknown_p (t))
        {
          mark_used (t);
@@ -1385,7 +1386,7 @@ build_offset_ref (tree type, tree name, bool address_p)
   if (TREE_CODE (name) == BIT_NOT_EXPR)
     {
       if (! check_dtor_name (type, name))
-       error ("qualified type `%T' does not match destructor name `~%T'",
+       error ("qualified type %qT does not match destructor name %<~%T%>",
                  type, TREE_OPERAND (name, 0));
       name = dtor_identifier;
     }
@@ -1393,11 +1394,11 @@ build_offset_ref (tree type, tree name, bool address_p)
   if (!COMPLETE_TYPE_P (complete_type (type))
       && !TYPE_BEING_DEFINED (type))
     {
-      error ("incomplete type `%T' does not have member `%D'", type,
-               name);
+      error ("incomplete type %qT does not have member %qD", type, name);
       return error_mark_node;
     }
 
+  /* Set up BASEBINFO for member lookup.  */
   decl = maybe_dummy_object (type, &basebinfo);
 
   if (BASELINK_P (name) || DECL_P (name))
@@ -1412,10 +1413,18 @@ build_offset_ref (tree type, tree name, bool address_p)
 
   if (!member)
     {
-      error ("`%D' is not a member of type `%T'", name, type);
+      error ("%qD is not a member of type %qT", name, type);
       return error_mark_node;
     }
 
+  if (processing_template_decl)
+    {
+      if (TREE_CODE (orig_name) == TEMPLATE_ID_EXPR)
+       return build_min (SCOPE_REF, TREE_TYPE (member), type, orig_name);
+      else
+       return build_min (SCOPE_REF, TREE_TYPE (member), type, name);
+    }
+
   if (TREE_CODE (member) == TYPE_DECL)
     {
       TREE_USED (member) = 1;
@@ -1431,7 +1440,7 @@ build_offset_ref (tree type, tree name, bool address_p)
 
   if (TREE_CODE (member) == FIELD_DECL && DECL_C_BIT_FIELD (member))
     {
-      error ("invalid pointer to bit-field `%D'", member);
+      error ("invalid pointer to bit-field %qD", member);
       return error_mark_node;
     }
 
@@ -1456,9 +1465,9 @@ build_offset_ref (tree type, tree name, bool address_p)
               expects to encounter OVERLOADs, not raw functions.  */
            t = ovl_cons (t, NULL_TREE);
 
-          t = build (TEMPLATE_ID_EXPR, TREE_TYPE (t), t,
-                    TREE_OPERAND (orig_name, 1));
-         t = build (OFFSET_REF, unknown_type_node, decl, t);
+          t = build2 (TEMPLATE_ID_EXPR, TREE_TYPE (t), t,
+                     TREE_OPERAND (orig_name, 1));
+         t = build2 (OFFSET_REF, unknown_type_node, decl, t);
           
           PTRMEM_OK_P (t) = 1;
                  
@@ -1525,7 +1534,7 @@ build_offset_ref (tree type, tree name, bool address_p)
          /* Build a representation of a the qualified name suitable
             for use as the operand to "&" -- even though the "&" is
             not actually present.  */
-         member = build (OFFSET_REF, TREE_TYPE (member), decl, member);
+         member = build2 (OFFSET_REF, TREE_TYPE (member), decl, member);
          /* In Microsoft mode, treat a non-static member function as if
             it were a pointer-to-member.  */
          if (flag_ms_extensions)
@@ -1533,13 +1542,13 @@ build_offset_ref (tree type, tree name, bool address_p)
              PTRMEM_OK_P (member) = 1;
              return build_unary_op (ADDR_EXPR, member, 0);
            }
-         error ("invalid use of non-static member function `%D'", 
+         error ("invalid use of non-static member function %qD", 
                 TREE_OPERAND (member, 1));
          return member;
        }
       else if (TREE_CODE (member) == FIELD_DECL)
        {
-         error ("invalid use of non-static data member `%D'", member);
+         error ("invalid use of non-static data member %qD", member);
          return error_mark_node;
        }
       return member;
@@ -1548,51 +1557,53 @@ build_offset_ref (tree type, tree name, bool address_p)
   /* In member functions, the form `type::name' is no longer
      equivalent to `this->type::name', at least not until
      resolve_offset_ref.  */
-  member = build (OFFSET_REF, TREE_TYPE (member), decl, member);
+  member = build2 (OFFSET_REF, TREE_TYPE (member), decl, member);
   PTRMEM_OK_P (member) = 1;
   return member;
 }
 
-/* If DECL is a `const' declaration, and its value is a known
-   constant, then return that value.  */
+/* If DECL is a CONST_DECL, or a constant VAR_DECL initialized by
+   constant of integral or enumeration type, then return that value.
+   These are those variables permitted in constant expressions by
+   [5.19/1].  FIXME:If we did lazy folding, this could be localized.  */
 
 tree
-decl_constant_value (tree decl)
+integral_constant_value (tree decl)
 {
-  /* When we build a COND_EXPR, we don't know whether it will be used
-     as an lvalue or as an rvalue.  If it is an lvalue, it's not safe
-     to replace the second and third operands with their
-     initializers.  So, we do that here.  */
-  if (TREE_CODE (decl) == COND_EXPR)
-    {
-      tree d1;
-      tree d2;
-
-      d1 = decl_constant_value (TREE_OPERAND (decl, 1));
-      d2 = decl_constant_value (TREE_OPERAND (decl, 2));
+  while ((TREE_CODE (decl) == CONST_DECL
+         || (TREE_CODE (decl) == VAR_DECL
+             /* And so are variables with a 'const' type -- unless they
+                are also 'volatile'.  */
+             && CP_TYPE_CONST_NON_VOLATILE_P (TREE_TYPE (decl))
+             && DECL_INITIALIZED_BY_CONSTANT_EXPRESSION_P (decl)))
+        && DECL_INITIAL (decl)
+        && DECL_INITIAL (decl) != error_mark_node
+        && TREE_TYPE (DECL_INITIAL (decl))
+        && INTEGRAL_OR_ENUMERATION_TYPE_P (TREE_TYPE (decl)))
+    decl = DECL_INITIAL (decl);
+  return decl;
+}
 
-      if (d1 != TREE_OPERAND (decl, 1) || d2 != TREE_OPERAND (decl, 2))
-       return build (COND_EXPR,
-                     TREE_TYPE (decl),
-                     TREE_OPERAND (decl, 0), d1, d2);
-    }
+/* A more relaxed version of integral_constant_value, for which type
+   is not considered.  This is used by the common C/C++ code, and not
+   directly by the C++ front end.  */
 
-  if (DECL_P (decl)
-      && (/* Enumeration constants are constant.  */
-         TREE_CODE (decl) == CONST_DECL
+tree
+decl_constant_value (tree decl)
+{
+  if ((TREE_CODE (decl) == CONST_DECL
+      || (TREE_CODE (decl) == VAR_DECL
          /* And so are variables with a 'const' type -- unless they
-            are also 'volatile'.  */
-         || CP_TYPE_CONST_NON_VOLATILE_P (TREE_TYPE (decl)))
-      && TREE_CODE (decl) != PARM_DECL
+             are also 'volatile'.  */
+         && CP_TYPE_CONST_NON_VOLATILE_P (TREE_TYPE (decl))))
       && DECL_INITIAL (decl)
       && DECL_INITIAL (decl) != error_mark_node
-      /* This is invalid if initial value is not constant.
-        If it has either a function call, a memory reference,
-        or a variable, then re-evaluating it could give different results.  */
-      && TREE_CONSTANT (DECL_INITIAL (decl))
-      /* Check for cases where this is sub-optimal, even though valid.  */
-      && TREE_CODE (DECL_INITIAL (decl)) != CONSTRUCTOR)
+      /* This is invalid if initial value is not constant.  If it has
+                either a function call, a memory reference, or a variable,
+                then re-evaluating it could give different results.  */
+      && TREE_CONSTANT (DECL_INITIAL (decl)))
     return DECL_INITIAL (decl);
+  
   return decl;
 }
 \f
@@ -1607,31 +1618,14 @@ build_builtin_delete_call (tree addr)
   return build_call (global_delete_fndecl, build_tree_list (NULL_TREE, addr));
 }
 \f
-/* Generate a C++ "new" expression. DECL is either a TREE_LIST
-   (which needs to go through some sort of groktypename) or it
-   is the name of the class we are newing. INIT is an initialization value.
-   It is either an EXPRLIST, an EXPR_NO_COMMAS, or something in braces.
-   If INIT is void_type_node, it means do *not* call a constructor
-   for this instance.
-
-   For types with constructors, the data returned is initialized
-   by the appropriate constructor.
-
-   Whether the type has a constructor or not, if it has a pointer
-   to a virtual function table, then that pointer is set up
-   here.
-
-   Unless I am mistaken, a call to new () will return initialized
-   data regardless of whether the constructor itself is private or
-   not.  NOPE; new fails if the constructor is private (jcm).
-
-   Note that build_new does nothing to assure that any special
-   alignment requirements of the type are met.  Rather, it leaves
-   it up to malloc to do the right thing.  Otherwise, folding to
-   the right alignment cal cause problems if the user tries to later
-   free the memory returned by `new'.
-
-   PLACEMENT is the `placement' list for user-defined operator new ().  */
+/* Generate a representation for a C++ "new" expression.  PLACEMENT is
+   a TREE_LIST of placement-new arguments (or NULL_TREE if none).  If
+   NELTS is NULL, TYPE is the type of the storage to be allocated.  If
+   NELTS is not NULL, then this is an array-new allocation; TYPE is
+   the type of the elements in the array and NELTS is the number of
+   elements in the array.  INIT, if non-NULL, is the initializer for
+   the new object.  If USE_GLOBAL_NEW is true, then the user
+   explicitly wrote "::new" rather than just "new".  */
 
 tree
 build_new (tree placement, tree type, tree nelts, tree init, 
@@ -1651,6 +1645,15 @@ build_new (tree placement, tree type, tree nelts, tree init,
       return rval;
     }
 
+  if (nelts)
+    {
+      if (!build_expr_type_conversion (WANT_INT | WANT_ENUM, nelts, false))
+       pedwarn ("size in array new must have integral type");
+      nelts = save_expr (cp_convert (sizetype, nelts));
+      if (nelts == integer_zero_node)
+       warning ("zero size array reserves no space");
+    }
+
   /* ``A reference cannot be created by the new operator.  A reference
      is not an object (8.2.2, 8.4.3), so a pointer to it could not be
      returned by new.'' ARM 5.3.3 */
@@ -1666,8 +1669,8 @@ build_new (tree placement, tree type, tree nelts, tree init,
       return error_mark_node;
     }
 
-  rval = build (NEW_EXPR, build_pointer_type (type), placement, type,
-               nelts, init);
+  rval = build4 (NEW_EXPR, build_pointer_type (type), placement, type,
+                nelts, init);
   NEW_EXPR_USE_GLOBAL (rval) = use_global_new;
   TREE_SIDE_EFFECTS (rval) = 1;
   rval = build_new_1 (rval);
@@ -1694,7 +1697,7 @@ build_java_class_ref (tree type)
     {
       jclass_node = IDENTIFIER_GLOBAL_VALUE (get_identifier ("jclass"));
       if (jclass_node == NULL_TREE)
-       fatal_error ("call to Java constructor, while `jclass' undefined");
+       fatal_error ("call to Java constructor, while %<jclass%> undefined");
 
       jclass_node = TREE_TYPE (jclass_node);
     }
@@ -1723,7 +1726,7 @@ build_java_class_ref (tree type)
       DECL_ARTIFICIAL (class_decl) = 1;
       DECL_IGNORED_P (class_decl) = 1;
       pushdecl_top_level (class_decl);
-      make_decl_rtl (class_decl, NULL);
+      make_decl_rtl (class_decl);
     }
   return class_decl;
 }
@@ -1736,15 +1739,29 @@ static tree
 build_new_1 (tree exp)
 {
   tree placement, init;
-  tree true_type, size, rval;
+  tree size, rval;
+  /* True iff this is a call to "operator new[]" instead of just
+     "operator new".  */   
+  bool array_p = false;
+  /* True iff ARRAY_P is true and the bound of the array type is
+     not necessarily a compile time constant.  For example, VLA_P is
+     true for "new int[f()]".  */
+  bool vla_p = false;
+  /* The type being allocated.  If ARRAY_P is true, this will be an 
+     ARRAY_TYPE.  */
+  tree full_type;
+  /* If ARRAY_P is true, the element type of the array.  This is an
+     never ARRAY_TYPE; for something like "new int[3][4]", the
+     ELT_TYPE is "int".  If ARRAY_P is false, this is the same type as
+     FULL_TYPE.  */
+  tree elt_type;
   /* The type of the new-expression.  (This type is always a pointer
      type.)  */
   tree pointer_type;
-  /* The type pointed to by POINTER_TYPE.  */
+  /* The type pointed to by POINTER_TYPE.  This type may be different
+     from ELT_TYPE for a multi-dimensional array; ELT_TYPE is never an
+     ARRAY_TYPE, but TYPE may be an ARRAY_TYPE.  */
   tree type;
-  /* The type being allocated.  For "new T[...]" this will be an
-     ARRAY_TYPE.  */
-  tree full_type;
   /* A pointer type pointing to to the FULL_TYPE.  */
   tree full_pointer_type;
   tree outer_nelts = NULL_TREE;
@@ -1755,8 +1772,6 @@ build_new_1 (tree exp)
   tree alloc_node;
   tree alloc_fn;
   tree cookie_expr, init_expr;
-  int has_array = 0;
-  enum tree_code code;
   int nothrow, check_new;
   /* Nonzero if the user wrote `::new' rather than just `new'.  */
   int globally_qualified_p;
@@ -1789,76 +1804,80 @@ build_new_1 (tree exp)
     {
       tree index;
 
-      has_array = 1;
       outer_nelts = nelts;
+      array_p = true;
 
       /* ??? The middle-end will error on us for building a VLA outside a 
         function context.  Methinks that's not it's purvey.  So we'll do
         our own VLA layout later.  */
-
+      vla_p = true;
       full_type = build_cplus_array_type (type, NULL_TREE);
-
       index = convert (sizetype, nelts);
       index = size_binop (MINUS_EXPR, index, size_one_node);
       TYPE_DOMAIN (full_type) = build_index_type (index);
     }
   else
-    full_type = type;
-
-  true_type = type;
-
-  code = has_array ? VEC_NEW_EXPR : NEW_EXPR;
+    {
+      full_type = type;
+      if (TREE_CODE (type) == ARRAY_TYPE)
+       {
+         array_p = true;
+         nelts = array_type_nelts_top (type);
+         outer_nelts = nelts;
+         type = TREE_TYPE (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 = cp_build_binary_op (MULT_EXPR, nelts, this_nelts);
-      true_type = TREE_TYPE (true_type);
-    }
+  for (elt_type = type;
+       TREE_CODE (elt_type) == ARRAY_TYPE;
+       elt_type = TREE_TYPE (elt_type))
+    nelts = cp_build_binary_op (MULT_EXPR, nelts, 
+                               array_type_nelts_top (elt_type));
 
-  if (!complete_type_or_else (true_type, exp))
+  if (!complete_type_or_else (elt_type, exp))
     return error_mark_node;
 
-  if (TREE_CODE (true_type) == VOID_TYPE)
+  if (TREE_CODE (elt_type) == VOID_TYPE)
     {
-      error ("invalid type `void' for new");
+      error ("invalid type %<void%> for new");
       return error_mark_node;
     }
 
-  if (abstract_virtuals_error (NULL_TREE, true_type))
+  if (abstract_virtuals_error (NULL_TREE, elt_type))
     return error_mark_node;
 
-  is_initialized = (TYPE_NEEDS_CONSTRUCTING (type) || init);
-  if (CP_TYPE_CONST_P (true_type) && !is_initialized)
+  is_initialized = (TYPE_NEEDS_CONSTRUCTING (elt_type) || init);
+  if (CP_TYPE_CONST_P (elt_type) && !is_initialized)
     {
-      error ("uninitialized const in `new' of `%#T'", true_type);
+      error ("uninitialized const in %<new%> of %q#T", elt_type);
       return error_mark_node;
     }
 
-  size = size_in_bytes (true_type);
-  if (has_array)
+  size = size_in_bytes (elt_type);
+  if (array_p)
     {
-      tree n, bitsize;
-
-      /* Do our own VLA layout.  Setting TYPE_SIZE/_UNIT is necessary in
-        order for the <INIT_EXPR <*foo> <CONSTRUCTOR ...>> to be valid.  */
-
-      n = convert (sizetype, nelts);
-      size = size_binop (MULT_EXPR, size, n);
-      TYPE_SIZE_UNIT (full_type) = size;
-
-      n = convert (bitsizetype, nelts);
-      bitsize = size_binop (MULT_EXPR, TYPE_SIZE (true_type), n);
-      TYPE_SIZE (full_type) = bitsize;
+      size = size_binop (MULT_EXPR, size, convert (sizetype, nelts));
+      if (vla_p)
+       {
+         tree n, bitsize;
+
+         /* Do our own VLA layout.  Setting TYPE_SIZE/_UNIT is
+            necessary in order for the <INIT_EXPR <*foo> <CONSTRUCTOR
+            ...>> to be valid.  */
+         TYPE_SIZE_UNIT (full_type) = size;
+         n = convert (bitsizetype, nelts);
+         bitsize = size_binop (MULT_EXPR, TYPE_SIZE (elt_type), n);
+         TYPE_SIZE (full_type) = bitsize;
+       }
     }
 
   /* Allocate the object.  */
-  if (! placement && TYPE_FOR_JAVA (true_type))
+  if (! placement && TYPE_FOR_JAVA (elt_type))
     {
       tree class_addr, alloc_decl;
-      tree class_decl = build_java_class_ref (true_type);
+      tree class_decl = build_java_class_ref (elt_type);
       static const char alloc_name[] = "_Jv_AllocObject";
 
       use_java_new = 1;
@@ -1866,12 +1885,12 @@ build_new_1 (tree exp)
       if (!get_global_value_if_present (get_identifier (alloc_name), 
                                        &alloc_decl))
        {
-         error ("call to Java constructor with `%s' undefined", alloc_name);
+         error ("call to Java constructor with %qs undefined", alloc_name);
          return error_mark_node;
        }
       else if (really_overloaded_fn (alloc_decl))
        {
-         error ("`%D' should never be overloaded", alloc_decl);
+         error ("%qD should never be overloaded", alloc_decl);
          return error_mark_node;
        }
       alloc_decl = OVL_CURRENT (alloc_decl);
@@ -1885,32 +1904,32 @@ build_new_1 (tree exp)
       tree fnname;
       tree fns;
 
-      fnname = ansi_opname (code);
+      fnname = ansi_opname (array_p ? VEC_NEW_EXPR : NEW_EXPR);
 
       if (!globally_qualified_p 
-         && CLASS_TYPE_P (true_type)
-         && (has_array
-             ? TYPE_HAS_ARRAY_NEW_OPERATOR (true_type)
-             : TYPE_HAS_NEW_OPERATOR (true_type)))
+         && CLASS_TYPE_P (elt_type)
+         && (array_p
+             ? TYPE_HAS_ARRAY_NEW_OPERATOR (elt_type)
+             : TYPE_HAS_NEW_OPERATOR (elt_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))
+         if (array_p && TYPE_VEC_NEW_USES_COOKIE (elt_type))
            {
-             cookie_size = targetm.cxx.get_cookie_size (true_type);
+             cookie_size = targetm.cxx.get_cookie_size (elt_type);
              size = size_binop (PLUS_EXPR, size, cookie_size);
            }
          /* Create the argument list.  */
          args = tree_cons (NULL_TREE, size, placement);
          /* Do name-lookup to find the appropriate operator.  */
-         fns = lookup_fnfields (true_type, fnname, /*protect=*/2);
+         fns = lookup_fnfields (elt_type, fnname, /*protect=*/2);
          if (TREE_CODE (fns) == TREE_LIST)
            {
-             error ("request for member `%D' is ambiguous", fnname);
+             error ("request for member %qD is ambiguous", fnname);
              print_candidates (fns);
              return error_mark_node;
            }
-         alloc_call = build_new_method_call (build_dummy_object (true_type),
+         alloc_call = build_new_method_call (build_dummy_object (elt_type),
                                              fns, args,
                                              /*conversion_path=*/NULL_TREE,
                                              LOOKUP_NORMAL);
@@ -1919,8 +1938,8 @@ build_new_1 (tree exp)
        {
          /* Use a global operator new.  */
          /* See if a cookie might be required.  */
-         if (has_array && TYPE_VEC_NEW_USES_COOKIE (true_type))
-           cookie_size = targetm.cxx.get_cookie_size (true_type);
+         if (array_p && TYPE_VEC_NEW_USES_COOKIE (elt_type))
+           cookie_size = targetm.cxx.get_cookie_size (elt_type);
          else
            cookie_size = NULL_TREE;
 
@@ -1948,7 +1967,7 @@ build_new_1 (tree exp)
   while (TREE_CODE (alloc_call) == COMPOUND_EXPR) 
     alloc_call = TREE_OPERAND (alloc_call, 1);
   alloc_fn = get_callee_fndecl (alloc_call);
-  my_friendly_assert (alloc_fn != NULL_TREE, 20020325);
+  gcc_assert (alloc_fn != NULL_TREE);
 
   /* Now, check to see if this function is actually a placement
      allocation function.  This can happen even when PLACEMENT is NULL
@@ -1971,8 +1990,8 @@ build_new_1 (tree exp)
       tree inits;
       stabilize_call (alloc_call, &inits);
       if (inits)
-       alloc_expr = build (COMPOUND_EXPR, TREE_TYPE (alloc_expr), inits,
-                           alloc_expr);
+       alloc_expr = build2 (COMPOUND_EXPR, TREE_TYPE (alloc_expr), inits,
+                            alloc_expr);
     }
 
   /*        unless an allocation function is declared with an empty  excep-
@@ -1994,28 +2013,28 @@ build_new_1 (tree exp)
       tree cookie_ptr;
 
       /* Adjust so we're pointing to the start of the object.  */
-      data_addr = get_target_expr (build (PLUS_EXPR, full_pointer_type,
-                                         alloc_node, cookie_size));
+      data_addr = get_target_expr (build2 (PLUS_EXPR, full_pointer_type,
+                                          alloc_node, cookie_size));
 
       /* Store the number of bytes allocated so that we can know how
         many elements to destroy later.  We use the last sizeof
         (size_t) bytes to store the number of elements.  */
-      cookie_ptr = build (MINUS_EXPR, build_pointer_type (sizetype),
-                     data_addr, size_in_bytes (sizetype));
+      cookie_ptr = build2 (MINUS_EXPR, build_pointer_type (sizetype),
+                          data_addr, size_in_bytes (sizetype));
       cookie = build_indirect_ref (cookie_ptr, NULL);
 
-      cookie_expr = build (MODIFY_EXPR, sizetype, cookie, nelts);
+      cookie_expr = build2 (MODIFY_EXPR, sizetype, cookie, nelts);
 
       if (targetm.cxx.cookie_has_size ())
        {
          /* Also store the element size.  */
-         cookie_ptr = build (MINUS_EXPR, build_pointer_type (sizetype),
-                             cookie_ptr, size_in_bytes (sizetype));
+         cookie_ptr = build2 (MINUS_EXPR, build_pointer_type (sizetype),
+                              cookie_ptr, size_in_bytes (sizetype));
          cookie = build_indirect_ref (cookie_ptr, NULL);
-         cookie = build (MODIFY_EXPR, sizetype, cookie,
-                         size_in_bytes(true_type));
-         cookie_expr = build (COMPOUND_EXPR, TREE_TYPE (cookie_expr),
-                              cookie, cookie_expr);
+         cookie = build2 (MODIFY_EXPR, sizetype, cookie,
+                          size_in_bytes(elt_type));
+         cookie_expr = build2 (COMPOUND_EXPR, TREE_TYPE (cookie_expr),
+                               cookie, cookie_expr);
        }
       data_addr = TARGET_EXPR_SLOT (data_addr);
     }
@@ -2038,10 +2057,10 @@ build_new_1 (tree exp)
 
       if (init == void_zero_node)
        init = build_default_init (full_type, nelts);
-      else if (init && has_array)
+      else if (init && array_p)
        pedwarn ("ISO C++ forbids initialization in array new");
 
-      if (has_array)
+      if (array_p)
        {
          init_expr
            = build_vec_init (init_expr,
@@ -2058,7 +2077,7 @@ build_new_1 (tree exp)
        {
          init_expr = build_special_member_call (init_expr, 
                                                 complete_ctor_identifier,
-                                                init, true_type,
+                                                init, elt_type,
                                                 LOOKUP_NORMAL);
          stable = stabilize_init (init_expr, &init_preeval_expr);
        }
@@ -2070,9 +2089,9 @@ build_new_1 (tree exp)
          if (TREE_CODE (init) == TREE_LIST)
            init = build_x_compound_expr_from_list (init, "new initializer");
 
-         else if (TREE_CODE (init) == CONSTRUCTOR
-                  && TREE_TYPE (init) == NULL_TREE)
-           abort ();
+         else
+           gcc_assert (TREE_CODE (init) != CONSTRUCTOR
+                       || TREE_TYPE (init) != NULL_TREE);
 
          init_expr = build_modify_expr (init_expr, INIT_EXPR, init);
          stable = stabilize_init (init_expr, &init_preeval_expr);
@@ -2091,7 +2110,7 @@ build_new_1 (tree exp)
         freed.  */
       if (flag_exceptions && ! use_java_new)
        {
-         enum tree_code dcode = has_array ? VEC_DELETE_EXPR : DELETE_EXPR;
+         enum tree_code dcode = array_p ? VEC_DELETE_EXPR : DELETE_EXPR;
          tree cleanup;
 
          /* The Standard is unclear here, but the right thing to do
@@ -2107,8 +2126,8 @@ build_new_1 (tree exp)
          else if (stable)
            /* This is much simpler if we were able to preevaluate all of
               the arguments to the constructor call.  */
-           init_expr = build (TRY_CATCH_EXPR, void_type_node,
-                              init_expr, cleanup);
+           init_expr = build2 (TRY_CATCH_EXPR, void_type_node,
+                               init_expr, cleanup);
          else
            /* Ack!  First we allocate the memory.  Then we set our sentry
               variable to true, and expand a cleanup that deletes the
@@ -2129,16 +2148,16 @@ build_new_1 (tree exp)
              sentry = TARGET_EXPR_SLOT (begin);
 
              TARGET_EXPR_CLEANUP (begin)
-               = build (COND_EXPR, void_type_node, sentry,
-                        cleanup, void_zero_node);
+               = build3 (COND_EXPR, void_type_node, sentry,
+                         cleanup, void_zero_node);
 
-             end = build (MODIFY_EXPR, TREE_TYPE (sentry),
-                          sentry, boolean_false_node);
+             end = build2 (MODIFY_EXPR, TREE_TYPE (sentry),
+                           sentry, boolean_false_node);
 
              init_expr
-               = build (COMPOUND_EXPR, void_type_node, begin,
-                        build (COMPOUND_EXPR, void_type_node, init_expr,
-                               end));
+               = build2 (COMPOUND_EXPR, void_type_node, begin,
+                         build2 (COMPOUND_EXPR, void_type_node, init_expr,
+                                 end));
            }
            
        }
@@ -2151,9 +2170,9 @@ build_new_1 (tree exp)
   rval = data_addr;
 
   if (init_expr)
-    rval = build (COMPOUND_EXPR, TREE_TYPE (rval), init_expr, rval);
+    rval = build2 (COMPOUND_EXPR, TREE_TYPE (rval), init_expr, rval);
   if (cookie_expr)
-    rval = build (COMPOUND_EXPR, TREE_TYPE (rval), cookie_expr, rval);
+    rval = build2 (COMPOUND_EXPR, TREE_TYPE (rval), cookie_expr, rval);
 
   if (rval == alloc_node)
     /* If we don't have an initializer or a cookie, strip the TARGET_EXPR
@@ -2170,11 +2189,11 @@ build_new_1 (tree exp)
 
       /* Perform the allocation before anything else, so that ALLOC_NODE
         has been initialized before we start using it.  */
-      rval = build (COMPOUND_EXPR, TREE_TYPE (rval), alloc_expr, rval);
+      rval = build2 (COMPOUND_EXPR, TREE_TYPE (rval), alloc_expr, rval);
     }
 
   if (init_preeval_expr)
-    rval = build (COMPOUND_EXPR, TREE_TYPE (rval), init_preeval_expr, rval);
+    rval = build2 (COMPOUND_EXPR, TREE_TYPE (rval), init_preeval_expr, rval);
 
   /* Convert to the final type.  */
   rval = build_nop (pointer_type, rval);
@@ -2214,8 +2233,7 @@ build_vec_delete_1 (tree base, tree maxindex, tree type,
   tree controller = NULL_TREE;
 
   /* We should only have 1-D arrays here.  */
-  if (TREE_CODE (type) == ARRAY_TYPE)
-    abort ();
+  gcc_assert (TREE_CODE (type) != ARRAY_TYPE);
 
   if (! IS_AGGR_TYPE (type) || TYPE_HAS_TRIVIAL_DESTRUCTOR (type))
     goto no_destructor;
@@ -2226,23 +2244,24 @@ build_vec_delete_1 (tree base, tree maxindex, tree type,
 
   tbase = create_temporary_var (ptype);
   tbase_init = build_modify_expr (tbase, NOP_EXPR,
-                                 fold (build (PLUS_EXPR, ptype,
-                                              base,
-                                              virtual_size)));
+                                 fold (build2 (PLUS_EXPR, ptype,
+                                               base,
+                                               virtual_size)));
   DECL_REGISTER (tbase) = 1;
-  controller = build (BIND_EXPR, void_type_node, tbase, NULL_TREE, NULL_TREE);
+  controller = build3 (BIND_EXPR, void_type_node, tbase,
+                      NULL_TREE, NULL_TREE);
   TREE_SIDE_EFFECTS (controller) = 1;
 
-  body = build (EXIT_EXPR, void_type_node,
-               build (EQ_EXPR, boolean_type_node, base, tbase));
+  body = build1 (EXIT_EXPR, void_type_node,
+                build2 (EQ_EXPR, boolean_type_node, base, tbase));
   body = build_compound_expr
     (body, build_modify_expr (tbase, NOP_EXPR,
-                             build (MINUS_EXPR, ptype, tbase, size_exp)));
+                             build2 (MINUS_EXPR, ptype, tbase, size_exp)));
   body = build_compound_expr
     (body, build_delete (ptype, tbase, sfk_complete_destructor,
                         LOOKUP_NORMAL|LOOKUP_DESTRUCTOR, 1));
 
-  loop = build (LOOP_EXPR, void_type_node, body);
+  loop = build1 (LOOP_EXPR, void_type_node, body);
   loop = build_compound_expr (tbase_init, loop);
 
  no_destructor:
@@ -2292,11 +2311,11 @@ build_vec_delete_1 (tree base, tree maxindex, tree type,
     body = integer_zero_node;
   
   /* Outermost wrapper: If pointer is null, punt.  */
-  body = fold (build (COND_EXPR, void_type_node,
-                     fold (build (NE_EXPR, boolean_type_node, base,
-                                  convert (TREE_TYPE (base),
-                                           integer_zero_node))),
-                     body, integer_zero_node));
+  body = fold (build3 (COND_EXPR, void_type_node,
+                      fold (build2 (NE_EXPR, boolean_type_node, base,
+                                    convert (TREE_TYPE (base),
+                                             integer_zero_node))),
+                      body, integer_zero_node));
   body = build1 (NOP_EXPR, void_type_node, body);
 
   if (controller)
@@ -2307,7 +2326,7 @@ build_vec_delete_1 (tree base, tree maxindex, tree type,
 
   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);
+    body = build2 (COMPOUND_EXPR, void_type_node, base, body);
 
   return convert_to_void (body, /*implicit=*/NULL);
 }
@@ -2322,8 +2341,8 @@ create_temporary_var (tree type)
   decl = build_decl (VAR_DECL, NULL_TREE, type);
   TREE_USED (decl) = 1;
   DECL_ARTIFICIAL (decl) = 1;
-  DECL_SOURCE_LOCATION (decl) = input_location;
   DECL_IGNORED_P (decl) = 1;
+  DECL_SOURCE_LOCATION (decl) = input_location;
   DECL_CONTEXT (decl) = current_function_decl;
 
   return decl;
@@ -2407,7 +2426,7 @@ build_vec_init (tree base, tree maxindex, tree init, int from_array)
         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);
+      stmt_expr = build2 (INIT_EXPR, atype, base, init);
       return stmt_expr;
     }
 
@@ -2533,8 +2552,8 @@ build_vec_init (tree base, tree maxindex, tree init, int from_array)
 
       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),
+      finish_for_cond (build2 (NE_EXPR, boolean_type_node,
+                              iterator, integer_minus_one_node),
                       for_stmt);
       finish_for_expr (build_unary_op (PREDECREMENT_EXPR, iterator, 0),
                       for_stmt);
@@ -2556,7 +2575,7 @@ build_vec_init (tree base, tree maxindex, tree init, int from_array)
          else if (from)
            elt_init = build_modify_expr (to, NOP_EXPR, from);
          else
-           abort ();
+           gcc_unreachable ();
        }
       else if (TREE_CODE (type) == ARRAY_TYPE)
        {
@@ -2666,10 +2685,8 @@ build_dtor_call (tree exp, special_function_kind dtor_kind, int flags)
       break;
 
     default:
-      abort ();
+      gcc_unreachable ();
     }
-
-  exp = convert_from_reference (exp);
   fn = lookup_fnfields (TREE_TYPE (exp), name, /*protect=*/2);
   return build_new_method_call (exp, fn, 
                                /*args=*/NULL_TREE,
@@ -2763,7 +2780,7 @@ build_delete (tree type, tree addr, special_function_kind auto_delete,
       addr = convert_force (build_pointer_type (type), addr, 0);
     }
 
-  my_friendly_assert (IS_AGGR_TYPE (type), 220);
+  gcc_assert (IS_AGGR_TYPE (type));
 
   if (TYPE_HAS_TRIVIAL_DESTRUCTOR (type))
     {
@@ -2779,7 +2796,8 @@ build_delete (tree type, tree addr, special_function_kind auto_delete,
       tree do_delete = NULL_TREE;
       tree ifexp;
 
-      my_friendly_assert (TYPE_HAS_DESTRUCTOR (type), 20011213);
+      if (CLASSTYPE_LAZY_DESTRUCTOR (type))
+       lazily_declare_fn (sfk_destructor, type);
 
       /* For `::delete x', we must not use the deleting destructor
         since then we would not be sure to get the global `operator
@@ -2823,7 +2841,7 @@ build_delete (tree type, tree addr, special_function_kind auto_delete,
       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);
+       expr = build2 (COMPOUND_EXPR, void_type_node, expr, do_delete);
 
       if (flags & LOOKUP_DESTRUCTOR)
        /* Explicit destructor call; don't check for null pointer.  */
@@ -2833,8 +2851,8 @@ build_delete (tree type, tree addr, special_function_kind auto_delete,
        ifexp = fold (cp_build_binary_op (NE_EXPR, addr, integer_zero_node));
 
       if (ifexp != integer_one_node)
-       expr = build (COND_EXPR, void_type_node,
-                     ifexp, expr, void_zero_node);
+       expr = build3 (COND_EXPR, void_type_node,
+                      ifexp, expr, void_zero_node);
 
       return expr;
     }
@@ -2848,47 +2866,44 @@ build_delete (tree type, tree addr, special_function_kind auto_delete,
 void
 push_base_cleanups (void)
 {
-  tree binfos;
-  int i, n_baseclasses;
+  tree binfo, base_binfo;
+  int i;
   tree member;
   tree expr;
+  VEC (tree) *vbases;
 
   /* Run destructors for all virtual baseclasses.  */
-  if (TYPE_USES_VIRTUAL_BASECLASSES (current_class_type))
+  if (CLASSTYPE_VBASECLASSES (current_class_type))
     {
       tree cond = (condition_conversion
-                  (build (BIT_AND_EXPR, integer_type_node,
-                          current_in_charge_parm,
-                          integer_two_node)));
+                  (build2 (BIT_AND_EXPR, integer_type_node,
+                           current_in_charge_parm,
+                           integer_two_node)));
 
       /* The CLASSTYPE_VBASECLASSES vector is in initialization
         order, which is also the right order for pushing cleanups.  */
-      for (i = 0; (binfos = VEC_iterate
-                  (tree, CLASSTYPE_VBASECLASSES (current_class_type), i));
-          i++)
+      for (vbases = CLASSTYPE_VBASECLASSES (current_class_type), i = 0;
+          VEC_iterate (tree, vbases, i, base_binfo); i++)
        {
-         if (TYPE_HAS_NONTRIVIAL_DESTRUCTOR (BINFO_TYPE (binfos)))
+         if (TYPE_HAS_NONTRIVIAL_DESTRUCTOR (BINFO_TYPE (base_binfo)))
            {
              expr = build_special_member_call (current_class_ref, 
                                                base_dtor_identifier,
                                                NULL_TREE,
-                                               binfos,
+                                               base_binfo,
                                                (LOOKUP_NORMAL 
                                                 | LOOKUP_NONVIRTUAL));
-             expr = build (COND_EXPR, void_type_node, cond,
-                           expr, void_zero_node);
+             expr = build3 (COND_EXPR, void_type_node, cond,
+                            expr, void_zero_node);
              finish_decl_cleanup (NULL_TREE, expr);
            }
        }
     }
 
-  binfos = BINFO_BASE_BINFOS (TYPE_BINFO (current_class_type));
-  n_baseclasses = BINFO_N_BASE_BINFOS (TYPE_BINFO (current_class_type));
-
   /* Take care of the remaining baseclasses.  */
-  for (i = 0; i < n_baseclasses; i++)
+  for (binfo = TYPE_BINFO (current_class_type), i = 0;
+       BINFO_BASE_ITERATE (binfo, i, base_binfo); i++)
     {
-      tree base_binfo = TREE_VEC_ELT (binfos, i);
       if (TYPE_HAS_TRIVIAL_DESTRUCTOR (BINFO_TYPE (base_binfo))
          || BINFO_VIRTUAL_P (base_binfo))
        continue;
@@ -2921,33 +2936,6 @@ push_base_cleanups (void)
     }
 }
 
-/* For type TYPE, delete the virtual baseclass objects of DECL.  */
-
-tree
-build_vbase_delete (tree type, tree decl)
-{
-  unsigned ix;
-  tree binfo;
-  tree result;
-  tree addr = build_unary_op (ADDR_EXPR, decl, 0);
-
-  my_friendly_assert (addr != error_mark_node, 222);
-
-  result = convert_to_void (integer_zero_node, NULL);
-  for (ix = 0; (binfo = VEC_iterate
-               (tree, CLASSTYPE_VBASECLASSES (type), ix)); ix++)
-    {
-      tree base_addr = convert_force
-       (build_pointer_type (BINFO_TYPE (binfo)), addr, 0);
-      tree base_delete = build_delete
-       (TREE_TYPE (base_addr), base_addr, sfk_base_destructor,
-        LOOKUP_NORMAL|LOOKUP_DESTRUCTOR, 0);
-      
-      result = build_compound_expr (result, base_delete);
-    }
-  return result;
-}
-
 /* Build a C++ vector delete expression.
    MAXINDEX is the number of elements to be deleted.
    ELT_SIZE is the nominal size of each element in the vector.
@@ -2985,10 +2973,10 @@ build_vec_delete (tree base, tree maxindex,
          base = TARGET_EXPR_SLOT (base_init);
        }
       type = strip_array_types (TREE_TYPE (type));
-      cookie_addr = build (MINUS_EXPR,
-                          build_pointer_type (sizetype),
-                          base,
-                          TYPE_SIZE_UNIT (sizetype));
+      cookie_addr = build2 (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)
@@ -3014,7 +3002,7 @@ build_vec_delete (tree base, tree maxindex,
   rval = build_vec_delete_1 (base, maxindex, type, auto_delete_vec,
                             use_global_delete);
   if (base_init)
-    rval = build (COMPOUND_EXPR, TREE_TYPE (rval), base_init, rval);
+    rval = build2 (COMPOUND_EXPR, TREE_TYPE (rval), base_init, rval);
 
   return rval;
 }