Better initialization of ir2-component-constants on x86-64.
[sbcl.git] / src / compiler / x86-64 / call.lisp
index 13a4511..58538fe 100644 (file)
 ;;; additional noise in the code object header.
 (!def-vm-support-routine select-component-format (component)
   (declare (type component component))
-  ;; The 1+ here is because for the x86 the first constant is a
-  ;; pointer to a list of fixups, or NIL if the code object has none.
-  ;; (If I understand correctly, the fixups are needed at GC copy
-  ;; time because the X86 code isn't relocatable.)
-  ;;
-  ;; KLUDGE: It'd be cleaner to have the fixups entry be a named
-  ;; element of the CODE (aka component) primitive object. However,
-  ;; it's currently a large, tricky, error-prone chore to change
-  ;; the layout of any primitive object, so for the foreseeable future
-  ;; we'll just live with this ugliness. -- WHN 2002-01-02
-  (dotimes (i (1+ code-constants-offset))
+  (dotimes (i code-constants-offset)
     (vector-push-extend nil
                         (ir2-component-constants (component-info component))))
   (values))
   (:generator 1
     nil))
 
+;;; Accessing a slot from an earlier stack frame is definite hackery.
+(define-vop (ancestor-frame-ref)
+  (:args (frame-pointer :scs (descriptor-reg))
+         (variable-home-tn :load-if nil))
+  (:results (value :scs (descriptor-reg any-reg)))
+  (:policy :fast-safe)
+  (:generator 4
+    (aver (sc-is variable-home-tn control-stack))
+    (loadw value frame-pointer
+           (frame-word-offset (tn-offset variable-home-tn)))))
+(define-vop (ancestor-frame-set)
+  (:args (frame-pointer :scs (descriptor-reg))
+         (value :scs (descriptor-reg any-reg)))
+  (:results (variable-home-tn :load-if nil))
+  (:policy :fast-safe)
+  (:generator 4
+    (aver (sc-is variable-home-tn control-stack))
+    (storew value frame-pointer
+            (frame-word-offset (tn-offset variable-home-tn)))))
+
+(macrolet ((define-frame-op
+               (suffix sc stack-sc instruction
+                &optional (ea
+                           `(make-ea :qword
+                                     :base frame-pointer
+                                     :disp (frame-byte-offset
+                                            (tn-offset variable-home-tn)))))
+               (let ((reffer (symbolicate 'ancestor-frame-ref '/ suffix))
+                     (setter (symbolicate 'ancestor-frame-set '/ suffix)))
+                 `(progn
+                    (define-vop (,reffer ancestor-frame-ref)
+                      (:results (value :scs (,sc)))
+                      (:generator 4
+                        (aver (sc-is variable-home-tn ,stack-sc))
+                        (inst ,instruction value
+                              ,ea)))
+                    (define-vop (,setter ancestor-frame-set)
+                      (:args (frame-pointer :scs (descriptor-reg))
+                             (value :scs (,sc)))
+                      (:generator 4
+                        (aver (sc-is variable-home-tn ,stack-sc))
+                        (inst ,instruction ,ea value)))))))
+  (define-frame-op double-float double-reg double-stack movsd)
+  (define-frame-op single-float single-reg single-stack movss)
+  (define-frame-op complex-double-float complex-double-reg complex-double-stack
+    movupd (ea-for-cdf-data-stack variable-home-tn frame-pointer))
+  (define-frame-op complex-single-float complex-single-reg complex-single-stack
+    movq   (ea-for-csf-data-stack variable-home-tn frame-pointer))
+  (define-frame-op signed-byte-64 signed-reg signed-stack mov)
+  (define-frame-op unsigned-byte-64 unsigned-reg unsigned-stack mov)
+  (define-frame-op system-area-pointer sap-reg sap-stack mov))
+
+(defun primitive-type-indirect-cell-type (ptype)
+  (declare (type primitive-type ptype))
+  (macrolet ((foo (&body data)
+                 `(case (primitive-type-name ptype)
+                    ,@(loop for (name stack-sc ref set) in data
+                            collect
+                            `(,name
+                               (load-time-value
+                                (list (primitive-type-or-lose ',name)
+                                      (sc-or-lose ',stack-sc)
+                                      (lambda (node block fp value res)
+                                        (sb!c::vop ,ref node block
+                                                   fp value res))
+                                      (lambda (node block fp new-val value)
+                                        (sb!c::vop ,set node block
+                                                   fp new-val value)))))))))
+    (foo (double-float double-stack
+                       ancestor-frame-ref/double-float
+                       ancestor-frame-set/double-float)
+         (single-float single-stack
+                       ancestor-frame-ref/single-float
+                       ancestor-frame-set/single-float)
+         (complex-double-float complex-double-stack
+                               ancestor-frame-ref/complex-double-float
+                               ancestor-frame-set/complex-double-float)
+         (complex-single-float complex-single-stack
+                               ancestor-frame-ref/complex-single-float
+                               ancestor-frame-set/complex-single-float)
+         (signed-byte-64 signed-stack
+                         ancestor-frame-ref/signed-byte-64
+                         ancestor-frame-set/signed-byte-64)
+         (unsigned-byte-64 unsigned-stack
+                           ancestor-frame-ref/unsigned-byte-64
+                           ancestor-frame-set/unsigned-byte-64)
+         (unsigned-byte-63 unsigned-stack
+                           ancestor-frame-ref/unsigned-byte-64
+                           ancestor-frame-set/unsigned-byte-64)
+         (system-area-pointer sap-stack
+                              ancestor-frame-ref/system-area-pointer
+                              ancestor-frame-set/system-area-pointer))))
+
 (define-vop (xep-allocate-frame)
   (:info start-lab copy-more-arg-follows)
   (:vop-var vop)
       (inst lea rsp-tn
             (make-ea :qword :base rbp-tn
                      :disp (- (* n-word-bytes
-                                 (max 3 (sb-allocated-size 'stack)))))))
+                                 (- (max 3 (sb-allocated-size 'stack))
+                                    sp->fp-offset))))))
 
     (trace-table-entry trace-table-normal)))
 
 ;;; or a multiple-call-local. All it does is allocate stack space for the
 ;;; callee (who has the same size stack as us).
 (define-vop (allocate-frame)
-  (:results (res :scs (any-reg control-stack))
+  (:results (res :scs (any-reg))
             (nfp))
   (:info callee)
   (:ignore nfp callee)
   (:generator 2
-    (move res rsp-tn)
+    (inst lea res (make-ea :qword :base rsp-tn
+                           :disp (- (* sp->fp-offset n-word-bytes))))
     (inst sub rsp-tn (* n-word-bytes (sb-allocated-size 'stack)))))
 
 ;;; Allocate a partial frame for passing stack arguments in a full
 ;;; before it can extend the stack.
 (define-vop (allocate-full-call-frame)
   (:info nargs)
-  (:results (res :scs (any-reg control-stack)))
+  (:results (res :scs (any-reg)))
   (:generator 2
-    (move res rsp-tn)
+    (inst lea res (make-ea :qword :base rsp-tn
+                           :disp (- (* sp->fp-offset n-word-bytes))))
     (inst sub rsp-tn (* (max nargs 3) n-word-bytes))))
 \f
 ;;; Emit code needed at the return-point from an unknown-values call
 ;;;     there are stack values.
 ;;;  -- Reset SP. This must be done whenever other than 1 value is
 ;;;     returned, regardless of the number of values desired.
-(defun default-unknown-values (vop values nvals)
+(defun default-unknown-values (vop values nvals node)
   (declare (type (or tn-ref null) values)
            (type unsigned-byte nvals))
-  (cond
-   ((<= nvals 1)
-    (note-this-location vop :single-value-return)
-    (inst cmov :c rsp-tn rbx-tn))
-   ((<= nvals register-arg-count)
-    (let ((regs-defaulted (gen-label)))
-      (note-this-location vop :unknown-return)
-      (inst jmp :c regs-defaulted)
-      ;; Default the unsupplied registers.
-      (let* ((2nd-tn-ref (tn-ref-across values))
-             (2nd-tn (tn-ref-tn 2nd-tn-ref)))
-        (inst mov 2nd-tn nil-value)
-        (when (> nvals 2)
-          (loop
-            for tn-ref = (tn-ref-across 2nd-tn-ref)
-            then (tn-ref-across tn-ref)
-            for count from 2 below register-arg-count
-            do (inst mov (tn-ref-tn tn-ref) 2nd-tn))))
-      (inst mov rbx-tn rsp-tn)
-      (emit-label regs-defaulted)
-      (inst mov rsp-tn rbx-tn)))
-   ((<= nvals 7)
-    ;; The number of bytes depends on the relative jump instructions.
-    ;; Best case is 31+(n-3)*14, worst case is 35+(n-3)*18. For
-    ;; NVALS=6 that is 73/89 bytes, and for NVALS=7 that is 87/107
-    ;; bytes which is likely better than using the blt below.
-    (let ((regs-defaulted (gen-label))
-          (defaulting-done (gen-label))
-          (default-stack-slots (gen-label)))
-      (note-this-location vop :unknown-return)
-      ;; Branch off to the MV case.
-      (inst jmp :c regs-defaulted)
-      ;; Do the single value case.
-      ;; Default the register args
-      (inst mov rax-tn nil-value)
-      (do ((i 1 (1+ i))
-           (val (tn-ref-across values) (tn-ref-across val)))
-          ((= i (min nvals register-arg-count)))
-        (inst mov (tn-ref-tn val) rax-tn))
-
-      ;; Fake other registers so it looks like we returned with all the
-      ;; registers filled in.
-      (move rbx-tn rsp-tn)
-      (inst jmp default-stack-slots)
-
-      (emit-label regs-defaulted)
-
-      (inst mov rax-tn nil-value)
-      (collect ((defaults))
-        (do ((i register-arg-count (1+ i))
-             (val (do ((i 0 (1+ i))
-                       (val values (tn-ref-across val)))
-                      ((= i register-arg-count) val))
-                  (tn-ref-across val)))
-            ((null val))
-          (let ((default-lab (gen-label))
-                (tn (tn-ref-tn val))
-                (first-stack-arg-p (= i register-arg-count)))
-            (defaults (cons default-lab (cons tn first-stack-arg-p)))
-
-            (inst cmp rcx-tn (fixnumize i))
-            (inst jmp :be default-lab)
-            (when first-stack-arg-p
-              ;; There are stack args so the frame of the callee is
-              ;; still there, save RDX in its first slot temporalily.
-              (storew rdx-tn rbx-tn -1))
-            (loadw rdx-tn rbx-tn (frame-word-offset i))
-            (inst mov tn rdx-tn)))
-
-        (emit-label defaulting-done)
-        (loadw rdx-tn rbx-tn -1)
-        (move rsp-tn rbx-tn)
-
-        (let ((defaults (defaults)))
-          (when defaults
-            (assemble (*elsewhere*)
-              (trace-table-entry trace-table-fun-prologue)
-              (emit-label default-stack-slots)
-              (dolist (default defaults)
-                (emit-label (car default))
-                (when (cddr default)
-                  ;; We are setting the first stack argument to NIL.
-                  ;; The callee's stack frame is dead, save RDX by
-                  ;; pushing it to the stack, it will end up at same
-                  ;; place as in the (STOREW RDX-TN RBX-TN -1) case
-                  ;; above.
-                  (inst push rdx-tn))
-                (inst mov (second default) rax-tn))
-              (inst jmp defaulting-done)
-              (trace-table-entry trace-table-normal)))))))
-   (t
-    (let ((regs-defaulted (gen-label))
-          (restore-edi (gen-label))
-          (no-stack-args (gen-label))
-          (default-stack-vals (gen-label))
-          (count-okay (gen-label)))
-      (note-this-location vop :unknown-return)
-      ;; Branch off to the MV case.
-      (inst jmp :c regs-defaulted)
-
-      ;; Default the register args, and set up the stack as if we
-      ;; entered the MV return point.
-      (inst mov rbx-tn rsp-tn)
-      (inst mov rdi-tn nil-value)
-      (inst mov rsi-tn rdi-tn)
-      ;; Compute a pointer to where to put the [defaulted] stack values.
-      (emit-label no-stack-args)
-      (inst push rdx-tn)
-      (inst push rdi-tn)
-      (inst lea rdi-tn
-            (make-ea :qword :base rbp-tn
-                     :disp (frame-byte-offset register-arg-count)))
-      ;; Load RAX with NIL so we can quickly store it, and set up
-      ;; stuff for the loop.
-      (inst mov rax-tn nil-value)
-      (inst std)
-      (inst mov rcx-tn (- nvals register-arg-count))
-      ;; Jump into the default loop.
-      (inst jmp default-stack-vals)
-
-      ;; The regs are defaulted. We need to copy any stack arguments,
-      ;; and then default the remaining stack arguments.
-      (emit-label regs-defaulted)
-      ;; Save EDI.
-      (storew rdi-tn rbx-tn (frame-word-offset 1))
-      ;; Compute the number of stack arguments, and if it's zero or
-      ;; less, don't copy any stack arguments.
-      (inst sub rcx-tn (fixnumize register-arg-count))
-      (inst jmp :le no-stack-args)
-
-      ;; Throw away any unwanted args.
-      (inst cmp rcx-tn (fixnumize (- nvals register-arg-count)))
-      (inst jmp :be count-okay)
-      (inst mov rcx-tn (fixnumize (- nvals register-arg-count)))
-      (emit-label count-okay)
-      ;; Save the number of stack values.
-      (inst mov rax-tn rcx-tn)
-      ;; Compute a pointer to where the stack args go.
-      (inst lea rdi-tn
-            (make-ea :qword :base rbp-tn
-                     :disp (frame-byte-offset register-arg-count)))
-      ;; Save ESI, and compute a pointer to where the args come from.
-      (storew rsi-tn rbx-tn (frame-word-offset 2))
-      (inst lea rsi-tn
-            (make-ea :qword :base rbx-tn
-                     :disp (frame-byte-offset register-arg-count)))
-      ;; Do the copy.
-      (inst shr rcx-tn word-shift)              ; make word count
-      (inst std)
-      (inst rep)
-      (inst movs :qword)
-      ;; Restore RSI.
-      (loadw rsi-tn rbx-tn (frame-word-offset 2))
-      ;; Now we have to default the remaining args. Find out how many.
-      (inst sub rax-tn (fixnumize (- nvals register-arg-count)))
-      (inst neg rax-tn)
-      ;; If none, then just blow out of here.
-      (inst jmp :le restore-edi)
-      (inst mov rcx-tn rax-tn)
-      (inst shr rcx-tn word-shift)      ; word count
-      ;; Load RAX with NIL for fast storing.
-      (inst mov rax-tn nil-value)
-      ;; Do the store.
-      (emit-label default-stack-vals)
-      (inst rep)
-      (inst stos rax-tn)
-      ;; Restore EDI, and reset the stack.
-      (emit-label restore-edi)
-      (loadw rdi-tn rbx-tn (frame-word-offset 1))
-      (inst mov rsp-tn rbx-tn)
-      (inst cld))))
+  (let ((type (sb!c::basic-combination-derived-type node)))
+    (cond
+      ((<= nvals 1)
+       (note-this-location vop :single-value-return)
+       (cond
+         ((<= (sb!kernel:values-type-max-value-count type)
+              register-arg-count)
+          (when (and (named-type-p type)
+                     (eq nil (named-type-name type)))
+            ;; The function never returns, it may happen that the code
+            ;; ends right here leavig the :SINGLE-VALUE-RETURN note
+            ;; dangling. Let's emit a NOP.
+            (inst nop)))
+         ((not (sb!kernel:values-type-may-be-single-value-p type))
+          (inst mov rsp-tn rbx-tn))
+         (t
+          (inst cmov :c rsp-tn rbx-tn))))
+      ((<= nvals register-arg-count)
+       (note-this-location vop :unknown-return)
+       (when (sb!kernel:values-type-may-be-single-value-p type)
+         (let ((regs-defaulted (gen-label)))
+           (inst jmp :c regs-defaulted)
+           ;; Default the unsupplied registers.
+           (let* ((2nd-tn-ref (tn-ref-across values))
+                  (2nd-tn (tn-ref-tn 2nd-tn-ref)))
+             (inst mov 2nd-tn nil-value)
+             (when (> nvals 2)
+               (loop
+                for tn-ref = (tn-ref-across 2nd-tn-ref)
+                then (tn-ref-across tn-ref)
+                for count from 2 below register-arg-count
+                do (inst mov (tn-ref-tn tn-ref) 2nd-tn))))
+           (inst mov rbx-tn rsp-tn)
+           (emit-label regs-defaulted)))
+       (when (< register-arg-count
+                (sb!kernel:values-type-max-value-count type))
+         (inst mov rsp-tn rbx-tn)))
+      ((<= nvals 7)
+       ;; The number of bytes depends on the relative jump instructions.
+       ;; Best case is 31+(n-3)*14, worst case is 35+(n-3)*18. For
+       ;; NVALS=6 that is 73/89 bytes, and for NVALS=7 that is 87/107
+       ;; bytes which is likely better than using the blt below.
+       (let ((regs-defaulted (gen-label))
+             (defaulting-done (gen-label))
+             (default-stack-slots (gen-label)))
+         (note-this-location vop :unknown-return)
+         ;; Branch off to the MV case.
+         (inst jmp :c regs-defaulted)
+         ;; Do the single value case.
+         ;; Default the register args
+         (inst mov rax-tn nil-value)
+         (do ((i 1 (1+ i))
+              (val (tn-ref-across values) (tn-ref-across val)))
+             ((= i (min nvals register-arg-count)))
+           (inst mov (tn-ref-tn val) rax-tn))
+         ;; Fake other registers so it looks like we returned with all the
+         ;; registers filled in.
+         (move rbx-tn rsp-tn)
+         (inst jmp default-stack-slots)
+         (emit-label regs-defaulted)
+         (inst mov rax-tn nil-value)
+         (collect ((defaults))
+           (do ((i register-arg-count (1+ i))
+                (val (do ((i 0 (1+ i))
+                          (val values (tn-ref-across val)))
+                         ((= i register-arg-count) val))
+                     (tn-ref-across val)))
+               ((null val))
+             (let ((default-lab (gen-label))
+                   (tn (tn-ref-tn val))
+                   (first-stack-arg-p (= i register-arg-count)))
+               (defaults (cons default-lab
+                               (cons tn first-stack-arg-p)))
+               (inst cmp rcx-tn (fixnumize i))
+               (inst jmp :be default-lab)
+               (when first-stack-arg-p
+                 ;; There are stack args so the frame of the callee is
+                 ;; still there, save RDX in its first slot temporalily.
+                 (storew rdx-tn rbx-tn (frame-word-offset sp->fp-offset)))
+               (loadw rdx-tn rbx-tn (frame-word-offset (+ sp->fp-offset i)))
+               (inst mov tn rdx-tn)))
+           (emit-label defaulting-done)
+           (loadw rdx-tn rbx-tn (frame-word-offset sp->fp-offset))
+           (move rsp-tn rbx-tn)
+           (let ((defaults (defaults)))
+             (when defaults
+               (assemble (*elsewhere*)
+                 (trace-table-entry trace-table-fun-prologue)
+                 (emit-label default-stack-slots)
+                 (dolist (default defaults)
+                   (emit-label (car default))
+                   (when (cddr default)
+                     ;; We are setting the first stack argument to NIL.
+                     ;; The callee's stack frame is dead, save RDX by
+                     ;; pushing it to the stack, it will end up at same
+                     ;; place as in the (STOREW RDX-TN RBX-TN -1) case
+                     ;; above.
+                     (inst push rdx-tn))
+                   (inst mov (second default) rax-tn))
+                 (inst jmp defaulting-done)
+                 (trace-table-entry trace-table-normal)))))))
+      (t
+       (let ((regs-defaulted (gen-label))
+             (restore-edi (gen-label))
+             (no-stack-args (gen-label))
+             (default-stack-vals (gen-label))
+             (count-okay (gen-label)))
+         (note-this-location vop :unknown-return)
+         ;; Branch off to the MV case.
+         (inst jmp :c regs-defaulted)
+         ;; Default the register args, and set up the stack as if we
+         ;; entered the MV return point.
+         (inst mov rbx-tn rsp-tn)
+         (inst mov rdi-tn nil-value)
+         (inst mov rsi-tn rdi-tn)
+         ;; Compute a pointer to where to put the [defaulted] stack values.
+         (emit-label no-stack-args)
+         (inst push rdx-tn)
+         (inst push rdi-tn)
+         (inst lea rdi-tn
+               (make-ea :qword :base rbp-tn
+                        :disp (frame-byte-offset register-arg-count)))
+         ;; Load RAX with NIL so we can quickly store it, and set up
+         ;; stuff for the loop.
+         (inst mov rax-tn nil-value)
+         (inst std)
+         (inst mov rcx-tn (- nvals register-arg-count))
+         ;; Jump into the default loop.
+         (inst jmp default-stack-vals)
+         ;; The regs are defaulted. We need to copy any stack arguments,
+         ;; and then default the remaining stack arguments.
+         (emit-label regs-defaulted)
+         ;; Compute the number of stack arguments, and if it's zero or
+         ;; less, don't copy any stack arguments.
+         (inst sub rcx-tn (fixnumize register-arg-count))
+         (inst jmp :le no-stack-args)
+         ;; Save EDI.
+         (storew rdi-tn rbx-tn (frame-word-offset (+ sp->fp-offset 1)))
+         ;; Throw away any unwanted args.
+         (inst cmp rcx-tn (fixnumize (- nvals register-arg-count)))
+         (inst jmp :be count-okay)
+         (inst mov rcx-tn (fixnumize (- nvals register-arg-count)))
+         (emit-label count-okay)
+         ;; Save the number of stack values.
+         (inst mov rax-tn rcx-tn)
+         ;; Compute a pointer to where the stack args go.
+         (inst lea rdi-tn
+               (make-ea :qword :base rbp-tn
+                        :disp (frame-byte-offset register-arg-count)))
+         ;; Save ESI, and compute a pointer to where the args come from.
+         (storew rsi-tn rbx-tn (frame-word-offset (+ sp->fp-offset 2)))
+         (inst lea rsi-tn
+               (make-ea :qword :base rbx-tn
+                        :disp (frame-byte-offset
+                               (+ sp->fp-offset register-arg-count))))
+         ;; Do the copy.
+         (inst shr rcx-tn n-fixnum-tag-bits)   ; make word count
+         (inst std)
+         (inst rep)
+         (inst movs :qword)
+         ;; Restore RSI.
+         (loadw rsi-tn rbx-tn (frame-word-offset (+ sp->fp-offset 2)))
+         ;; Now we have to default the remaining args. Find out how many.
+         (inst sub rax-tn (fixnumize (- nvals register-arg-count)))
+         (inst neg rax-tn)
+         ;; If none, then just blow out of here.
+         (inst jmp :le restore-edi)
+         (inst mov rcx-tn rax-tn)
+         (inst shr rcx-tn n-fixnum-tag-bits)   ; word count
+         ;; Load RAX with NIL for fast storing.
+         (inst mov rax-tn nil-value)
+         ;; Do the store.
+         (emit-label default-stack-vals)
+         (inst rep)
+         (inst stos rax-tn)
+         ;; Restore EDI, and reset the stack.
+         (emit-label restore-edi)
+         (loadw rdi-tn rbx-tn (frame-word-offset (+ sp->fp-offset 1)))
+         (inst mov rsp-tn rbx-tn)
+         (inst cld)))))
   (values))
 \f
 ;;;; unknown values receiving
 ;;; explicitly allocate these TNs, since their lifetimes overlap with
 ;;; the results start and count. (Also, it's nice to be able to target
 ;;; them.)
-(defun receive-unknown-values (args nargs start count)
+(defun receive-unknown-values (args nargs start count node)
   (declare (type tn args nargs start count))
-  (let ((variable-values (gen-label))
+  (let ((type (sb!c::basic-combination-derived-type node))
+        (variable-values (gen-label))
         (stack-values (gen-label))
         (done (gen-label)))
-    (inst jmp :c variable-values)
-
-    (cond ((location= start (first *register-arg-tns*))
-           (inst push (first *register-arg-tns*))
-           (inst lea start (make-ea :qword :base rsp-tn :disp n-word-bytes)))
-          (t (inst mov start rsp-tn)
-             (inst push (first *register-arg-tns*))))
-    (inst mov count (fixnumize 1))
-    (inst jmp done)
-
-    (emit-label variable-values)
+    (when (sb!kernel:values-type-may-be-single-value-p type)
+      (inst jmp :c variable-values)
+      (cond ((location= start (first *register-arg-tns*))
+             (inst push (first *register-arg-tns*))
+             (inst lea start (make-ea :qword :base rsp-tn :disp n-word-bytes)))
+            (t (inst mov start rsp-tn)
+               (inst push (first *register-arg-tns*))))
+      (inst mov count (fixnumize 1))
+      (inst jmp done)
+      (emit-label variable-values))
     ;; The stack frame is burnt and RETurned from if there are no
     ;; stack values. In this case quickly reallocate sufficient space.
-    (inst cmp nargs (fixnumize register-arg-count))
-    (inst jmp :g stack-values)
-    (inst sub rsp-tn nargs)
-    (emit-label stack-values)
+    (when (<= (sb!kernel:values-type-min-value-count type)
+              register-arg-count)
+      (inst cmp nargs (fixnumize register-arg-count))
+      (inst jmp :g stack-values)
+      #!+#.(cl:if (cl:= sb!vm:word-shift sb!vm:n-fixnum-tag-bits) '(and) '(or))
+      (inst sub rsp-tn nargs)
+      #!-#.(cl:if (cl:= sb!vm:word-shift sb!vm:n-fixnum-tag-bits) '(and) '(or))
+      (progn
+        ;; FIXME: This can't be efficient, but LEA (my first choice)
+        ;; doesn't do subtraction.
+        (inst shl nargs (- word-shift n-fixnum-tag-bits))
+        (inst sub rsp-tn nargs)
+        (inst shr nargs (- word-shift n-fixnum-tag-bits)))
+      (emit-label stack-values))
     ;; dtc: this writes the registers onto the stack even if they are
     ;; not needed, only the number specified in rcx are used and have
     ;; stack allocated to them. No harm is done.
     (loop
       for arg in *register-arg-tns*
       for i downfrom -1
+      for j below (sb!kernel:values-type-max-value-count type)
       do (storew arg args i))
     (move start args)
     (move count nargs)
                (= (tn-offset return-pc) return-pc-save-offset))
     (error "return-pc not on stack in standard save location?")))
 
+;;; The local call convention doesn't fit that well with x86-style
+;;; calls. Emit a header for local calls to pop the return address
+;;; in the right place.
+(defun emit-block-header (start-label trampoline-label fall-thru-p alignp)
+  (when (and fall-thru-p trampoline-label)
+    (inst jmp start-label))
+  (when trampoline-label
+    (emit-label trampoline-label)
+    (popw rbp-tn (frame-word-offset return-pc-save-offset)))
+  (when alignp
+    (emit-alignment n-lowtag-bits :long-nop))
+  (emit-label start-label))
+
 ;;; Non-TR local call for a fixed number of values passed according to
 ;;; the unknown values convention.
 ;;;
   (:args (fp)
          (nfp)
          (args :more t))
-  (:temporary (:sc unsigned-reg) return-label)
   (:results (values :more t))
   (:save-p t)
   (:move-args :local-call)
   (:info arg-locs callee target nvals)
   (:vop-var vop)
-  (:ignore nfp arg-locs args #+nil callee)
+  (:ignore nfp arg-locs args callee)
+  (:node-var node)
   (:generator 5
     (trace-table-entry trace-table-call-site)
     (move rbp-tn fp)
-
-    (let ((ret-tn (callee-return-pc-tn callee)))
-      #+nil
-      (format t "*call-local ~S; tn-kind ~S; tn-save-tn ~S; its tn-kind ~S~%"
-              ret-tn (sb!c::tn-kind ret-tn) (sb!c::tn-save-tn ret-tn)
-              (sb!c::tn-kind (sb!c::tn-save-tn ret-tn)))
-
-      ;; Is the return-pc on the stack or in a register?
-      (sc-case ret-tn
-        ((sap-stack)
-         (unless (= (tn-offset ret-tn) return-pc-save-offset)
-           (error "ret-tn ~A in wrong stack slot" ret-tn))
-         #+nil (format t "*call-local: ret-tn on stack; offset=~S~%"
-                       (tn-offset ret-tn))
-         (inst lea return-label (make-fixup nil :code-object RETURN))
-         (storew return-label rbp-tn (frame-word-offset (tn-offset ret-tn))))
-        (t
-         (error "ret-tn ~A in sap-reg" ret-tn))))
-
     (note-this-location vop :call-site)
-    (inst jmp target)
-    RETURN
-    (default-unknown-values vop values nvals)
+    (inst call target)
+    (default-unknown-values vop values nvals node)
     (trace-table-entry trace-table-normal)))
 
 ;;; Non-TR local call for a variable number of return values passed according
   (:args (fp)
          (nfp)
          (args :more t))
-  (:temporary (:sc unsigned-reg) return-label)
   (:save-p t)
   (:move-args :local-call)
   (:info save callee target)
-  (:ignore args save nfp #+nil callee)
+  (:ignore args save nfp callee)
   (:vop-var vop)
+  (:node-var node)
   (:generator 20
     (trace-table-entry trace-table-call-site)
     (move rbp-tn fp)
-
-    (let ((ret-tn (callee-return-pc-tn callee)))
-      #+nil
-      (format t "*multiple-call-local ~S; tn-kind ~S; tn-save-tn ~S; its tn-kind ~S~%"
-              ret-tn (sb!c::tn-kind ret-tn) (sb!c::tn-save-tn ret-tn)
-              (sb!c::tn-kind (sb!c::tn-save-tn ret-tn)))
-
-      ;; Is the return-pc on the stack or in a register?
-      (sc-case ret-tn
-        ((sap-stack)
-         #+nil (format t "*multiple-call-local: ret-tn on stack; offset=~S~%"
-                       (tn-offset ret-tn))
-         ;; Stack
-         (inst lea return-label (make-fixup nil :code-object RETURN))
-         (storew return-label rbp-tn (frame-word-offset (tn-offset ret-tn))))
-        (t
-         (error "multiple-call-local: return-pc not on stack."))))
-
     (note-this-location vop :call-site)
-    (inst jmp target)
-    RETURN
+    (inst call target)
     (note-this-location vop :unknown-return)
-    (receive-unknown-values values-start nvals start count)
+    (receive-unknown-values values-start nvals start count node)
     (trace-table-entry trace-table-normal)))
 \f
 ;;;; local call with known values return
   (:args (fp)
          (nfp)
          (args :more t))
-  (:temporary (:sc unsigned-reg) return-label)
   (:results (res :more t))
   (:move-args :local-call)
   (:save-p t)
   (:info save callee target)
-  (:ignore args res save nfp #+nil callee)
+  (:ignore args res save nfp callee)
   (:vop-var vop)
   (:generator 5
     (trace-table-entry trace-table-call-site)
     (move rbp-tn fp)
-
-    (let ((ret-tn (callee-return-pc-tn callee)))
-
-      #+nil
-      (format t "*known-call-local ~S; tn-kind ~S; tn-save-tn ~S; its tn-kind ~S~%"
-              ret-tn (sb!c::tn-kind ret-tn) (sb!c::tn-save-tn ret-tn)
-              (sb!c::tn-kind (sb!c::tn-save-tn ret-tn)))
-
-      ;; Is the return-pc on the stack or in a register?
-      (sc-case ret-tn
-        ((sap-stack)
-         #+nil (format t "*known-call-local: ret-tn on stack; offset=~S~%"
-                       (tn-offset ret-tn))
-         ;; Stack
-         (inst lea return-label (make-fixup nil :code-object RETURN))
-         (storew return-label rbp-tn (frame-word-offset (tn-offset ret-tn))))
-        (t
-         (error "known-call-local: return-pc not on stack."))))
-
     (note-this-location vop :call-site)
-    (inst jmp target)
-    RETURN
+    (inst call target)
     (note-this-location vop :known-return)
     (trace-table-entry trace-table-normal)))
 \f
     (check-ocfp-and-return-pc old-fp return-pc)
     (trace-table-entry trace-table-fun-epilogue)
     ;; Zot all of the stack except for the old-fp and return-pc.
-    (inst lea rsp-tn
-          (make-ea :qword :base rbp-tn
-                   :disp (frame-byte-offset ocfp-save-offset)))
+    (inst mov rsp-tn rbp-tn)
     (inst pop rbp-tn)
     (inst ret)
     (trace-table-entry trace-table-normal)))
                                       :from (:argument 1)
                                       :to (:argument 2))
                                  old-fp-tmp)))
+               ,@(unless (eq return :tail)
+                   '((:node-var node)))
 
                (:generator ,(+ (if named 5 0)
                                (if variable 19 1)
                               ;; Compute the number of arguments.
                               (noise '(inst mov rcx new-fp))
                               (noise '(inst sub rcx rsp-tn))
+                              #.(unless (= word-shift n-fixnum-tag-bits)
+                                  '(noise '(inst shr rcx
+                                            (- word-shift n-fixnum-tag-bits))))
                               ;; Move the necessary args to registers,
                               ;; this moves them all even if they are
                               ;; not all needed.
                           ;; there are at least 3 slots. This hack
                           ;; just adds 3 more.
                           ,(if variable
-                               '(inst sub rsp-tn (fixnumize 3)))
+                               '(inst sub rsp-tn (* 3 n-word-bytes)))
+
+                          ;; Bias the new-fp for use as an fp
+                          ,(if variable
+                               '(inst sub new-fp (* sp->fp-offset n-word-bytes)))
 
                           ;; Save the fp
                           (storew rbp-tn new-fp
                                            fun-pointer-lowtag))))
                ,@(ecase return
                    (:fixed
-                    '((default-unknown-values vop values nvals)))
+                    '((default-unknown-values vop values nvals node)))
                    (:unknown
                     '((note-this-location vop :unknown-return)
-                      (receive-unknown-values values-start nvals start count)))
+                      (receive-unknown-values values-start nvals start count
+                                              node)))
                    (:tail))
                (trace-table-entry trace-table-normal)))))
 
     (check-ocfp-and-return-pc old-fp return-pc)
     (trace-table-entry trace-table-fun-epilogue)
     ;; Drop stack above old-fp
