https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96204

--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Patrick Palka <ppa...@gcc.gnu.org>:

https://gcc.gnu.org/g:9f26e34a5a9614a5b66f146752ecef9ea67b3e2d

commit r12-1829-g9f26e34a5a9614a5b66f146752ecef9ea67b3e2d
Author: Patrick Palka <ppa...@redhat.com>
Date:   Sat Jun 26 11:05:54 2021 -0400

    c++: access scope during partial spec matching [PR96204]

    Here, when determining whether the partial specialization matches
    has_type_member<Child>, we do so from the scope of where the template-id
    appears rather than from the scope of the specialization, and this
    causes us to select the partial specialization (since Child::type is
    accessible from Parent).  When we later instantiate this partial
    specialization, we've entered the scope of the specialization and so
    substitution into e.g. the DECL_CONTEXT of has_type_member::value fails
    with access errors since the friend declaration that we relied on to
    choose the partial specialization no longer applies.

    It seems the appropriate access scope from which to perform partial
    specialization matching is the specialization itself (similar to how
    we check access of base-clauses), which is what this patch implements.

            PR c++/96204

    gcc/cp/ChangeLog:

            * pt.c (instantiate_class_template_1): Enter the scope of the
            type when calling most_specialized_partial_spec.

    gcc/testsuite/ChangeLog:

            * g++.dg/template/access40.C: New test.
            * g++.dg/template/access40a.C: New test.

Reply via email to