OSDN Git Service

* call.c (null_ptr_cst_p): Use maybe_constant_value.
[pf3gnuchains/gcc-fork.git] / gcc / cp / typeck2.c
index a3d9201..f1bd16e 100644 (file)
@@ -1,7 +1,7 @@
 /* Report error messages, build initializers, and perform
    some front-end optimizations for C++ compiler.
    Copyright (C) 1987, 1988, 1989, 1992, 1993, 1994, 1995, 1996, 1997, 1998,
-   1999, 2000, 2001, 2002, 2004, 2005
+   1999, 2000, 2001, 2002, 2004, 2005, 2006, 2007, 2008, 2009, 2010
    Free Software Foundation, Inc.
    Hacked by Michael Tiemann (tiemann@cygnus.com)
 
@@ -9,7 +9,7 @@ This file is part of GCC.
 
 GCC is free software; you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2, or (at your option)
+the Free Software Foundation; either version 3, or (at your option)
 any later version.
 
 GCC is distributed in the hope that it will be useful,
@@ -18,9 +18,8 @@ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 GNU General Public License for more details.
 
 You should have received a copy of the GNU General Public License
-along with GCC; see the file COPYING.  If not, write to
-the Free Software Foundation, 59 Temple Place - Suite 330,
-Boston, MA 02111-1307, USA.  */
+along with GCC; see the file COPYING3.  If not see
+<http://www.gnu.org/licenses/>.  */
 
 
 /* This file is part of the C++ front end.
@@ -33,13 +32,16 @@ Boston, MA 02111-1307, USA.  */
 #include "coretypes.h"
 #include "tm.h"
 #include "tree.h"
+#include "intl.h"
 #include "cp-tree.h"
 #include "flags.h"
 #include "toplev.h"
 #include "output.h"
-#include "diagnostic.h"
+#include "diagnostic-core.h"
+
+static tree
+process_init_constructor (tree type, tree init);
 
-static tree process_init_constructor (tree, tree, tree *);
 
 /* Print an error message stemming from an attempt to use
    BASETYPE as a base class for TYPE.  */
@@ -66,60 +68,146 @@ binfo_or_else (tree base, tree type)
 }
 
 /* According to ARM $7.1.6, "A `const' object may be initialized, but its
-   value may not be changed thereafter.  Thus, we emit hard errors for these,
-   rather than just pedwarns.  If `SOFT' is 1, then we just pedwarn.  (For
-   example, conversions to references.)  */
+   value may not be changed thereafter.  */
 
 void
-readonly_error (tree arg, const char* string, int soft)
+readonly_error (tree arg, readonly_error_kind errstring)
 {
-  const char *fmt;
-  void (*fn) (const char *, ...);
-
-  if (soft)
-    fn = pedwarn;
-  else
-    fn = error;
+/* This macro is used to emit diagnostics to ensure that all format
+   strings are complete sentences, visible to gettext and checked at
+   compile time.  */
+#define ERROR_FOR_ASSIGNMENT(AS, ASM, IN, DE, ARG)                      \
+  do {                                                                  \
+    switch (errstring)                                                  \
+      {                                                                 \
+      case REK_ASSIGNMENT:                                              \
+        error(AS, ARG);                                                 \
+        break;                                                          \
+      case REK_ASSIGNMENT_ASM:                                          \
+        error(ASM, ARG);                                                \
+        break;                                                          \
+      case REK_INCREMENT:                                               \
+        error (IN, ARG);                                                \
+        break;                                                          \
+      case REK_DECREMENT:                                               \
+        error (DE, ARG);                                                \
+        break;                                                          \
+      default:                                                          \
+        gcc_unreachable ();                                             \
+      }                                                                 \
+  } while (0)
 
   if (TREE_CODE (arg) == COMPONENT_REF)
     {
       if (TYPE_READONLY (TREE_TYPE (TREE_OPERAND (arg, 0))))
-        fmt = "%s of data-member %qD in read-only structure";
+        ERROR_FOR_ASSIGNMENT (G_("assignment of "
+                                 "data-member %qD in read-only structure"),
+                              G_("assignment (via 'asm' output) of "
+                                 "data-member %qD in read-only structure"),
+                              G_("increment of "
+                                 "data-member %qD in read-only structure"),
+                              G_("decrement of "
+                                 "data-member %qD in read-only structure"),
+                              TREE_OPERAND (arg, 1));
       else
-        fmt = "%s of read-only data-member %qD";
-      (*fn) (fmt, string, TREE_OPERAND (arg, 1));
+        ERROR_FOR_ASSIGNMENT (G_("assignment of "
+                                 "read-only data-member %qD"),
+                              G_("assignment (via 'asm' output) of "
+                                 "read-only data-member %qD"),
+                              G_("increment of "
+                                 "read-only data-member %qD"),
+                              G_("decrement of "
+                                 "read-only data-member %qD"),
+                              TREE_OPERAND (arg, 1));
     }
   else if (TREE_CODE (arg) == VAR_DECL)
     {
       if (DECL_LANG_SPECIFIC (arg)
          && DECL_IN_AGGR_P (arg)
          && !TREE_STATIC (arg))
-       fmt = "%s of constant field %qD";
+        ERROR_FOR_ASSIGNMENT (G_("assignment of "
+                              "constant field %qD"),
+                              G_("assignment (via 'asm' output) of "
+                              "constant field %qD"),
+                              G_("increment of "
+                              "constant field %qD"),
+                              G_("decrement of "
+                              "constant field %qD"),
+                              arg);
       else
-       fmt = "%s of read-only variable %qD";
-      (*fn) (fmt, string, arg);
+        ERROR_FOR_ASSIGNMENT (G_("assignment of "
+                              "read-only variable %qD"),
+                              G_("assignment (via 'asm' output) of "
+                              "read-only variable %qD"),
+                              G_("increment of "
+                              "read-only variable %qD"),
+                              G_("decrement of "
+                              "read-only variable %qD"),
+                              arg);
+
     }
   else if (TREE_CODE (arg) == PARM_DECL)
-    (*fn) ("%s of read-only parameter %qD", string, arg);
+    ERROR_FOR_ASSIGNMENT (G_("assignment of "
+                             "read-only parameter %qD"),
+                          G_("assignment (via 'asm' output) of "
+                             "read-only parameter %qD"),
+                          G_("increment of "
+                             "read-only parameter %qD"),
+                          G_("decrement of "
+                             "read-only parameter %qD"),
+                          arg);  
   else if (TREE_CODE (arg) == INDIRECT_REF
-           && TREE_CODE (TREE_TYPE (TREE_OPERAND (arg, 0))) == REFERENCE_TYPE
-           && (TREE_CODE (TREE_OPERAND (arg, 0)) == VAR_DECL
-               || TREE_CODE (TREE_OPERAND (arg, 0)) == PARM_DECL))
-    (*fn) ("%s of read-only reference %qD", string, TREE_OPERAND (arg, 0));
+          && TREE_CODE (TREE_TYPE (TREE_OPERAND (arg, 0))) == REFERENCE_TYPE
+          && (TREE_CODE (TREE_OPERAND (arg, 0)) == VAR_DECL
+              || TREE_CODE (TREE_OPERAND (arg, 0)) == PARM_DECL))
+    ERROR_FOR_ASSIGNMENT (G_("assignment of "
+                             "read-only reference %qD"),
+                          G_("assignment (via 'asm' output) of "
+                             "read-only reference %qD"), 
+                          G_("increment of "
+                             "read-only reference %qD"),
+                          G_("decrement of "
+                             "read-only reference %qD"),
+                          TREE_OPERAND (arg, 0));
   else if (TREE_CODE (arg) == RESULT_DECL)
-    (*fn) ("%s of read-only named return value %qD", string, arg);
+    ERROR_FOR_ASSIGNMENT (G_("assignment of "
+                             "read-only named return value %qD"),
+                          G_("assignment (via 'asm' output) of "
+                             "read-only named return value %qD"),
+                          G_("increment of "
+                             "read-only named return value %qD"),
+                          G_("decrement of "
+                             "read-only named return value %qD"),
+                          arg);
   else if (TREE_CODE (arg) == FUNCTION_DECL)
-    (*fn) ("%s of function %qD", string, arg);
+    ERROR_FOR_ASSIGNMENT (G_("assignment of "
+                             "function %qD"),
+                          G_("assignment (via 'asm' output) of "
+                             "function %qD"),
+                          G_("increment of "
+                             "function %qD"),
+                          G_("decrement of "
+                             "function %qD"),
+                          arg);
   else
-    (*fn) ("%s of read-only location", string);
+    ERROR_FOR_ASSIGNMENT (G_("assignment of "
+                             "read-only location %qE"),
+                          G_("assignment (via 'asm' output) of "
+                             "read-only location %qE"),
+                          G_("increment of "
+                             "read-only location %qE"),
+                          G_("decrement of "
+                             "read-only location %qE"),
+                          arg);
 }
 
 \f
 /* Structure that holds information about declarations whose type was
    incomplete and we could not check whether it was abstract or not.  */
 