-    (inst lea rsp-tn (make-ea :qword :base rbp-tn
-                              :disp (frame-byte-offset (tn-offset old-fp))))
+    (inst mov rsp-tn rbp-tn)
     ;; Clear the multiple-value return flag
     (inst clc)
     ;; Restore the old frame pointer
       (error "nvalues is 1"))
     (trace-table-entry trace-table-fun-epilogue)
     ;; Establish the values pointer and values count.
-    (move rbx rbp-tn)
+    (inst lea rbx (make-ea :qword :base rbp-tn
+                           :disp (* sp->fp-offset n-word-bytes)))
     (if (zerop nvals)
         (zeroize rcx) ; smaller
         (inst mov rcx (fixnumize nvals)))
-    ;; Clear as much of the stack as possible, but not past the old
-    ;; frame address.
-    (inst lea rsp-tn
-          (make-ea :qword :base rbx
-                   :disp (frame-byte-offset
-                          (if (< register-arg-count nvals)
-                              (1- nvals)
-                              ocfp-save-offset))))
     ;; Pre-default any argument register that need it.
     (when (< nvals register-arg-count)
       (let* ((arg-tns (nthcdr nvals (list a0 a1 a2)))
     ;; stack and we've changed the stack pointer. So we have to
     ;; tell it to index off of RBX instead of RBP.
     (cond ((<= nvals register-arg-count)
+           (inst mov rsp-tn rbp-tn)
            (inst pop rbp-tn)
            (inst ret))
           (t
            ;; Some values are on the stack after RETURN-PC and OLD-FP,
            ;; can't return normally and some slots of the frame will
            ;; be used as temporaries by the receiver.
+           ;;
+           ;; Clear as much of the stack as possible, but not past the
+           ;; old frame address.
+           (inst lea rsp-tn
+                 (make-ea :qword :base rbp-tn
+                          :disp (frame-byte-offset (1- nvals))))
            (move rbp-tn old-fp)
            (inst push (make-ea :qword :base rbx
-                               :disp (frame-byte-offset (tn-offset return-pc))))
+                               :disp (frame-byte-offset
+                                      (+ sp->fp-offset
+                                         (tn-offset return-pc)))))
            (inst ret)))
 
     (trace-table-entry trace-table-normal)))
 ;;; assembly-routine.
 ;;;
 ;;; The assembly routine takes the following args:
-;;;  RAX -- the return-pc to finally jump to.
-;;;  RBX -- pointer to where to put the values.
 ;;;  RCX -- number of values to find there.
 ;;;  RSI -- pointer to where to find the values.
 (define-vop (return-multiple)
         (inst jmp :ne not-single)
         ;; Return with one value.
         (loadw a0 vals -1)
-        (inst lea rsp-tn (make-ea :qword :base rbp-tn
-                                  :disp (frame-byte-offset ocfp-save-offset)))
+        ;; Clear the stack until ocfp.
+        (inst mov rsp-tn rbp-tn)
         ;; clear the multiple-value return flag
         (inst clc)
         ;; Out of here.
            (inst cmp rcx-tn (fixnumize fixed))
            (inst jmp :be JUST-ALLOC-FRAME)))
 
+    ;; Create a negated copy of the number of arguments to allow us to
+    ;; use EA calculations in order to do scaled subtraction.
+    (inst mov temp rcx-tn)
+    (inst neg temp)
+
     ;; Allocate the space on the stack.
-    ;; stack = rbp - (max 3 frame-size) - (nargs - fixed)
-    (inst lea rbx-tn
+    ;; stack = rbp + sp->fp-offset - (max 3 frame-size) - (nargs - fixed)
+    (inst lea rsp-tn
           (make-ea :qword :base rbp-tn
-                   :disp (- (fixnumize fixed)
-                            (* n-word-bytes
+                   :index temp :scale (ash 1 (- word-shift n-fixnum-tag-bits))
+                   :disp (* n-word-bytes
+                            (- (+ sp->fp-offset fixed)
                                (max 3 (sb-allocated-size 'stack))))))
-    (inst sub rbx-tn rcx-tn)  ; Got the new stack in rbx
-    (inst mov rsp-tn rbx-tn)
 
     ;; Now: nargs>=1 && nargs>fixed
 
            (inst sub rcx-tn (fixnumize fixed))))
 
     ;; Initialize R8 to be the end of args.
-    (inst mov source rbp-tn)
-    (inst sub source rbx-tn)
+    (inst lea source (make-ea :qword :base rbp-tn
+                              :index temp :scale (ash 1 (- word-shift n-fixnum-tag-bits))
+                              :disp (* sp->fp-offset n-word-bytes)))
 
     ;; We need to copy from downwards up to avoid overwriting some of
     ;; the yet uncopied args. So we need to use R9 as the copy index
     (inst mov temp (make-ea :qword :base source :index copy-index))
     (inst mov (make-ea :qword :base rsp-tn :index copy-index) temp)
     (inst add copy-index n-word-bytes)
-    (inst sub rcx-tn n-word-bytes)
+    (inst sub rcx-tn (fixnumize 1))
     (inst jmp :nz COPY-LOOP)
 
     DO-REGS
           ( nil )
         ;; Store it relative to rbp
         (inst mov (make-ea :qword :base rbp-tn
-                           :disp (- (* n-word-bytes
-                                       (+ 1 (- i fixed)
-                                          (max 3 (sb-allocated-size 'stack))))))
+                           :disp (* n-word-bytes
+                                    (- sp->fp-offset
+                                       (+ 1
+                                          (- i fixed)
+                                          (max 3 (sb-allocated-size
+                                                  'stack))))))
               (nth i *register-arg-tns*))
 
         (incf i)
     JUST-ALLOC-FRAME
     (inst lea rsp-tn
           (make-ea :qword :base rbp-tn
-                   :disp (- (* n-word-bytes
+                   :disp (* n-word-bytes
+                            (- sp->fp-offset
                                (max 3 (sb-allocated-size 'stack))))))
 
     DONE))
             (keyword :scs (descriptor-reg any-reg)))
   (:result-types * *)
   (:generator 4
-     (inst mov value (make-ea :qword :base object :index index))
+     (inst mov value (make-ea :qword :base object :index index
+                              :scale (ash 1 (- word-shift n-fixnum-tag-bits))))
      (inst mov keyword (make-ea :qword :base object :index index
+                                :scale (ash 1 (- word-shift n-fixnum-tag-bits))
                                 :disp n-word-bytes))))
 
 (define-vop (more-arg)
   (:generator 4
     (move value index)
     (inst neg value)
-    (inst mov value (make-ea :qword :base object :index value))))
+    (inst mov value (make-ea :qword :base object :index value
+                             :scale (ash 1 (- word-shift n-fixnum-tag-bits))))))
 
 ;;; Turn more arg (context, count) into a list.
 (define-vop (listify-rest-args)
       ;; Check to see whether there are no args, and just return NIL if so.
       (inst mov result nil-value)
       (inst jrcxz done)
-      (inst lea dst (make-ea :qword :base rcx :index rcx))
+      (inst lea dst (make-ea :qword :index rcx :scale (ash 2 (- word-shift n-fixnum-tag-bits))))
       (maybe-pseudo-atomic stack-allocate-p
        (allocation dst dst node stack-allocate-p list-pointer-lowtag)
-       (inst shr rcx (1- n-lowtag-bits))
        ;; Set decrement mode (successive args at lower addresses)
        (inst std)
        ;; Set up the result.
        (storew dst dst -1 list-pointer-lowtag)
        (emit-label enter)
        ;; Grab one value and stash it in the car of this cons.
-       (inst lods rax)
+       (inst mov rax (make-ea :qword :base src))
+       (inst sub src n-word-bytes)
        (storew rax dst 0 list-pointer-lowtag)
        ;; Go back for more.
-       (inst sub rcx 1)
+       (inst sub rcx (fixnumize 1))
        (inst jmp :nz loop)
        ;; NIL out the last cons.
        (storew nil-value dst 1 list-pointer-lowtag)
     ;; SP at this point points at the last arg pushed.
     ;; Point to the first more-arg, not above it.
     (inst lea context (make-ea :qword :base rsp-tn
-                               :index count :scale 1
-                               :disp (- (+ (fixnumize fixed) n-word-bytes))))
+                               :index count
+                               :scale (ash 1 (- word-shift n-fixnum-tag-bits))
+                               :disp (- (* (1+ fixed) n-word-bytes))))
     (unless (zerop fixed)
       (inst sub count (fixnumize fixed)))))