[Bug c++/105351] [concepts] Constraint checking does correctly match static member attributes

["cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>]

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

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

https://gcc.gnu.org/g:a16fc9f1c96c7361d0b7a83a06f3649ace6c440d

commit r13-95-ga16fc9f1c96c7361d0b7a83a06f3649ace6c440d
Author: Patrick Palka <ppalka@redhat.com>
Date:   Tue May 3 15:21:26 2022 -0400

    c++: make finish_non_static_data_member SFINAE enabled [PR105351]

    Here since finish_non_static_data_member isn't SFINAE enabled, we
    incorrectly emit an error when considering the first overload rather
    than silently discarding it:

    sfinae33.C: In substitution of âtemplate<class T> A<T::value> f() [with T
= B]â:
    sfinae33.C:11:7:   required from here
    sfinae33.C:5:31: error: invalid use of non-static data member
âB::valueâ
        5 | template<class T> A<T::value> f();
          |                               ^

    This patch makes the function SFINAE enabled in the usual way: give it a
    complain parameter, check it before emitting an error, and pass it through
    appropriately.

            PR c++/105351

    gcc/cp/ChangeLog:

            * cp-tree.h (finish_non_static_data_member): Add defaulted
            complain parameter.
            * pt.cc (tsubst_copy_and_build): Pass complain to
            finish_non_static_data_member.
            * semantics.cc (finish_non_static_data_member): Respect complain
            parameter.
            (finish_qualified_id_expr): Pass complain to
            finish_non_static_data_member.

    gcc/testsuite/ChangeLog:

            * g++.dg/template/sfinae33.C: New test.