* i386-linux-tdep.c (i386_linux_sigtramp_saved_pc,

i386_linux_sigtramp_saved_sp): Make static.
(i386_linux_frame_saved_pc): New function based on the old
FRAME_SAVED_PC macro, but use read_memory_unsigned_integer instead
of read_memory_integer.
* config/i386/tm-linux.h (sigtramp_saved_pc): Remove definition.
(i386_linux_sigtramp_saved_pc): Remove prototype.
(FRAME_SAVED_PC): Redefine in terms of i386_linux_frame_saved_pc.
(i386_linux_frame_saved_pc): New prototype.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 760e6b8dd792abcecb46b1e99407014272d368fc..1a21bc91fbcdfe8c4af532bc49d6979eda865cb9 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,15 @@
+2001-03-26  Mark Kettenis  <kettenis@gnu.org>
+
+       * i386-linux-tdep.c (i386_linux_sigtramp_saved_pc,
+       i386_linux_sigtramp_saved_sp): Make static.
+       (i386_linux_frame_saved_pc): New function based on the old
+       FRAME_SAVED_PC macro, but use read_memory_unsigned_integer instead
+       of read_memory_integer.
+       * config/i386/tm-linux.h (sigtramp_saved_pc): Remove definition.
+       (i386_linux_sigtramp_saved_pc): Remove prototype.
+       (FRAME_SAVED_PC): Redefine in terms of i386_linux_frame_saved_pc.
+       (i386_linux_frame_saved_pc): New prototype.
+
 2001-03-26  Andrew Cagney  <ac131313@redhat.com>
 
        * MAINTAINERS: Document m88k as a ``Known problem in 5.1''.

diff --git a/gdb/config/i386/tm-linux.h b/gdb/config/i386/tm-linux.h
index f72b3c982bb78b935b0ce6f3142042ed5c49fa5c..6c34498cbf1a966ba0b76c16dc6dd3f84c027803 100644 (file)
--- a/gdb/config/i386/tm-linux.h
+++ b/gdb/config/i386/tm-linux.h
@@ -46,12 +46,6 @@ extern struct link_map_offsets *i386_linux_svr4_fetch_link_map_offsets (void);
 #define IN_SIGTRAMP(pc, name) i386_linux_in_sigtramp (pc, name)
 extern int i386_linux_in_sigtramp (CORE_ADDR, char *);
 
-/* We need our own version of sigtramp_saved_pc to get the saved PC in
-   a sigtramp routine.  */
-
-#define sigtramp_saved_pc i386_linux_sigtramp_saved_pc
-extern CORE_ADDR i386_linux_sigtramp_saved_pc (struct frame_info *);
-
 /* Signal trampolines don't have a meaningful frame.  As in tm-i386.h,
    the frame pointer value we use is actually the frame pointer of the
    calling frame--that is, the frame which was in progress when the
@@ -89,14 +83,8 @@ extern CORE_ADDR i386_linux_sigtramp_saved_pc (struct frame_info *);
          : 0)))
 
 #undef FRAME_SAVED_PC
-#define FRAME_SAVED_PC(FRAME)                                  \
-  ((FRAME)->signal_handler_caller                              \
-   ? sigtramp_saved_pc (FRAME)                                 \
-   : (FRAMELESS_SIGNAL (FRAME)                                 \
-      ? read_memory_integer (i386_linux_sigtramp_saved_sp ((FRAME)->next), 4) \
-      : read_memory_integer ((FRAME)->frame + 4, 4)))
-
-extern CORE_ADDR i386_linux_sigtramp_saved_sp (struct frame_info *);
+#define FRAME_SAVED_PC(frame) i386_linux_frame_saved_pc (frame)
+extern CORE_ADDR i386_linux_frame_saved_pc (struct frame_info *frame);
 
 #undef SAVED_PC_AFTER_CALL
 #define SAVED_PC_AFTER_CALL(frame) i386_linux_saved_pc_after_call (frame)

diff --git a/gdb/i386-linux-tdep.c b/gdb/i386-linux-tdep.c
index a973ac97f411257c88ed05c14d2422929415f425..56a5eadecd93c9281d4e05cdd9aa5f013bf4bd71 100644 (file)
--- a/gdb/i386-linux-tdep.c
+++ b/gdb/i386-linux-tdep.c
@@ -255,7 +255,7 @@ i386_linux_sigcontext_addr (struct frame_info *frame)
 /* Assuming FRAME is for a Linux sigtramp routine, return the saved
    program counter.  */
 
-CORE_ADDR
+static CORE_ADDR
 i386_linux_sigtramp_saved_pc (struct frame_info *frame)
 {
   CORE_ADDR addr;
@@ -269,7 +269,7 @@ i386_linux_sigtramp_saved_pc (struct frame_info *frame)
 /* Assuming FRAME is for a Linux sigtramp routine, return the saved
    stack pointer.  */
 
-CORE_ADDR
+static CORE_ADDR
 i386_linux_sigtramp_saved_sp (struct frame_info *frame)
 {
   CORE_ADDR addr;
@@ -277,6 +277,28 @@ i386_linux_sigtramp_saved_sp (struct frame_info *frame)
   return read_memory_integer (addr + LINUX_SIGCONTEXT_SP_OFFSET, 4);
 }
 
+/* Return the saved program counter for FRAME.  */
+
+CORE_ADDR
+i386_linux_frame_saved_pc (struct frame_info *frame)
+{
+  if (frame->signal_handler_caller)
+    return i386_linux_sigtramp_saved_pc (frame);
+
+  /* See comment in "i386/tm-linux.h" for an explanation what this
+     "FRAMELESS_SIGNAL" stuff is supposed to do.
+
+     FIXME: kettenis/2001-03-26: That comment should eventually be
+     moved to this file.  */
+  if (FRAMELESS_SIGNAL (frame))
+    {
+      CORE_ADDR sp = i386_linux_sigtramp_saved_sp (frame->next);
+      return read_memory_unsigned_integer (sp, 4);
+    }
+
+  return read_memory_unsigned_integer (frame->frame + 4, 4);
+}
+
 /* Immediately after a function call, return the saved pc.  */
 
 CORE_ADDR