0.pre7.58:

[sbcl.git] / src / runtime / breakpoint.c

diff --git a/src/runtime/breakpoint.c b/src/runtime/breakpoint.c
index 9e95a8a..1ad416c 100644 (file)
--- a/src/runtime/breakpoint.c
+++ b/src/runtime/breakpoint.c
@@ -54,7 +54,15 @@ void breakpoint_remove(lispobj code_obj, int pc_offset,
 void breakpoint_do_displaced_inst(os_context_t* context,
                                  unsigned long orig_inst)
 {
-#if !defined(hpux) && !defined(irix) && !defined(__i386__)
+    /* on platforms with sigreturn(), we go directly back from
+     * arch_do_displaced_inst() to lisp code, so we need to clean up
+     * our bindings now.  (side note: I'd love to know in exactly what
+     * scenario the speed of breakpoint handling is critical enough to
+     * justify this maintenance mess)
+     *
+     * -dan 2001.08.09 */
+
+#if !(defined(hpux) || defined(irix) || defined(__i386__) || defined(alpha))
     undo_fake_foreign_function_call(context);
 #endif
     arch_do_displaced_inst(context, orig_inst);
@@ -67,12 +75,12 @@ static lispobj find_code(os_context_t *context)
     lispobj code = *os_context_register_addr(context, reg_CODE);
     lispobj header;
 
-    if (LowtagOf(code) != type_OtherPointer)
+    if (lowtag_of(code) != OTHER_POINTER_LOWTAG)
        return NIL;
 
-    header = *(lispobj *)(code-type_OtherPointer);
+    header = *(lispobj *)(code-OTHER_POINTER_LOWTAG);
 
-    if (TypeOf(header) == type_CodeHeader)
+    if (widetag_of(header) == CODE_HEADER_WIDETAG)
        return code;
     else
        return code - HeaderValue(header)*sizeof(lispobj);
@@ -91,7 +99,7 @@ static lispobj find_code(os_context_t *context)
   if (codeptr == 0) {
       return NIL;
   } else {
-      return codeptr + type_OtherPointer;
+      return codeptr + OTHER_POINTER_LOWTAG;
   }
 }
 #endif
@@ -122,7 +130,10 @@ static int compute_offset(os_context_t *context, lispobj code)
        }
     }
 }
-
+/* FIXME: I can see no really good reason these couldn't be merged, but haven't
+ * tried.  The sigprocmask() call would work just as well on alpha as it
+ * presumably does on x86   -dan 2001.08.10
+ */
 #ifndef __i386__
 void handle_breakpoint(int signal, siginfo_t *info, os_context_t *context)
 {
@@ -162,8 +173,8 @@ void handle_breakpoint(int signal, siginfo_t* info, os_context_t *context)
 #endif
 
 #ifndef __i386__
-void *handle_function_end_breakpoint(int signal, siginfo_t *info,
-                                    os_context_t *context)
+void *handle_fun_end_breakpoint(int signal, siginfo_t *info,
+                               os_context_t *context)
 {
     lispobj code, lra;
     struct code *codeptr;
@@ -185,11 +196,11 @@ void *handle_function_end_breakpoint(int signal, siginfo_t *info,
     }
 #endif
     undo_fake_foreign_function_call(context);
-    return (void *)(lra-type_OtherPointer+sizeof(lispobj));
+    return (void *)(lra-OTHER_POINTER_LOWTAG+sizeof(lispobj));
 }
 #else
-void *handle_function_end_breakpoint(int signal, siginfo_t *info,
-                                    os_context_t *context)
+void *handle_fun_end_breakpoint(int signal, siginfo_t *info,
+                               os_context_t *context)
 {
     lispobj code, context_sap = alloc_sap(context);
     struct code *codeptr;