From ac9face8d26ea4b6aa72902ecc22e89ef00763c5 Mon Sep 17 00:00:00 2001 From: Nathan Sidwell Date: Wed, 27 May 2020 09:50:15 -0400 Subject: [PATCH] c++: Revert alias template change [pr95263] Turns out templates are more complicated than you think, even when you know they are more complicated than you think. Reverting this change. PR c++/95263 * pt.c (lookup_template_class_1): Restore alias template mutation. --- gcc/cp/pt.c | 17 ++--------------- gcc/testsuite/g++.dg/template/pr95263.C | 23 +++++++++++++++++++++++ 2 files changed, 25 insertions(+), 15 deletions(-) create mode 100644 gcc/testsuite/g++.dg/template/pr95263.C diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c index c17a038c6d0..4d9651acee6 100644 --- a/gcc/cp/pt.c +++ b/gcc/cp/pt.c @@ -10062,21 +10062,8 @@ lookup_template_class_1 (tree d1, tree arglist, tree in_decl, tree context, } } - /* Build template info for the new specialization. This can - overwrite the existing TEMPLATE_INFO for T (that points to - its instantiated TEMPLATE_DECL), with this one that points to - the most general template, but that's what we want. */ - - if (TYPE_ALIAS_P (t)) - { - /* This should already have been constructed during - instantiation of the alias decl. */ - tree ti = DECL_TEMPLATE_INFO (TYPE_NAME (t)); - gcc_checking_assert (template_args_equal (TI_ARGS (ti), arglist) - && TI_TEMPLATE (ti) == found); - } - else - SET_TYPE_TEMPLATE_INFO (t, build_template_info (found, arglist)); + // Build template info for the new specialization. + SET_TYPE_TEMPLATE_INFO (t, build_template_info (found, arglist)); elt.spec = t; slot = type_specializations->find_slot_with_hash (&elt, hash, INSERT); diff --git a/gcc/testsuite/g++.dg/template/pr95263.C b/gcc/testsuite/g++.dg/template/pr95263.C new file mode 100644 index 00000000000..08a1b8730c0 --- /dev/null +++ b/gcc/testsuite/g++.dg/template/pr95263.C @@ -0,0 +1,23 @@ +// { dg-do compile { target c++11 } } +// PR C++/95263 +// ICE on alias template instantiation + +template class TPL { + template using INT = int; +}; + +template class Klass +{ +public: + template using ALIAS = typename TPL::INT; + + template static void FUNC (); // OK + + template static ALIAS FUNC (); // SFINAE ICE +}; + +void Fn () +{ + Klass::FUNC<0> (); +} + -- 2.30.2