[gcc(refs/vendors/redhat/heads/gcc-9-branch)] c++: Fix value-init crash in template [PR93676]

[Jakub Jelinek]

https://gcc.gnu.org/g:b38d6242be6aeaf83cdf1f990ff3297a697e4488

commit b38d6242be6aeaf83cdf1f990ff3297a697e4488
Author: Marek Polacek <polacek@redhat.com>
Date:   Wed Mar 4 18:57:08 2020 -0500

    c++: Fix value-init crash in template [PR93676]
    
            PR c++/93676 - value-init crash in template.
            * init.c (build_new_1): Don't call build_vec_init in a template.
    
            * g++.dg/cpp0x/nsdmi-template19.C: New test.

Diff:
---
 gcc/cp/ChangeLog                              |  8 ++++++++
 gcc/cp/init.c                                 |  8 ++++++--
 gcc/testsuite/g++.dg/cpp0x/nsdmi-template19.C | 13 +++++++++++++
 3 files changed, 27 insertions(+), 2 deletions(-)

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index cb5211e5789..26d9a08d9ef 100644
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,11 @@
+2020-03-04  Marek Polacek  <polacek@redhat.com>
+
+	Backported from mainline
+	2020-02-26  Marek Polacek  <polacek@redhat.com>
+
+	PR c++/93676 - value-init crash in template.
+	* init.c (build_new_1): Don't call build_vec_init in a template.
+
 2020-03-04  Jason Merrill  <jason@redhat.com>
 	    Marek Polacek  <polacek@redhat.com>
 
diff --git a/gcc/cp/init.c b/gcc/cp/init.c
index 44ddc8cd144..fcc91fc9172 100644
--- a/gcc/cp/init.c
+++ b/gcc/cp/init.c
@@ -3448,13 +3448,17 @@ build_new_1 (vec<tree, va_gc> **placement, tree type, tree nelts,
 	  explicit_value_init_p = true;
 	}
 
-      if (processing_template_decl && explicit_value_init_p)
+      if (processing_template_decl)
 	{
+	  /* Avoid an ICE when converting to a base in build_simple_base_path.
+	     We'll throw this all away anyway, and build_new will create
+	     a NEW_EXPR.  */
+	  tree t = fold_convert (build_pointer_type (elt_type), data_addr);
 	  /* build_value_init doesn't work in templates, and we don't need
 	     the initializer anyway since we're going to throw it away and
 	     rebuild it at instantiation time, so just build up a single
 	     constructor call to get any appropriate diagnostics.  */
-	  init_expr = cp_build_fold_indirect_ref (data_addr);
+	  init_expr = cp_build_fold_indirect_ref (t);
 	  if (type_build_ctor_call (elt_type))
 	    init_expr = build_special_member_call (init_expr,
 						   complete_ctor_identifier,
diff --git a/gcc/testsuite/g++.dg/cpp0x/nsdmi-template19.C b/gcc/testsuite/g++.dg/cpp0x/nsdmi-template19.C
new file mode 100644
index 00000000000..f3e2cb87fd6
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp0x/nsdmi-template19.C
@@ -0,0 +1,13 @@
+// PR c++/93676 - value-init crash in template.
+// { dg-do compile { target c++11 } }
+
+struct P {
+  int x = 0;
+};
+
+template<class T>
+struct S {
+  S() { new P[2][2]; }
+};
+
+S<int> s;