-struct pending_abstract_type GTY((chain_next ("%h.next")))
-{
+struct GTY((chain_next ("%h.next"))) pending_abstract_type {
   /* Declaration which we are checking for abstractness. It is either
      a DECL node, or an IDENTIFIER_NODE if we do not have a full
      declaration available.  */
@@ -143,7 +231,8 @@ struct pending_abstract_type GTY((chain_next ("%h.next")))
 static hashval_t
 pat_calc_hash (const void* val)
 {
-  const struct pending_abstract_type* pat = val;
+  const struct pending_abstract_type *pat =
+     (const struct pending_abstract_type *) val;
   return (hashval_t) TYPE_UID (pat->type);
 }
 
@@ -154,8 +243,9 @@ pat_calc_hash (const void* val)
 static int
 pat_compare (const void* val1, const void* val2)
 {
-  const struct pending_abstract_type* pat1 = val1;
-  tree type2 = (tree)val2;
+  const struct pending_abstract_type *const pat1 =
+     (const struct pending_abstract_type *) val1;
+  const_tree const type2 = (const_tree)val2;
 
   return (pat1->type == type2);
 }
@@ -214,7 +304,7 @@ complete_type_check_abstract (tree type)
 
          /* Tweak input_location so that the diagnostic appears at the correct
            location. Notice that this is only needed if the decl is an
-           IDENTIFIER_NODE, otherwise cp_error_at.  */
+           IDENTIFIER_NODE.  */
          input_location = pat->locus;
          abstract_virtuals_error (pat->decl, pat->type);
          pat = pat->next;
@@ -241,6 +331,7 @@ abstract_virtuals_error (tree decl, tree type)
      be abstract.  */
   if (!CLASS_TYPE_P (type))
     return 0;
+  type = TYPE_MAIN_VARIANT (type);
 
   /* If the type is incomplete, we register it within a hash table,
      so that we can check again once it is completed. This makes sense
@@ -261,14 +352,14 @@ abstract_virtuals_error (tree decl, tree type)
       slot = htab_find_slot_with_hash (abstract_pending_vars, type,
                                      (hashval_t)TYPE_UID (type), INSERT);
 
-      pat = GGC_NEW (struct pending_abstract_type);
+      pat = ggc_alloc_pending_abstract_type ();
       pat->type = type;
       pat->decl = decl;
       pat->locus = ((decl && DECL_P (decl))
                    ? DECL_SOURCE_LOCATION (decl)
                    : input_location);
 
-      pat->next = *slot;
+      pat->next = (struct pending_abstract_type *) *slot;
       *slot = pat;
 
       return 0;
@@ -289,27 +380,24 @@ abstract_virtuals_error (tree decl, tree type)
        return 0;
 
       if (TREE_CODE (decl) == VAR_DECL)
-       cp_error_at ("cannot declare variable %q+D to be of abstract "
-                    "type %qT", decl, type);
+       error ("cannot declare variable %q+D to be of abstract "
+              "type %qT", decl, type);
       else if (TREE_CODE (decl) == PARM_DECL)
-       cp_error_at ("cannot declare parameter %q+D to be of abstract "
-                    "type %qT", decl, type);
+       error ("cannot declare parameter %q+D to be of abstract type %qT",
+              decl, type);
       else if (TREE_CODE (decl) == FIELD_DECL)
-       cp_error_at ("cannot declare field %q+D to be of abstract "
-                    "type %qT", decl, type);
+       error ("cannot declare field %q+D to be of abstract type %qT",
+              decl, type);
       else if (TREE_CODE (decl) == FUNCTION_DECL
               && TREE_CODE (TREE_TYPE (decl)) == METHOD_TYPE)
-       cp_error_at ("invalid abstract return type for member function %q+#D",
-                    decl);
+       error ("invalid abstract return type for member function %q+#D", decl);
       else if (TREE_CODE (decl) == FUNCTION_DECL)
-       cp_error_at ("invalid abstract return type for function %q+#D",
-                    decl);
+       error ("invalid abstract return type for function %q+#D", decl);
       else if (TREE_CODE (decl) == IDENTIFIER_NODE)
-       /* Here we do not have location information, so use error instead
-          of cp_error_at.  */
+       /* Here we do not have location information.  */
        error ("invalid abstract type %qT for %qE", type, decl);
       else
-       cp_error_at ("invalid abstract type for %q+D", decl);
+       error ("invalid abstract type for %q+D", decl);
     }
   else
     error ("cannot allocate an object of abstract type %qT", type);
@@ -320,51 +408,39 @@ abstract_virtuals_error (tree decl, tree type)
       unsigned ix;
       tree fn;
 
-      inform ("%J  because the following virtual functions are pure "
-             "within %qT:", TYPE_MAIN_DECL (type), type);
+      inform (DECL_SOURCE_LOCATION (TYPE_MAIN_DECL (type)),
+             "  because the following virtual functions are pure within %qT:",
+             type);
 
-      for (ix = 0; VEC_iterate (tree, pure, ix, fn); ix++)
-       inform ("%J\t%#D", fn, fn);
+      FOR_EACH_VEC_ELT (tree, pure, ix, fn)
+       inform (input_location, "\t%+#D", fn);
       /* Now truncate the vector.  This leaves it non-null, so we know
-         there are pure virtuals, but empty so we don't list them out
-         again.  */
+        there are pure virtuals, but empty so we don't list them out
+        again.  */
       VEC_truncate (tree, pure, 0);
     }
   else
-    inform ("%J  since type %qT has pure virtual functions",
-           TYPE_MAIN_DECL (type), type);
+    inform (DECL_SOURCE_LOCATION (TYPE_MAIN_DECL (type)),
+           "  since type %qT has pure virtual functions",
+           type);
 
   return 1;
 }
 
 /* Print an error message for invalid use of an incomplete type.
    VALUE is the expression that was used (or 0 if that isn't known)
-   and TYPE is the type that was invalid.  DIAG_TYPE indicates the
-   type of diagnostic:  0 for an error, 1 for a warning, 2 for a
-   pedwarn.  */
+   and TYPE is the type that was invalid.  DIAG_KIND indicates the
+   type of diagnostic (see diagnostic.def).  */
 
 void
-cxx_incomplete_type_diagnostic (tree value, tree type, int diag_type)
+cxx_incomplete_type_diagnostic (const_tree value, const_tree type, 
+                               diagnostic_t diag_kind)
 {
   int decl = 0;
-  void (*p_msg) (const char *, ...);
-  void (*p_msg_at) (const char *, ...);
 
-  if (diag_type == 1)
-    {
-      p_msg = warning0;
-      p_msg_at = cp_warning_at;
-    }
-  else if (diag_type == 2)
-    {
-      p_msg = pedwarn;
-      p_msg_at = cp_pedwarn_at;
-    }
-  else
-    {
-      p_msg = error;
-      p_msg_at = cp_error_at;
-    }
+  gcc_assert (diag_kind == DK_WARNING 
+             || diag_kind == DK_PEDWARN 
+             || diag_kind == DK_ERROR);
 
   /* Avoid duplicate error message.  */
   if (TREE_CODE (type) == ERROR_MARK)
@@ -374,7 +450,8 @@ cxx_incomplete_type_diagnostic (tree value, tree type, int diag_type)
                     || TREE_CODE (value) == PARM_DECL
                     || TREE_CODE (value) == FIELD_DECL))
     {
-      (*p_msg_at) ("%qD has incomplete type", value);
+      emit_diagnostic (diag_kind, input_location, 0,
+                      "%q+D has incomplete type", value);
       decl = 1;
     }
  retry:
@@ -386,45 +463,71 @@ cxx_incomplete_type_diagnostic (tree value, tree type, int diag_type)
     case UNION_TYPE:
     case ENUMERAL_TYPE:
       if (!decl)
-        (*p_msg) ("invalid use of undefined type %q#T", type);
+       emit_diagnostic (diag_kind, input_location, 0,
+                        "invalid use of incomplete type %q#T", type);
       if (!TYPE_TEMPLATE_INFO (type))
-       (*p_msg_at) ("forward declaration of %q#T", type);
+       emit_diagnostic (diag_kind, input_location, 0,
+                        "forward declaration of %q+#T", type);
       else
-       (*p_msg_at) ("declaration of %q#T", type);
+       emit_diagnostic (diag_kind, input_location, 0,
+                        "declaration of %q+#T", type);
       break;
 
     case VOID_TYPE:
-      (*p_msg) ("invalid use of %qT", type);
+      emit_diagnostic (diag_kind, input_location, 0,
+                      "invalid use of %qT", type);
       break;
 
     case ARRAY_TYPE:
       if (TYPE_DOMAIN (type))
-        {
-          type = TREE_TYPE (type);
-          goto retry;
-        }
-      (*p_msg) ("invalid use of array with unspecified bounds");
+       {
+         type = TREE_TYPE (type);
+         goto retry;
+       }
+      emit_diagnostic (diag_kind, input_location, 0,
+                      "invalid use of array with unspecified bounds");
       break;
 
     case OFFSET_TYPE:
     bad_member:
-      (*p_msg) ("invalid use of member (did you forget the %<&%> ?)");
+      emit_diagnostic (diag_kind, input_location, 0,
+                      "invalid use of member (did you forget the %<&%> ?)");
       break;
 
     case TEMPLATE_TYPE_PARM:
-      (*p_msg) ("invalid use of template type parameter");
+      if (is_auto (type))
+       emit_diagnostic (diag_kind, input_location, 0,
+                        "invalid use of %<auto%>");
+      else
+       emit_diagnostic (diag_kind, input_location, 0,
+                        "invalid use of template type parameter %qT", type);
+      break;
+
+    case BOUND_TEMPLATE_TEMPLATE_PARM:
+      emit_diagnostic (diag_kind, input_location, 0,
+                      "invalid use of template template parameter %qT",
+                      TYPE_NAME (type));
       break;
 
-    case UNKNOWN_TYPE:
+    case TYPENAME_TYPE:
+      emit_diagnostic (diag_kind, input_location, 0,
+                      "invalid use of dependent type %qT", type);
+      break;
+
+    case LANG_TYPE:
+      gcc_assert (type == unknown_type_node);
       if (value && TREE_CODE (value) == COMPONENT_REF)
-        goto bad_member;
+       goto bad_member;
       else if (value && TREE_CODE (value) == ADDR_EXPR)
-        (*p_msg) ("address of overloaded function with no contextual "
-                  "type information");
+       emit_diagnostic (diag_kind, input_location, 0,
+                        "address of overloaded function with no contextual "
+                        "type information");
       else if (value && TREE_CODE (value) == OVERLOAD)
-        (*p_msg) ("overloaded function with no contextual type information");
+       emit_diagnostic (diag_kind, input_location, 0,
+                        "overloaded function with no contextual type information");
       else
-        (*p_msg) ("insufficient contextual information to determine type");
+       emit_diagnostic (diag_kind, input_location, 0,
+                        "insufficient contextual information to determine type");
       break;
 
     default:
@@ -436,9 +539,9 @@ cxx_incomplete_type_diagnostic (tree value, tree type, int diag_type)
    required by ../tree.c.  */
 #undef cxx_incomplete_type_error
 void
