[C++] PR c++/90101 - dependent class non-type parameter.

Message ID 20190719072859.10788-1-jason@redhat.com
State New
Headers show
Series
  • [C++] PR c++/90101 - dependent class non-type parameter.
Related show

Commit Message

Jason Merrill July 19, 2019, 7:28 a.m.
We shouldn't complain that a dependent type is incomplete.

Tested x86_64-pc-linux-gnu, applying to trunk.

	* pt.c (invalid_nontype_parm_type_p): Check for dependent class type.
---
 gcc/cp/pt.c                                  |  2 ++
 gcc/testsuite/g++.dg/cpp2a/nontype-class21.C | 10 ++++++++++
 gcc/testsuite/g++.dg/cpp2a/nontype-class22.C | 21 ++++++++++++++++++++
 gcc/cp/ChangeLog                             |  5 +++++
 4 files changed, 38 insertions(+)
 create mode 100644 gcc/testsuite/g++.dg/cpp2a/nontype-class21.C
 create mode 100644 gcc/testsuite/g++.dg/cpp2a/nontype-class22.C


base-commit: d4f6160a8abbf666fc86fc9c60763b4b93da8fb9
-- 
2.21.0

Patch

diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c
index 53aaad1800a..e433413827a 100644
--- a/gcc/cp/pt.c
+++ b/gcc/cp/pt.c
@@ -25228,6 +25228,8 @@  invalid_nontype_parm_type_p (tree type, tsubst_flags_t complain)
 		 "with %<-std=c++2a%> or %<-std=gnu++2a%>");
 	  return true;
 	}
+      if (dependent_type_p (type))
+	return false;
       if (!complete_type_or_else (type, NULL_TREE))
 	return true;
       if (!literal_type_p (type))
diff --git a/gcc/testsuite/g++.dg/cpp2a/nontype-class21.C b/gcc/testsuite/g++.dg/cpp2a/nontype-class21.C
new file mode 100644
index 00000000000..c58fe05b9dd
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp2a/nontype-class21.C
@@ -0,0 +1,10 @@ 
+// PR c++/90101
+// { dg-do compile { target c++2a } }
+
+template<int N>
+struct A{};
+
+template<int N, A<N>>
+struct B {};
+
+B<2,A<2>{}> b;
diff --git a/gcc/testsuite/g++.dg/cpp2a/nontype-class22.C b/gcc/testsuite/g++.dg/cpp2a/nontype-class22.C
new file mode 100644
index 00000000000..026855f0bc6
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp2a/nontype-class22.C
@@ -0,0 +1,21 @@ 
+// PR c++/90100
+// { dg-do compile { target c++2a } }
+
+template<typename T>
+inline constexpr bool is_nontype_list = false;
+
+template<template<auto...> typename T, auto... NonTypes>
+inline constexpr bool is_nontype_list<T<NonTypes...>> = true;
+
+// works
+template<auto...>
+struct A {};
+
+static_assert(is_nontype_list<A<1, 2, 3>>);
+
+// fails
+struct X {
+    int v;
+};
+
+static_assert(is_nontype_list<A<X{1}, X{2}, X{3}>>);
diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index cef36b2d1b2..c1fc980e287 100644
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,8 @@ 
+2019-07-19  Jason Merrill  <jason@redhat.com>
+
+	PR c++/90101 - dependent class non-type parameter.
+	* pt.c (invalid_nontype_parm_type_p): Check for dependent class type.
+
 2019-07-18  Jason Merrill  <jason@redhat.com>
 
 	PR c++/90098 - partial specialization and class non-type parms.