On 6/25/25 10:22 AM, Iain Sandoe wrote:
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.
OK then.
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