-cxx_incomplete_type_error (tree value, tree type)
+cxx_incomplete_type_error (const_tree value, const_tree type)
 {
-  cxx_incomplete_type_diagnostic (value, type, 0);
+  cxx_incomplete_type_diagnostic (value, type, DK_ERROR);
 }
 
 \f
@@ -446,13 +549,16 @@ cxx_incomplete_type_error (tree value, tree type)
    expression to which INIT should be assigned.  INIT is a CONSTRUCTOR.  */
 
 static void
-split_nonconstant_init_1 (tree dest, tree init)
+split_nonconstant_init_1 (tree dest, tree *initp)
 {
-  tree *pelt, elt, type = TREE_TYPE (dest);
-  tree sub, code, inner_type = NULL;
+  unsigned HOST_WIDE_INT idx;
+  tree init = *initp;
+  tree field_index, value;
+  tree type = TREE_TYPE (dest);
+  tree inner_type = NULL;
   bool array_type_p = false;
+  HOST_WIDE_INT num_type_elements, num_initialized_elements;
 
-  pelt = &CONSTRUCTOR_ELTS (init);
   switch (TREE_CODE (type))
     {
     case ARRAY_TYPE:
@@ -463,53 +569,84 @@ split_nonconstant_init_1 (tree dest, tree init)
     case RECORD_TYPE:
     case UNION_TYPE:
     case QUAL_UNION_TYPE:
-      while ((elt = *pelt))
+      num_initialized_elements = 0;
+      FOR_EACH_CONSTRUCTOR_ELT (CONSTRUCTOR_ELTS (init), idx,
+                               field_index, value)
        {
-         tree field_index = TREE_PURPOSE (elt);
-         tree value = TREE_VALUE (elt);
+         /* The current implementation of this algorithm assumes that
+            the field was set for all the elements. This is usually done
+            by process_init_constructor.  */
+         gcc_assert (field_index);
 
          if (!array_type_p)
            inner_type = TREE_TYPE (field_index);
 
          if (TREE_CODE (value) == CONSTRUCTOR)
            {
+             tree sub;
+
              if (array_type_p)
-               sub = build4 (ARRAY_REF, inner_type, dest, field_index,
+               sub = build4 (ARRAY_REF, inner_type, dest, field_index,
                              NULL_TREE, NULL_TREE);
              else
-               sub = build3 (COMPONENT_REF, inner_type, dest, field_index,
+               sub = build3 (COMPONENT_REF, inner_type, dest, field_index,
                              NULL_TREE);
 
-             split_nonconstant_init_1 (sub, value);
+             split_nonconstant_init_1 (sub, &value);
            }
          else if (!initializer_constant_valid_p (value, inner_type))
            {
-             *pelt = TREE_CHAIN (elt);
+             tree code;
+             tree sub;
+             HOST_WIDE_INT inner_elements;
+
+             /* FIXME: Ordered removal is O(1) so the whole function is
+                worst-case quadratic. This could be fixed using an aside
+                bitmap to record which elements must be removed and remove
+                them all at the same time. Or by merging
+                split_non_constant_init into process_init_constructor_array,
+                that is separating constants from non-constants while building
+                the vector.  */
+             VEC_ordered_remove (constructor_elt, CONSTRUCTOR_ELTS (init),
+                                 idx);
+             --idx;
 
              if (array_type_p)
-               sub = build4 (ARRAY_REF, inner_type, dest, field_index,
+               sub = build4 (ARRAY_REF, inner_type, dest, field_index,
                              NULL_TREE, NULL_TREE);
              else
-               sub = build3 (COMPONENT_REF, inner_type, dest, field_index,
+               sub = build3 (COMPONENT_REF, inner_type, dest, field_index,
                              NULL_TREE);
 
-             code = build2 (MODIFY_EXPR, inner_type, sub, value);
-             code = build_stmt (EXPR_STMT, code);
+             code = build2 (INIT_EXPR, inner_type, sub, value);
+             code = build_stmt (input_location, EXPR_STMT, code);
              add_stmt (code);
+
+             inner_elements = count_type_elements (inner_type, true);
+             if (inner_elements < 0)
+               num_initialized_elements = -1;
+             else if (num_initialized_elements >= 0)
+               num_initialized_elements += inner_elements;
              continue;
            }
-
-         pelt = &TREE_CHAIN (elt);
        }
+
+      num_type_elements = count_type_elements (type, true);
+      /* If all elements of the initializer are non-constant and
+        have been split out, we don't need the empty CONSTRUCTOR.  */
+      if (num_type_elements > 0
+         && num_type_elements == num_initialized_elements)
+       *initp = NULL;
       break;
 
     case VECTOR_TYPE:
       if (!initializer_constant_valid_p (init, type))
        {
+         tree code;
          tree cons = copy_node (init);
          CONSTRUCTOR_ELTS (init) = NULL;
          code = build2 (MODIFY_EXPR, type, dest, cons);
-         code = build_stmt (EXPR_STMT, code);
+         code = build_stmt (input_location, EXPR_STMT, code);
          add_stmt (code);
        }
       break;
@@ -517,6 +654,9 @@ split_nonconstant_init_1 (tree dest, tree init)
     default:
       gcc_unreachable ();
     }
+
+  /* The rest of the initializer is now a constant. */
+  TREE_CONSTANT (init) = 1;
 }
 
 /* A subroutine of store_init_value.  Splits non-constant static
@@ -532,7 +672,7 @@ split_nonconstant_init (tree dest, tree init)
   if (TREE_CODE (init) == CONSTRUCTOR)
     {
       code = push_stmt_list ();
-      split_nonconstant_init_1 (dest, init);
+      split_nonconstant_init_1 (dest, &init);
       code = pop_stmt_list (code);
       DECL_INITIAL (dest) = init;
       TREE_READONLY (dest) = 0;
@@ -562,7 +702,7 @@ split_nonconstant_init (tree dest, tree init)
    for static variable.  In that case, caller must emit the code.  */
 
 tree
-store_init_value (tree decl, tree init)
+store_init_value (tree decl, tree init, int flags)
 {
   tree value, type;
 
@@ -572,25 +712,21 @@ store_init_value (tree decl, tree init)
   if (TREE_CODE (type) == ERROR_MARK)
     return NULL_TREE;
 
-  if (IS_AGGR_TYPE (type))
+  if (MAYBE_CLASS_TYPE_P (type))
     {
-      gcc_assert (TYPE_HAS_TRIVIAL_INIT_REF (type)
-                 || TREE_CODE (init) == CONSTRUCTOR);
-
       if (TREE_CODE (init) == TREE_LIST)
        {
          error ("constructor syntax used, but no constructor declared "
-                 "for type %qT", type);
-         init = build_constructor (NULL_TREE, nreverse (init));
+                "for type %qT", type);
+         init = build_constructor_from_list (init_list_type_node, nreverse (init));
        }
     }
   else if (TREE_CODE (init) == TREE_LIST
           && TREE_TYPE (init) != unknown_type_node)
     {
-      if (TREE_CODE (decl) == RESULT_DECL)
-       init = build_x_compound_expr_from_list (init,
-                                               "return value initializer");
-      else if (TREE_CODE (init) == TREE_LIST
+      gcc_assert (TREE_CODE (decl) != RESULT_DECL);
+
+      if (TREE_CODE (init) == TREE_LIST
               && TREE_CODE (TREE_TYPE (decl)) == ARRAY_TYPE)
        {
          error ("cannot initialize arrays using this syntax");
@@ -598,13 +734,38 @@ store_init_value (tree decl, tree init)
        }
       else
        /* We get here with code like `int a (2);' */
-       init = build_x_compound_expr_from_list (init, "initializer");
+       init = build_x_compound_expr_from_list (init, ELK_INIT,
+                                               tf_warning_or_error);
     }
 
   /* End of special C++ code.  */
 
-  /* Digest the specified initializer into an expression.  */
-  value = digest_init (type, init, (tree *) 0);
+  if (flags & LOOKUP_ALREADY_DIGESTED)
+    value = init;
+  else
+    /* Digest the specified initializer into an expression.  */
+    value = digest_init_flags (type, init, flags);
+
+  /* In C++0x constant expression is a semantic, not syntactic, property.
+     In C++98, make sure that what we thought was a constant expression at
+     template definition time is still constant.  */
+  if ((cxx_dialect >= cxx0x
+       || DECL_INITIALIZED_BY_CONSTANT_EXPRESSION_P (decl))
+      && (decl_maybe_constant_var_p (decl)
+         || TREE_STATIC (decl)))
+    {
+      bool const_init;
+      value = fold_non_dependent_expr (value);
+      value = maybe_constant_init (value);
+      if (DECL_DECLARED_CONSTEXPR_P (decl))
+       /* Diagnose a non-constant initializer for constexpr.  */
+       value = cxx_constant_value (value);
+      const_init = (reduced_constant_expression_p (value)
+                   || error_operand_p (value));
+      DECL_INITIALIZED_BY_CONSTANT_EXPRESSION_P (decl) = const_init;
+      TREE_CONSTANT (decl) = const_init && decl_maybe_constant_var_p (decl);
+    }
+
   /* If the initializer is not a constant, fill in DECL_INITIAL with
      the bits that are constant, and then return an expression that
      will perform the dynamic initialization.  */
@@ -620,39 +781,82 @@ store_init_value (tree decl, tree init)
 }
 
 \f
-/* Digest the parser output INIT as an initializer for type TYPE.
-   Return a C expression of type TYPE to represent the initial value.
-
-   If TAIL is nonzero, it points to a variable holding a list of elements
-   of which INIT is the first.  We update the list stored there by
-   removing from the head all the elements that we use.
-   Normally this is only one; we use more than one element only if
-   TYPE is an aggregate and INIT is not a constructor.  */
+/* Give errors about narrowing conversions within { }.  */
 
-tree
-digest_init (tree type, tree init, tree* tail)
+void
+check_narrowing (tree type, tree init)
 {
-  enum tree_code code = TREE_CODE (type);
-  tree element = NULL_TREE;
-  tree old_tail_contents = NULL_TREE;
+  tree ftype = unlowered_expr_type (init);
+  bool ok = true;
+  REAL_VALUE_TYPE d;
 
-  /* By default, assume we use one element from a list.
-     We correct this later in the sole case where it is not true.  */
+  init = maybe_constant_value (init);
 
-  if (tail)
+  if (TREE_CODE (type) == INTEGER_TYPE
+      && TREE_CODE (ftype) == REAL_TYPE)
+    ok = false;
+  else if (INTEGRAL_OR_ENUMERATION_TYPE_P (ftype)
+          && CP_INTEGRAL_TYPE_P (type))
+    {
+      if (TYPE_PRECISION (type) < TYPE_PRECISION (ftype)
+         && (TREE_CODE (init) != INTEGER_CST
+             || !int_fits_type_p (init, type)))
+       ok = false;
+    }
+  else if (TREE_CODE (ftype) == REAL_TYPE
+          && TREE_CODE (type) == REAL_TYPE)
+    {
+      if (TYPE_PRECISION (type) < TYPE_PRECISION (ftype))
+       {
+         if (TREE_CODE (init) == REAL_CST)
+           {
+             /* Issue 703: Loss of precision is OK as long as the value is
+                within the representable range of the new type.  */
+             REAL_VALUE_TYPE r;
+             d = TREE_REAL_CST (init);
+             real_convert (&r, TYPE_MODE (type), &d);
+             if (real_isinf (&r))
+               ok = false;
+           }
+         else
+           ok = false;
+       }
+    }
+  else if (INTEGRAL_OR_ENUMERATION_TYPE_P (ftype)
+          && TREE_CODE (type) == REAL_TYPE)
     {
-      old_tail_contents = *tail;
-      *tail = TREE_CHAIN (*tail);
+      ok = false;
+      if (TREE_CODE (init) == INTEGER_CST)
+       {
+         d = real_value_from_int_cst (0, init);
+         if (exact_real_truncate (TYPE_MODE (type), &d))
+           ok = true;
+       }
     }
 
-  if (init == error_mark_node || (TREE_CODE (init) == TREE_LIST
-                                 && TREE_VALUE (init) == error_mark_node))
+  if (!ok)
+    permerror (input_location, "narrowing conversion of %qE from %qT to %qT inside { }",
+              init, ftype, type);
+}
+
+/* Process the initializer INIT for a variable of type TYPE, emitting
+   diagnostics for invalid initializers and converting the initializer as
+   appropriate.
+
+   For aggregate types, it assumes that reshape_init has already run, thus the
+   initializer will have the right shape (brace elision has been undone).
+
+   NESTED is true iff we are being called for an element of a CONSTRUCTOR.  */
+
+static tree
+digest_init_r (tree type, tree init, bool nested, int flags)
+{
+  enum tree_code code = TREE_CODE (type);
+
+  if (error_operand_p (init))
     return error_mark_node;
 
-  if (TREE_CODE (init) == ERROR_MARK)
-    /* __PRETTY_FUNCTION__'s initializer is a bogus expression inside
-       a template function. This gets substituted during instantiation.  */
-    return init;
+  gcc_assert (init);
 
   /* We must strip the outermost array type when completing the type,
      because the its bounds might be incomplete at the moment.  */
@@ -660,60 +864,47 @@ digest_init (tree type, tree init, tree* tail)
                              ? TREE_TYPE (type) : type, NULL_TREE))
     return error_mark_node;
 
-  /* Strip NON_LVALUE_EXPRs since we aren't using as an lvalue.  */
+  /* Strip NON_LVALUE_EXPRs since we aren't using as an lvalue
+     (g++.old-deja/g++.law/casts2.C).  */
   if (TREE_CODE (init) == NON_LVALUE_EXPR)
     init = TREE_OPERAND (init, 0);
 
-  if (BRACE_ENCLOSED_INITIALIZER_P (init)
-      && CONSTRUCTOR_ELTS (init) != 0
-      && TREE_CHAIN (CONSTRUCTOR_ELTS (init)) == 0)
-    {
-      element = TREE_VALUE (CONSTRUCTOR_ELTS (init));
-      /* Strip NON_LVALUE_EXPRs since we aren't using as an lvalue.  */
-      if (element && TREE_CODE (element) == NON_LVALUE_EXPR)
-       element = TREE_OPERAND (element, 0);
-      if (element == error_mark_node)
-       return element;
-    }
-
-  /* Initialization of an array of chars from a string constant
-     optionally enclosed in braces.  */
-
+  /* Initialization of an array of chars from a string constant. The initializer
+     can be optionally enclosed in braces, but reshape_init has already removed
+     them if they were present.  */
   if (code == ARRAY_TYPE)
     {
-      tree typ1;
-
-      if (TREE_CODE (init) == TREE_LIST)
-       {
-         error ("initializing array with parameter list");
-         return error_mark_node;
-       }
-
-      typ1 = TYPE_MAIN_VARIANT (TREE_TYPE (type));
+      tree typ1 = TYPE_MAIN_VARIANT (TREE_TYPE (type));
       if (char_type_p (typ1)
-         && ((init && TREE_CODE (init) == STRING_CST)
-             || (element && TREE_CODE (element) == STRING_CST)))
+         /*&& init */
+         && TREE_CODE (init) == STRING_CST)
        {
-         tree string = element ? element : init;
+         tree char_type = TYPE_MAIN_VARIANT (TREE_TYPE (TREE_TYPE (init)));
 
-         if ((TYPE_MAIN_VARIANT (TREE_TYPE (TREE_TYPE (string)))
-              != char_type_node)
-             && TYPE_PRECISION (typ1) == BITS_PER_UNIT)
+         if (TYPE_PRECISION (typ1) == BITS_PER_UNIT)
            {
-             error ("char-array initialized from wide string");
-             return error_mark_node;
+             if (char_type != char_type_node)
+               {
+                 error ("char-array initialized from wide string");
+                 return error_mark_node;
+               }
            }
-         if ((TYPE_MAIN_VARIANT (TREE_TYPE (TREE_TYPE (string)))
-              == char_type_node)
-             && TYPE_PRECISION (typ1) != BITS_PER_UNIT)
+         else
            {
-             error ("int-array initialized from non-wide string");
-             return error_mark_node;
+             if (char_type == char_type_node)
+               {
+                 error ("int-array initialized from non-wide string");
+                 return error_mark_node;
+               }
+             else if (char_type != typ1)
+               {
+                 error ("int-array initialized from incompatible wide string");
+                 return error_mark_node;
+               }
            }
 
-         TREE_TYPE (string) = type;
-         if (TYPE_DOMAIN (type) != 0
-             && TREE_CONSTANT (TYPE_SIZE (type)))
+         TREE_TYPE (init) = type;
+         if (TYPE_DOMAIN (type) != 0 && TREE_CONSTANT (TYPE_SIZE (type)))
            {
              int size = TREE_INT_CST_LOW (TYPE_SIZE (type));
              size = (size + BITS_PER_UNIT - 1) / BITS_PER_UNIT;
@@ -721,419 +912,447 @@ digest_init (tree type, tree init, tree* tail)
                 because it's ok to ignore the terminating null char that is
                 counted in the length of the constant, but in C++ this would
                 be invalid.  */
-             if (size < TREE_STRING_LENGTH (string))
-               pedwarn ("initializer-string for array of chars is too long");
+             if (size < TREE_STRING_LENGTH (init))
+               permerror (input_location, "initializer-string for array of chars is too long");
            }
-         return string;
+         return init;
        }
     }
 
-  /* Handle scalar types, including conversions,
-     and signature pointers and references.  */
-
-  if (code == INTEGER_TYPE || code == REAL_TYPE || code == POINTER_TYPE
-      || code == ENUMERAL_TYPE || code == REFERENCE_TYPE
-      || code == BOOLEAN_TYPE || code == COMPLEX_TYPE
-      || TYPE_PTR_TO_MEMBER_P (type))
+  /* Handle scalar types (including conversions) and references.  */
+  if ((TREE_CODE (type) != COMPLEX_TYPE
+       || BRACE_ENCLOSED_INITIALIZER_P (init))
+      && (SCALAR_TYPE_P (type) || code == REFERENCE_TYPE))
     {
-      if (BRACE_ENCLOSED_INITIALIZER_P (init))
-       {
-         if (element == 0)
-           {
-             error ("initializer for scalar variable requires one element");
-             return error_mark_node;
-           }
-         init = element;
-       }
-      while (BRACE_ENCLOSED_INITIALIZER_P (init))
-       {
-         pedwarn ("braces around scalar initializer for %qT", type);
-         init = CONSTRUCTOR_ELTS (init);
-         if (TREE_CHAIN (init))
-           pedwarn ("ignoring extra initializers for %qT", type);
-         init = TREE_VALUE (init);
-       }
+      tree *exp;
 
-      return convert_for_initialization (0, type, init, LOOKUP_NORMAL,
-                                        "initialization", NULL_TREE, 0);
-    }
+      if (cxx_dialect != cxx98 && nested)
+       check_narrowing (type, init);
+      init = convert_for_initialization (0, type, init, flags,
+                                        ICR_INIT, NULL_TREE, 0,
+                                        tf_warning_or_error);
+      exp = &init;
 
-  /* Come here only for records and arrays (and unions with constructors).  */
+      /* Skip any conversions since we'll be outputting the underlying
+        constant.  */
+      while (CONVERT_EXPR_P (*exp)
+            || TREE_CODE (*exp) == NON_LVALUE_EXPR)
+       exp = &TREE_OPERAND (*exp, 0);
 
-  if (COMPLETE_TYPE_P (type) && ! TREE_CONSTANT (TYPE_SIZE (type)))
-    {
-      error ("variable-sized object of type %qT may not be initialized",
-               type);
-      return error_mark_node;
+      *exp = cplus_expand_constant (*exp);
+
+      return init;
     }
 
-  if (code == ARRAY_TYPE || code == VECTOR_TYPE || IS_AGGR_TYPE_CODE (code))
+  /* Come here only for aggregates: records, arrays, unions, complex numbers
+     and vectors.  */
+  gcc_assert (TREE_CODE (type) == ARRAY_TYPE
+             || TREE_CODE (type) == VECTOR_TYPE
+             || TREE_CODE (type) == RECORD_TYPE
+             || TREE_CODE (type) == UNION_TYPE
+             || TREE_CODE (type) == COMPLEX_TYPE);
+
+  if (BRACE_ENCLOSED_INITIALIZER_P (init)
+      && !TYPE_NON_AGGREGATE_CLASS (type))
+    return process_init_constructor (type, init);
+  else
     {
-      if (BRACE_ENCLOSED_INITIALIZER_P (init))
-       {
-         if (TYPE_NON_AGGREGATE_CLASS (type))
-           {
-             error ("subobject of type %qT must be initialized by "
-                     "constructor, not by %qE",
-                    type, init);
-             return error_mark_node;
-           }
-         return process_init_constructor (type, init, (tree *)0);
-       }
-      else if (can_convert_arg (type, TREE_TYPE (init), init)
-              || TYPE_NON_AGGREGATE_CLASS (type))
-       /* These are never initialized from multiple constructor elements.  */;
-      else if (tail != 0)
+      if (COMPOUND_LITERAL_P (init) && TREE_CODE (type) == ARRAY_TYPE)
        {
-         *tail = old_tail_contents;
-         return process_init_constructor (type, 0, tail);
+         error ("cannot initialize aggregate of type %qT with "
+                "a compound literal", type);
+
+         return error_mark_node;
        }
 
-      if (code != ARRAY_TYPE)
+      if (TREE_CODE (type) == ARRAY_TYPE
+         && !BRACE_ENCLOSED_INITIALIZER_P (init))
        {
-         int flags = LOOKUP_NORMAL;
-         /* Initialization from { } is copy-initialization.  */
-         if (tail)
-           flags |= LOOKUP_ONLYCONVERTING;
-
-         return convert_for_initialization (NULL_TREE, type, init, flags,
-                                            "initialization", NULL_TREE, 0);
+         /* Allow the result of build_array_copy and of
+            build_value_init_noctor.  */
+         if ((TREE_CODE (init) == TARGET_EXPR
+              || TREE_CODE (init) == CONSTRUCTOR)
+             && (same_type_ignoring_top_level_qualifiers_p
+                 (type, TREE_TYPE (init))))
+           return init;
+
+         error ("array must be initialized with a brace-enclosed"
+                " initializer");
+         return error_mark_node;
        }
+
+      return convert_for_initialization (NULL_TREE, type, init,
+                                        flags,
+                                        ICR_INIT, NULL_TREE, 0,
+                                         tf_warning_or_error);
     }
+}
 
-  error ("invalid initializer");
-  return error_mark_node;
+tree
+digest_init (tree type, tree init)
+{
+  return digest_init_r (type, init, false, LOOKUP_IMPLICIT);
+}
+
+tree
+digest_init_flags (tree type, tree init, int flags)
+{
+  return digest_init_r (type, init, false, flags);
 }
 \f
-/* Process a constructor for a variable of type TYPE.
-   The constructor elements may be specified either with INIT or with ELTS,
-   only one of which should be non-null.
+/* Set of flags used within process_init_constructor to describe the
+   initializers.  */
+#define PICFLAG_ERRONEOUS 1
+#define PICFLAG_NOT_ALL_CONSTANT 2
+#define PICFLAG_NOT_ALL_SIMPLE 4
 
-   If INIT is specified, it is a CONSTRUCTOR node which is specifically
-   and solely for initializing this datum.
+/* Given an initializer INIT, return the flag (PICFLAG_*) which better
+   describe it.  */
 
-   If ELTS is specified, it is the address of a variable containing
-   a list of expressions.  We take as many elements as we need
-   from the head of the list and update the list.
+static int
+picflag_from_initializer (tree init)
+{
+  if (init == error_mark_node)
+    return PICFLAG_ERRONEOUS;
+  else if (!TREE_CONSTANT (init))
+    return PICFLAG_NOT_ALL_CONSTANT;
+  else if (!initializer_constant_valid_p (init, TREE_TYPE (init)))
+    return PICFLAG_NOT_ALL_SIMPLE;
+  return 0;
+}
 
-   In the resulting constructor, TREE_CONSTANT is set if all elts are
-   constant, and TREE_STATIC is set if, in addition, all elts are simple enough
-   constants that the assembler and linker can compute them.  */
+/* Subroutine of process_init_constructor, which will process an initializer
+   INIT for an array or vector of type TYPE. Returns the flags (PICFLAG_*)
+   which describe the initializers.  */
 
-static tree
-process_init_constructor (tree type, tree init, tree* elts)
+static int
+process_init_constructor_array (tree type, tree init)
 {
-  tree tail;
-  /* List of the elements of the result constructor,
-     in reverse order.  */
-  tree members = NULL;
-  tree next1;
-  tree result;
-  int allconstant = 1;
-  int allsimple = 1;
-  int erroneous = 0;
-
-  /* Make TAIL be the list of elements to use for the initialization,
-     no matter how the data was given to us.  */
-
-  if (elts)
+  unsigned HOST_WIDE_INT i, len = 0;
+  int flags = 0;
+  bool unbounded = false;
+  constructor_elt *ce;
+  VEC(constructor_elt,gc) *v = CONSTRUCTOR_ELTS (init);
+
+  gcc_assert (TREE_CODE (type) == ARRAY_TYPE
+             || TREE_CODE (type) == VECTOR_TYPE);
+
+  if (TREE_CODE (type) == ARRAY_TYPE)
     {
-      if (warn_missing_braces)
-       warning (0, "aggregate has a partly bracketed initializer");
-      tail = *elts;
+      tree domain = TYPE_DOMAIN (type);
+      if (domain)
+       len = (TREE_INT_CST_LOW (TYPE_MAX_VALUE (domain))
+             - TREE_INT_CST_LOW (TYPE_MIN_VALUE (domain))
+             + 1);
+      else
+       unbounded = true;  /* Take as many as there are.  */
     }
   else
-    tail = CONSTRUCTOR_ELTS (init);
+    /* Vectors are like simple fixed-size arrays.  */
+    len = TYPE_VECTOR_SUBPARTS (type);
 
-  /* Gobble as many elements as needed, and make a constructor or initial value
-     for each element of this aggregate.  Chain them together in result.
-     If there are too few, use 0 for each scalar ultimate component.  */
+  /* There must not be more initializers than needed.  */
+  if (!unbounded && VEC_length (constructor_elt, v)  > len)
+    error ("too many initializers for %qT", type);
 
-  if (TREE_CODE (type) == ARRAY_TYPE || TREE_CODE (type) == VECTOR_TYPE)
+  FOR_EACH_VEC_ELT (constructor_elt, v, i, ce)
     {
-      long len;
-      int i;
-
-      if (TREE_CODE (type) == ARRAY_TYPE)
+      if (ce->index)
        {
-         tree domain = TYPE_DOMAIN (type);
-         if (domain)
-           len = (TREE_INT_CST_LOW (TYPE_MAX_VALUE (domain))
-                  - TREE_INT_CST_LOW (TYPE_MIN_VALUE (domain))
-                  + 1);
-         else
-           len = -1;  /* Take as many as there are.  */
+         gcc_assert (TREE_CODE (ce->index) == INTEGER_CST);
+         if (compare_tree_int (ce->index, i) != 0)
+           {
+             ce->value = error_mark_node;
+             sorry ("non-trivial designated initializers not supported");
+           }
        }
       else
+       ce->index = size_int (i);
+      gcc_assert (ce->value);
+      ce->value = digest_init_r (TREE_TYPE (type), ce->value, true, LOOKUP_IMPLICIT);
+
+      if (ce->value != error_mark_node)
+       gcc_assert (same_type_ignoring_top_level_qualifiers_p
+                     (TREE_TYPE (type), TREE_TYPE (ce->value)));
+
+      flags |= picflag_from_initializer (ce->value);
+    }
+
+  /* No more initializers. If the array is unbounded, we are done. Otherwise,
+     we must add initializers ourselves.  */
+  if (!unbounded)
+    for (; i < len; ++i)
+      {
+       tree next;
+
+       if (TYPE_NEEDS_CONSTRUCTING (TREE_TYPE (type)))
+         {
+           /* If this type needs constructors run for default-initialization,
+             we can't rely on the back end to do it for us, so build up
+             TARGET_EXPRs.  If the type in question is a class, just build
+             one up; if it's an array, recurse.  */
+           if (MAYBE_CLASS_TYPE_P (TREE_TYPE (type)))
+              next = build_functional_cast (TREE_TYPE (type), NULL_TREE,
+                                            tf_warning_or_error);
+           else
+             next = build_constructor (init_list_type_node, NULL);
+           next = digest_init (TREE_TYPE (type), next);
+         }
+       else if (!zero_init_p (TREE_TYPE (type)))
+         next = build_zero_init (TREE_TYPE (type),
+                                 /*nelts=*/NULL_TREE,
+                                 /*static_storage_p=*/false);
+       else
+         /* The default zero-initialization is fine for us; don't
+            add anything to the CONSTRUCTOR.  */
+         break;
+
+       flags |= picflag_from_initializer (next);
+       CONSTRUCTOR_APPEND_ELT (v, size_int (i), next);
+      }
+
+  CONSTRUCTOR_ELTS (init) = v;
+  return flags;
+}
+
+/* Subroutine of process_init_constructor, which will process an initializer
+   INIT for a class of type TYPE. Returns the flags (PICFLAG_*) which describe
+   the initializers.  */
+
+static int
+process_init_constructor_record (tree type, tree init)
+{
+  VEC(constructor_elt,gc) *v = NULL;
+  int flags = 0;
+  tree field;
+  unsigned HOST_WIDE_INT idx = 0;
+
+  gcc_assert (TREE_CODE (type) == RECORD_TYPE);
+  gcc_assert (!CLASSTYPE_VBASECLASSES (type));
+  gcc_assert (!TYPE_BINFO (type)
+             || !BINFO_N_BASE_BINFOS (TYPE_BINFO (type)));
+  gcc_assert (!TYPE_POLYMORPHIC_P (type));
+
+  /* Generally, we will always have an index for each initializer (which is
+     a FIELD_DECL, put by reshape_init), but compound literals don't go trough
+     reshape_init. So we need to handle both cases.  */
+  for (field = TYPE_FIELDS (type); field; field = DECL_CHAIN (field))
+    {
+      tree next;
+      tree type;
+
+      if (!DECL_NAME (field) && DECL_C_BIT_FIELD (field))
        {
-         /* Vectors are like simple fixed-size arrays.  */
-         len = TYPE_VECTOR_SUBPARTS (type);
+         flags |= picflag_from_initializer (integer_zero_node);
+         CONSTRUCTOR_APPEND_ELT (v, field, integer_zero_node);
+         continue;
        }
 
-      for (i = 0; len < 0 || i < len; i++)
+      if (TREE_CODE (field) != FIELD_DECL || DECL_ARTIFICIAL (field))
+       continue;
+
+      /* If this is a bitfield, first convert to the declared type.  */
+      type = TREE_TYPE (field);
+      if (DECL_BIT_FIELD_TYPE (field))
+       type = DECL_BIT_FIELD_TYPE (field);
+
+      if (idx < VEC_length (constructor_elt, CONSTRUCTOR_ELTS (init)))
        {
-         if (tail)
+         constructor_elt *ce = VEC_index (constructor_elt,
+                                          CONSTRUCTOR_ELTS (init), idx);
+         if (ce->index)
            {
-             if (TREE_PURPOSE (tail)
-                 && (TREE_CODE (TREE_PURPOSE (tail)) != INTEGER_CST
-                     || compare_tree_int (TREE_PURPOSE (tail), i) != 0))
-               sorry ("non-trivial labeled initializers");
-
-             if (TREE_VALUE (tail) != 0)
+             /* We can have either a FIELD_DECL or an IDENTIFIER_NODE. The
+                latter case can happen in templates where lookup has to be
+                deferred.  */
+             gcc_assert (TREE_CODE (ce->index) == FIELD_DECL
+                         || TREE_CODE (ce->index) == IDENTIFIER_NODE);
+             if (ce->index != field
+                 && ce->index != DECL_NAME (field))
                {
-                 tree tail1 = tail;
-                 next1 = digest_init (TREE_TYPE (type),
-                                      TREE_VALUE (tail), &tail1);
-                 if (next1 == error_mark_node)
-                   return next1;
-                 gcc_assert (same_type_ignoring_top_level_qualifiers_p
-                             (TREE_TYPE (type), TREE_TYPE (next1)));
-                 gcc_assert (!tail1 || TREE_CODE (tail1) == TREE_LIST);
-                 if (tail == tail1 && len < 0)
-                   {
-                     error ("non-empty initializer for array of empty elements");
-                     /* Just ignore what we were supposed to use.  */
-                     tail1 = NULL_TREE;
-                   }
-                 tail = tail1;
-               }
-             else
-               {
-                 next1 = error_mark_node;
-                 tail = TREE_CHAIN (tail);
+                 ce->value = error_mark_node;
+                 sorry ("non-trivial designated initializers not supported");
                }
            }
-         else if (len < 0)
-           /* We're done.  */
-           break;
-         else if (TYPE_NEEDS_CONSTRUCTING (TREE_TYPE (type)))
-           {
-             /* If this type needs constructors run for
-                default-initialization, we can't rely on the backend to do it
-                for us, so build up TARGET_EXPRs.  If the type in question is
-                a class, just build one up; if it's an array, recurse.  */
 
-             if (IS_AGGR_TYPE (TREE_TYPE (type)))
-               next1 = build_functional_cast (TREE_TYPE (type), NULL_TREE);
-             else
-               next1 = build_constructor (NULL_TREE, NULL_TREE);
-             next1 = digest_init (TREE_TYPE (type), next1, 0);
+         gcc_assert (ce->value);
+         next = digest_init_r (type, ce->value, true, LOOKUP_IMPLICIT);
+         ++idx;
+       }
+      else if (TYPE_NEEDS_CONSTRUCTING (TREE_TYPE (field)))
+       {
+         /* If this type needs constructors run for
+            default-initialization, we can't rely on the back end to do it
+            for us, so build up TARGET_EXPRs.  If the type in question is
+            a class, just build one up; if it's an array, recurse.  */
+         next = build_constructor (init_list_type_node, NULL);
+         if (MAYBE_CLASS_TYPE_P (TREE_TYPE (field)))
+           {
+             next = finish_compound_literal (TREE_TYPE (field), next);
+             /* direct-initialize the target. No temporary is going
+                 to be involved.  */
+             if (TREE_CODE (next) == TARGET_EXPR)
+               TARGET_EXPR_DIRECT_INIT_P (next) = true;
            }
-         else if (! zero_init_p (TREE_TYPE (type)))
-           next1 = build_zero_init (TREE_TYPE (type),
-                                    /*nelts=*/NULL_TREE,
-                                    /*static_storage_p=*/false);
+
+         next = digest_init_r (TREE_TYPE (field), next, true, LOOKUP_IMPLICIT);
+
+         /* Warn when some struct elements are implicitly initialized.  */
+         warning (OPT_Wmissing_field_initializers,
+                  "missing initializer for member %qD", field);
+       }
+      else
+       {
+         if (TREE_READONLY (field))
+           error ("uninitialized const member %qD", field);
+         else if (CLASSTYPE_READONLY_FIELDS_NEED_INIT (TREE_TYPE (field)))
+           error ("member %qD with uninitialized const fields", field);
+         else if (TREE_CODE (TREE_TYPE (field)) == REFERENCE_TYPE)
+           error ("member %qD is uninitialized reference", field);
+
+         /* Warn when some struct elements are implicitly initialized
+            to zero.  */
+         warning (OPT_Wmissing_field_initializers,
+                  "missing initializer for member %qD", field);
+
+         if (!zero_init_p (TREE_TYPE (field)))
+           next = build_zero_init (TREE_TYPE (field), /*nelts=*/NULL_TREE,
+                                   /*static_storage_p=*/false);
          else
            /* The default zero-initialization is fine for us; don't
-              add anything to the CONSTRUCTOR.  */
-           break;
-
-         if (next1 == error_mark_node)
-           erroneous = 1;
-         else if (!TREE_CONSTANT (next1))
-           allconstant = 0;
-         else if (! initializer_constant_valid_p (next1, TREE_TYPE (next1)))
-           allsimple = 0;
-         members = tree_cons (size_int (i), next1, members);
+           add anything to the CONSTRUCTOR.  */
+           continue;
        }
+
+      /* If this is a bitfield, now convert to the lowered type.  */
+      if (type != TREE_TYPE (field))
+       next = cp_convert_and_check (TREE_TYPE (field), next);
+      flags |= picflag_from_initializer (next);
+      CONSTRUCTOR_APPEND_ELT (v, field, next);
     }
-  else if (TREE_CODE (type) == RECORD_TYPE)
-    {
-      tree field;
 
-      if (tail)
-       {
-         gcc_assert (!CLASSTYPE_VBASECLASSES (type));
-         gcc_assert (!TYPE_BINFO (type)
-                     || !BINFO_N_BASE_BINFOS (TYPE_BINFO (type)));
-         gcc_assert (!TYPE_POLYMORPHIC_P (type));
-       }
+  if (idx < VEC_length (constructor_elt, CONSTRUCTOR_ELTS (init)))
+    error ("too many initializers for %qT", type);
+    
+  CONSTRUCTOR_ELTS (init) = v;
+  return flags;
+}
 
-      for (field = TYPE_FIELDS (type); field;
-          field = TREE_CHAIN (field))
-       {
-         if (! DECL_NAME (field) && DECL_C_BIT_FIELD (field))
-           {
-             members = tree_cons (field, integer_zero_node, members);
-             continue;
-           }
+/* Subroutine of process_init_constructor, which will process a single
+   initializer INIT for a union of type TYPE. Returns the flags (PICFLAG_*)
+   which describe the initializer.  */
 
-         if (TREE_CODE (field) != FIELD_DECL || DECL_ARTIFICIAL (field))
-           continue;
+static int
+process_init_constructor_union (tree type, tree init)
+{
+  constructor_elt *ce;
+  int len;
 
-         if (tail)
-           {
-             if (TREE_PURPOSE (tail)
-                 && TREE_PURPOSE (tail) != field
-                 && TREE_PURPOSE (tail) != DECL_NAME (field))
-               sorry ("non-trivial labeled initializers");
+  /* If the initializer was empty, use default zero initialization.  */
+  if (VEC_empty (constructor_elt, CONSTRUCTOR_ELTS (init)))
+    return 0;
 
-             if (TREE_VALUE (tail) != 0)
-               {
-                 tree tail1 = tail;
+  len = VEC_length (constructor_elt, CONSTRUCTOR_ELTS (init));
+  if (len > 1)
+    {
+      error ("too many initializers for %qT", type);
+      VEC_block_remove (constructor_elt, CONSTRUCTOR_ELTS (init), 1, len-1);
+    }
 
-                 next1 = digest_init (TREE_TYPE (field),
-                                      TREE_VALUE (tail), &tail1);
-                 gcc_assert (!tail1 || TREE_CODE (tail1) == TREE_LIST);
-                 tail = tail1;
-               }
-             else
-               {
-                 next1 = error_mark_node;
-                 tail = TREE_CHAIN (tail);
-               }
-           }
-         else if (TYPE_NEEDS_CONSTRUCTING (TREE_TYPE (field)))
-           {
-             /* If this type needs constructors run for
-                default-initialization, we can't rely on the backend to do it
-                for us, so build up TARGET_EXPRs.  If the type in question is
-                a class, just build one up; if it's an array, recurse.  */
-
-             if (IS_AGGR_TYPE (TREE_TYPE (field)))
-               next1 = build_functional_cast (TREE_TYPE (field),
-                                              NULL_TREE);
-             else
-               {
-                 next1 = build_constructor (NULL_TREE, NULL_TREE);
-                  if (init)
-                    TREE_HAS_CONSTRUCTOR (next1)
-                       = TREE_HAS_CONSTRUCTOR (init);
-                }
-             next1 = digest_init (TREE_TYPE (field), next1, 0);
-
-             /* Warn when some struct elements are implicitly initialized.  */
-             if (warn_missing_field_initializers
-                 && (!init || BRACE_ENCLOSED_INITIALIZER_P (init)))
-               warning (0, "missing initializer for member %qD", field);
-           }
-         else
+  ce = VEC_index (constructor_elt, CONSTRUCTOR_ELTS (init), 0);
+
+  /* If this element specifies a field, initialize via that field.  */
+  if (ce->index)
+    {
+      if (TREE_CODE (ce->index) == FIELD_DECL)
+       ;
+      else if (TREE_CODE (ce->index) == IDENTIFIER_NODE)
+       {
+         /* This can happen within a cast, see g++.dg/opt/cse2.C.  */
+         tree name = ce->index;
+         tree field;
+         for (field = TYPE_FIELDS (type); field; field = TREE_CHAIN (field))
+           if (DECL_NAME (field) == name)
+             break;
+         if (!field)
            {
-             if (TREE_READONLY (field))
-               error ("uninitialized const member %qD", field);
-             else if (CLASSTYPE_READONLY_FIELDS_NEED_INIT (TREE_TYPE (field)))
-               error ("member %qD with uninitialized const fields", field);
-             else if (TREE_CODE (TREE_TYPE (field)) == REFERENCE_TYPE)
-               error ("member %qD is uninitialized reference", field);
-
-             /* Warn when some struct elements are implicitly initialized
-                to zero.  */
-             if (warn_missing_field_initializers
-                 && (!init || BRACE_ENCLOSED_INITIALIZER_P (init)))
-               warning (0, "missing initializer for member %qD", field);
-
-             if (! zero_init_p (TREE_TYPE (field)))
-               next1 = build_zero_init (TREE_TYPE (field),
-                                        /*nelts=*/NULL_TREE,
-                                        /*static_storage_p=*/false);
-             else
-               /* The default zero-initialization is fine for us; don't
-                  add anything to the CONSTRUCTOR.  */
-               continue;
+             error ("no field %qD found in union being initialized", field);
+             ce->value = error_mark_node;
            }
-
-         if (next1 == error_mark_node)
-           erroneous = 1;
-         else if (!TREE_CONSTANT (next1))
-           allconstant = 0;
-         else if (! initializer_constant_valid_p (next1, TREE_TYPE (next1)))
-           allsimple = 0;
-         members = tree_cons (field, next1, members);
+         ce->index = field;
+       }
+      else
+       {
+         gcc_assert (TREE_CODE (ce->index) == INTEGER_CST
+                     || TREE_CODE (ce->index) == RANGE_EXPR);
+         error ("index value instead of field name in union initializer");
+         ce->value = error_mark_node;
        }
     }
-  else if (TREE_CODE (type) == UNION_TYPE
-          /* If the initializer was empty, use default zero initialization.  */
-          && tail)
+  else
     {
-      tree field = TYPE_FIELDS (type);
-
       /* Find the first named field.  ANSI decided in September 1990
         that only named fields count here.  */
+      tree field = TYPE_FIELDS (type);
       while (field && (!DECL_NAME (field) || TREE_CODE (field) != FIELD_DECL))
        field = TREE_CHAIN (field);
-
-      /* If this element specifies a field, initialize via that field.  */
-      if (TREE_PURPOSE (tail) != NULL_TREE)
-       {
-         int win = 0;
-
-         if (TREE_CODE (TREE_PURPOSE (tail)) == FIELD_DECL)
-           /* Handle the case of a call by build_c_cast.  */
-           field = TREE_PURPOSE (tail), win = 1;
-         else if (TREE_CODE (TREE_PURPOSE (tail)) != IDENTIFIER_NODE)
-           error ("index value instead of field name in union initializer");
-         else
-           {
-             tree temp;
-             for (temp = TYPE_FIELDS (type);
-                  temp;
-                  temp = TREE_CHAIN (temp))
-               if (DECL_NAME (temp) == TREE_PURPOSE (tail))
-                 break;
-             if (temp)
-               field = temp, win = 1;
-             else
-               error ("no field %qD in union being initialized",
-                       TREE_PURPOSE (tail));
-           }
-         if (!win)
-           TREE_VALUE (tail) = error_mark_node;
-       }
-      else if (field == 0)
+      if (field == NULL_TREE)
        {
-         error ("union %qT with no named members cannot be initialized",
-                 type);
-         TREE_VALUE (tail) = error_mark_node;
+         error ("too many initializers for %qT", type);
+         ce->value = error_mark_node;
        }
+      ce->index = field;
+    }
 
-      if (TREE_VALUE (tail) != 0)
-       {
-         tree tail1 = tail;
+  if (ce->value && ce->value != error_mark_node)
+    ce->value = digest_init_r (TREE_TYPE (ce->index), ce->value, true, LOOKUP_IMPLICIT);
 
-         next1 = digest_init (TREE_TYPE (field),
-                              TREE_VALUE (tail), &tail1);
-         gcc_assert (!tail1 || TREE_CODE (tail1) == TREE_LIST);
-         tail = tail1;
-       }
-      else
-       {
-         next1 = error_mark_node;
-         tail = TREE_CHAIN (tail);
-       }
+  return picflag_from_initializer (ce->value);
+}
 
-      if (next1 == error_mark_node)
-       erroneous = 1;
-      else if (!TREE_CONSTANT (next1))
-       allconstant = 0;
-      else if (initializer_constant_valid_p (next1, TREE_TYPE (next1)) == 0)
-       allsimple = 0;
-      members = tree_cons (field, next1, members);
-    }
+/* Process INIT, a constructor for a variable of aggregate type TYPE. The
+   constructor is a brace-enclosed initializer, and will be modified in-place.
 
-  /* If arguments were specified as a list, just remove the ones we used.  */
-  if (elts)
-    *elts = tail;
-  /* If arguments were specified as a constructor,
-     complain unless we used all the elements of the constructor.  */
-  else if (tail)
-    pedwarn ("excess elements in aggregate initializer");
+   Each element is converted to the right type through digest_init, and
+   missing initializers are added following the language rules (zero-padding,
+   etc.).
 
-  if (erroneous)
+   After the execution, the initializer will have TREE_CONSTANT if all elts are
+   constant, and TREE_STATIC set if, in addition, all elts are simple enough
+   constants that the assembler and linker can compute them.
+
+   The function returns the initializer itself, or error_mark_node in case
+   of error.  */
+
+static tree
+process_init_constructor (tree type, tree init)
+{
+  int flags;
+
+  gcc_assert (BRACE_ENCLOSED_INITIALIZER_P (init));
+
+  if (TREE_CODE (type) == ARRAY_TYPE || TREE_CODE (type) == VECTOR_TYPE)
+    flags = process_init_constructor_array (type, init);
+  else if (TREE_CODE (type) == RECORD_TYPE)
+    flags = process_init_constructor_record (type, init);
+  else if (TREE_CODE (type) == UNION_TYPE)
+    flags = process_init_constructor_union (type, init);
+  else
+    gcc_unreachable ();
+
+  if (flags & PICFLAG_ERRONEOUS)
     return error_mark_node;
 
-  result = build_constructor (type, nreverse (members));
+  TREE_TYPE (init) = type;
   if (TREE_CODE (type) == ARRAY_TYPE && TYPE_DOMAIN (type) == NULL_TREE)
-    cp_complete_array_type (&TREE_TYPE (result), result, /*do_default=*/0);
-  if (init)
-    TREE_HAS_CONSTRUCTOR (result) = TREE_HAS_CONSTRUCTOR (init);
-  if (allconstant)
+    cp_complete_array_type (&TREE_TYPE (init), init, /*do_default=*/0);
+  if (!(flags & PICFLAG_NOT_ALL_CONSTANT))
     {
-      TREE_CONSTANT (result) = 1;
-      TREE_INVARIANT (result) = 1;
-      if (allsimple)
-       TREE_STATIC (result) = 1;
+      TREE_CONSTANT (init) = 1;
+      if (!(flags & PICFLAG_NOT_ALL_SIMPLE))
+       TREE_STATIC (init) = 1;
     }
-  return result;
+  return init;
 }
 \f
 /* Given a structure or union value DATUM, construct and return
@@ -1199,9 +1418,9 @@ tree
 build_x_arrow (tree expr)
 {
   tree orig_expr = expr;
-  tree types_memoized = NULL_TREE;
   tree type = TREE_TYPE (expr);
   tree last_rval = NULL_TREE;
+  VEC(tree,gc) *types_memoized = NULL;
 
   if (type == error_mark_node)
     return error_mark_node;
@@ -1213,25 +1432,23 @@ build_x_arrow (tree expr)
       expr = build_non_dependent_expr (expr);
     }
 
-  if (IS_AGGR_TYPE (type))
+  if (MAYBE_CLASS_TYPE_P (type))
     {
       while ((expr = build_new_op (COMPONENT_REF, LOOKUP_NORMAL, expr,
                                   NULL_TREE, NULL_TREE,
-                                  /*overloaded_p=*/NULL)))
+                                  /*overloaded_p=*/NULL, 
+                                  tf_warning_or_error)))
        {
          if (expr == error_mark_node)
            return error_mark_node;
 
-         if (value_member (TREE_TYPE (expr), types_memoized))
+         if (vec_member (TREE_TYPE (expr), types_memoized))
            {
              error ("circular pointer delegation detected");
              return error_mark_node;
            }
-         else
-           {
-             types_memoized = tree_cons (NULL_TREE, TREE_TYPE (expr),
-                                         types_memoized);
-           }
+
+         VEC_safe_push (tree, gc, types_memoized, TREE_TYPE (expr));
          last_rval = expr;
        }
 
