From: Nathan Sidwell Date: Wed, 27 May 2020 13:50:15 +0000 (-0400) Subject: c++: Revert alias template change [pr95263] X-Git-Tag: upstream/12.2.0~16293 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ac9face8d26ea4b6aa72902ecc22e89ef00763c5;p=platform%2Fupstream%2Fgcc.git 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. --- diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c index c17a038..4d9651a 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 0000000..08a1b87 --- /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> (); +} +