gdb: remove reggroup_next and reggroup_prev
authorAndrew Burgess <aburgess@redhat.com>
Thu, 31 Mar 2022 10:43:13 +0000 (11:43 +0100)
committerAndrew Burgess <aburgess@redhat.com>
Thu, 7 Apr 2022 15:01:18 +0000 (16:01 +0100)
commit1bca9b1e6be9cb45684f38dba0d4d323447a653a
tree1dd7781dc5cc99264f61bb158ef3329ef5edfbd0
parentb89f77be528605fb1781bdd481c3ba1b9d6afab7
gdb: remove reggroup_next and reggroup_prev

Add a new function gdbarch_reggroups that returns a reference to a
vector containing all the reggroups for an architecture.

Make use of this function throughout GDB instead of the existing
reggroup_next and reggroup_prev functions.

Finally, delete the reggroup_next and reggroup_prev functions.

Most of these changes are pretty straight forward, using range based
for loops instead of the old style look using reggroup_next.  There
are two places where the changes are less straight forward.

In gdb/python/py-registers.c, the register group iterator needed to
change slightly.  As the iterator is tightly coupled to the gdbarch, I
just fetch the register group vector from the gdbarch when needed, and
use an index counter to find the next item from the vector when
needed.

In gdb/tui/tui-regs.c the tui_reg_next and tui_reg_prev functions are
just wrappers around reggroup_next and reggroup_prev respectively.
I've just inlined the logic of the old functions into the tui
functions.  As the tui function had its own special twist (wrap around
behaviour) I think this is OK.

There should be no user visible changes after this commit.
gdb/completer.c
gdb/infcmd.c
gdb/python/py-registers.c
gdb/regcache-dump.c
gdb/reggroups.c
gdb/reggroups.h
gdb/tui/tui-regs.c