PR c++/92582 - ICE with member template as requirement.

Here, we weren't recognizing that the template parameter of A is used by the
reference to d in the requires-clause of f.  Fixed by passing down the
active template parameters in the context of normalization, and adding to
the mapping any such parameters shared by a member template used in the
constraint-expression.

	* pt.c (struct find_template_parameter_info): Add ctx_parms.
	(any_template_parm_r): Handle TEMPLATE_DECL.
	(find_template_parameters): Take parms instead of their depth.
	* constraint.cc (build_parameter_mapping): Pass them.

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 146c280..59646c7 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,5 +1,11 @@
 2020-01-13  Jason Merrill  <jason@redhat.com>
 
+       PR c++/92582 - ICE with member template as requirement.
+       * pt.c (struct find_template_parameter_info): Add ctx_parms.
+       (any_template_parm_r): Handle TEMPLATE_DECL.
+       (find_template_parameters): Take parms instead of their depth.
+       * constraint.cc (build_parameter_mapping): Pass them.
+
        PR c++/33799 - destroy return value if local cleanup throws.
        * cp-tree.h (current_retval_sentinel): New macro.
        * decl.c (start_preparsed_function): Set up cleanup for retval.

diff --git a/gcc/cp/constraint.cc b/gcc/cp/constraint.cc
index 11f5807..128ab8a 100644 (file)
--- a/gcc/cp/constraint.cc
+++ b/gcc/cp/constraint.cc
@@ -559,12 +559,11 @@ map_arguments (tree parms, tree args)
 static tree
 build_parameter_mapping (tree expr, tree args, tree decl)
 {
-  int depth = 0;
+  tree ctx_parms = NULL_TREE;
   if (decl)
     {
       gcc_assert (TREE_CODE (decl) == TEMPLATE_DECL);
-      tree parms = DECL_TEMPLATE_PARMS (decl);
-      depth = TREE_INT_CST_LOW (TREE_PURPOSE (parms));
+      ctx_parms = DECL_TEMPLATE_PARMS (decl);
     }
   else if (current_template_parms)
     {
@@ -572,10 +571,10 @@ build_parameter_mapping (tree expr, tree args, tree decl)
         point of declaration of concepts is currently set after the
         initializer, the template parameter lists are not available
         when normalizing concept definitions, hence the case above.  */
-      depth = TMPL_PARMS_DEPTH (current_template_parms);
+      ctx_parms = current_template_parms;
     }
 
-  tree parms = find_template_parameters (expr, depth);
+  tree parms = find_template_parameters (expr, ctx_parms);
   tree map = map_arguments (parms, args);
   return map;
 }

diff --git a/gcc/cp/cp-tree.h b/gcc/cp/cp-tree.h
index c0f780d..01fcf66 100644 (file)
--- a/gcc/cp/cp-tree.h
+++ b/gcc/cp/cp-tree.h
@@ -7865,7 +7865,7 @@ extern bool constraints_satisfied_p               (tree, tree);
 extern void clear_satisfaction_cache           ();
 extern bool* lookup_subsumption_result          (tree, tree);
 extern bool save_subsumption_result             (tree, tree, bool);
-extern tree find_template_parameters           (tree, int);
+extern tree find_template_parameters           (tree, tree);
 extern bool equivalent_constraints              (tree, tree);
 extern bool equivalently_constrained            (tree, tree);
 extern bool subsumes_constraints                (tree, tree);

diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c
index 29c3aec..fa82eca 100644 (file)
--- a/gcc/cp/pt.c
+++ b/gcc/cp/pt.c
@@ -10364,12 +10364,14 @@ for_each_template_parm (tree t, tree_fn_t fn, void* data,
 
 struct find_template_parameter_info
 {
-  explicit find_template_parameter_info (int d)
-    : max_depth (d)
+  explicit find_template_parameter_info (tree ctx_parms)
+    : ctx_parms (ctx_parms),
+      max_depth (TMPL_PARMS_DEPTH (ctx_parms))
   {}
 
   hash_set<tree> visited;
   hash_set<tree> parms;
+  tree ctx_parms;
   int max_depth;
 };
 
@@ -10459,6 +10461,29 @@ any_template_parm_r (tree t, void *data)
       WALK_SUBTREE (TREE_TYPE (t));
       break;
 
+    case TEMPLATE_DECL:
+      {
+       /* If T is a member template that shares template parameters with
+          ctx_parms, we need to mark all those parameters for mapping.  */
+       tree dparms = DECL_TEMPLATE_PARMS (t);
+       tree cparms = ftpi->ctx_parms;
+       while (TMPL_PARMS_DEPTH (dparms) > ftpi->max_depth)
+         dparms = TREE_CHAIN (dparms);
+       while (dparms
+              && (TREE_TYPE (TREE_VALUE (dparms))
+                  != TREE_TYPE (TREE_VALUE (cparms))))
+         dparms = TREE_CHAIN (dparms),
+           cparms = TREE_CHAIN (cparms);
+       if (dparms)
+         {
+           int ddepth = TMPL_PARMS_DEPTH (dparms);
+           tree dargs = TI_ARGS (get_template_info (DECL_TEMPLATE_RESULT (t)));
+           for (int i = 0; i < ddepth; ++i)
+             WALK_SUBTREE (TMPL_ARGS_LEVEL (dargs, i+1));
+         }
+      }
+      break;
+
     default:
       break;
     }
@@ -10467,12 +10492,16 @@ any_template_parm_r (tree t, void *data)
   return 0;
 }
 
-/* Returns a list of unique template parameters found within T.  */
+/* Returns a list of unique template parameters found within T, where CTX_PARMS
+   are the template parameters in scope.  */
 
 tree
-find_template_parameters (tree t, int depth)
+find_template_parameters (tree t, tree ctx_parms)
 {
-  find_template_parameter_info ftpi (depth);
+  if (!ctx_parms)
+    return NULL_TREE;
+
+  find_template_parameter_info ftpi (ctx_parms);
   for_each_template_parm (t, keep_template_parm, &ftpi, &ftpi.visited,
                          /*include_nondeduced*/true, any_template_parm_r);
   tree list = NULL_TREE;

diff --git a/gcc/testsuite/g++.dg/cpp2a/concepts-memtmpl3.C b/gcc/testsuite/g++.dg/cpp2a/concepts-memtmpl3.C
new file mode 100644 (file)
index 0000000..353937b
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp2a/concepts-memtmpl3.C
@@ -0,0 +1,12 @@
+// PR c++/92582
+// { dg-do compile { target concepts } }
+
+template <class a, class> concept b = true;
+template <typename> struct A {
+  template <typename c> static constexpr bool d = b<c, int>;
+  template <typename c> static void f(c) requires d<c>;
+};
+int main()
+{
+  A<void>::f(0);
+}