public inbox for gdb-prs@sourceware.org
help / color / mirror / Atom feed
From: "cvs-commit at gcc dot gnu.org" <sourceware-bugzilla@sourceware.org>
To: gdb-prs@sourceware.org
Subject: [Bug record/29178] GDB wont print frame when reverse stepping out of recursive function
Date: Mon, 20 Nov 2023 09:54:39 +0000	[thread overview]
Message-ID: <bug-29178-4717-7TLL9Bepa0@http.sourceware.org/bugzilla/> (raw)
In-Reply-To: <bug-29178-4717@http.sourceware.org/bugzilla/>

https://sourceware.org/bugzilla/show_bug.cgi?id=29178

--- Comment #1 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Bruno Larsen <blarsen@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=bf2813aff8f2988ad3d53e819a0415abf295c91f

commit bf2813aff8f2988ad3d53e819a0415abf295c91f
Author: Guinevere Larsen <blarsen@redhat.com>
Date:   Fri Sep 1 13:47:32 2023 +0200

    gdb/record: print frame information when exiting a recursive call

    Currently,  when GDB is reverse stepping out of a function into the same
    function due to a recursive call, it doesn't print frame information, as
    reported by PR record/29178. This happens because when the inferior
    leaves the current frame, GDB decides to refresh the step information,
    clobbering the original step_frame_id, making it impossible to figure
    out later on that the frame has been changed.

    This commit changes GDB so that, if we notice we're in this exact
    situation, we won't refresh the step information.

    Because of implementation details, this change can cause some debug
    information to be read when it normally wouldn't before, which showed up
    as a regression on gdb.dwarf2/dw2-out-of-range-end-of-seq. Since that
    isn't a problem, the test was changed to allow for the new output.

    Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29178
    Approved-By: Tom Tromey <tom@tromey.com>

-- 
You are receiving this mail because:
You are on the CC list for the bug.

  reply	other threads:[~2023-11-20  9:54 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-25 13:08 [Bug record/29178] New: " blarsen at redhat dot com
2023-11-20  9:54 ` cvs-commit at gcc dot gnu.org [this message]
2023-11-20 10:01 ` [Bug record/29178] " blarsen at redhat dot com
2024-01-21 16:25 ` cvs-commit at gcc dot gnu.org
2024-05-09 20:07 ` blarsen at redhat dot com

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-29178-4717-7TLL9Bepa0@http.sourceware.org/bugzilla/ \
    --to=sourceware-bugzilla@sourceware.org \
    --cc=gdb-prs@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).