2003-03-27 Andrew Cagney <cagney@redhat.com>

	* regcache.c (write_sp): Delete function and references.
	* inferior.h (write_sp): Delete declaration.
	* valops.c (hand_function_call): Replace write_sp with
	TARGET_WRITE_SP.
	* sparc-tdep.c (sparc_push_dummy_frame): Ditto.
	(sparc_pop_frame): Ditto.

Index: doc/ChangeLog
2003-03-27  Andrew Cagney  <cagney@redhat.com>

	* gdbint.texinfo (Target Architecture Definition): Remove
	references to write_sp.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index dc0d2560f4cbac317eec1ff5979169aa3166d867..6ab01d8248360161d9dbe56e140d6322dbf0bd4f 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,12 @@
+2003-03-27  Andrew Cagney  <cagney@redhat.com>
+
+       * regcache.c (write_sp): Delete function and references.
+       * inferior.h (write_sp): Delete declaration.
+       * valops.c (hand_function_call): Replace write_sp with
+       TARGET_WRITE_SP.
+       * sparc-tdep.c (sparc_push_dummy_frame): Ditto.
+       (sparc_pop_frame): Ditto.
+       
 2003-03-27  Andrew Cagney  <cagney@redhat.com>
 
        * NEWS: Mention removal of support for hppa*-*-bsd* and

diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog
index ab8a56dc39395aceb81cb44f0c2e75eec9bed684..1cc2b36d2256086ebf7f10d2cf59352906d789c4 100644 (file)
--- a/gdb/doc/ChangeLog
+++ b/gdb/doc/ChangeLog
@@ -1,3 +1,8 @@
+2003-03-27  Andrew Cagney  <cagney@redhat.com>
+
+       * gdbint.texinfo (Target Architecture Definition): Remove
+       references to write_sp.
+       
 2003-03-27  Andrew Cagney  <cagney@redhat.com>
 
        * gdb.texinfo (GDB/MI Variable Objects): Replace @include with

diff --git a/gdb/doc/gdbint.texinfo b/gdb/doc/gdbint.texinfo
index bd2a722fedcaa0a129efdf01aee7b599791683f0..46bbe088321770b5e20bfa28adf40f63a4f390f4 100644 (file)
--- a/gdb/doc/gdbint.texinfo
+++ b/gdb/doc/gdbint.texinfo
@@ -3894,12 +3894,11 @@ Number of bits in a short integer; defaults to @code{2 * TARGET_CHAR_BIT}.
 @findex read_pc
 @findex write_pc
 @findex read_sp
-@findex write_sp
 @findex read_fp
 These change the behavior of @code{read_pc}, @code{write_pc},
-@code{read_sp}, @code{write_sp} and @code{read_fp}.  For most targets,
-these may be left undefined.  @value{GDBN} will call the read and write
-register functions with the relevant @code{_REGNUM} argument.
+@code{read_sp} and @code{read_fp}.  For most targets, these may be left
+undefined.  @value{GDBN} will call the read and write register functions
+with the relevant @code{_REGNUM} argument.
 
 These macros are useful when a target keeps one of these registers in a
 hard to get at place; for example, part in a segment register and part

diff --git a/gdb/inferior.h b/gdb/inferior.h
index 10da8ee54080eb90d38ee857a23203d32cd291c3..99316ccff9d8ff2461ca034f5631935ce42bfc95 100644 (file)
--- a/gdb/inferior.h
+++ b/gdb/inferior.h
@@ -178,8 +178,6 @@ extern CORE_ADDR read_sp (void);
 
 extern CORE_ADDR generic_target_read_sp (void);
 
-extern void write_sp (CORE_ADDR);
-
 extern void generic_target_write_sp (CORE_ADDR);
 
 extern CORE_ADDR read_fp (void);

diff --git a/gdb/regcache.c b/gdb/regcache.c
index c66901780aedc5ad54e7a5f13cd63307fe671ea6..70af06f2fab2b66e581b4522cdaadee470404c21 100644 (file)
--- a/gdb/regcache.c
+++ b/gdb/regcache.c
@@ -1328,16 +1328,16 @@ regcache_collect (int regnum, void *buf)
 }
 
 
-/* read_pc, write_pc, read_sp, write_sp, read_fp, etc.  Special
-   handling for registers PC, SP, and FP.  */
+/* read_pc, write_pc, read_sp, read_fp, etc.  Special handling for
+   registers PC, SP, and FP.  */
 
 /* NOTE: cagney/2001-02-18: The functions generic_target_read_pc(),
    read_pc_pid(), read_pc(), generic_target_write_pc(),
    write_pc_pid(), write_pc(), generic_target_read_sp(), read_sp(),
-   generic_target_write_sp(), write_sp(), generic_target_read_fp() and
-   read_fp(), will eventually be moved out of the reg-cache into
-   either frame.[hc] or to the multi-arch framework.  The are not part
-   of the raw register cache.  */
+   generic_target_write_sp(), generic_target_read_fp() and read_fp(),
+   will eventually be moved out of the reg-cache into either
+   frame.[hc] or to the multi-arch framework.  The are not part of the
+   raw register cache.  */
 
 /* This routine is getting awfully cluttered with #if's.  It's probably
    time to turn this into READ_PC and define it in the tm.h file.
@@ -1456,12 +1456,6 @@ generic_target_write_sp (CORE_ADDR val)
                  "generic_target_write_sp");
 }
 
-void
-write_sp (CORE_ADDR val)
-{
-  TARGET_WRITE_SP (val);
-}
-
 CORE_ADDR
 generic_target_read_fp (void)
 {

diff --git a/gdb/sparc-tdep.c b/gdb/sparc-tdep.c
index c638bd65ac46fbcfcc8624bdc72e887582211411..59079b0a013ba329b52ede7bcc8cda1134a8fdbf 100644 (file)
--- a/gdb/sparc-tdep.c
+++ b/gdb/sparc-tdep.c
@@ -1040,7 +1040,7 @@ sparc_push_dummy_frame (void)
 
   sp -= DUMMY_STACK_SIZE;
 
-  write_sp (sp);
+  TARGET_WRITE_SP (sp);
 
   write_memory (sp + DUMMY_REG_SAVE_OFFSET, &register_temp[0],
                DUMMY_STACK_REG_BUF_SIZE);
@@ -1310,7 +1310,7 @@ sparc_pop_frame (void)
                        read_memory_integer (fsr[O0_REGNUM + 7],
                                             SPARC_INTREG_SIZE));
 
-      write_sp (get_frame_base (frame));
+      TARGET_WRITE_SP (get_frame_base (frame));
     }
   else if (fsr[I0_REGNUM])
     {

diff --git a/gdb/valops.c b/gdb/valops.c
index a7453e97cb8fe9a46a6c2eae9b13b25ecd47a679..1a5956c38d77ca3b94b3a473bdc07c0fb3a6afcd 100644 (file)
--- a/gdb/valops.c
+++ b/gdb/valops.c
@@ -1697,7 +1697,7 @@ You must use a pointer to function type variable. Command ignored.", arg_name);
      frame), and none of the code following that code adjusts the
      stack-pointer value, the below call is entirely redundant.  */
   if (!gdbarch_push_dummy_call_p (current_gdbarch))
-    write_sp (sp);
+    TARGET_WRITE_SP (sp);
 
   if (SAVE_DUMMY_FRAME_TOS_P ())
     SAVE_DUMMY_FRAME_TOS (sp);