X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;ds=sidebyside;f=src%2Fcompiler%2Fx86%2Fcall.lisp;h=772f6f75035bc264c5445a19e430eb5c838c560d;hb=29a9ccc860532b32c566aec095f570e999a9c52c;hp=a0fb814838ba2aa4ab91d8c43d96e79ed3d268d6;hpb=adf0d51d2bde8b723276bacf94641df9aa5ae561;p=sbcl.git diff --git a/src/compiler/x86/call.lisp b/src/compiler/x86/call.lisp index a0fb814..772f6f7 100644 --- a/src/compiler/x86/call.lisp +++ b/src/compiler/x86/call.lisp @@ -10,136 +10,96 @@ ;;;; files for more information. (in-package "SB!VM") - -(file-comment - "$Header$") ;;;; interfaces to IR2 conversion ;;; Return a wired TN describing the N'th full call argument passing ;;; location. -(def-vm-support-routine standard-argument-location (n) +(!def-vm-support-routine standard-argument-location (n) (declare (type unsigned-byte n)) (if (< n register-arg-count) (make-wired-tn *backend-t-primitive-type* descriptor-reg-sc-number - (nth n register-arg-offsets)) + (nth n *register-arg-offsets*)) (make-wired-tn *backend-t-primitive-type* control-stack-sc-number n))) ;;; Make a passing location TN for a local call return PC. ;;; ;;; Always wire the return PC location to the stack in its standard ;;; location. -;;; -;;; No problems. -;#+nil -(def-vm-support-routine make-return-pc-passing-location (standard) +(!def-vm-support-routine make-return-pc-passing-location (standard) (declare (ignore standard)) (make-wired-tn (primitive-type-or-lose 'system-area-pointer) sap-stack-sc-number return-pc-save-offset)) -;;; If standard is true, then use the standard (full call) location, -;;; otherwise use any legal location. -;;; -;;; No problems. -#+nil -(def-vm-support-routine make-return-pc-passing-location (standard) - (let ((ptype (primitive-type-or-lose 'system-area-pointer))) - (if standard - (make-wired-tn ptype sap-stack-sc-number return-pc-save-offset) - (make-normal-tn ptype)))) - -;;; Similar to Make-Return-PC-Passing-Location, but makes a location to pass -;;; Old-FP in. -;;; -;;; This is wired in both the standard and the local-call -;;; conventions, because we want to be able to assume it's always there. -;;; Besides, the x86 doesn't have enough registers to really make it -;;; profitable to pass it in a register. + +;;; Similar to Make-Return-PC-Passing-Location, but makes a location +;;; to pass Old-FP in. ;;; -;;; No problems -;#+nil -(def-vm-support-routine make-old-fp-passing-location (standard) +;;; This is wired in both the standard and the local-call conventions, +;;; because we want to be able to assume it's always there. Besides, +;;; the x86 doesn't have enough registers to really make it profitable +;;; to pass it in a register. +(!def-vm-support-routine make-old-fp-passing-location (standard) (declare (ignore standard)) (make-wired-tn *fixnum-primitive-type* control-stack-sc-number ocfp-save-offset)) -;;; If standard is true, then use the standard (full call) location, -;;; otherwise use any legal location. -;;; -;;; No problems. -#+nil -(def-vm-support-routine make-old-fp-passing-location (standard) - (if standard - (make-wired-tn *fixnum-primitive-type* control-stack-sc-number - ocfp-save-offset) - (make-normal-tn *fixnum-primitive-type*))) ;;; Make the TNs used to hold Old-FP and Return-PC within the current -;;; function. We treat these specially so that the debugger can find them at a -;;; known location. +;;; function. We treat these specially so that the debugger can find +;;; them at a known location. ;;; ;;; Without using a save-tn - which does not make much sense if it is -;;; wire to the stack? No problems. -(def-vm-support-routine make-old-fp-save-location (env) - (environment-debug-live-tn (make-wired-tn *fixnum-primitive-type* - control-stack-sc-number - ocfp-save-offset) - env)) -;;; Using a save-tn. No problems. -#+nil -(def-vm-support-routine make-old-fp-save-location (env) - (specify-save-tn - (environment-debug-live-tn (make-normal-tn *fixnum-primitive-type*) env) - (make-wired-tn *fixnum-primitive-type* control-stack-sc-number - ocfp-save-offset))) - -;;; Without using a save-tn - which does not make much sense if it is -;;; wire to the stack? No problems. -(def-vm-support-routine make-return-pc-save-location (env) - (environment-debug-live-tn +;;; wire to the stack? +(!def-vm-support-routine make-old-fp-save-location (env) + (physenv-debug-live-tn (make-wired-tn *fixnum-primitive-type* + control-stack-sc-number + ocfp-save-offset) + env)) + +(!def-vm-support-routine make-return-pc-save-location (env) + (physenv-debug-live-tn (make-wired-tn (primitive-type-or-lose 'system-area-pointer) sap-stack-sc-number return-pc-save-offset) env)) -;;; Using a save-tn. No problems. -#+nil -(def-vm-support-routine make-return-pc-save-location (env) - (let ((ptype (primitive-type-or-lose 'system-area-pointer))) - (specify-save-tn - (environment-debug-live-tn (make-normal-tn ptype) env) - (make-wired-tn ptype sap-stack-sc-number return-pc-save-offset)))) ;;; Make a TN for the standard argument count passing location. We only ;;; need to make the standard location, since a count is never passed when we ;;; are using non-standard conventions. -(def-vm-support-routine make-argument-count-location () +(!def-vm-support-routine make-argument-count-location () (make-wired-tn *fixnum-primitive-type* any-reg-sc-number ecx-offset)) - ;;; Make a TN to hold the number-stack frame pointer. This is allocated ;;; once per component, and is component-live. -(def-vm-support-routine make-nfp-tn () +(!def-vm-support-routine make-nfp-tn () (make-restricted-tn *fixnum-primitive-type* ignore-me-sc-number)) -(def-vm-support-routine make-stack-pointer-tn () +(!def-vm-support-routine make-stack-pointer-tn () (make-normal-tn *fixnum-primitive-type*)) -(def-vm-support-routine make-number-stack-pointer-tn () +(!def-vm-support-routine make-number-stack-pointer-tn () (make-restricted-tn *fixnum-primitive-type* ignore-me-sc-number)) ;;; Return a list of TNs that can be used to represent an unknown-values ;;; continuation within a function. -(def-vm-support-routine make-unknown-values-locations () +(!def-vm-support-routine make-unknown-values-locations () (list (make-stack-pointer-tn) (make-normal-tn *fixnum-primitive-type*))) - -;;; This function is called by the Entry-Analyze phase, allowing -;;; VM-dependent initialization of the IR2-Component structure. We push -;;; placeholder entries in the Constants to leave room for additional -;;; noise in the code object header. -;;; -;;; For the x86 the first constant is a pointer to a list of fixups, -;;; or nil if the code object has none. -(def-vm-support-routine select-component-format (component) +;;; This function is called by the ENTRY-ANALYZE phase, allowing +;;; VM-dependent initialization of the IR2-COMPONENT structure. We +;;; push placeholder entries in the CONSTANTS to leave room for +;;; 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)) (vector-push-extend nil (ir2-component-constants (component-info component)))) @@ -147,7 +107,7 @@ ;;;; frame hackery -;;; Used for setting up the Old-FP in local call. +;;; This is used for setting up the Old-FP in local call. (define-vop (current-fp) (:results (val :scs (any-reg control-stack))) (:generator 1 @@ -160,19 +120,16 @@ (:generator 1 nil)) - (define-vop (xep-allocate-frame) (:info start-lab copy-more-arg-follows) (:vop-var vop) (:generator 1 - ;; Make sure the function is aligned, and drop a label pointing to this - ;; function header. - (align lowtag-bits) - (trace-table-entry trace-table-function-prologue) + (align n-lowtag-bits) + (trace-table-entry trace-table-fun-prologue) (emit-label start-lab) ;; Skip space for the function header. - (inst function-header-word) - (dotimes (i (1- sb!vm:function-code-offset)) + (inst simple-fun-header-word) + (dotimes (i (1- simple-fun-code-offset)) (inst dword 0)) ;; The start of the actual code. @@ -186,7 +143,7 @@ ;; The args fit within the frame so just allocate the frame. (inst lea esp-tn (make-ea :dword :base ebp-tn - :disp (- (* sb!vm:word-bytes + :disp (- (* n-word-bytes (max 3 (sb-allocated-size 'stack))))))) (trace-table-entry trace-table-normal))) @@ -201,42 +158,43 @@ (:ignore nfp callee) (:generator 2 (move res esp-tn) - (inst sub esp-tn (* sb!vm:word-bytes (sb-allocated-size 'stack))))) + (inst sub esp-tn (* n-word-bytes (sb-allocated-size 'stack))))) -;;; Allocate a partial frame for passing stack arguments in a full call. Nargs -;;; is the number of arguments passed. We allocate at least 3 slots, because -;;; the XEP noise is going to want to use them before it can extend the stack. +;;; Allocate a partial frame for passing stack arguments in a full +;;; call. NARGS is the number of arguments passed. We allocate at +;;; least 3 slots, because the XEP noise is going to want to use them +;;; before it can extend the stack. (define-vop (allocate-full-call-frame) (:info nargs) (:results (res :scs (any-reg control-stack))) (:generator 2 (move res esp-tn) - (inst sub esp-tn (* (max nargs 3) sb!vm:word-bytes)))) - - + (inst sub esp-tn (* (max nargs 3) n-word-bytes)))) -;;; Emit code needed at the return-point from an unknown-values call for a -;;; fixed number of values. Values is the head of the TN-Ref list for the -;;; locations that the values are to be received into. Nvals is the number of -;;; values that are to be received (should equal the length of Values). +;;; Emit code needed at the return-point from an unknown-values call +;;; for a fixed number of values. Values is the head of the TN-Ref +;;; list for the locations that the values are to be received into. +;;; Nvals is the number of values that are to be received (should +;;; equal the length of Values). ;;; ;;; Move-Temp is a Descriptor-Reg TN used as a temporary. ;;; -;;; This code exploits the fact that in the unknown-values convention, a -;;; single value return returns at the return PC + 2, whereas a return of other -;;; than one value returns directly at the return PC. +;;; This code exploits the fact that in the unknown-values convention, +;;; a single value return returns at the return PC + 2, whereas a +;;; return of other than one value returns directly at the return PC. ;;; -;;; If 0 or 1 values are expected, then we just emit an instruction to reset -;;; the SP (which will only be executed when other than 1 value is returned.) +;;; If 0 or 1 values are expected, then we just emit an instruction to +;;; reset the SP (which will only be executed when other than 1 value +;;; is returned.) ;;; ;;; In the general case we have to do three things: -;;; -- Default unsupplied register values. This need only be done when a -;;; single value is returned, since register values are defaulted by the -;;; called in the non-single case. -;;; -- Default unsupplied stack values. This needs to be done whenever there -;;; are stack values. -;;; -- Reset SP. This must be done whenever other than 1 value is returned, -;;; regardless of the number of values desired. +;;; -- Default unsupplied register values. This need only be done +;;; when a single value is returned, since register values are +;;; defaulted by the called in the non-single case. +;;; -- Default unsupplied stack values. This needs to be done whenever +;;; 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) (declare (type (or tn-ref null) values) (type unsigned-byte nvals)) @@ -257,15 +215,15 @@ for tn-ref = (tn-ref-across 2nd-tn-ref) then (tn-ref-across tn-ref) for count from 2 below register-arg-count - do count (inst mov (tn-ref-tn tn-ref) 2nd-tn)))) + do (inst mov (tn-ref-tn tn-ref) 2nd-tn)))) (inst mov ebx-tn esp-tn) (emit-label regs-defaulted) (inst mov esp-tn ebx-tn))) ((<= nvals 7) - ;; 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. + ;; 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))) @@ -313,7 +271,7 @@ (let ((defaults (defaults))) (when defaults (assemble (*elsewhere*) - (trace-table-entry trace-table-function-prologue) + (trace-table-entry trace-table-fun-prologue) (emit-label default-stack-slots) (dolist (default defaults) (emit-label (car default)) @@ -331,8 +289,8 @@ ;; Branch off to the MV case. (inst jmp-short regs-defaulted) - ;; Default the register args, and set up the stack as if we entered - ;; the MV return point. + ;; Default the register args, and set up the stack as if we + ;; entered the MV return point. (inst mov ebx-tn esp-tn) (inst push edx-tn) (inst mov edi-tn nil-value) @@ -342,9 +300,9 @@ (emit-label no-stack-args) (inst lea edi-tn (make-ea :dword :base ebp-tn - :disp (* (- (1+ register-arg-count)) word-bytes))) - ;; Load EAX with NIL so we can quickly store it, and set up stuff - ;; for the loop. + :disp (* (- (1+ register-arg-count)) n-word-bytes))) + ;; Load EAX with NIL so we can quickly store it, and set up + ;; stuff for the loop. (inst mov eax-tn nil-value) (inst std) (inst mov ecx-tn (- nvals register-arg-count)) @@ -356,8 +314,8 @@ (emit-label regs-defaulted) ;; Save EDI. (storew edi-tn ebx-tn (- (1+ 1))) - ;; Compute the number of stack arguments, and if it's zero or less, - ;; don't copy any stack arguments. + ;; Compute the number of stack arguments, and if it's zero or + ;; less, don't copy any stack arguments. (inst sub ecx-tn (fixnumize register-arg-count)) (inst jmp :le no-stack-args) @@ -371,12 +329,12 @@ ;; Compute a pointer to where the stack args go. (inst lea edi-tn (make-ea :dword :base ebp-tn - :disp (* (- (1+ register-arg-count)) word-bytes))) + :disp (* (- (1+ register-arg-count)) n-word-bytes))) ;; Save ESI, and compute a pointer to where the args come from. (storew esi-tn ebx-tn (- (1+ 2))) (inst lea esi-tn (make-ea :dword :base ebx-tn - :disp (* (- (1+ register-arg-count)) word-bytes))) + :disp (* (- (1+ register-arg-count)) n-word-bytes))) ;; Do the copy. (inst shr ecx-tn word-shift) ; make word count (inst std) @@ -405,22 +363,23 @@ ;;;; unknown values receiving -;;; Emit code needed at the return point for an unknown-values call for an -;;; arbitrary number of values. +;;; Emit code needed at the return point for an unknown-values call +;;; for an arbitrary number of values. ;;; -;;; We do the single and non-single cases with no shared code: there doesn't -;;; seem to be any potential overlap, and receiving a single value is more -;;; important efficiency-wise. +;;; We do the single and non-single cases with no shared code: there +;;; doesn't seem to be any potential overlap, and receiving a single +;;; value is more important efficiency-wise. ;;; -;;; When there is a single value, we just push it on the stack, returning -;;; the old SP and 1. +;;; When there is a single value, we just push it on the stack, +;;; returning the old SP and 1. ;;; -;;; When there is a variable number of values, we move all of the argument -;;; registers onto the stack, and return Args and Nargs. +;;; When there is a variable number of values, we move all of the +;;; argument registers onto the stack, and return ARGS and NARGS. ;;; -;;; Args and Nargs are TNs wired to the named locations. We must -;;; explicitly allocate these TNs, since their lifetimes overlap with the -;;; results Start and Count (also, it's nice to be able to target them). +;;; ARGS and NARGS are TNs wired to the named locations. We must +;;; 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) (declare (type tn args nargs start count)) (let ((variable-values (gen-label)) @@ -460,24 +419,25 @@ ;;;; local call with unknown values convention return -;;; Non-TR local call for a fixed number of values passed according to the -;;; unknown values convention. +;;; Non-TR local call for a fixed number of values passed according to +;;; the unknown values convention. ;;; ;;; FP is the frame pointer in install before doing the call. ;;; -;;; NFP would be the number-stack frame pointer if we had a separate number -;;; stack. +;;; NFP would be the number-stack frame pointer if we had a separate +;;; number stack. ;;; -;;; Args are the argument passing locations, which are specified only to -;;; terminate their lifetimes in the caller. +;;; Args are the argument passing locations, which are specified only +;;; to terminate their lifetimes in the caller. ;;; -;;; Values are the return value locations (wired to the standard passing -;;; locations). -;;; Nvals is the number of values received. +;;; VALUES are the return value locations (wired to the standard +;;; passing locations). NVALS is the number of values received. ;;; -;;; Save is the save info, which we can ignore since saving has been done. +;;; Save is the save info, which we can ignore since saving has been +;;; done. ;;; -;;; Target is a continuation pointing to the start of the called function. +;;; TARGET is a continuation pointing to the start of the called +;;; function. (define-vop (call-local) (:args (fp) (nfp) @@ -557,12 +517,12 @@ ;;;; local call with known values return -;;; Non-TR local call with known return locations. Known-value return works -;;; just like argument passing in local call. +;;; Non-TR local call with known return locations. Known-value return +;;; works just like argument passing in local call. ;;; -;;; Note: we can't use normal load-tn allocation for the fixed args, since all -;;; registers may be tied up by the more operand. Instead, we use -;;; MAYBE-LOAD-STACK-TN. +;;; Note: we can't use normal load-tn allocation for the fixed args, +;;; since all registers may be tied up by the more operand. Instead, +;;; we use MAYBE-LOAD-STACK-TN. (define-vop (known-call-local) (:args (fp) (nfp) @@ -620,9 +580,10 @@ (:ignore val-locs vals) (:vop-var vop) (:generator 6 - (trace-table-entry trace-table-function-epilogue) - ;; Save the return-pc in a register 'cause the frame-pointer is going away. - ;; Note this not in the usual stack location so we can't use RET + (trace-table-entry trace-table-fun-epilogue) + ;; Save the return-pc in a register 'cause the frame-pointer is + ;; going away. Note this not in the usual stack location so we + ;; can't use RET (move rpc return-pc) ;; Restore the stack. (move esp-tn ebp-tn) @@ -652,14 +613,15 @@ (:ignore val-locs vals) (:vop-var vop) (:generator 6 - (trace-table-entry trace-table-function-epilogue) + (trace-table-entry trace-table-fun-epilogue) #+nil (format t "*known-return: old-fp ~S, tn-kind ~S; ~S ~S~%" old-fp (sb!c::tn-kind old-fp) (sb!c::tn-save-tn old-fp) (sb!c::tn-kind (sb!c::tn-save-tn old-fp))) #+nil (format t "*known-return: return-pc ~S, tn-kind ~S; ~S ~S~%" - return-pc (sb!c::tn-kind return-pc) (sb!c::tn-save-tn return-pc) + return-pc (sb!c::tn-kind return-pc) + (sb!c::tn-save-tn return-pc) (sb!c::tn-kind (sb!c::tn-save-tn return-pc))) ;; return-pc may be either in a register or on the stack. @@ -675,7 +637,7 @@ ;; Zot all of the stack except for the old-fp. (inst lea esp-tn (make-ea :dword :base ebp-tn :disp (- (* (1+ ocfp-save-offset) - word-bytes)))) + n-word-bytes)))) ;; Restore the old fp from its save location on the stack, ;; and zot the stack. (inst pop ebp-tn)) @@ -702,56 +664,57 @@ ;; Zot all of the stack except for the old-fp and return-pc. (inst lea esp-tn (make-ea :dword :base ebp-tn - :disp (- (* (1+ (tn-offset return-pc)) word-bytes)))) + :disp (- (* (1+ (tn-offset return-pc)) n-word-bytes)))) ;; Restore the old fp. old-fp may be either on the stack in its ;; save location or in a register, in either case this restores it. (move ebp-tn old-fp) ;; The return pops the return address (4 bytes), then we need ;; to pop all the slots before the return-pc which includes the ;; 4 bytes for the old-fp. - (inst ret (* (tn-offset return-pc) word-bytes)))) + (inst ret (* (tn-offset return-pc) n-word-bytes)))) (trace-table-entry trace-table-normal))) ;;;; full call ;;; -;;; There is something of a cross-product effect with full calls. Different -;;; versions are used depending on whether we know the number of arguments or -;;; the name of the called function, and whether we want fixed values, unknown -;;; values, or a tail call. +;;; There is something of a cross-product effect with full calls. +;;; Different versions are used depending on whether we know the +;;; number of arguments or the name of the called function, and +;;; whether we want fixed values, unknown values, or a tail call. ;;; -;;; In full call, the arguments are passed creating a partial frame on the -;;; stack top and storing stack arguments into that frame. On entry to the -;;; callee, this partial frame is pointed to by FP. +;;; In full call, the arguments are passed creating a partial frame on +;;; the stack top and storing stack arguments into that frame. On +;;; entry to the callee, this partial frame is pointed to by FP. -;;; This macro helps in the definition of full call VOPs by avoiding code -;;; replication in defining the cross-product VOPs. +;;; This macro helps in the definition of full call VOPs by avoiding +;;; code replication in defining the cross-product VOPs. ;;; -;;; Name is the name of the VOP to define. +;;; NAME is the name of the VOP to define. ;;; -;;; Named is true if the first argument is an fdefinition object whose +;;; NAMED is true if the first argument is an fdefinition object whose ;;; definition is to be called. ;;; -;;; Return is either :Fixed, :Unknown or :Tail: -;;; -- If :Fixed, then the call is for a fixed number of values, returned in +;;; RETURN is either :FIXED, :UNKNOWN or :TAIL: +;;; -- If :FIXED, then the call is for a fixed number of values, returned in ;;; the standard passing locations (passed as result operands). -;;; -- If :Unknown, then the result values are pushed on the stack, and the +;;; -- If :UNKNOWN, then the result values are pushed on the stack, and the ;;; result values are specified by the Start and Count as in the ;;; unknown-values continuation representation. -;;; -- If :Tail, then do a tail-recursive call. No values are returned. +;;; -- If :TAIL, then do a tail-recursive call. No values are returned. ;;; The Old-Fp and Return-PC are passed as the second and third arguments. ;;; -;;; In non-tail calls, the pointer to the stack arguments is passed as the last -;;; fixed argument. If Variable is false, then the passing locations are -;;; passed as a more arg. Variable is true if there are a variable number of -;;; arguments passed on the stack. Variable cannot be specified with :Tail -;;; return. TR variable argument call is implemented separately. +;;; In non-tail calls, the pointer to the stack arguments is passed as +;;; the last fixed argument. If Variable is false, then the passing +;;; locations are passed as a more arg. Variable is true if there are +;;; a variable number of arguments passed on the stack. Variable +;;; cannot be specified with :Tail return. TR variable argument call +;;; is implemented separately. ;;; -;;; In tail call with fixed arguments, the passing locations are passed as a -;;; more arg, but there is no new-FP, since the arguments have been set up in -;;; the current frame. +;;; In tail call with fixed arguments, the passing locations are +;;; passed as a more arg, but there is no new-FP, since the arguments +;;; have been set up in the current frame. (macrolet ((define-full-call (name named return variable) - (assert (not (and variable (eq return :tail)))) + (aver (not (and variable (eq return :tail)))) `(define-vop (,name ,@(when (eq return :unknown) '(unknown-values-receiver))) @@ -786,33 +749,40 @@ ,@(unless (or variable (eq return :tail)) '(arg-locs)) ,@(unless variable '(args))) - ;; We pass either the fdefn object (for named call) or the actual - ;; function object (for unnamed call) in EAX. With named call, - ;; closure-tramp will replace it with the real function and invoke - ;; the real function for closures. Non-closures do not need this - ;; value, so don't care what shows up in it. + ;; We pass either the fdefn object (for named call) or + ;; the actual function object (for unnamed call) in + ;; EAX. With named call, closure-tramp will replace it + ;; with the real function and invoke the real function + ;; for closures. Non-closures do not need this value, + ;; so don't care what shows up in it. (:temporary - (:sc descriptor-reg :offset eax-offset :from (:argument 0) :to :eval) + (:sc descriptor-reg + :offset eax-offset + :from (:argument 0) + :to :eval) eax) ;; We pass the number of arguments in ECX. (:temporary (:sc unsigned-reg :offset ecx-offset :to :eval) ecx) - ;; With variable call, we have to load the register-args out - ;; of the (new) stack frame before doing the call. Therefore, - ;; we have to tell the lifetime stuff that we need to use them. + ;; With variable call, we have to load the + ;; register-args out of the (new) stack frame before + ;; doing the call. Therefore, we have to tell the + ;; lifetime stuff that we need to use them. ,@(when variable - (mapcar #'(lambda (name offset) - `(:temporary (:sc descriptor-reg - :offset ,offset - :from (:argument 0) - :to :eval) - ,name)) - register-arg-names register-arg-offsets)) + (mapcar (lambda (name offset) + `(:temporary (:sc descriptor-reg + :offset ,offset + :from (:argument 0) + :to :eval) + ,name)) + *register-arg-names* *register-arg-offsets*)) ,@(when (eq return :tail) - '((:temporary (:sc unsigned-reg - :from (:argument 1) :to (:argument 2)) old-fp-tmp))) + '((:temporary (:sc unsigned-reg + :from (:argument 1) + :to (:argument 2)) + old-fp-tmp))) (:generator ,(+ (if named 5 0) (if variable 19 1) @@ -821,22 +791,25 @@ (if (eq return :unknown) 25 0)) (trace-table-entry trace-table-call-site) - ;; This has to be done before the frame pointer is changed! - ;; eax stores the 'lexical environment' needed for closures + ;; This has to be done before the frame pointer is + ;; changed! EAX stores the 'lexical environment' needed + ;; for closures. (move eax fun) ,@(if variable - ;; For variable call, compute the number of arguments and - ;; move some of the arguments to registers. + ;; For variable call, compute the number of + ;; arguments and move some of the arguments to + ;; registers. (collect ((noise)) ;; Compute the number of arguments. (noise '(inst mov ecx new-fp)) (noise '(inst sub ecx esp-tn)) - ;; Move the necessary args to registers, this - ;; moves them all even if they are not all needed. + ;; Move the necessary args to registers, + ;; this moves them all even if they are + ;; not all needed. (loop - for name in register-arg-names + for name in *register-arg-names* for index downfrom -1 do (noise `(loadw ,name new-fp ,index))) (noise)) @@ -844,24 +817,26 @@ (inst xor ecx ecx) (inst mov ecx (fixnumize nargs))))) ,@(cond ((eq return :tail) - '(;; Python has figured out what frame we should return - ;; to so might as well use that clue. This seems - ;; really important to the implementation of things - ;; like (without-interrupts ...) - + '(;; Python has figured out what frame we should + ;; return to so might as well use that clue. + ;; This seems really important to the + ;; implementation of things like + ;; (without-interrupts ...) + ;; ;; dtc; Could be doing a tail call from a - ;; known-local-call etc in which the old-fp or ret-pc - ;; are in regs or in non-standard places. If the - ;; passing location were wired to the stack in - ;; standard locations then these moves will be - ;; un-necessary; this is probably best for the x86. + ;; known-local-call etc in which the old-fp + ;; or ret-pc are in regs or in non-standard + ;; places. If the passing location were + ;; wired to the stack in standard locations + ;; then these moves will be un-necessary; + ;; this is probably best for the x86. (sc-case old-fp ((control-stack) (unless (= ocfp-save-offset (tn-offset old-fp)) - ;; FIXME: FORMAT T for stale diagnostic - ;; output (several of them around here), - ;; ick + ;; FIXME: FORMAT T for stale + ;; diagnostic output (several of + ;; them around here), ick (format t "** tail-call old-fp not S0~%") (move old-fp-tmp old-fp) (storew old-fp-tmp @@ -873,23 +848,26 @@ ebp-tn (- (1+ ocfp-save-offset))))) - ;; For tail call, we have to push the return-pc so - ;; that it looks like we CALLed despite the fact that - ;; we are going to JMP. + ;; For tail call, we have to push the + ;; return-pc so that it looks like we CALLed + ;; despite the fact that we are going to JMP. (inst push return-pc) )) (t - ;; For non-tail call, we have to save our frame pointer - ;; and install the new frame pointer. We can't load - ;; stack tns after this point. - `(;; Python doesn't seem to allocate a frame here which - ;; doesn't leave room for the ofp/ret stuff. + ;; For non-tail call, we have to save our + ;; frame pointer and install the new frame + ;; pointer. We can't load stack tns after this + ;; point. + `(;; Python doesn't seem to allocate a frame + ;; here which doesn't leave room for the + ;; ofp/ret stuff. - ;; The variable args are on the stack and become the - ;; frame, but there may be <3 args and 3 stack slots - ;; are assumed allocate on the call. So need to - ;; ensure there are at least 3 slots. This hack just - ;; adds 3 more. + ;; The variable args are on the stack and + ;; become the frame, but there may be <3 + ;; args and 3 stack slots are assumed + ;; allocate on the call. So need to ensure + ;; there are at least 3 slots. This hack + ;; just adds 3 more. ,(if variable '(inst sub esp-tn (fixnumize 3))) @@ -904,10 +882,11 @@ (inst ,(if (eq return :tail) 'jmp 'call) (make-ea :dword :base eax :disp ,(if named - '(- (* fdefn-raw-addr-slot word-bytes) - other-pointer-type) - '(- (* closure-function-slot word-bytes) - function-pointer-type)))) + '(- (* fdefn-raw-addr-slot + n-word-bytes) + other-pointer-lowtag) + '(- (* closure-fun-slot n-word-bytes) + fun-pointer-lowtag)))) ,@(ecase return (:fixed '((default-unknown-values vop values nvals))) @@ -927,9 +906,9 @@ (define-full-call call-variable nil :fixed t) (define-full-call multiple-call-variable nil :unknown t)) -;;; This is defined separately, since it needs special code that BLT's the -;;; arguments down. All the real work is done in the assembly routine. We just -;;; set things up so that it can find what it needs. +;;; This is defined separately, since it needs special code that BLT's +;;; the arguments down. All the real work is done in the assembly +;;; routine. We just set things up so that it can find what it needs. (define-vop (tail-call-variable) (:args (args :scs (any-reg control-stack) :target esi) (function :scs (descriptor-reg control-stack) :target eax) @@ -975,7 +954,7 @@ (:temporary (:sc unsigned-reg) ret) (:ignore value) (:generator 6 - (trace-table-entry trace-table-function-epilogue) + (trace-table-entry trace-table-fun-epilogue) (move ret return-pc) ;; Clear the control stack (move ofp old-fp) @@ -987,13 +966,13 @@ ;; Out of here. (inst jmp ret))) -;;; Do unknown-values return of a fixed (other than 1) number of values. The -;;; Values are required to be set up in the standard passing locations. Nvals -;;; is the number of values returned. +;;; Do unknown-values return of a fixed (other than 1) number of +;;; values. The VALUES are required to be set up in the standard +;;; passing locations. NVALS is the number of values returned. ;;; -;;; Basically, we just load ECX with the number of values returned and EBX -;;; with a pointer to the values, set ESP to point to the end of the values, -;;; and jump directly to return-pc. +;;; Basically, we just load ECX with the number of values returned and +;;; EBX with a pointer to the values, set ESP to point to the end of +;;; the values, and jump directly to return-pc. (define-vop (return) (:args (old-fp) (return-pc :to (:eval 1)) @@ -1001,35 +980,35 @@ (:ignore values) (:info nvals) - ;; In the case of other than one value, we need these registers to tell - ;; the caller where they are and how many there are. + ;; In the case of other than one value, we need these registers to + ;; tell the caller where they are and how many there are. (:temporary (:sc unsigned-reg :offset ebx-offset) ebx) (:temporary (:sc unsigned-reg :offset ecx-offset) ecx) ;; We need to stretch the lifetime of return-pc past the argument ;; registers so that we can default the argument registers without ;; trashing return-pc. - (:temporary (:sc unsigned-reg :offset (first register-arg-offsets) + (:temporary (:sc unsigned-reg :offset (first *register-arg-offsets*) :from :eval) a0) - (:temporary (:sc unsigned-reg :offset (second register-arg-offsets) + (:temporary (:sc unsigned-reg :offset (second *register-arg-offsets*) :from :eval) a1) - (:temporary (:sc unsigned-reg :offset (third register-arg-offsets) + (:temporary (:sc unsigned-reg :offset (third *register-arg-offsets*) :from :eval) a2) (:generator 6 - (trace-table-entry trace-table-function-epilogue) + (trace-table-entry trace-table-fun-epilogue) ;; Establish the values pointer and values count. (move ebx ebp-tn) (if (zerop nvals) (inst xor ecx ecx) ; smaller (inst mov ecx (fixnumize nvals))) - ;; restore the frame pointer. + ;; Restore the frame pointer. (move ebp-tn old-fp) - ;; clear as much of the stack as possible, but not past the return + ;; Clear as much of the stack as possible, but not past the return ;; address. (inst lea esp-tn (make-ea :dword :base ebx - :disp (- (* (max nvals 2) word-bytes)))) - ;; pre-default any argument register that need it. + :disp (- (* (max nvals 2) n-word-bytes)))) + ;; Pre-default any argument register that need it. (when (< nvals register-arg-count) (let* ((arg-tns (nthcdr nvals (list a0 a1 a2))) (first (first arg-tns))) @@ -1041,7 +1020,7 @@ ;; tell it to index off of EBX instead of EBP. (cond ((zerop nvals) ;; Return popping the return address and the OCFP. - (inst ret word-bytes)) + (inst ret n-word-bytes)) ((= nvals 1) ;; Return popping the return, leaving 1 slot. Can this ;; happen, or is a single value return handled elsewhere? @@ -1049,14 +1028,15 @@ (t (inst jmp (make-ea :dword :base ebx :disp (- (* (1+ (tn-offset return-pc)) - word-bytes)))))) + n-word-bytes)))))) (trace-table-entry trace-table-normal))) -;;; Do unknown-values return of an arbitrary number of values (passed on the -;;; stack.) We check for the common case of a single return value, and do that -;;; inline using the normal single value return convention. Otherwise, we -;;; branch off to code that calls an assembly-routine. +;;; Do unknown-values return of an arbitrary number of values (passed +;;; on the stack.) We check for the common case of a single return +;;; value, and do that inline using the normal single value return +;;; convention. Otherwise, we branch off to code that calls an +;;; assembly-routine. ;;; ;;; The assembly routine takes the following args: ;;; EAX -- the return-pc to finally jump to. @@ -1073,13 +1053,13 @@ (:temporary (:sc unsigned-reg :offset esi-offset :from (:argument 2)) esi) (:temporary (:sc unsigned-reg :offset ecx-offset :from (:argument 3)) ecx) (:temporary (:sc unsigned-reg :offset ebx-offset :from (:eval 0)) ebx) - (:temporary (:sc descriptor-reg :offset (first register-arg-offsets) + (:temporary (:sc descriptor-reg :offset (first *register-arg-offsets*) :from (:eval 0)) a0) (:temporary (:sc unsigned-reg :from (:eval 1)) old-fp-temp) (:node-var node) (:generator 13 - (trace-table-entry trace-table-function-epilogue) + (trace-table-entry trace-table-fun-epilogue) ;; Load the return-pc. (move eax return-pc) (unless (policy node (> space speed)) @@ -1128,13 +1108,13 @@ ;; Get result. (move closure eax-tn))) -;;; Copy a more arg from the argument area to the end of the current frame. -;;; Fixed is the number of non-more arguments. +;;; Copy a &MORE arg from the argument area to the end of the current +;;; frame. FIXED is the number of non-&MORE arguments. ;;; ;;; The tricky part is doing this without trashing any of the calling -;;; convention registers that are still needed. This vop is emitted directly -;;; after the xep-allocate frame. That means the registers are in use as -;;; follows: +;;; convention registers that are still needed. This vop is emitted +;;; directly after the xep-allocate frame. That means the registers +;;; are in use as follows: ;;; ;;; EAX -- The lexenv. ;;; EBX -- Available. @@ -1145,8 +1125,9 @@ ;;; ;;; So basically, we have one register available for our use: EBX. ;;; -;;; What we can do is push the other regs onto the stack, and then restore -;;; their values by looking directly below where we put the more-args. +;;; What we can do is push the other regs onto the stack, and then +;;; restore their values by looking directly below where we put the +;;; more-args. (define-vop (copy-more-arg) (:info fixed) (:generator 20 @@ -1162,7 +1143,7 @@ (inst lea ebx-tn (make-ea :dword :base ebp-tn :disp (- (fixnumize fixed) - (* sb!vm:word-bytes + (* n-word-bytes (max 3 (sb-allocated-size 'stack)))))) (inst sub ebx-tn ecx-tn) ; Got the new stack in ebx (inst mov esp-tn ebx-tn) @@ -1240,13 +1221,14 @@ JUST-ALLOC-FRAME (inst lea esp-tn (make-ea :dword :base ebp-tn - :disp (- (* sb!vm:word-bytes + :disp (- (* n-word-bytes (max 3 (sb-allocated-size 'stack)))))) DONE)) -;;; More args are stored contiguously on the stack, starting immediately at the -;;; context pointer. The context pointer is not typed, so the lowtag is 0. +;;; &MORE args are stored contiguously on the stack, starting +;;; immediately at the context pointer. The context pointer is not +;;; typed, so the lowtag is 0. (define-vop (more-arg) (:translate %more-arg) (:policy :fast-safe) @@ -1271,7 +1253,7 @@ (:result-types *) (:generator 4 (inst mov value - (make-ea :dword :base object :disp (- (* index word-bytes)))))) + (make-ea :dword :base object :disp (- (* index n-word-bytes)))))) ;;; Turn more arg (context, count) into a list. @@ -1299,8 +1281,9 @@ (inst lea dst (make-ea :dword :index ecx :scale 2)) (pseudo-atomic (allocation dst dst node) - (inst lea dst (make-ea :byte :base dst :disp list-pointer-type)) - ;; Convert the count into a raw value, so that we can use the LOOP inst. + (inst lea dst (make-ea :byte :base dst :disp list-pointer-lowtag)) + ;; Convert the count into a raw value, so that we can use the + ;; LOOP instruction. (inst shr ecx 2) ;; Set decrement mode (successive args at lower addresses) (inst std) @@ -1311,28 +1294,29 @@ (inst jmp enter) (emit-label loop) ;; Compute a pointer to the next cons. - (inst add dst (* cons-size word-bytes)) + (inst add dst (* cons-size n-word-bytes)) ;; Store a pointer to this cons in the CDR of the previous cons. - (storew dst dst -1 list-pointer-type) + (storew dst dst -1 list-pointer-lowtag) (emit-label enter) ;; Grab one value and stash it in the car of this cons. (inst lods eax) - (storew eax dst 0 list-pointer-type) + (storew eax dst 0 list-pointer-lowtag) ;; Go back for more. (inst loop loop) ;; NIL out the last cons. - (storew nil-value dst 1 sb!vm:list-pointer-type)) + (storew nil-value dst 1 list-pointer-lowtag)) (emit-label done)))) -;;; Return the location and size of the more arg glob created by Copy-More-Arg. -;;; Supplied is the total number of arguments supplied (originally passed in -;;; ECX.) Fixed is the number of non-rest arguments. +;;; Return the location and size of the &MORE arg glob created by +;;; COPY-MORE-ARG. SUPPLIED is the total number of arguments supplied +;;; (originally passed in ECX). FIXED is the number of non-rest +;;; arguments. ;;; -;;; We must duplicate some of the work done by Copy-More-Arg, since at that -;;; time the environment is in a pretty brain-damaged state, preventing this -;;; info from being returned as values. What we do is compute -;;; supplied - fixed, and return a pointer that many words below the current -;;; stack top. +;;; We must duplicate some of the work done by COPY-MORE-ARG, since at +;;; that time the environment is in a pretty brain-damaged state, +;;; preventing this info from being returned as values. What we do is +;;; compute supplied - fixed, and return a pointer that many words +;;; below the current stack top. (define-vop (more-arg-context) (:policy :fast-safe) (:translate sb!c::%more-arg-context) @@ -1353,7 +1337,7 @@ (unless (zerop fixed) (inst sub count (fixnumize fixed))))) -;;; Signal wrong argument count error if Nargs isn't = to Count. +;;; Signal wrong argument count error if NARGS isn't equal to COUNT. (define-vop (verify-argument-count) (:policy :fast-safe) (:translate sb!c::%verify-argument-count) @@ -1371,26 +1355,26 @@ (inst jmp :ne err-lab)))) ;;; Various other error signallers. -(macrolet ((frob (name error translate &rest args) +(macrolet ((def (name error translate &rest args) `(define-vop (,name) ,@(when translate `((:policy :fast-safe) (:translate ,translate))) - (:args ,@(mapcar #'(lambda (arg) - `(,arg :scs (any-reg descriptor-reg))) + (:args ,@(mapcar (lambda (arg) + `(,arg :scs (any-reg descriptor-reg))) args)) (:vop-var vop) (:save-p :compute-only) (:generator 1000 (error-call vop ,error ,@args))))) - (frob argument-count-error invalid-argument-count-error + (def argument-count-error invalid-argument-count-error sb!c::%argument-count-error nargs) - (frob type-check-error object-not-type-error sb!c::%type-check-error + (def type-check-error object-not-type-error sb!c::%type-check-error object type) - (frob layout-invalid-error layout-invalid-error sb!c::%layout-invalid-error + (def layout-invalid-error layout-invalid-error sb!c::%layout-invalid-error object layout) - (frob odd-keyword-arguments-error odd-keyword-arguments-error - sb!c::%odd-keyword-arguments-error) - (frob unknown-keyword-argument-error unknown-keyword-argument-error - sb!c::%unknown-keyword-argument-error key) - (frob nil-function-returned-error nil-function-returned-error nil fun)) + (def odd-key-arguments-error odd-key-arguments-error + sb!c::%odd-key-arguments-error) + (def unknown-key-argument-error unknown-key-argument-error + sb!c::%unknown-key-argument-error key) + (def nil-fun-returned-error nil-fun-returned-error nil fun))