> On 25 Jun 2025, at 15:17, Martin Jambor <mjam...@suse.cz> wrote:
>
> Hi,
>
> when building GCC with clang, it warns that the private member suffix
> in class cp_coroutine_transform (defined in gcc/cp/coroutines.h) is
> not used which indeed looks like it is the case. This patch therefore
> removes it.
>
> Bootstrapped and tested on x86_64-linx. OK for master?
LGTM and presumably in the “trivial / obvious” category. If we need to
preserve the original fn body in upcoming changes, we can always add
it back.
thanks
Iain
>
> Alternatively, as with all of these clang warning issues, I'm
> perfectly happy to add an entry to contrib/filter-clang-warnings.py to
> ignore the warning instead.
>
> Thanks,
>
> Martin
>
>
> gcc/cp/ChangeLog:
>
> 2025-06-24 Martin Jambor <mjam...@suse.cz>
>
> * coroutines.h (class cp_coroutine_transform): Remove member
> orig_fn_body.
> ---
> gcc/cp/coroutines.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/gcc/cp/coroutines.h b/gcc/cp/coroutines.h
> index 919dc9ab06b..fcc46457915 100644
> --- a/gcc/cp/coroutines.h
> +++ b/gcc/cp/coroutines.h
> @@ -100,7 +100,6 @@ public:
>
> private:
> tree orig_fn_decl; /* The original function decl. */
> - tree orig_fn_body = NULL_TREE; /* The original function body. */
> location_t fn_start = UNKNOWN_LOCATION;
> location_t fn_end = UNKNOWN_LOCATION;
> tree resumer = error_mark_node;
> --
> 2.49.0
>
