On 1/19/23 11:46, Tom de Vries via Gdb-patches wrote:
> In test-case gdb.base/unwind-on-each-insn.exp, we try to determine the last
> insn in function foo.
> 
> This in it self is fragile, as demonstrated by commit 91836f41e20 ("Powerpc
> fix for gdb.base/unwind-on-each-insn.exp").
> 
> But the purpose of finding the last insn is to stop stepping in foo when
> arriving at that last insn.
> 
> There is however no guarantee that:
> - the last insn is actually executed, nor
> - that the last insn is executed last, nor
> - that the last insn is executed once.
> 
> Fix this by simplying the test-case to continue stepping till stepping out of
> foo.
> 

I re-read the commit log, found it not clear enough, and decided to make 
the difference between:
- last disassembled insn, and
- last insn executed before returning to main.
a bit more explicit.

Committed as attached.

Thanks,
- Tom