PR c++/91353 - P1331R2: Allow trivial default init in constexpr contexts.

This patch implements C++20 P1331, allowing trivial default initialization in
constexpr contexts.

	* c-cppbuiltin.c (c_cpp_builtins): Adjust the value of __cpp_constexpr.

	* class.c (trivial_default_constructor_is_constexpr): Return true in
	C++20.
	* constexpr.c (cx_check_missing_mem_inits): Allow missing field
	initializers in C++20.
	(cxx_eval_call_expression): Don't clear CONSTRUCTOR_NO_CLEARING for
	constexpr constructors in C++20.
	(reduced_constant_expression_p): Don't set FIELD for union and array
	types.  Skip empty class fields without initializers.
	* decl.c (check_for_uninitialized_const_var): Permit trivial default
	initialization in constexpr.
	(next_initializable_field): Don't skip vptr fields.
	* method.c (walk_field_subobs): Still consider a constructor that
	doesn't initialize all the members constexpr.

	* g++.dg/cpp0x/constexpr-array6.C: Adjust dg-error.
	* g++.dg/cpp0x/constexpr-ctor.C: Likewise.
	* g++.dg/cpp0x/constexpr-diag3.C: Likewise.
	* g++.dg/cpp0x/constexpr-diag4.C: Likewise.
	* g++.dg/cpp0x/constexpr-ex3.C: Likewise.
	* g++.dg/cpp0x/constexpr-template2.C: Likewise.
	* g++.dg/cpp0x/constexpr-union2.C: Likewise.
	* g++.dg/cpp0x/lambda/lambda-mangle.C: Rip out a piece of code ...
	* g++.dg/cpp0x/lambda/lambda-mangle6.C: ... and put it here.
	* g++.dg/cpp0x/pr79118.C: Adjust dg-error.
	* g++.dg/cpp1y/constexpr-83921-3.C: Likewise.
	* g++.dg/cpp1y/constexpr-neg1.C: Likewise.
	* g++.dg/cpp1z/constexpr-lambda12.C: Likewise.
	* g++.dg/cpp1z/feat-cxx1z.C: Use -std=c++17.
	* g++.dg/cpp2a/constexpr-init1.C: New test.
	* g++.dg/cpp2a/constexpr-init2.C: New test.
	* g++.dg/cpp2a/constexpr-init3.C: New test.
	* g++.dg/cpp2a/constexpr-init4.C: New test.
	* g++.dg/cpp2a/constexpr-init5.C: New test.
	* g++.dg/cpp2a/constexpr-init6.C: New test.
	* g++.dg/cpp2a/constexpr-init7.C: New test.
	* g++.dg/cpp2a/constexpr-init8.C: New test.
	* g++.dg/cpp2a/constexpr-init9.C: New test.
	* g++.dg/cpp2a/constexpr-init10.C: New test.
	* g++.dg/cpp2a/constexpr-init11.C: New test.
	* g++.dg/cpp2a/constexpr-init12.C: New test.
	* g++.dg/cpp2a/constexpr-init13.C: New test.
	* g++.dg/cpp2a/constexpr-init14.C: New test.
	* g++.dg/cpp2a/constexpr-init15.C: New test.
	* g++.dg/cpp2a/constexpr-try5.C: Adjust dg-error.
	* g++.dg/cpp2a/feat-cxx2a.C: Test __cpp_constexpr.
	* g++.dg/cpp2a/lambda-mangle.C: New test.
	* g++.dg/debug/dwarf2/pr44641.C: Skip for c++2a.
	* g++.dg/ext/stmtexpr21.C: Adjust dg-error.

Co-Authored-By: Jakub Jelinek <jakub@redhat.com>

From-SVN: r279019

1 files changed, 36 insertions, 9 deletions

