summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--gdb/ChangeLog31
-rw-r--r--gdb/corelow.c7
-rw-r--r--gdb/findvar.c15
-rw-r--r--gdb/frame.c10
-rw-r--r--gdb/mt-tdep.c4
-rw-r--r--gdb/regcache.c61
-rw-r--r--gdb/regcache.h10
-rw-r--r--gdb/remote-sim.c1
-rw-r--r--gdb/remote.c4
-rw-r--r--gdb/sh-tdep.c2
-rw-r--r--gdb/tui/tui-regs.c43
11 files changed, 74 insertions, 114 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 2420767d92a..cd81b4b0426 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,36 @@
2007-05-06 Ulrich Weigand <uweigand@de.ibm.com>
+ * regcache.c (regcache_invalidate): New function.
+ (register_cached): Remove.
+ (set_register_cached): Remove.
+ (deprecated_registers_fetched): Remove.
+ (registers_changed): Use regcache_invalidate instead
+ of set_register_cached.
+ (regcache_raw_read): Update comment.
+
+ * regcache.h (regcache_invalidate): Add prototype.
+ (register_cached): Remove.
+ (set_register_cached): Remove.
+ (deprecated_registers_fetched): Remove.
+
+ * findvar.c (value_of_register): Do not call register_cached.
+ * frame.c (frame_register): Likewise.
+ * tui/tui-regs.c (tui_get_register): Likewise.
+
+ * remote.c (fetch_register_using_p): Do not call set_register_cached.
+ (process_g_packet): Likewise.
+ (remote_fetch_registers): Likewise.
+ * remote-sim.c (gdbsim_fetch_register): Likewise.
+ * mt-tdep.c (mt_select_coprocessor): Replace set_register_cached call
+ by regcache_invalidate.
+ (mt_pseudo_register_write): Likewise.
+ * sh-tdep.c (sh_pseudo_register_write): Likewise.
+
+ * corelow.c (get_core_registers): Replace deprecated_registers_fetched
+ call by loop over regcache_raw_supply (..., NULL).
+
+2007-05-06 Ulrich Weigand <uweigand@de.ibm.com>
+
* target.h (struct target_ops): Add REGCACHE parameter to
to_prepare_to_store.
(target_prepare_to_store): Likewise.
diff --git a/gdb/corelow.c b/gdb/corelow.c
index ea2d3868fdb..7be9655f8b3 100644
--- a/gdb/corelow.c
+++ b/gdb/corelow.c
@@ -485,7 +485,7 @@ get_core_register_section (struct regcache *regcache,
static void
get_core_registers (struct regcache *regcache, int regno)
{
- int status;
+ int i;
if (!(core_gdbarch && gdbarch_regset_from_core_section_p (core_gdbarch))
&& (core_vec == NULL || core_vec->core_read_registers == NULL))
@@ -502,7 +502,10 @@ get_core_registers (struct regcache *regcache, int regno)
get_core_register_section (regcache,
".reg-xfp", 3, "extended floating-point", 0);
- deprecated_registers_fetched ();
+ /* Supply dummy value for all registers not found in the core. */
+ for (i = 0; i < NUM_REGS; i++)
+ if (!regcache_valid_p (regcache, i))
+ regcache_raw_supply (regcache, i, NULL);
}
static void
diff --git a/gdb/findvar.c b/gdb/findvar.c
index 09693888edc..a05ee1f8286 100644
--- a/gdb/findvar.c
+++ b/gdb/findvar.c
@@ -250,10 +250,7 @@ store_typed_address (gdb_byte *buf, struct type *type, CORE_ADDR addr)
/* Return a `value' with the contents of (virtual or cooked) register
REGNUM as found in the specified FRAME. The register's type is
- determined by register_type().
-
- NOTE: returns NULL if register value is not available. Caller will
- check return value or die! */
+ determined by register_type(). */
struct value *
value_of_register (int regnum, struct frame_info *frame)
@@ -272,16 +269,6 @@ value_of_register (int regnum, struct frame_info *frame)
frame_register (frame, regnum, &optim, &lval, &addr, &realnum, raw_buffer);
- /* FIXME: cagney/2002-05-15: This test is just bogus.
-
- It indicates that the target failed to supply a value for a
- register because it was "not available" at this time. Problem
- is, the target still has the register and so get saved_register()
- may be returning a value saved on the stack. */
-
- if (register_cached (regnum) < 0)
- return NULL; /* register value not available */
-
reg_val = allocate_value (register_type (current_gdbarch, regnum));
memcpy (value_contents_raw (reg_val), raw_buffer,
diff --git a/gdb/frame.c b/gdb/frame.c
index 8a14b29ede3..2c62d441a6b 100644
--- a/gdb/frame.c
+++ b/gdb/frame.c
@@ -738,16 +738,6 @@ frame_register_read (struct frame_info *frame, int regnum,
int realnum;
frame_register (frame, regnum, &optimized, &lval, &addr, &realnum, myaddr);
- /* FIXME: cagney/2002-05-15: This test is just bogus.
-
- It indicates that the target failed to supply a value for a
- register because it was "not available" at this time. Problem
- is, the target still has the register and so get saved_register()
- may be returning a value saved on the stack. */
-
- if (register_cached (regnum) < 0)
- return 0; /* register value not available */
-
return !optimized;
}
diff --git a/gdb/mt-tdep.c b/gdb/mt-tdep.c
index b9b865330a3..3bf0a55a7d2 100644
--- a/gdb/mt-tdep.c
+++ b/gdb/mt-tdep.c
@@ -495,7 +495,7 @@ mt_select_coprocessor (struct gdbarch *gdbarch,
/* We must flush the cache, as it is now invalid. */
for (ix = MT_NUM_CPU_REGS; ix != MT_NUM_REGS; ix++)
- set_register_cached (ix, 0);
+ regcache_invalidate (regcache, ix);
}
return index;
@@ -573,7 +573,7 @@ mt_pseudo_register_write (struct gdbarch *gdbarch,
case MT_COPRO_PSEUDOREG_REGNUM:
regcache_raw_write (regcache, MT_COPRO_REGNUM, buf);
for (i = MT_NUM_CPU_REGS; i < MT_NUM_REGS; i++)
- set_register_cached (i, 0);
+ regcache_invalidate (regcache, i);
break;
case MT_MAC_REGNUM:
case MT_MAC_PSEUDOREG_REGNUM:
diff --git a/gdb/regcache.c b/gdb/regcache.c
index c54585c98cd..daa412f5541 100644
--- a/gdb/regcache.c
+++ b/gdb/regcache.c
@@ -393,6 +393,17 @@ regcache_valid_p (const struct regcache *regcache, int regnum)
return regcache->register_valid_p[regnum];
}
+void
+regcache_invalidate (struct regcache *regcache, int regnum)
+{
+ gdb_assert (regcache != NULL);
+ gdb_assert (regnum >= 0);
+ gdb_assert (!regcache->readonly_p);
+ gdb_assert (regnum < regcache->descr->nr_raw_registers);
+ regcache->register_valid_p[regnum] = 0;
+}
+
+
/* Global structure containing the current regcache. */
/* FIXME: cagney/2002-05-11: The two global arrays registers[] and
deprecated_register_valid[] currently point into this structure. */
@@ -407,33 +418,6 @@ struct regcache *current_regcache;
static ptid_t registers_ptid;
-/*
- * FUNCTIONS:
- */
-
-/* REGISTER_CACHED()
-
- Returns 0 if the value is not in the cache (needs fetch).
- >0 if the value is in the cache.
- <0 if the value is permanently unavailable (don't ask again). */
-
-int
-register_cached (int regnum)
-{
- return current_regcache->register_valid_p[regnum];
-}
-
-/* Record that REGNUM's value is cached if STATE is >0, uncached but
- fetchable if STATE is 0, and uncached and unfetchable if STATE is <0. */
-
-void
-set_register_cached (int regnum, int state)
-{
- gdb_assert (regnum >= 0);
- gdb_assert (regnum < current_regcache->descr->nr_raw_registers);
- current_regcache->register_valid_p[regnum] = state;
-}
-
/* Observer for the target_changed event. */
void
@@ -468,28 +452,9 @@ registers_changed (void)
alloca (0);
for (i = 0; i < current_regcache->descr->nr_raw_registers; i++)
- set_register_cached (i, 0);
+ regcache_invalidate (current_regcache, i);
}
-/* DEPRECATED_REGISTERS_FETCHED ()
-
- Indicate that all registers have been fetched, so mark them all valid. */
-
-/* FIXME: cagney/2001-12-04: This function is DEPRECATED. The target
- code was blatting the registers[] array and then calling this.
- Since targets should only be using regcache_raw_supply() the need for
- this function/hack is eliminated. */
-
-void
-deprecated_registers_fetched (void)
-{
- int i;
-
- for (i = 0; i < NUM_REGS; i++)
- set_register_cached (i, 1);
- /* Do not assume that the pseudo-regs have also been fetched.
- Fetching all real regs NEVER accounts for pseudo-regs. */
-}
void
regcache_raw_read (struct regcache *regcache, int regnum, gdb_byte *buf)
@@ -517,7 +482,7 @@ regcache_raw_read (struct regcache *regcache, int regnum, gdb_byte *buf)
that a register is in one of the possible states: valid,
undefined, unknown. The last of which isn't yet
possible. */
- gdb_assert (register_cached (regnum));
+ gdb_assert (regcache_valid_p (regcache, regnum));
#endif
}
/* Copy the value directly into the register cache. */
diff --git a/gdb/regcache.h b/gdb/regcache.h
index c8ceca6e78e..4ced1fcd79b 100644
--- a/gdb/regcache.h
+++ b/gdb/regcache.h
@@ -61,6 +61,8 @@ void regcache_raw_write_part (struct regcache *regcache, int regnum,
int regcache_valid_p (const struct regcache *regcache, int regnum);
+void regcache_invalidate (struct regcache *regcache, int regnum);
+
/* Transfer a cooked register [0..NUM_REGS+NUM_PSEUDO_REGS). */
void regcache_cooked_read (struct regcache *regcache, int rawnum,
gdb_byte *buf);
@@ -157,14 +159,6 @@ extern struct regcache *regcache_dup_no_passthrough (struct regcache *regcache);
extern void regcache_cpy (struct regcache *dest, struct regcache *src);
extern void regcache_cpy_no_passthrough (struct regcache *dest, struct regcache *src);
-/* NOTE: cagney/2002-11-05: This function has been superseeded by
- regcache_raw_supply(). */
-extern void deprecated_registers_fetched (void);
-
-extern int register_cached (int regnum);
-
-extern void set_register_cached (int regnum, int state);
-
extern void registers_changed (void);
diff --git a/gdb/remote-sim.c b/gdb/remote-sim.c
index 0a13713c14c..af2238f6a5c 100644
--- a/gdb/remote-sim.c
+++ b/gdb/remote-sim.c
@@ -300,7 +300,6 @@ gdbsim_fetch_register (struct regcache *regcache, int regno)
int nr_bytes;
memset (buf, 0, MAX_REGISTER_SIZE);
regcache_raw_supply (regcache, regno, buf);
- set_register_cached (regno, -1);
break;
}
diff --git a/gdb/remote.c b/gdb/remote.c
index 8e7999d6583..f1d071091e7 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -3522,7 +3522,6 @@ fetch_register_using_p (struct regcache *regcache, struct packet_reg *reg)
if (buf[0] == 'x')
{
regcache_raw_supply (regcache, reg->regnum, NULL);
- set_register_cached (reg->regnum, -1);
return 1;
}
@@ -3661,7 +3660,6 @@ process_g_packet (struct regcache *regcache)
/* The register isn't available, mark it as such (at
the same time setting the value to zero). */
regcache_raw_supply (regcache, r->regnum, NULL);
- set_register_cached (i, -1);
}
else
regcache_raw_supply (regcache, r->regnum,
@@ -3708,7 +3706,6 @@ remote_fetch_registers (struct regcache *regcache, int regnum)
/* This register is not available. */
regcache_raw_supply (regcache, reg->regnum, NULL);
- set_register_cached (reg->regnum, -1);
return;
}
@@ -3721,7 +3718,6 @@ remote_fetch_registers (struct regcache *regcache, int regnum)
{
/* This register is not available. */
regcache_raw_supply (regcache, i, NULL);
- set_register_cached (i, -1);
}
}
diff --git a/gdb/sh-tdep.c b/gdb/sh-tdep.c
index 61ca104e9c6..204a29653bc 100644
--- a/gdb/sh-tdep.c
+++ b/gdb/sh-tdep.c
@@ -2055,7 +2055,7 @@ sh_pseudo_register_write (struct gdbarch *gdbarch, struct regcache *regcache,
regcache_raw_write (regcache, BANK_REGNUM, buffer);
for (bregnum = R0_BANK0_REGNUM; bregnum < MACLB_REGNUM; ++bregnum)
- set_register_cached (bregnum, 0);
+ regcache_invalidate (regcache, bregnum);
}
else if (reg_nr >= DR0_REGNUM && reg_nr <= DR_LAST_REGNUM)
{
diff --git a/gdb/tui/tui-regs.c b/gdb/tui/tui-regs.c
index 4be83dc69d1..e40f3405850 100644
--- a/gdb/tui/tui-regs.c
+++ b/gdb/tui/tui-regs.c
@@ -710,32 +710,27 @@ tui_get_register (struct gdbarch *gdbarch, struct frame_info *frame,
if (target_has_registers)
{
gdb_byte buf[MAX_REGISTER_SIZE];
-
get_frame_register (frame, regnum, buf);
- /* NOTE: cagney/2003-03-13: This is bogus. It is refering to
- the register cache and not the frame which could have pulled
- the register value off the stack. */
- if (register_cached (regnum) >= 0)
- {
- if (changedp)
- {
- int size = register_size (gdbarch, regnum);
- char *old = (char*) data->value;
- int i;
-
- for (i = 0; i < size; i++)
- if (buf[i] != old[i])
- {
- *changedp = TRUE;
- old[i] = buf[i];
- }
- }
- /* Reformat the data content if the value changed. */
- if (changedp == 0 || *changedp == TRUE)
- tui_register_format (gdbarch, frame, data, regnum);
- ret = TUI_SUCCESS;
- }
+ if (changedp)
+ {
+ int size = register_size (gdbarch, regnum);
+ char *old = (char*) data->value;
+ int i;
+
+ for (i = 0; i < size; i++)
+ if (buf[i] != old[i])
+ {
+ *changedp = TRUE;
+ old[i] = buf[i];
+ }
+ }
+
+ /* Reformat the data content if the value changed. */
+ if (changedp == 0 || *changedp == TRUE)
+ tui_register_format (gdbarch, frame, data, regnum);
+
+ ret = TUI_SUCCESS;
}
return ret;
}
View Lorry Controller Status