@@ -1257,7 +1474,7 @@ build_x_arrow (tree expr)
          return expr;
        }
 
-      return build_indirect_ref (last_rval, NULL);
+      return cp_build_indirect_ref (last_rval, RO_NULL, tf_warning_or_error);
     }
 
   if (types_memoized)
@@ -1279,26 +1496,27 @@ build_m_component_ref (tree datum, tree component)
   tree binfo;
   tree ctype;
 
-  datum = decay_conversion (datum);
-
-  if (datum == error_mark_node || component == error_mark_node)
+  if (error_operand_p (datum) || error_operand_p (component))
     return error_mark_node;
 
+  datum = mark_lvalue_use (datum);
+  component = mark_rvalue_use (component);
+
   ptrmem_type = TREE_TYPE (component);
   if (!TYPE_PTR_TO_MEMBER_P (ptrmem_type))
     {
       error ("%qE cannot be used as a member pointer, since it is of "
-             "type %qT",
+            "type %qT",
             component, ptrmem_type);
       return error_mark_node;
     }
 
   objtype = TYPE_MAIN_VARIANT (TREE_TYPE (datum));
-  if (! IS_AGGR_TYPE (objtype))
+  if (! MAYBE_CLASS_TYPE_P (objtype))
     {
       error ("cannot apply member pointer %qE to %qE, which is of "
-             "non-aggregate type %qT",
-             component, datum, objtype);
+            "non-class type %qT",
+            component, datum, objtype);
       return error_mark_node;
     }
 
