gdb: Remove redundant frame switching
authorRichard Bunt <richard.bunt@linaro.org>
Fri, 19 May 2023 07:17:20 +0000 (08:17 +0100)
committerRichard Bunt <richard.bunt@linaro.org>
Fri, 19 May 2023 10:29:43 +0000 (11:29 +0100)
commit712872748deffff984c79b875550e46359ba053b
tree930e192e593447304b2616311306c67fcbe64256
parente84060b489746d031ed1ec9e7b6b39fdf4b6cfe3
gdb: Remove redundant frame switching

547ce8f00b fixed an issue where dynamic types were not being resolved
correctly prior to printing a value. The same issue was discovered when
printing the value using mi-mode, which was not covered by the fix.
Porting the fix to the mi-mode code path resolved the issue.

However, it was discovered that a later patch series, ending
2fc3b8a4cb8, independently fixed the issue in both the cli- and mi-mode
code paths, making the original fix unneeded.

This commit removes this extra frame switch and adds test coverage for
the mi-mode scenario to protect against any future divergence in this
area.

GDB built with GCC 11.

No test suite regressions detected. Compilers: GCC 12.1.0, ACfL 22.1,
Intel 22.1; Platforms: x86_64, aarch64.

Approved-By: Tom Tromey <tom@tromey.com>
gdb/stack.c
gdb/testsuite/gdb.fortran/mixed-lang-stack.exp