* findvar.c ({read,write}_register): Use REGISTER_RAW_SIZE
authorJim Kingdon <jkingdon@engr.sgi.com>
Tue, 18 May 1993 15:06:47 +0000 (15:06 +0000)
committerJim Kingdon <jkingdon@engr.sgi.com>
Tue, 18 May 1993 15:06:47 +0000 (15:06 +0000)
not typo RAW_REGISTER_SIZE.

* frame.h, inferior.h: Doc fixes.

gdb/ChangeLog
gdb/inferior.h

index 4b69cc33d2d43cd70033bf22139b3b79c62e3f0d..573666c4d9e829ab7e41ec253e8e43c41c87ba11 100644 (file)
@@ -1,3 +1,10 @@
+Tue May 18 09:03:37 1993  Jim Kingdon  (kingdon@lioth.cygnus.com)
+
+       * findvar.c ({read,write}_register): Use REGISTER_RAW_SIZE
+       not typo RAW_REGISTER_SIZE.
+
+       * frame.h, inferior.h: Doc fixes.
+
 Mon May 17 15:43:03 1993  Stu Grossman  (grossman@cygnus.com)
 
        * findvar.c (write_register):  Add sanity check for register size.
index 5db853eaca451dd45d5d3307cd2033b7b04d689a..576fcff8b01f8a72e82aa9d9e9c276422f8e271f 100644 (file)
@@ -176,9 +176,6 @@ call_ptrace PARAMS ((int, int, PTRACE_ARG3_TYPE, int));
 extern int
 proc_iterate_over_mappings PARAMS ((int (*) (int, CORE_ADDR)));
 
-extern void
-proc_signal_handling_change PARAMS ((void));
-
 /* From fork-child.c */
 
 extern void
@@ -328,8 +325,19 @@ extern CORE_ADDR text_end;
   ((pc) >= text_end   \
    && (pc) <= text_end + CALL_DUMMY_LENGTH + DECR_PC_AFTER_BREAK)
 #else /* On stack.  */
+
+/* This assumes that frame_address is the value of SP_REGNUM before
+   the dummy frame was pushed.  The only known machine for which this
+   isn't true is the 29k, which doesn't use ON_STACK.  Machines for
+   which it isn't true who want to put stack dummies on the stack
+   could provide their own PC_IN_CALL_DUMMY, or perhaps this macro
+   could be re-written to check for the end of the stack instead
+   (using the target_ops->sections).  Are there user programs, libraries,
+   kernel routines, etc. which also execute on the stack?  If so, the
+   latter would be a bad idea.  */
+
 #define PC_IN_CALL_DUMMY(pc, sp, frame_address) \
-  ((sp) INNER_THAN (pc) && (pc) INNER_THAN (frame_address))
+  ((sp) INNER_THAN (pc) && (frame_address != 0) && (pc) INNER_THAN (frame_address))
 #endif /* On stack.  */
 #endif /* Not before text_end.  */
 #endif /* No PC_IN_CALL_DUMMY.  */