@@ -1319,7 +1537,7 @@ build_m_component_ref (tree datum, tree component)
        {
        mismatch:
          error ("pointer to member type %qT incompatible with object "
-                 "type %qT",
+                "type %qT",
                 type, objtype);
          return error_mark_node;
        }
@@ -1329,6 +1547,8 @@ build_m_component_ref (tree datum, tree component)
 
   if (TYPE_PTRMEM_P (ptrmem_type))
     {
+      tree ptype;
+
       /* Compute the type of the field, as described in [expr.ref].
         There's no such thing as a mutable pointer-to-member, so
         things are not as complex as they are for references to
@@ -1345,9 +1565,11 @@ build_m_component_ref (tree datum, tree component)
 
       /* Build an expression for "object + offset" where offset is the
         value stored in the pointer-to-data-member.  */
-      datum = build2 (PLUS_EXPR, build_pointer_type (type),
-                     datum, build_nop (ptrdiff_type_node, component));
-      return build_indirect_ref (datum, 0);
+      ptype = build_pointer_type (type);
+      datum = build2 (POINTER_PLUS_EXPR, ptype,
+                     fold_convert (ptype, datum),
+                     build_nop (sizetype, component));
+      return cp_build_indirect_ref (datum, RO_NULL, tf_warning_or_error);
     }
   else
     return build2 (OFFSET_REF, type, datum, component);
