c++: requires-expr and access checking [PR107179]
commit40c34beef620ed13c4113c893ed4335ccc1b8f92
authorPatrick Palka <ppalka@redhat.com>
Thu, 3 Nov 2022 19:35:18 +0000 (3 15:35 -0400)
committerPatrick Palka <ppalka@redhat.com>
Thu, 3 Nov 2022 19:35:18 +0000 (3 15:35 -0400)
tree08e929a363fa50b11f45169d0ea92b8507491e30
parent095de980339f2fe0ff2add112c4d5ab9175b8f58
c++: requires-expr and access checking [PR107179]

Like during satisfaction, we also need to avoid deferring access checks
during substitution of a requires-expr because the outcome of an access
check can determine the value of the requires-expr.  Otherwise (in
deferred access checking contexts such as within a base-clause), the
requires-expr may evaluate to the wrong result, and along the way a
failed access check may leak out from it into a non-SFINAE context and
cause a hard error (as in the below testcase).

PR c++/107179

gcc/cp/ChangeLog:

* constraint.cc (tsubst_requires_expr): Make sure we're not
deferring access checks.

gcc/testsuite/ChangeLog:

* g++.dg/cpp2a/concepts-requires31.C: New test.
gcc/cp/constraint.cc
gcc/testsuite/g++.dg/cpp2a/concepts-requires31.C [new file with mode: 0644]