On Thu, Apr 28, 2022 at 11:22:45PM +0100, Iain Sandoe wrote:
> how about the following, which uniques the names by bind scope, scope nest
> and then
> sequence within that?
That LGTM.
> --- a/gcc/cp/coroutines.cc
> +++ b/gcc/cp/coroutines.cc
> @@ -3913,6 +3913,7 @@ register_local_var_uses (tree
> On 28 Apr 2022, at 20:26, Jakub Jelinek wrote:
>
> On Thu, Apr 28, 2022 at 08:17:04PM +0100, Iain Sandoe wrote:
>> Signed-off-by: Iain Sandoe
>>
>> PR c++/105426
>>
>> gcc/cp/ChangeLog:
>>
>> * coroutines.cc (register_local_var_uses): Allow promotion of unnamed
>>
On Thu, Apr 28, 2022 at 08:17:04PM +0100, Iain Sandoe wrote:
> Signed-off-by: Iain Sandoe
>
> PR c++/105426
>
> gcc/cp/ChangeLog:
>
> * coroutines.cc (register_local_var_uses): Allow promotion of unnamed
> temporaries to coroutine frame copies.
> ---
> gcc/cp/coroutines.cc |
The changes to fix PR 105287 included a tightening of the constraints on which
variables are promoted to frame copies. This has exposed that we are failing
to name some variables that should be promoted.
The long-term fix is to address the cases where the naming has been missed,
but for the