From: Jason Merrill Date: Tue, 25 Jan 2022 20:12:16 +0000 (-0500) Subject: c++: alias template and typename [PR103057] X-Git-Tag: basepoints/gcc-13~1441 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f3e6ef7d873885ffaa0db7f4185364bbd7a70505;p=thirdparty%2Fgcc.git c++: alias template and typename [PR103057] Usually we handle DR1558 substitution near the top of tsubst, but in this case while substituting TYPENAME_TYPE we were passing an alias specialization to tsubst_aggr_type, which ignored its aliasness. PR c++/103057 gcc/cp/ChangeLog: * pt.cc (tsubst_aggr_type): Call tsubst for alias template specialization. gcc/testsuite/ChangeLog: * g++.dg/cpp0x/alias-decl-void1.C: New test. --- diff --git a/gcc/cp/pt.cc b/gcc/cp/pt.cc index 8f50b9c4d3c2..6fbda676527a 100644 --- a/gcc/cp/pt.cc +++ b/gcc/cp/pt.cc @@ -13585,6 +13585,14 @@ tsubst_aggr_type (tree t, if (t == NULL_TREE) return NULL_TREE; + /* If T is an alias template specialization, we want to substitute that + rather than strip it, especially if it's dependent_alias_template_spec_p. + It should be OK not to handle entering_scope in this case, since + DECL_CONTEXT will never be an alias template specialization. We only get + here with an alias when tsubst calls us for TYPENAME_TYPE. */ + if (alias_template_specialization_p (t, nt_transparent)) + return tsubst (t, args, complain, in_decl); + switch (TREE_CODE (t)) { case RECORD_TYPE: diff --git a/gcc/testsuite/g++.dg/cpp0x/alias-decl-void1.C b/gcc/testsuite/g++.dg/cpp0x/alias-decl-void1.C new file mode 100644 index 000000000000..accc1a45abcc --- /dev/null +++ b/gcc/testsuite/g++.dg/cpp0x/alias-decl-void1.C @@ -0,0 +1,18 @@ +// PR c++/103057 +// { dg-do compile { target c++11 } } + +template struct A { }; +template struct B { using type = A; }; +template struct C { + using type = typename T::foo; // { dg-error "int" } +}; +template using L = B; + +template +typename L::type>::type +f(T) { }; + +int main() +{ + f(42); // { dg-error "no match" } +}