public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] Adjust C++ destructor type tests
@ 2024-05-10 20:33 Jason Merrill
  2024-05-11 21:56 ` Kevin Buettner
  2024-05-14 14:40 ` Tom Tromey
  0 siblings, 2 replies; 3+ messages in thread
From: Jason Merrill @ 2024-05-10 20:33 UTC (permalink / raw)
  To: gdb-patches

In gcc-15-95-ga12cae97390 I dropped the unnecessary artificial "in-charge"
parameter from destructors of classes with no virtual bases; Linaro's CI
informed me that the gdb testsuite needs to be adjusted to match.

Teested against GCC 13.2 and GCC 15 trunk.
---
 gdb/testsuite/gdb.cp/m-static.exp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gdb/testsuite/gdb.cp/m-static.exp b/gdb/testsuite/gdb.cp/m-static.exp
index 19eb70b7d71..45bc090d01f 100644
--- a/gdb/testsuite/gdb.cp/m-static.exp
+++ b/gdb/testsuite/gdb.cp/m-static.exp
@@ -79,14 +79,14 @@ if { [is_aarch32_target] } {
 	"simple object class, ptype constructor"
 
     gdb_test "print test1.~gnu_obj_1" \
-	{ = {void \*\(gnu_obj_1 \* const, int\)} 0x[0-9a-f]+ <gnu_obj_1::~gnu_obj_1\(\)>} \
+	{ = {void \*\(gnu_obj_1 \* const(, int)?\)} 0x[0-9a-f]+ <gnu_obj_1::~gnu_obj_1\(\)>} \
 	"simple object instance, print destructor"
     gdb_test "ptype test1.~gnu_obj_1" \
-	{type = void \*\(gnu_obj_1 \* const, int\)} \
+	{type = void \*\(gnu_obj_1 \* const(, int)?\)} \
 	"simple object instance, ptype destructor"
 
     gdb_test "print test1.'~gnu_obj_1'" \
-	{ = {void \*\(gnu_obj_1 \*( const)?, int\)} 0x[0-9a-f]+ <gnu_obj_1::~gnu_obj_1\(\)>} \
+	{ = {void \*\(gnu_obj_1 \*( const)?(, int)?\)} 0x[0-9a-f]+ <gnu_obj_1::~gnu_obj_1\(\)>} \
 	"simple object instance, print quoted destructor"
 
     gdb_test "ptype gnu_obj_1::'~gnu_obj_1'" \

base-commit: e14f6ec969d788b73ac61f61e8c623536cda1cad
-- 
2.44.0


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Adjust C++ destructor type tests
  2024-05-10 20:33 [PATCH] Adjust C++ destructor type tests Jason Merrill
@ 2024-05-11 21:56 ` Kevin Buettner
  2024-05-14 14:40 ` Tom Tromey
  1 sibling, 0 replies; 3+ messages in thread
From: Kevin Buettner @ 2024-05-11 21:56 UTC (permalink / raw)
  To: Jason Merrill; +Cc: gdb-patches

On Fri, 10 May 2024 16:33:20 -0400
Jason Merrill <jason@redhat.com> wrote:

> In gcc-15-95-ga12cae97390 I dropped the unnecessary artificial "in-charge"
> parameter from destructors of classes with no virtual bases; Linaro's CI
> informed me that the gdb testsuite needs to be adjusted to match.
> 
> Teested against GCC 13.2 and GCC 15 trunk.

LGTM.

Approved-by: Kevin Buettner <kevinb@redhat.com>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Adjust C++ destructor type tests
  2024-05-10 20:33 [PATCH] Adjust C++ destructor type tests Jason Merrill
  2024-05-11 21:56 ` Kevin Buettner
@ 2024-05-14 14:40 ` Tom Tromey
  1 sibling, 0 replies; 3+ messages in thread
From: Tom Tromey @ 2024-05-14 14:40 UTC (permalink / raw)
  To: Jason Merrill; +Cc: gdb-patches

>>>>> "Jason" == Jason Merrill <jason@redhat.com> writes:

Jason> In gcc-15-95-ga12cae97390 I dropped the unnecessary artificial "in-charge"
Jason> parameter from destructors of classes with no virtual bases; Linaro's CI
Jason> informed me that the gdb testsuite needs to be adjusted to match.

Jason> Teested against GCC 13.2 and GCC 15 trunk.

I went ahead & pushed this.

Tom

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-05-14 14:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-05-10 20:33 [PATCH] Adjust C++ destructor type tests Jason Merrill
2024-05-11 21:56 ` Kevin Buettner
2024-05-14 14:40 ` Tom Tromey

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).