From 0ef8a082c32f34827479f2df8fbd431e08acd2d8 Mon Sep 17 00:00:00 2001 From: Alan Hayward Date: Wed, 20 Feb 2019 11:56:23 +0000 Subject: [PATCH] AArch64: Add default reggroups AArch64 does not define any reggroups. This causes "maintenance print reggroups" to dump the default set (which is ok). However, if a new group is added via an xml file, then this now becomes the only group. Fixes gdb.xml/tdesc-regs.exp on AArch64. gdb/ChangeLog: * aarch64-tdep.c (aarch64_add_reggroups): New function (aarch64_gdbarch_init): Call aarch64_add_reggroups. --- gdb/ChangeLog | 5 +++++ gdb/aarch64-tdep.c | 16 ++++++++++++++++ 2 files changed, 21 insertions(+) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index eb855819e7c..721e636fdca 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2019-02-20 Alan Hayward + + * aarch64-tdep.c (aarch64_add_reggroups): New function. + (aarch64_gdbarch_init): Call aarch64_add_reggroups. + 2019-02-19 Simon Marchi * top.h (source_file_name): Change to std::string. diff --git a/gdb/aarch64-tdep.c b/gdb/aarch64-tdep.c index bc928e14e9f..f85539313fe 100644 --- a/gdb/aarch64-tdep.c +++ b/gdb/aarch64-tdep.c @@ -2955,6 +2955,19 @@ aarch64_get_tdesc_vq (const struct target_desc *tdesc) return sve_vq_from_vl (vl); } +/* Add all the expected register sets into GDBARCH. */ + +static void +aarch64_add_reggroups (struct gdbarch *gdbarch) +{ + reggroup_add (gdbarch, general_reggroup); + reggroup_add (gdbarch, float_reggroup); + reggroup_add (gdbarch, system_reggroup); + reggroup_add (gdbarch, vector_reggroup); + reggroup_add (gdbarch, all_reggroup); + reggroup_add (gdbarch, save_reggroup); + reggroup_add (gdbarch, restore_reggroup); +} /* Initialize the current architecture based on INFO. If possible, re-use an architecture from ARCHES, which is a list of @@ -3137,6 +3150,9 @@ aarch64_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) /* Virtual tables. */ set_gdbarch_vbit_in_delta (gdbarch, 1); + /* Register architecture. */ + aarch64_add_reggroups (gdbarch); + /* Hook in the ABI-specific overrides, if they have been registered. */ info.target_desc = tdesc; info.tdesc_data = tdesc_data; -- 2.30.2