c++: zero_init_expr_p of dependent expression

This fixes an ICE coming from mangle.c:write_expression when building the
testsuite of range-v3; the added testcase is a reduced reproducer for the ICE.

gcc/cp/ChangeLog:

	* tree.c (zero_init_expr_p): Use uses_template_parms instead of
	dependent_type_p.

gcc/testsuite/ChangeLog:

	* g++.dg/cpp0x/dependent3.C: New test.

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 6506f288e0598f54882df9b12fa5722996f85e78..043e6aaddf49295684ec28bf9d56e30c88a7f9a4 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,5 +1,8 @@
 2020-04-23  Patrick Palka  <ppalka@redhat.com>
 
+       * tree.c (zero_init_expr_p): Use uses_template_parms instead of
+       dependent_type_p.
+
        PR c++/94645
        * pt.c (template_class_depth): Walk into the DECL_FRIEND_CONTEXT of a
        friend declaration rather than into its CP_DECL_CONTEXT.

diff --git a/gcc/cp/tree.c b/gcc/cp/tree.c
index 090c565c093436f0d462c0fb7f4b9226f36ecca4..8840932dba255f74869a5034c2beaea17b47e80d 100644 (file)
--- a/gcc/cp/tree.c
+++ b/gcc/cp/tree.c
@@ -4486,7 +4486,7 @@ bool
 zero_init_expr_p (tree t)
 {
   tree type = TREE_TYPE (t);
-  if (!type || dependent_type_p (type))
+  if (!type || uses_template_parms (type))
     return false;
   if (zero_init_p (type))
     return initializer_zerop (t);

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index e0df5bc0a99b0eaa74c9a202257d75c1dc79e310..7c6fc644332bcea35f94bf2f8a21808b37dde346 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,5 +1,7 @@
 2020-04-23  Patrick Palka  <ppalka@redhat.com>
 
+       * g++.dg/cpp0x/dependent3.C: New test.
+
        PR c++/94645
        * g++.dg/cpp2a/concepts-lambda6.C: New test.
 

diff --git a/gcc/testsuite/g++.dg/cpp0x/dependent3.C b/gcc/testsuite/g++.dg/cpp0x/dependent3.C
new file mode 100644 (file)
index 0000000..caf7e1c
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp0x/dependent3.C
@@ -0,0 +1,28 @@
+// { dg-do compile { target c++11 } }
+
+template<typename c>
+struct d
+{
+  using e = c;
+};
+
+template<class f>
+struct g
+{
+  using h = typename d<f>::e;
+
+  template<class i, class j>
+  auto operator()(i, j k) -> decltype(h{k});
+};
+
+template<class l>
+void m()
+{
+  int a[1];
+  l{}(a, a);
+}
+
+int main()
+{
+  m<g<int *>>();
+}