X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=src%2Fcompiler%2Fir1opt.lisp;h=35720894ff38948146f6700881528b85f4ae792a;hb=ba38798a5ca26b90647a1993f348806cb32f2d1b;hp=bab067f21ba8a0072cd758a47e770f60e29fe5a3;hpb=f1efc390c46d7b0054b504981b36baf928259ab6;p=sbcl.git diff --git a/src/compiler/ir1opt.lisp b/src/compiler/ir1opt.lisp index bab067f..3572089 100644 --- a/src/compiler/ir1opt.lisp +++ b/src/compiler/ir1opt.lisp @@ -56,9 +56,9 @@ (node-derived-type (continuation-use cont))))) ;;; Our best guess for the type of this continuation's value. Note -;;; that this may be Values or Function type, which cannot be passed +;;; that this may be VALUES or FUNCTION type, which cannot be passed ;;; as an argument to the normal type operations. See -;;; Continuation-Type. This may be called on deleted continuations, +;;; CONTINUATION-TYPE. This may be called on deleted continuations, ;;; always returning *. ;;; ;;; What we do is call CONTINUATION-PROVEN-TYPE and check whether the @@ -84,6 +84,18 @@ (cond ((values-subtypep proven asserted) (setf (continuation-%type-check cont) nil) (setf (continuation-%derived-type cont) proven)) + ((and (values-subtypep proven (specifier-type 'function)) + (values-subtypep asserted (specifier-type 'function))) + ;; It's physically impossible for a runtime type check to + ;; distinguish between the various subtypes of FUNCTION, so + ;; it'd be pointless to do more type checks here. + (setf (continuation-%type-check cont) nil) + (setf (continuation-%derived-type cont) + ;; FIXME: This should depend on optimization + ;; policy. This is for SPEED > SAFETY: + #+nil (values-type-intersection asserted proven) + ;; and this is for SAFETY >= SPEED: + #-nil proven)) (t (unless (or (continuation-%type-check cont) (not (continuation-dest cont)) @@ -159,9 +171,9 @@ (eq int *empty-type*) (not (eq rtype *empty-type*))) (let ((*compiler-error-context* node)) - (compiler-warning + (compiler-warn "New inferred type ~S conflicts with old type:~ - ~% ~S~%*** Bug?" + ~% ~S~%*** possible internal error? Please report this." (type-specifier rtype) (type-specifier node-type)))) (setf (node-derived-type node) int) (reoptimize-continuation (node-cont node)))))) @@ -221,26 +233,33 @@ ;;; and doing IR1 optimizations. We can ignore all blocks that don't ;;; have the REOPTIMIZE flag set. If COMPONENT-REOPTIMIZE is true when ;;; we are done, then another iteration would be beneficial. -;;; -;;; We delete blocks when there is either no predecessor or the block -;;; is in a lambda that has been deleted. These blocks would -;;; eventually be deleted by DFO recomputation, but doing it here -;;; immediately makes the effect available to IR1 optimization. (defun ir1-optimize (component) (declare (type component component)) (setf (component-reoptimize component) nil) (do-blocks (block component) (cond ((or (block-delete-p block) - (null (block-pred block)) - (eq (functional-kind (block-home-lambda block)) :deleted)) + (null (block-pred block))) (delete-block block)) + ((eq (functional-kind (block-home-lambda block)) :deleted) + ;; Preserve the BLOCK-SUCC invariant that almost every block has + ;; one successor (and a block with DELETE-P set is an acceptable + ;; exception). + (labels ((mark-blocks (block) + (dolist (pred (block-pred block)) + (when (and (not (block-delete-p pred)) + (eq (functional-kind (block-home-lambda pred)) + :deleted)) + (setf (block-delete-p pred) t) + (mark-blocks pred))))) + (mark-blocks block) + (delete-block block))) (t (loop (let ((succ (block-succ block))) (unless (and succ (null (rest succ))) (return))) - + (let ((last (block-last block))) (typecase last (cif @@ -250,56 +269,66 @@ (exit (when (maybe-delete-exit last) (return))))) - - (unless (join-successor-if-possible block) + + (unless (join-successor-if-possible block) (return))) (when (and (block-reoptimize block) (block-component block)) (aver (not (block-delete-p block))) (ir1-optimize-block block)) + ;; We delete blocks when there is either no predecessor or the + ;; block is in a lambda that has been deleted. These blocks + ;; would eventually be deleted by DFO recomputation, but doing + ;; it here immediately makes the effect available to IR1 + ;; optimization. (when (and (block-flush-p block) (block-component block)) (aver (not (block-delete-p block))) (flush-dead-code block))))) (values)) -;;; Loop over the nodes in Block, looking for stuff that needs to be -;;; optimized. We dispatch off of the type of each node with its -;;; reoptimize flag set: - -;;; -- With a combination, we call Propagate-Function-Change whenever -;;; the function changes, and call IR1-Optimize-Combination if any -;;; argument changes. -;;; -- With an Exit, we derive the node's type from the Value's type. -;;; We don't propagate Cont's assertion to the Value, since if we -;;; did, this would move the checking of Cont's assertion to the -;;; exit. This wouldn't work with Catch and UWP, where the Exit -;;; node is just a placeholder for the actual unknown exit. +;;; Loop over the nodes in BLOCK, acting on (and clearing) REOPTIMIZE +;;; flags. ;;; -;;; Note that we clear the node & block reoptimize flags *before* -;;; doing the optimization. This ensures that the node or block will -;;; be reoptimized if necessary. We leave the NODE-OPTIMIZE flag set -;;; going into IR1-OPTIMIZE-RETURN, since IR1-OPTIMIZE-RETURN wants to -;;; clear the flag itself. +;;; Note that although they are cleared here, REOPTIMIZE flags might +;;; still be set upon return from this function, meaning that further +;;; optimization is wanted (as a consequence of optimizations we did). (defun ir1-optimize-block (block) (declare (type cblock block)) + ;; We clear the node and block REOPTIMIZE flags before doing the + ;; optimization, not after. This ensures that the node or block will + ;; be reoptimized if necessary. (setf (block-reoptimize block) nil) (do-nodes (node cont block :restart-p t) (when (node-reoptimize node) + ;; As above, we clear the node REOPTIMIZE flag before optimizing. (setf (node-reoptimize node) nil) (typecase node (ref) (combination + ;; With a COMBINATION, we call PROPAGATE-FUN-CHANGE whenever + ;; the function changes, and call IR1-OPTIMIZE-COMBINATION if + ;; any argument changes. (ir1-optimize-combination node)) (cif (ir1-optimize-if node)) (creturn + ;; KLUDGE: We leave the NODE-OPTIMIZE flag set going into + ;; IR1-OPTIMIZE-RETURN, since IR1-OPTIMIZE-RETURN wants to + ;; clear the flag itself. -- WHN 2002-02-02, quoting original + ;; CMU CL comments (setf (node-reoptimize node) t) (ir1-optimize-return node)) (mv-combination (ir1-optimize-mv-combination node)) (exit + ;; With an EXIT, we derive the node's type from the VALUE's + ;; type. We don't propagate CONT's assertion to the VALUE, + ;; since if we did, this would move the checking of CONT's + ;; assertion to the exit. This wouldn't work with CATCH and + ;; UWP, where the EXIT node is just a placeholder for the + ;; actual unknown exit. (let ((value (exit-value node))) (when value (derive-node-type node (continuation-derived-type value))))) @@ -307,22 +336,8 @@ (ir1-optimize-set node))))) (values)) -;;; We cannot combine with a successor block if: -;;; 1. The successor has more than one predecessor. -;;; 2. The last node's CONT is also used somewhere else. -;;; 3. The successor is the current block (infinite loop). -;;; 4. The next block has a different cleanup, and thus we may want -;;; to insert cleanup code between the two blocks at some point. -;;; 5. The next block has a different home lambda, and thus the -;;; control transfer is a non-local exit. -;;; -;;; If we succeed, we return true, otherwise false. -;;; -;;; Joining is easy when the successor's Start continuation is the -;;; same from our Last's Cont. If they differ, then we can still join -;;; when the last continuation has no next and the next continuation -;;; has no uses. In this case, we replace the next continuation with -;;; the last before joining the blocks. +;;; Try to join with a successor block. If we succeed, we return true, +;;; otherwise false. (defun join-successor-if-possible (block) (declare (type cblock block)) (let ((next (first (block-succ block)))) @@ -330,22 +345,39 @@ (let* ((last (block-last block)) (last-cont (node-cont last)) (next-cont (block-start next))) - (cond ((or (rest (block-pred next)) - (not (eq (continuation-use last-cont) last)) - (eq next block) - (not (eq (block-end-cleanup block) - (block-start-cleanup next))) - (not (eq (block-home-lambda block) - (block-home-lambda next)))) + (cond (;; We cannot combine with a successor block if: + (or + ;; The successor has more than one predecessor. + (rest (block-pred next)) + ;; The last node's CONT is also used somewhere else. + (not (eq (continuation-use last-cont) last)) + ;; The successor is the current block (infinite loop). + (eq next block) + ;; The next block has a different cleanup, and thus + ;; we may want to insert cleanup code between the + ;; two blocks at some point. + (not (eq (block-end-cleanup block) + (block-start-cleanup next))) + ;; The next block has a different home lambda, and + ;; thus the control transfer is a non-local exit. + (not (eq (block-home-lambda block) + (block-home-lambda next)))) nil) + ;; Joining is easy when the successor's START + ;; continuation is the same from our LAST's CONT. ((eq last-cont next-cont) (join-blocks block next) t) + ;; If they differ, then we can still join when the last + ;; continuation has no next and the next continuation + ;; has no uses. ((and (null (block-start-uses next)) (eq (continuation-kind last-cont) :inside-block)) + ;; In this case, we replace the next + ;; continuation with the last before joining the blocks. (let ((next-node (continuation-next next-cont))) - ;; If next-cont does have a dest, it must be - ;; unreachable, since there are no uses. + ;; If NEXT-CONT does have a dest, it must be + ;; unreachable, since there are no USES. ;; DELETE-CONTINUATION will mark the dest block as ;; DELETE-P [and also this block, unless it is no ;; longer backward reachable from the dest block.] @@ -359,7 +391,7 @@ nil)))))) ;;; Join together two blocks which have the same ending/starting -;;; continuation. The code in Block2 is moved into Block1 and Block2 +;;; continuation. The code in BLOCK2 is moved into BLOCK1 and BLOCK2 ;;; is deleted from the DFO. We combine the optimize flags for the two ;;; blocks so that any indicated optimization gets done. (defun join-blocks (block1 block2) @@ -394,13 +426,9 @@ (values)) -;;; Delete any nodes in BLOCK whose value is unused and have no -;;; side-effects. We can delete sets of lexical variables when the set +;;; Delete any nodes in BLOCK whose value is unused and which have no +;;; side effects. We can delete sets of lexical variables when the set ;;; variable has no references. -;;; -;;; [### For now, don't delete potentially flushable calls when they -;;; have the CALL attribute. Someday we should look at the funcitonal -;;; args to determine if they have any side-effects.] (defun flush-dead-code (block) (declare (type cblock block)) (do-nodes-backwards (node cont block) @@ -411,9 +439,14 @@ (unlink-node node)) (combination (let ((info (combination-kind node))) - (when (function-info-p info) - (let ((attr (function-info-attributes info))) + (when (fun-info-p info) + (let ((attr (fun-info-attributes info))) (when (and (ir1-attributep attr flushable) + ;; ### For now, don't delete potentially + ;; flushable calls when they have the CALL + ;; attribute. Someday we should look at the + ;; functional args to determine if they have + ;; any side effects. (not (ir1-attributep attr call))) (flush-dest (combination-fun node)) (dolist (arg (combination-args node)) @@ -605,9 +638,9 @@ ;;; This function attempts to delete an exit node, returning true if ;;; it deletes the block as a consequence: -;;; -- If the exit is degenerate (has no Entry), then we don't do +;;; -- If the exit is degenerate (has no ENTRY), then we don't do ;;; anything, since there is nothing to be done. -;;; -- If the exit node and its Entry have the same home lambda then +;;; -- If the exit node and its ENTRY have the same home lambda then ;;; we know the exit is local, and can delete the exit. We change ;;; uses of the Exit-Value to be uses of the original continuation, ;;; then unlink the node. If the exit is to a TR context, then we @@ -650,7 +683,7 @@ (declaim (ftype (function (combination) (values)) ir1-optimize-combination)) (defun ir1-optimize-combination (node) (when (continuation-reoptimize (basic-combination-fun node)) - (propagate-function-change node)) + (propagate-fun-change node)) (let ((args (basic-combination-args node)) (kind (basic-combination-kind node))) (case kind @@ -668,7 +701,7 @@ (when arg (setf (continuation-reoptimize arg) nil))) - (let ((attr (function-info-attributes kind))) + (let ((attr (fun-info-attributes kind))) (when (and (ir1-attributep attr foldable) ;; KLUDGE: The next test could be made more sensitive, ;; only suppressing constant-folding of functions with @@ -686,22 +719,20 @@ ;; cross-compiler can't fold it because the ;; cross-compiler doesn't know how to evaluate it. #+sb-xc-host - (let* ((ref (continuation-use (combination-fun node))) - (fun-name (leaf-source-name (ref-leaf ref)))) - (fboundp fun-name))) + (fboundp (combination-fun-source-name node))) (constant-fold-call node) (return-from ir1-optimize-combination))) - (let ((fun (function-info-derive-type kind))) + (let ((fun (fun-info-derive-type kind))) (when fun (let ((res (funcall fun node))) (when res (derive-node-type node res) (maybe-terminate-block node nil))))) - (let ((fun (function-info-optimizer kind))) + (let ((fun (fun-info-optimizer kind))) (unless (and fun (funcall fun node)) - (dolist (x (function-info-transforms kind)) + (dolist (x (fun-info-transforms kind)) #!+sb-show (when *show-transforms-p* (let* ((cont (basic-combination-fun node)) @@ -778,15 +809,10 @@ ;;; if necessary. We claim that the parent form is LABELS for ;;; context declarations, since we don't want it to be considered ;;; a real global function. -;;; -- In addition to a direct check for the function name in the -;;; table, we also must check for slot accessors. If the function -;;; is a slot accessor, then we set the combination kind to the -;;; function info of %SLOT-SETTER or %SLOT-ACCESSOR, as -;;; appropriate. ;;; -- If it is a known function, mark it as such by setting the KIND. ;;; ;;; We return the leaf referenced (NIL if not a leaf) and the -;;; FUNCTION-INFO assigned. +;;; FUN-INFO assigned. ;;; ;;; FIXME: The IR1-CONVERTING-NOT-OPTIMIZING-P argument is what the ;;; old CMU CL code called IR1-P, without explanation. My (WHN @@ -836,13 +862,7 @@ (values (ref-leaf (continuation-use (basic-combination-fun call))) nil)) (t - (let* ((name (leaf-source-name leaf)) - (info (info :function :info - (if (slot-accessor-p leaf) - (if (consp source-name) ; i.e. if SETF function - '%slot-setter - '%slot-accessor) - name)))) + (let ((info (info :function :info (leaf-source-name leaf)))) (if info (values leaf (setf (basic-combination-kind call) info)) (values leaf nil))))))) @@ -861,27 +881,27 @@ (csubtypep type (specifier-type 'function)) (or val (not win)))) (recognize-known-call call ir1-converting-not-optimizing-p)) - ((valid-function-use call type - :argument-test #'always-subtypep - :result-test #'always-subtypep - ;; KLUDGE: Common Lisp is such a dynamic - ;; language that all we can do here in - ;; general is issue a STYLE-WARNING. It - ;; would be nice to issue a full WARNING - ;; in the special case of of type - ;; mismatches within a compilation unit - ;; (as in section 3.2.2.3 of the spec) - ;; but at least as of sbcl-0.6.11, we - ;; don't keep track of whether the - ;; mismatched data came from the same - ;; compilation unit, so we can't do that. - ;; -- WHN 2001-02-11 - ;; - ;; FIXME: Actually, I think we could - ;; issue a full WARNING if the call - ;; violates a DECLAIM FTYPE. - :error-function #'compiler-style-warning - :warning-function #'compiler-note) + ((valid-fun-use call type + :argument-test #'always-subtypep + :result-test #'always-subtypep + ;; KLUDGE: Common Lisp is such a dynamic + ;; language that all we can do here in + ;; general is issue a STYLE-WARNING. It + ;; would be nice to issue a full WARNING + ;; in the special case of of type + ;; mismatches within a compilation unit + ;; (as in section 3.2.2.3 of the spec) + ;; but at least as of sbcl-0.6.11, we + ;; don't keep track of whether the + ;; mismatched data came from the same + ;; compilation unit, so we can't do that. + ;; -- WHN 2001-02-11 + ;; + ;; FIXME: Actually, I think we could + ;; issue a full WARNING if the call + ;; violates a DECLAIM FTYPE. + :lossage-fun #'compiler-style-warn + :unwinnage-fun #'compiler-note) (assert-call-type call type) (maybe-terminate-block call ir1-converting-not-optimizing-p) (recognize-known-call call ir1-converting-not-optimizing-p)) @@ -896,7 +916,7 @@ ;;; expansion, etc. If a call to a predicate in a non-conditional ;;; position or to a function with a source transform, then we ;;; reconvert the form to give IR1 another chance. -(defun propagate-function-change (call) +(defun propagate-fun-change (call) (declare (type combination call)) (let ((*compiler-error-context* call) (fun-cont (basic-combination-fun call))) @@ -917,7 +937,7 @@ ((not leaf)) ((or (info :function :source-transform (leaf-source-name leaf)) (and info - (ir1-attributep (function-info-attributes info) + (ir1-attributep (fun-info-attributes info) predicate) (let ((dest (continuation-dest (node-cont call)))) (and dest (not (if-p dest)))))) @@ -938,7 +958,8 @@ (transform-call call `(lambda ,dummies (,(leaf-source-name leaf) - ,@dummies))))))))))) + ,@dummies)) + (leaf-source-name leaf)))))))))) (values)) ;;;; known function optimization @@ -973,24 +994,13 @@ (policy node (>= speed inhibit-warnings)) (policy node (> speed inhibit-warnings)))) (*compiler-error-context* node)) - (cond ((not (member (transform-when transform) - '(:native :both))) - ;; FIXME: Make sure that there's a transform for - ;; (MEMBER SYMBOL ..) into MEMQ. - ;; FIXME: Note that when/if I make SHARE operation to shared - ;; constant data between objects in the system, remember that a - ;; SHAREd list, or other SHAREd compound object, can be processed - ;; recursively, so that e.g. the two lists above can share their - ;; '(:BOTH) tail sublists. - (let ((when (transform-when transform))) - (not (or (eq when :both) - (eq when :native)))) - t) - ((or (not constrained) - (valid-function-use node type :strict-result t)) + (cond ((or (not constrained) + (valid-fun-use node type :strict-result t)) (multiple-value-bind (severity args) (catch 'give-up-ir1-transform - (transform-call node (funcall fun node)) + (transform-call node + (funcall fun node) + (combination-fun-source-name node)) (values :none nil)) (ecase severity (:none @@ -999,7 +1009,7 @@ (:aborted (setf (combination-kind node) :error) (when args - (apply #'compiler-warning args)) + (apply #'compiler-warn args)) (remhash node table) nil) (:failure @@ -1013,11 +1023,10 @@ (remhash node table) nil)))) ((and flame - (valid-function-use node - type - :argument-test #'types-equal-or-intersect - :result-test - #'values-types-equal-or-intersect)) + (valid-fun-use node + type + :argument-test #'types-equal-or-intersect + :result-test #'values-types-equal-or-intersect)) (record-optimization-failure node transform type) t) (t @@ -1087,21 +1096,31 @@ (setf (component-reoptimize (block-component block)) t))))))) reoptimize)) - ;;; Take the lambda-expression RES, IR1 convert it in the proper ;;; environment, and then install it as the function for the call ;;; NODE. We do local call analysis so that the new function is ;;; integrated into the control flow. -(defun transform-call (node res) +;;; +;;; We require the original function source name in order to generate +;;; a meaningful debug name for the lambda we set up. (It'd be +;;; possible to do this starting from debug names as well as source +;;; names, but as of sbcl-0.7.1.5, there was no need for this +;;; generality, since source names are always known to our callers.) +(defun transform-call (node res source-name) (declare (type combination node) (list res)) + (aver (and (legal-fun-name-p source-name) + (not (eql source-name '.anonymous.)))) (with-ir1-environment-from-node node - (let ((new-fun (ir1-convert-inline-lambda - res - :debug-name "something inlined in TRANSFORM-CALL")) - (ref (continuation-use (combination-fun node)))) - (change-ref-leaf ref new-fun) - (setf (combination-kind node) :full) - (locall-analyze-component *current-component*))) + (let ((new-fun (ir1-convert-inline-lambda + res + :debug-name (debug-namify "LAMBDA-inlined ~A" + (as-debug-name + source-name + "")))) + (ref (continuation-use (combination-fun node)))) + (change-ref-leaf ref new-fun) + (setf (combination-kind node) :full) + (locall-analyze-component *current-component*))) (values)) ;;; Replace a call to a foldable function of constant arguments with @@ -1113,24 +1132,21 @@ ;;; call a :ERROR call. ;;; ;;; If there is more than one value, then we transform the call into a -;;; values form. +;;; VALUES form. (defun constant-fold-call (call) - (declare (type combination call)) - (let* ((args (mapcar #'continuation-value (combination-args call))) - (ref (continuation-use (combination-fun call))) - (fun-name (leaf-source-name (ref-leaf ref)))) - + (let ((args (mapcar #'continuation-value (combination-args call))) + (fun-name (combination-fun-source-name call))) (multiple-value-bind (values win) (careful-call fun-name args call "constant folding") (if (not win) - (setf (combination-kind call) :error) - (let ((dummies (make-gensym-list (length args)))) - (transform-call - call - `(lambda ,dummies - (declare (ignore ,@dummies)) - (values ,@(mapcar #'(lambda (x) `',x) values)))))))) - + (setf (combination-kind call) :error) + (let ((dummies (make-gensym-list (length args)))) + (transform-call + call + `(lambda ,dummies + (declare (ignore ,@dummies)) + (values ,@(mapcar (lambda (x) `',x) values))) + fun-name))))) (values)) ;;;; local call optimization @@ -1204,7 +1220,7 @@ ;;; -- the var's DEST has a different policy than the ARG's (think safety). ;;; ;;; We change the REF to be a reference to NIL with unused value, and -;;; let it be flushed as dead code. A side-effect of this substitution +;;; let it be flushed as dead code. A side effect of this substitution ;;; is to delete the variable. (defun substitute-single-use-continuation (arg var) (declare (type continuation arg) (type lambda-var var)) @@ -1238,15 +1254,15 @@ ;;; any unreferenced variables. Note that FLUSH-DEAD-CODE will come ;;; along right away and delete the REF and then the lambda, since we ;;; flush the FUN continuation. -(defun delete-let (fun) - (declare (type clambda fun)) - (aver (member (functional-kind fun) '(:let :mv-let))) - (note-unreferenced-vars fun) - (let ((call (let-combination fun))) +(defun delete-let (clambda) + (declare (type clambda clambda)) + (aver (functional-letlike-p clambda)) + (note-unreferenced-vars clambda) + (let ((call (let-combination clambda))) (flush-dest (basic-combination-fun call)) (unlink-node call) - (unlink-node (lambda-bind fun)) - (setf (lambda-bind fun) nil)) + (unlink-node (lambda-bind clambda)) + (setf (lambda-bind clambda) nil)) (values)) ;;; This function is called when one of the arguments to a LET @@ -1289,12 +1305,12 @@ (propagate-to-refs var (continuation-type arg)) (let ((use-component (node-component use))) (substitute-leaf-if - #'(lambda (ref) - (cond ((eq (node-component ref) use-component) - t) - (t - (aver (lambda-toplevelish-p (lambda-home fun))) - nil))) + (lambda (ref) + (cond ((eq (node-component ref) use-component) + t) + (t + (aver (lambda-toplevelish-p (lambda-home fun))) + nil))) leaf var)) t))))) ((and (null (rest (leaf-refs var))) @@ -1325,11 +1341,11 @@ (unless (or (functional-entry-fun fun) (lambda-optional-dispatch fun)) (let* ((vars (lambda-vars fun)) - (union (mapcar #'(lambda (arg var) - (when (and arg - (continuation-reoptimize arg) - (null (basic-var-sets var))) - (continuation-type arg))) + (union (mapcar (lambda (arg var) + (when (and arg + (continuation-reoptimize arg) + (null (basic-var-sets var))) + (continuation-type arg))) (basic-combination-args call) vars)) (this-ref (continuation-use (basic-combination-fun call)))) @@ -1342,16 +1358,16 @@ (let ((dest (continuation-dest (node-cont ref)))) (unless (or (eq ref this-ref) (not dest)) (setq union - (mapcar #'(lambda (this-arg old) - (when old - (setf (continuation-reoptimize this-arg) nil) - (type-union (continuation-type this-arg) old))) + (mapcar (lambda (this-arg old) + (when old + (setf (continuation-reoptimize this-arg) nil) + (type-union (continuation-type this-arg) old))) (basic-combination-args dest) union))))) - (mapc #'(lambda (var type) - (when type - (propagate-to-refs var type))) + (mapc (lambda (var type) + (when type + (propagate-to-refs var type))) vars union))) (values)) @@ -1413,11 +1429,11 @@ (multiple-value-bind (types nvals) (values-types (continuation-derived-type arg)) (unless (eq nvals :unknown) - (mapc #'(lambda (var type) - (if (basic-var-sets var) - (propagate-from-sets var type) - (propagate-to-refs var type))) - vars + (mapc (lambda (var type) + (if (basic-var-sets var) + (propagate-from-sets var type) + (propagate-to-refs var type))) + vars (append types (make-list (max (- (length vars) nvals) 0) :initial-element (specifier-type 'null)))))) @@ -1471,14 +1487,14 @@ (when total-nvals (when (and min (< total-nvals min)) - (compiler-warning + (compiler-warn "MULTIPLE-VALUE-CALL with ~R values when the function expects ~ at least ~R." total-nvals min) (setf (basic-combination-kind node) :error) (return-from ir1-optimize-mv-call)) (when (and max (> total-nvals max)) - (compiler-warning + (compiler-warn "MULTIPLE-VALUE-CALL with ~R values when the function expects ~ at most ~R." total-nvals max) @@ -1578,7 +1594,7 @@ (eq (continuation-fun-name (combination-fun use)) 'list)) (change-ref-leaf (continuation-use (combination-fun node)) - (find-free-function 'values "in a strange place")) + (find-free-fun 'values "in a strange place")) (setf (combination-kind node) :full) (let ((args (combination-args use))) (dolist (arg args)