This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[rfc] [14/15] Add gdbarch to register_name callback in frv-tdep.c
- From: Markus Deuling <deuling at de dot ibm dot com>
- To: GDB Patches <gdb-patches at sourceware dot org>
- Cc: Ulrich Weigand <uweigand at de dot ibm dot com>
- Date: Wed, 31 Oct 2007 11:46:13 +0100
- Subject: [rfc] [14/15] Add gdbarch to register_name callback in frv-tdep.c
Hi,
this patch adds gdbarch to register_name callback in frv-tdep.c.
The macro CURRENT_VARIANT included current_gdbarch. This patch replaces its occurences by its expression and then
replaces current_gdbarch.
Is this ok to commit?
ChangeLog:
* frv-tdep.c (frv_register_name): Add gdbarch parameter. Replace
current_gdbarch by gdbarch.
(frv_register_name, frv_check_watch_resources): Replace macro
CURRENT_VARIANT by its expression.
(CURRENT_VARIANT): Remove.
--
Markus Deuling
GNU Toolchain for Linux on Cell BE
deuling@de.ibm.com
diff -urpN src/gdb/frv-tdep.c dev/gdb/frv-tdep.c
--- src/gdb/frv-tdep.c 2007-10-24 23:08:23.000000000 +0200
+++ dev/gdb/frv-tdep.c 2007-10-31 06:57:30.000000000 +0100
@@ -91,7 +91,6 @@ struct gdbarch_tdep
char **register_names;
};
-#define CURRENT_VARIANT (gdbarch_tdep (current_gdbarch))
/* Return the FR-V ABI associated with GDBARCH. */
enum frv_abi
@@ -278,14 +277,14 @@ set_variant_scratch_registers (struct gd
}
static const char *
-frv_register_name (int reg)
+frv_register_name (struct gdbarch *gdbarch, int reg)
{
if (reg < 0)
return "?toosmall?";
if (reg >= frv_num_regs + frv_num_pseudo_regs)
return "?toolarge?";
- return CURRENT_VARIANT->register_names[reg];
+ return gdbarch_tdep (gdbarch)->register_names[reg];
}
@@ -1282,7 +1281,7 @@ frv_return_value (struct gdbarch *gdbarc
int
frv_check_watch_resources (int type, int cnt, int ot)
{
- struct gdbarch_tdep *var = CURRENT_VARIANT;
+ struct gdbarch_tdep *var = gdbarch_tdep (current_gdbarch);
/* Watchpoints not supported on simulator. */
if (strcmp (target_shortname, "sim") == 0)