@@ -1356,11 +1578,14 @@ build_m_component_ref (tree datum, tree component)
 /* Return a tree node for the expression TYPENAME '(' PARMS ')'.  */
 
 tree
-build_functional_cast (tree exp, tree parms)
+build_functional_cast (tree exp, tree parms, tsubst_flags_t complain)
 {
   /* This is either a call to a constructor,
      or a C cast in C++'s `functional' notation.  */
+
+  /* The type to which we are casting.  */
   tree type;
+  VEC(tree,gc) *parmvec;
 
   if (exp == error_mark_node || parms == error_mark_node)
     return error_mark_node;
@@ -1370,6 +1595,12 @@ build_functional_cast (tree exp, tree parms)
   else
     type = exp;
 
+  if (TREE_CODE (type) == REFERENCE_TYPE && !parms)
+    {
+      error ("invalid value-initialization of reference type");
+      return error_mark_node;
+    }
+
   if (processing_template_decl)
     {
       tree t = build_min (CAST_EXPR, type, parms);
@@ -1378,15 +1609,14 @@ build_functional_cast (tree exp, tree parms)
       return t;
     }
 
-  if (! IS_AGGR_TYPE (type))
+  if (! MAYBE_CLASS_TYPE_P (type))
     {
-      /* This must build a C cast.  */
       if (parms == NULL_TREE)
-       parms = integer_zero_node;
-      else
-       parms = build_x_compound_expr_from_list (parms, "functional cast");
+       return cp_convert (type, integer_zero_node);
 
-      return build_c_cast (type, parms);
+      /* This must build a C cast.  */
+      parms = build_x_compound_expr_from_list (parms, ELK_FUNC_CAST, complain);
+      return cp_build_c_cast (type, parms, complain);
     }
 
   /* Prepare to evaluate as a call to a constructor.  If this expression
@@ -1396,25 +1626,46 @@ build_functional_cast (tree exp, tree parms)
 
      then the slot being initialized will be filled in.  */
 
-  if (!complete_type_or_else (type, NULL_TREE))
+  if (!complete_type_or_maybe_complain (type, NULL_TREE, complain))
     return error_mark_node;
   if (abstract_virtuals_error (NULL_TREE, type))
     return error_mark_node;
 
+  /* [expr.type.conv]
+
+     If the expression list is a single-expression, the type
+     conversion is equivalent (in definedness, and if defined in
+     meaning) to the corresponding cast expression.  */
   if (parms && TREE_CHAIN (parms) == NULL_TREE)
-    return build_c_cast (type, TREE_VALUE (parms));
+    return cp_build_c_cast (type, TREE_VALUE (parms), complain);
 
-  /* We need to zero-initialize POD types.  Let's do that for everything
-     that doesn't need a constructor.  */
-  if (parms == NULL_TREE && !TYPE_NEEDS_CONSTRUCTING (type)
-      && TYPE_HAS_DEFAULT_CONSTRUCTOR (type))
+  /* [expr.type.conv]
+
+     The expression T(), where T is a simple-type-specifier for a
+     non-array complete object type or the (possibly cv-qualified)
+     void type, creates an rvalue of the specified type, which is
+     value-initialized.  */
+
+  if (parms == NULL_TREE
+      /* If there's a user-defined constructor, value-initialization is
+        just calling the constructor, so fall through.  */
+      && !TYPE_HAS_USER_CONSTRUCTOR (type))
     {
-      exp = build_constructor (type, NULL_TREE);
-      return get_target_expr (exp);
+      exp = build_value_init (type, complain);
+      exp = get_target_expr (exp);
+      /* FIXME this is wrong */
+      if (literal_type_p (type))
+       TREE_CONSTANT (exp) = true;
+      return exp;
     }
 
-  exp = build_special_member_call (NULL_TREE, complete_ctor_identifier, parms,
-                                  type, LOOKUP_NORMAL);
+  /* Call the constructor.  */
+  parmvec = make_tree_vector ();
+  for (; parms != NULL_TREE; parms = TREE_CHAIN (parms))
+    VEC_safe_push (tree, gc, parmvec, TREE_VALUE (parms));
+  exp = build_special_member_call (NULL_TREE, complete_ctor_identifier,
+                                  &parmvec, type, LOOKUP_NORMAL, complain);
+  release_tree_vector (parmvec);
 
   if (exp == error_mark_node)
     return error_mark_node;
@@ -1434,7 +1685,7 @@ add_exception_specifier (tree list, tree spec, int complain)
   bool ok;
   tree core = spec;
   bool is_ptr;
-  int diag_type = -1; /* none */
+  diagnostic_t diag_type = DK_UNSPECIFIED; /* none */
 
   if (spec == error_mark_node)
     return list;
@@ -1459,11 +1710,11 @@ add_exception_specifier (tree list, tree spec, int complain)
     {
       ok = true;
       /* 15.4/1 says that types in an exception specifier must be complete,
-         but it seems more reasonable to only require this on definitions
-         and calls.  So just give a pedwarn at this point; we will give an
-         error later if we hit one of those two cases.  */
+        but it seems more reasonable to only require this on definitions
+        and calls.  So just give a pedwarn at this point; we will give an
+        error later if we hit one of those two cases.  */
       if (!COMPLETE_TYPE_P (complete_type (core)))
-       diag_type = 2; /* pedwarn */
+       diag_type = DK_PEDWARN; /* pedwarn */
     }
 
   if (ok)
@@ -1471,15 +1722,15 @@ add_exception_specifier (tree list, tree spec, int complain)
       tree probe;
 
       for (probe = list; probe; probe = TREE_CHAIN (probe))
-        if (same_type_p (TREE_VALUE (probe), spec))
-          break;
+       if (same_type_p (TREE_VALUE (probe), spec))
+         break;
       if (!probe)
        list = tree_cons (NULL_TREE, spec, list);
     }
   else