diff --git a/gcc/cp/constexpr.c b/gcc/cp/constexpr.c
index 3706334..19e09c7 100644
--- a/gcc/cp/constexpr.c
+++ b/gcc/cp/constexpr.c
@@ -767,6 +767,10 @@ massage_constexpr_body (tree fun, tree body)
 static bool
 cx_check_missing_mem_inits (tree ctype, tree body, bool complain)
 {
+  /* We allow uninitialized bases/fields in C++20.  */
+  if (cxx_dialect >= cxx2a)
+    return false;
+
   unsigned nelts = 0;
   
   if (body)
@@ -815,7 +819,7 @@ cx_check_missing_mem_inits (tree ctype, tree body, bool complain)
 	    continue;
 	  if (ANON_AGGR_TYPE_P (TREE_TYPE (field)))
 	    {
-	      /* Recurse to check the anonummous aggregate member.  */
+	      /* Recurse to check the anonymous aggregate member.  */
 	      bad |= cx_check_missing_mem_inits
 		(TREE_TYPE (field), NULL_TREE, complain);
 	      if (bad && !complain)
@@ -2179,15 +2183,26 @@ cxx_eval_call_expression (const constexpr_ctx *ctx, tree t,
 	entry->result = result;
     }
 
-  /* The result of a constexpr function must be completely initialized.  */
-  if (TREE_CODE (result) == CONSTRUCTOR)
+  /* The result of a constexpr function must be completely initialized.
+
+     However, in C++20, a constexpr constructor doesn't necessarily have
+     to initialize all the fields, so we don't clear CONSTRUCTOR_NO_CLEARING
+     in order to detect reading an unitialized object in constexpr instead
+     of value-initializing it.  (reduced_constant_expression_p is expected to
+     take care of clearing the flag.)  */
+  if (TREE_CODE (result) == CONSTRUCTOR
+      && (cxx_dialect < cxx2a
+	  || !DECL_CONSTRUCTOR_P (fun)))
     clear_no_implicit_zero (result);
 
   pop_cx_call_context ();
   return result;
 }
 
-/* FIXME speed this up, it's taking 16% of compile time on sieve testcase.  */
+/* Return true if T is a valid constant initializer.  If a CONSTRUCTOR
+   initializes all the members, the CONSTRUCTOR_NO_CLEARING flag will be
+   cleared.
+   FIXME speed this up, it's taking 16% of compile time on sieve testcase.  */
 
 bool
 reduced_constant_expression_p (tree t)
@@ -2209,6 +2224,12 @@ reduced_constant_expression_p (tree t)
 	  if (TREE_CODE (TREE_TYPE (t)) == VECTOR_TYPE)
 	    /* An initialized vector would have a VECTOR_CST.  */
 	    return false;
+	  else if (cxx_dialect >= cxx2a
+		   /* An ARRAY_TYPE doesn't have any TYPE_FIELDS.  */
+		   && (TREE_CODE (TREE_TYPE (t)) == ARRAY_TYPE
+		       /* A union only initializes one member.  */
+		       || TREE_CODE (TREE_TYPE (t)) == UNION_TYPE))
+	    field = NULL_TREE;
 	  else
 	    field = next_initializable_field (TYPE_FIELDS (TREE_TYPE (t)));
 	}
@@ -2222,14 +2243,20 @@ reduced_constant_expression_p (tree t)
 	    return false;
 	  if (field)
 	    {
-	      if (idx != field)
-		return false;
+	      /* Empty class field may or may not have an initializer.  */
+	      for (; idx != field;
+		   field = next_initializable_field (DECL_CHAIN (field)))
+		if (!is_really_empty_class (TREE_TYPE (field),
+					    /*ignore_vptr*/false))
+		  return false;
 	      field = next_initializable_field (DECL_CHAIN (field));
 	    }
 	}
-      if (field)
-	return false;
-      else if (CONSTRUCTOR_NO_CLEARING (t))
+      /* There could be a non-empty field at the end.  */
+      for (; field; field = next_initializable_field (DECL_CHAIN (field)))
+	if (!is_really_empty_class (TREE_TYPE (field), /*ignore_vptr*/false))
+	  return false;
+      if (CONSTRUCTOR_NO_CLEARING (t))
 	/* All the fields are initialized.  */
 	CONSTRUCTOR_NO_CLEARING (t) = false;
       return true;