This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[PATCH 1/11] Add ARM_MAX_REGISTER_SIZE
- From: Alan Hayward <Alan dot Hayward at arm dot com>
- To: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Cc: nd <nd at arm dot com>
- Date: Tue, 4 Apr 2017 10:11:42 +0000
- Subject: [PATCH 1/11] Add ARM_MAX_REGISTER_SIZE
- Authentication-results: sourceware.org; auth=none
- Authentication-results: sourceware.org; dkim=none (message not signed) header.d=none;sourceware.org; dmarc=none action=none header.from=arm.com;
- Nodisclaimer: True
- Spamdiagnosticmetadata: NSPM
- Spamdiagnosticoutput: 1:99
Max size set to 96bits, which is taken from FP_REGISTER_SIZE
Tested on a --enable-targets=all build using make check with board files
unix and native-gdbserver.
Ok to commit?
Alan.
2017-04-04 Alan Hayward <alan.hayward@arm.com>
* arm-tdep.c (arm_store_return_value): Use ARM_MAX_REGISTER_SIZE.
* arm-tdep.h: Add ARM_MAX_REGISTER_SIZE.
diff --git a/gdb/arm-tdep.h b/gdb/arm-tdep.h
index 8125226362e5a9c155b60ffd1fbd4ed839270493..20bcda9dcc554c7d613d6534f5b3cf9a59dfa975 100644
--- a/gdb/arm-tdep.h
+++ b/gdb/arm-tdep.h
@@ -38,6 +38,9 @@ struct gdb_get_next_pcs;
purposes and code readability. These are fixed at 64 bits. */
#define VFP_REGISTER_SIZE 8
+/* Big enough to hold the size of the largest register in bytes. */
+#define ARM_MAX_REGISTER_SIZE FP_REGISTER_SIZE
+
/* Number of machine registers. The only define actually required
is gdbarch_num_regs. The other definitions are used for documentation
purposes and code readability. */
diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c
index 88ed391729813dc4ef8e4dccb6698fa798e0de08..6b544481627678f2b6442725573b02d4e4cd9f2d 100644
--- a/gdb/arm-tdep.c
+++ b/gdb/arm-tdep.c
@@ -8153,7 +8153,7 @@ arm_store_return_value (struct type *type, struct regcache *regs,
if (TYPE_CODE (type) == TYPE_CODE_FLT)
{
- gdb_byte buf[MAX_REGISTER_SIZE];
+ gdb_byte buf[ARM_MAX_REGISTER_SIZE];
switch (gdbarch_tdep (gdbarch)->fp_model)
{