-    diag_type = 0; /* error */
+    diag_type = DK_ERROR; /* error */
 
-  if (diag_type >= 0 && complain)
+  if (diag_type != DK_UNSPECIFIED && complain)
     cxx_incomplete_type_diagnostic (NULL_TREE, core, diag_type);
 
   return list;
@@ -1493,29 +1744,33 @@ merge_exception_specifiers (tree list, tree add)
 {
   if (!list || !add)
     return NULL_TREE;
-  else if (!TREE_VALUE (list))
-    return add;
+  /* For merging noexcept(true) and throw(), take the more recent one (LIST).
+     A throw(type-list) spec takes precedence over a noexcept(false) spec.
+     Any other noexcept-spec should only be merged with an equivalent one.
+     So the !TREE_VALUE code below is correct for all cases.  */
   else if (!TREE_VALUE (add))
     return list;
+  else if (!TREE_VALUE (list))
+    return add;
   else
     {
       tree orig_list = list;
 
       for (; add; add = TREE_CHAIN (add))
-        {
-          tree spec = TREE_VALUE (add);
-          tree probe;
-
-          for (probe = orig_list; probe; probe = TREE_CHAIN (probe))
-            if (same_type_p (TREE_VALUE (probe), spec))
-              break;
-          if (!probe)
-            {
-              spec = build_tree_list (NULL_TREE, spec);
-              TREE_CHAIN (spec) = list;
-              list = spec;
-            }
-        }
+       {
+         tree spec = TREE_VALUE (add);
+         tree probe;
+
+         for (probe = orig_list; probe; probe = TREE_CHAIN (probe))
+           if (same_type_p (TREE_VALUE (probe), spec))
+             break;
+         if (!probe)
+           {
+             spec = build_tree_list (NULL_TREE, spec);
+             TREE_CHAIN (spec) = list;
+             list = spec;
+           }
+       }
     }
   return list;
 }