Changeset 6478

Timestamp:

May 9, 2007, 12:42:03 AM (18 years ago)

Author:

Gary Byers

Message:

Handle new-style tras, other new-calling-sequence changes.

File:

• trunk/ccl/level-0/X86/x86-def.lisp (modified) (18 diffs)

trunk/ccl/level-0/X86/x86-def.lisp

@@ -92 +92 @@
   (jmp-subprim .SPvalues))
   
-
         
 
@@ -156 +155 @@
 
 (defx86lapfunction %fixnum-ref ((fixnum arg_y) #| &optional |# (offset arg_z))
+  (:arglist (fixnum &optional offset))
   (check-nargs 1 2)
   (cmpw ($ x8664::fixnumone) (% nargs))
@@ -167 +167 @@
 
 (defx86lapfunction %fixnum-ref-natural ((fixnum arg_y) #| &optional |# (offset arg_z))
+  (:arglist (fixnum &optional offset))
   (check-nargs 1 2)
   (cmpw ($ x8664::fixnumone) (% nargs))
@@ -178 +179 @@
 
 (defx86lapfunction %fixnum-set ((fixnum arg_x) (offset arg_y) #| &optional |# (new-value arg_z))
+  (:arglist (fixnum offset &optional newval))
   (check-nargs 2 3)
   (cmpw ($ '2) (% nargs))
@@ -190 +192 @@
 
 
-
 (defx86lapfunction %fixnum-set-natural ((fixnum arg_x) (offset arg_y) #| &optional |# (new-value arg_z))
+  (:arglist (fixnum offset &optional newval))
   (check-nargs 2 3)
   (save-simple-frame)
@@ -223 +225 @@
   (single-value-return))
 
-
+;;; Look for "lea -nnnn(%rip),%fn" AT the tra; if that's present, use
+;;; the dispacement -nnnn to find the function.  The end of the
+;;; encoded displacement is
+;;; x8664::recover-fn-from-rip-disp-offset (= 7) bytes from the tra.
 (defx86lapfunction %return-address-function ((r arg_z))
   (extract-lisptag r imm0)
   (cmpb ($ x8664::tag-tra) (% imm0.b))
   (jne @fail)
-  (movl (@ -4 (% r)) (% imm0.l))
-  (testl (% imm0.l) (% imm0.l))
-  (jle @fail)
-  (subq (% imm0) (% arg_z))
+  (cmpw ($ x8664::recover-fn-from-rip-word0) (@ (% r)))
+  (jne @fail)
+  (cmpb ($ x8664::recover-fn-from-rip-byte2) (@ 2 (% r)))
+  (movslq (@ x8664::recover-fn-from-rip-disp-offset (% r)) (% imm0))
+  (jne @fail)
+  (lea (@ x8664::recover-fn-from-rip-length (% imm0) (% r)) (% arg_z))
   (single-value-return)
   @fail
@@ -241 +248 @@
   (cmpb ($ x8664::tag-tra) (% imm0.b))
   (jne @fail)
-  (movl (@ -4 (% r)) (% imm0.l))
-  (testl (% imm0.l) (% imm0.l))
-  (jle @fail)
-  (box-fixnum imm0 arg_z)
+  (cmpw ($ x8664::recover-fn-from-rip-word0) (@ (% r)))
+  (jne @fail)
+  (cmpb ($ x8664::recover-fn-from-rip-byte2) (@ 2 (% r)))
+  (movslq (@ x8664::recover-fn-from-rip-disp-offset (% r)) (% imm0))
+  (jne @fail)
+  (negq (% imm0))
+  (leaq (@ (ash x8664::recover-fn-from-rip-length x8664::fixnumshift) (% imm0) 8) (% arg_z))
   (single-value-return)
   @fail
@@ -252 +262 @@
 ;;; It's always been the case that the function associated with a
 ;;; frame pointer is the caller of the function that "uses" that frame.
-(defx86lapfunction %cfp-lfun ((p arg_z))
-  (ref-global ret1valaddr imm0)
-  (movq (@ x8664::lisp-frame.return-address (% p)) (% arg_y))
-  (cmpq (% imm0) (% arg_y))
-  (cmoveq (@ x8664::lisp-frame.xtra (% p)) (% arg_y))
-  (extract-lisptag arg_y imm0)
-  (cmpb ($ x8664::tag-tra) (%b imm0))
-  (jne @no)
-  (movl (@ -4 (% arg_y)) (%l imm0))
-  (testl (% imm0.l) (% imm0.l))
-  (je @no)
-  (subq (% imm0) (% arg_y))
-  (box-fixnum imm0 arg_z)
-  (movq (% rsp) (% temp0))
-  (pushq (% arg_y))
-  (pushq (% arg_z))
-  (set-nargs 2)
-  (jmp-subprim .SPvalues)
-  @no
-  (movq (% rsp) (% temp0))
-  (pushq ($ x8664::nil-value))
-  (pushq ($ x8664::nil-value))
-  (set-nargs 2)
-  (jmp-subprim .SPvalues))
+(defun %cfp-lfun (p)
+  (let* ((ra (%fixnum-ref p x8664::lisp-frame.return-address)))
+    (if (eq ra (%get-kernel-global ret1valaddr))
+      (setq ra (%fixnum-ref p x8664::lisp-frame.xtra)))
+    (values (%return-address-function ra) (%return-address-offset ra))))
 
 
@@ -350 +341 @@
   ;;   but preserves x866::xfn/x8664::next-method-context.
   ;; Jump to the function in x8664::xfn.
+  (popq (% ra0))
   (movq (% magic) (% next-method-context))
   (movq (% function) (% xfn))
@@ -381 +373 @@
   (jmp @three)
   @go
-  (xchgq (% xfn) (% fn))
-  (jmp (% fn)))
+  (push (% ra0))
+  (jmp (% xfn)))
 
 (defx86lapfunction %apply-with-method-context ((magic arg_x)
@@ -393 +385 @@
   ;;   but preserves x8664::xfn/x8664::next-method-context.
   ;; Jump to the function in x8664::xfn.
-  ;; We need to inline the "spreadargz" operation, 'cause there's no
-  ;; good place to keep %ra0.
+  (pop (% ra0))  
   (movq (% magic) (% x8664::next-method-context))
   (movq (% function) (% x8664::xfn))
@@ -431 +422 @@
   (je @discard-and-go)
   @go
-  (xchgq (% xfn) (% fn))
-  (jmp (% fn))
+  (push (% ra0))
+  (jmp (% xfn))
   @bad
   (addq (% imm0) (% rsp))
@@ -447 +438 @@
 ;;; entry must be in %rbp.
 (defx86lapfunction %apply-lexpr-tail-wise ((method arg_y) (args arg_z))
+  (addq ($ x8664::node-size) (% rsp))   ; discard extra return address
   (movq (% method) (% xfn))
   (movq (% args) (% rsp))
@@ -478 +470 @@
   (discard-reserved-frame)
   @popped
-  (xchgq (% xfn) (% fn))
-  (jmp (% fn)))
+  (push (% ra0))
+  (jmp (% xfn)))
 
 
@@ -495 +487 @@
 (defun apply+ (&lap function arg1 arg2 &rest other-args)
   (x86-lap-function apply+ ()
+   (:arglist (function arg1 arg2 &rest other-args))
    (check-nargs 3 nil)
    (cmpw ($ '3) (% nargs))
+   (pop (% ra0))
    (ja @no-frame)
    (pushq ($ x8664::reserved-frame-marker))
@@ -533 +527 @@
    @no-discard
    (load-constant funcall temp0)
+   (push (% ra0))
    (jmp-subprim .SPfuncall)
    @bad                                 ; error spreading list.
@@ -541 +536 @@
    (jmp-subprim .SPksignalerr) ))
 
-(lfun-bits #'apply+ (logior $lfbits-rest-bit
-                            (dpb 3 $lfbits-numreq 0)))
+
 
 ;;; This needs to:
@@ -551 +545 @@
 ;;;     (%rax/%xmm0) there
 (defx86lapfunction %do-ff-call ((nfp 0) (frame arg_x) (fp-regs arg_y) (entry arg_z))
+  (popq (% ra0))
   (popq (% rax))
-  (movq (% rbp) (@ @ (% rsp)))
+  (movq (% rbp) (@  (% rsp)))
   (movq (% rsp) (% rbp))
   (movq (% ra0) (@ 8 (% rbp)))