From: Andrew Burgess <andrew.burgess@embecosm.com>
To: gdb-patches@sourceware.org
Cc: Andrew Burgess <andrew.burgess@embecosm.com>
Subject: [PATCHv2 0/2] Changes to frame selection.
Date: Tue, 08 May 2018 16:58:00 -0000 [thread overview]
Message-ID: <cover.1525797846.git.andrew.burgess@embecosm.com> (raw)
This is an update of this patch series:
https://sourceware.org/ml/gdb-patches/2015-09/msg00248.html
In summary this patch means that instead of simply:
(gdb) frame NUMBER|ADDRESS-OF-EXISTING-FRAME|ADDRESS-OF-NEW-FRAME
we now have:
(gdb) frame NUMBER
(gdb) frame level NUMBER
(gdb) frame address ADDRESS-OF-EXISTING-FRAME
(gdb) frame create ADDRESS-OF-NEW-FRAME
(gdb) frame function NAME
The same flexibility is added to 'select-frame' and 'info frame'.
From the original patch series, some of the patches are no longer
needed (due to other upstream changes) and some of the patches were
already approved and merged. It is only the last two patches that are
needed, and these are presented here.
Patch #1 is preperation work, and is just a rebase of what I had
before. This patch was actually approved previously, but I didn't
merge it as without the second patch this doesn't make much sense.
Patch #2 is the interesting stuff, and this has been rebased, and
updated inline with the feedback I originally got, specifically, the
frame specification is now implemented using sub-commands, rather than
using a single parse function.
I'm aware of this patch series:
https://sourceware.org/ml/gdb-patches/2018-05/msg00152.html
That also makes changes to the 'frame' command, and though there will
obviously be textual conflicts, I don't think there's any
functionality conflicts.
--
Andrew Burgess (2):
gdb: Split func_command into two parts.
gdb: Change how frames are selected for 'frame' and 'info frame'.
gdb/ChangeLog | 40 ++
gdb/NEWS | 29 ++
gdb/cli/cli-decode.c | 44 ++-
gdb/command.h | 14 +
gdb/doc/ChangeLog | 8 +
gdb/doc/gdb.texinfo | 91 +++--
gdb/mi/mi-cmd-stack.c | 4 +-
gdb/stack.c | 564 +++++++++++++++++++---------
gdb/stack.h | 2 +-
gdb/testsuite/ChangeLog | 7 +
gdb/testsuite/gdb.base/frame-selection.c | 52 +++
gdb/testsuite/gdb.base/frame-selection.exp | 156 ++++++++
gdb/testsuite/gdb.mi/mi-frame-selection.c | 34 ++
gdb/testsuite/gdb.mi/mi-frame-selection.exp | 90 +++++
14 files changed, 920 insertions(+), 215 deletions(-)
create mode 100644 gdb/testsuite/gdb.base/frame-selection.c
create mode 100644 gdb/testsuite/gdb.base/frame-selection.exp
create mode 100644 gdb/testsuite/gdb.mi/mi-frame-selection.c
create mode 100644 gdb/testsuite/gdb.mi/mi-frame-selection.exp
--
2.14.3
next reply other threads:[~2018-05-08 16:58 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-08 16:58 Andrew Burgess [this message]
2018-05-08 16:58 ` [PATCHv2 1/2] gdb: Split func_command into two parts Andrew Burgess
2018-05-18 19:57 ` Pedro Alves
2018-05-21 15:52 ` Andrew Burgess
2018-05-21 16:06 ` Pedro Alves
2018-05-08 16:59 ` [PATCHv2 2/2] gdb: Change how frames are selected for 'frame' and 'info frame' Andrew Burgess
2018-05-11 15:44 ` Eli Zaretskii
2018-05-21 12:16 ` Andrew Burgess
2018-05-21 17:46 ` Eli Zaretskii
2018-06-05 18:53 ` Andrew Burgess
2018-06-05 21:16 ` Philippe Waroquiers
[not found] ` <20180606082211.GF15881@embecosm.com>
2018-06-06 14:56 ` Eli Zaretskii
2018-06-07 16:19 ` [PATCHv3] " Andrew Burgess
2018-06-29 12:23 ` Andrew Burgess
2018-07-17 15:58 ` [PATCHv4] " Andrew Burgess
2018-07-23 20:46 ` Philippe Waroquiers
2018-07-25 18:14 ` Andrew Burgess
2018-08-13 22:20 ` [PATCHv5 0/2] " Andrew Burgess
2018-08-14 10:31 ` Philippe Waroquiers
2018-08-21 13:10 ` Joel Brobecker
2018-08-27 11:04 ` Andrew Burgess
2018-08-27 15:23 ` Eli Zaretskii
2018-08-28 8:43 ` Andrew Burgess
2018-08-28 9:08 ` Eli Zaretskii
2018-08-28 18:03 ` [PATCHv6] " Andrew Burgess
2018-08-28 18:20 ` Eli Zaretskii
2018-09-05 7:46 ` PING: " Andrew Burgess
2018-09-13 18:02 ` Pedro Alves
2018-09-18 23:01 ` Andrew Burgess
2018-09-19 16:26 ` Pedro Alves
2018-09-26 23:06 ` Andrew Burgess
2018-09-27 20:58 ` Pedro Alves
2018-08-13 22:20 ` [PATCHv5_A 1/2] " Andrew Burgess
[not found] ` <cover.1534197765.git.andrew.burgess@embecosm.com>
2018-08-13 22:20 ` [PATCHv5_B 2/2] " Andrew Burgess
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=cover.1525797846.git.andrew.burgess@embecosm.com \
--to=andrew.burgess@embecosm.com \
--cc=gdb-patches@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