RE: [PATCH 1/2] fix for gdb.reverse/finish-precsave.exp and gdb.reverse/finish-reverse.exp

Carl Love via Gdb-patches
2023-01-12T18:54:51Z

Tom:

On Thu, 2023-01-12 at 17:56 +0100, Tom de Vries wrote:
> On 1/11/23 19:27, Carl Love via Gdb-patches wrote:
> > This patch requires a number of regression tests to be
> > updated. Test
> > gdb.mi/mi-reverse.exp no longer needs to execute two steps to get
> > to the
> > previous line.
>
> Hi Carl,
>
> FYI, I stumbled upon a FAIL (not related to reverse-finish) in
> gdb.reverse/step-reverse.exp, see PR29962 (
> https://sourceware.org/bugzilla/show_bug.cgi?id=29962 ) and wrote
> a
> tentative patch for it (
> https://sourceware.org/bugzilla/attachment.cgi?id=14588 ), and ended
> up
> making the same fix in the gdb.mi/mi-reverse.exp test-case.
>
> I'll test the patch a bit further before posting it.

On PPC64 there are also issues with reverse stepping over a line that
has multiple executable statements in it. Not sure if this is related
to what you are seeing.

There is a patch that I have worked on with Luis Machado to fix. We
have posted versions of it to the mailing list. The subject line is:

Fix reverse stepping multiple contiguous PC ranges over the line table

which might be relevant. Unfortunately, work on this patch has been
stalled for awhile. The patch fixes 5 failures in gdb.reverse/solib-
precsave.exp and 5 failures in gdb.reverse/solib-reverse.exp.

You might want to take a look at it to see if it helps.

Carl