Message ID | 20231107124150.116140-2-blarsen@redhat.com |
---|---|
Headers |
Return-Path: <gdb-patches-bounces+patchwork=sourceware.org@sourceware.org> X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 1011238582A4 for <patchwork@sourceware.org>; Tue, 7 Nov 2023 12:47:13 +0000 (GMT) X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTPS id 1EBE43858D1E for <gdb-patches@sourceware.org>; Tue, 7 Nov 2023 12:46:59 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 1EBE43858D1E Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=redhat.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 1EBE43858D1E Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=170.10.129.124 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1699361220; cv=none; b=YEY/jYkr0IGJlMjyju3vUCr45Y/4zf9bdiY7xrcy5JTQ8pu3TnItvpiUEeAyGBRkUfzj8jw4GxlvTuvYfVGMN+PJQilQrH3+j085G84yrwzXPRPRcKQKC8RBwK/ShwuV8/f+tUbNsRGHnn9YLvx9v71smts4dhUJyJlbrfHv0Uw= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1699361220; c=relaxed/simple; bh=gq/rlfEzFvJJLAMKXu0PkSOU3XZKrVXjyHT68mygLfU=; h=DKIM-Signature:From:To:Subject:Date:Message-ID:MIME-Version; b=qv3HeUlNoPDtj03aRYplEL2dT5z5EtR0QGe2/znrcfVh2xpzC+0eP3BXLTs+h0KQP9zDAHqvcJCQ0HawqPmnxMYMMXamOxT59mahFUKqHOe1EGKMXx3EjjHyXF+c8tIoyWudiMnnyl1a3bcnXtY2WJzOuM/qdURHXaQJIACOBI8= ARC-Authentication-Results: i=1; server2.sourceware.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1699361218; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=pFlrITq9LLOazqdt+EOUfoSrG/yqmN7zmRxKgL+Hvgo=; b=I9rSgKIAyK+cSHz2p+tPRGZh0PCpqYNaV5YX8sLkFYo2qCLwCNnKhmOnCUQTiTCMm+4PZM +3Am0RgfROSwrL65aPjFdt6Txg/gnoh57Xfl4XjHHKgLA3yK+4CguITVPLO5PZdoXBuKQ0 g8Fi6XWSCNfja+pudf6yZUZBEc58rcc= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-213-wVSQTh-jOkSvGvb-_fm5oA-1; Tue, 07 Nov 2023 07:46:55 -0500 X-MC-Unique: wVSQTh-jOkSvGvb-_fm5oA-1 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.rdu2.redhat.com [10.11.54.4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 212E088D713 for <gdb-patches@sourceware.org>; Tue, 7 Nov 2023 12:46:55 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.45.224.103]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 9CB222026D66; Tue, 7 Nov 2023 12:46:54 +0000 (UTC) From: Guinevere Larsen <blarsen@redhat.com> To: gdb-patches@sourceware.org Cc: Guinevere Larsen <blarsen@redhat.com> Subject: [PATCH v4 0/2] Improving frame printing with recursive calls Date: Tue, 7 Nov 2023 13:41:49 +0100 Message-ID: <20231107124150.116140-2-blarsen@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.11.54.4 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="US-ASCII"; x-default=true X-Spam-Status: No, score=-5.5 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H4, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gdb-patches mailing list <gdb-patches.sourceware.org> List-Unsubscribe: <https://sourceware.org/mailman/options/gdb-patches>, <mailto:gdb-patches-request@sourceware.org?subject=unsubscribe> List-Archive: <https://sourceware.org/pipermail/gdb-patches/> List-Post: <mailto:gdb-patches@sourceware.org> List-Help: <mailto:gdb-patches-request@sourceware.org?subject=help> List-Subscribe: <https://sourceware.org/mailman/listinfo/gdb-patches>, <mailto:gdb-patches-request@sourceware.org?subject=subscribe> Errors-To: gdb-patches-bounces+patchwork=sourceware.org@sourceware.org |
Series |
Improving frame printing with recursive calls
|
|
Message
Guinevere Larsen
Nov. 7, 2023, 12:41 p.m. UTC
This started with the first patch, just fixing PR record/29178. However, Kevin pointed out that I could reuse some of the local variables to make the function as a whole simpler, which I did on patch 2. My changes on v3 introduced a regression in a dwarf2 test. I gave some time for people to comment on Kevin's solution (then forgot about this patch), but since no one disagreed, this new version changes thetest, rather than the code. Changelog for v4: * changed gdb.dwarf2/dw2-out-of-range... because it was reporting a failure, but it is just some symbols being read earlier than before. Changelog for v3: * Fix remaining style issues in the testsuite Changelog for v2: * Added second patch in the series * Fixed some tyle issues Guinevere Larsen (2): gdb/record: print frame information when exiting a recursive call gdb/infrun: simplify process_event_stop_test gdb/infrun.c | 36 ++++++++++----- .../dw2-out-of-range-end-of-seq.exp | 2 +- gdb/testsuite/gdb.reverse/recursion.c | 44 ++++++++++++++++++ gdb/testsuite/gdb.reverse/recursion.exp | 45 +++++++++++++++++++ 4 files changed, 114 insertions(+), 13 deletions(-) create mode 100644 gdb/testsuite/gdb.reverse/recursion.c create mode 100644 gdb/testsuite/gdb.reverse/recursion.exp
Comments
>>>>> "Guinevere" == Guinevere Larsen <blarsen@redhat.com> writes:
Guinevere> This started with the first patch, just fixing PR record/29178. However,
Guinevere> Kevin pointed out that I could reuse some of the local variables to make
Guinevere> the function as a whole simpler, which I did on patch 2.
Guinevere> My changes on v3 introduced a regression in a dwarf2 test. I gave some
Guinevere> time for people to comment on Kevin's solution (then forgot about this
Guinevere> patch), but since no one disagreed, this new version changes thetest,
Guinevere> rather than the code.
These look reasonable to me. Thank you.
Approved-By: Tom Tromey <tom@tromey.com>
Tom
On 17/11/2023 15:38, Tom Tromey wrote: >>>>>> "Guinevere" == Guinevere Larsen <blarsen@redhat.com> writes: > Guinevere> This started with the first patch, just fixing PR record/29178. However, > Guinevere> Kevin pointed out that I could reuse some of the local variables to make > Guinevere> the function as a whole simpler, which I did on patch 2. > Guinevere> My changes on v3 introduced a regression in a dwarf2 test. I gave some > Guinevere> time for people to comment on Kevin's solution (then forgot about this > Guinevere> patch), but since no one disagreed, this new version changes thetest, > Guinevere> rather than the code. > > These look reasonable to me. Thank you. > Approved-By: Tom Tromey <tom@tromey.com> > Thanks, I pushed these!
On 11/20/23 09:54, Guinevere Larsen wrote: > On 17/11/2023 15:38, Tom Tromey wrote: >>>>>>> "Guinevere" == Guinevere Larsen <blarsen@redhat.com> writes: >> Guinevere> This started with the first patch, just fixing PR record/29178. However, >> Guinevere> Kevin pointed out that I could reuse some of the local variables to make >> Guinevere> the function as a whole simpler, which I did on patch 2. >> Guinevere> My changes on v3 introduced a regression in a dwarf2 test. I gave some >> Guinevere> time for people to comment on Kevin's solution (then forgot about this >> Guinevere> patch), but since no one disagreed, this new version changes thetest, >> Guinevere> rather than the code. >> >> These look reasonable to me. Thank you. >> Approved-By: Tom Tromey <tom@tromey.com> >> > Thanks, I pushed these! > Sorry I'm late to the party, but looks like this regresses reverse debugging. I was just testing Carl's patches for step/next reverse fixes, and I notice a few new FAIL's that were not related to the fixes by Carl. I don't see a problem with aarch64-linux, but arm-linux seems to be affected. It was caught by the Linaro CI here: https://linaro.atlassian.net/browse/GNU-1026 Do you have any ideas on what's up?
On 29/11/2023 12:42, Luis Machado wrote: > On 11/20/23 09:54, Guinevere Larsen wrote: >> On 17/11/2023 15:38, Tom Tromey wrote: >>>>>>>> "Guinevere" == Guinevere Larsen <blarsen@redhat.com> writes: >>> Guinevere> This started with the first patch, just fixing PR record/29178. However, >>> Guinevere> Kevin pointed out that I could reuse some of the local variables to make >>> Guinevere> the function as a whole simpler, which I did on patch 2. >>> Guinevere> My changes on v3 introduced a regression in a dwarf2 test. I gave some >>> Guinevere> time for people to comment on Kevin's solution (then forgot about this >>> Guinevere> patch), but since no one disagreed, this new version changes thetest, >>> Guinevere> rather than the code. >>> >>> These look reasonable to me. Thank you. >>> Approved-By: Tom Tromey <tom@tromey.com> >>> >> Thanks, I pushed these! >> > Sorry I'm late to the party, but looks like this regresses reverse debugging. I was just > testing Carl's patches for step/next reverse fixes, and I notice a few new FAIL's that > were not related to the fixes by Carl. > > I don't see a problem with aarch64-linux, but arm-linux seems to be affected. > > It was caught by the Linaro CI here: https://linaro.atlassian.net/browse/GNU-1026 > > Do you have any ideas on what's up? > I have no idea what's up. I'll take a look at it, thanks for bringing it up