]> git.ipfire.org Git - thirdparty/gcc.git/commit
c++: ICE with nested default targ lambdas [PR119574]
authorPatrick Palka <ppalka@redhat.com>
Wed, 9 Apr 2025 21:47:34 +0000 (17:47 -0400)
committerPatrick Palka <ppalka@redhat.com>
Wed, 9 Apr 2025 21:47:34 +0000 (17:47 -0400)
commitf3862ab07943d1fc6e6a0416657ae4b7d1f3941d
treed4426a0845a5d79a52abe1007838dc78be3403bd
parent6704d95ec859d9e7480da130bff1e6b58fe37350
c++: ICE with nested default targ lambdas [PR119574]

Here we substitute into the inner lambda twice, first during default
argument substitution for the outer template parameters, then during
that for the inner template parameters.

For the second testcase (which is easier to follow/debug), the first
substitution into the inner lambda is with the template arguments
{0, NULL_TREE}, which we defer because it's an incremental substitution.
For the second and final substitution we have the template arguments
{1, NULL_TREE}, which we try combining via add_extra_args and ICE on
the checking assert because TREE_STATIC isn't set on the deferred
arguments but the template arguments are considered dependent.

The template arguments aren't dependent however -- they're just
incomplete because when we deferred them we were in the middle of
deduction, and we consider a NULL_TREE template argument as dependent.

If we remove this checking assert, we go on to correctly merge the
template arguments into {{0, NULL_TREE}, {1, NULL_TREE}}.  So this
patch just removes this imprecise assert.

PR c++/119574

gcc/cp/ChangeLog:

* pt.cc (add_extra_args): Remove checking assert.

gcc/testsuite/ChangeLog:

* g++.dg/cpp2a/lambda-targ13.C: New test.
* g++.dg/cpp2a/lambda-targ13a.C: New test.
* g++.dg/cpp2a/lambda-targ13b.C: New test.

Reviewed-by: Jason Merrill <jason@redhat.com>
gcc/cp/pt.cc
gcc/testsuite/g++.dg/cpp2a/lambda-targ13.C [new file with mode: 0644]
gcc/testsuite/g++.dg/cpp2a/lambda-targ13a.C [new file with mode: 0644]
gcc/testsuite/g++.dg/cpp2a/lambda-targ13b.C [new file with mode: 0644]