C++ PATCH for c++/70778 (ICE with member template template parameter)

Jason Merrill jason@redhat.com
Sun Jul 24 02:56:00 GMT 2016


On Sat, Jul 23, 2016 at 10:50 PM, Jason Merrill <jason@redhat.com> wrote:
> Here when we partially instantiated the declaration of the AddToFront
> partial specialization, which meant substituting into
> "Holder<Indexs>", we weren't replacing KeyType in the template
> parameter list of Holder, leading to confusion when we encounter it
> later and try to substitute in 0.
>
> Tested x86_64-pc-linux-gnu, applying to trunk.
-------------- next part --------------
commit fa7e4de043897f89467887f84e705cdd3a0cfb79
Author: Jason Merrill <jason@redhat.com>
Date:   Fri Jul 22 16:42:52 2016 -0400

    	PR c++/70778 - member template template parameter
    
    	* pt.c (tsubst): Also substitute into the template of a
    	BOUND_TEMPLATE_TEMPLATE_PARM.

diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c
index a44bead..65fa982 100644
--- a/gcc/cp/pt.c
+++ b/gcc/cp/pt.c
@@ -13232,13 +13232,20 @@ tsubst (tree t, tree args, tsubst_flags_t complain, tree in_decl)
 
 		if (code == BOUND_TEMPLATE_TEMPLATE_PARM)
 		  {
-		    tree argvec = tsubst (TYPE_TI_ARGS (t), args,
+		    tree tinfo = TYPE_TEMPLATE_INFO (t);
+		    /* We might need to substitute into the types of non-type
+		       template parameters.  */
+		    tree tmpl = tsubst (TI_TEMPLATE (tinfo), args,
+					complain, in_decl);
+		    if (tmpl == error_mark_node)
+		      return error_mark_node;
+		    tree argvec = tsubst (TI_ARGS (tinfo), args,
 					  complain, in_decl);
 		    if (argvec == error_mark_node)
 		      return error_mark_node;
 
 		    TEMPLATE_TEMPLATE_PARM_TEMPLATE_INFO (r)
-		      = build_template_info (TYPE_TI_TEMPLATE (t), argvec);
+		      = build_template_info (tmpl, argvec);
 		  }
 	      }
 	    break;
diff --git a/gcc/testsuite/g++.dg/template/ttp29.C b/gcc/testsuite/g++.dg/template/ttp29.C
new file mode 100644
index 0000000..7d4e03a
--- /dev/null
+++ b/gcc/testsuite/g++.dg/template/ttp29.C
@@ -0,0 +1,21 @@
+// PR c++/70778
+
+template <class KeyType>
+struct Stuff
+{
+  template <KeyType, class>
+  struct AddToFront;
+
+  template <KeyType ToAdd, template<KeyType> class Holder, KeyType Indexs>
+  struct AddToFront<ToAdd, Holder<Indexs> >
+  {
+  };
+};
+
+template <unsigned>
+struct Holder {};
+
+int main()
+{
+  Stuff<unsigned>::AddToFront<0, Holder<24> > t;
+}


More information about the Gcc-patches mailing list