0.7.9.41:
[sbcl.git] / src / compiler / locall.lisp
index 8867f28..304142d 100644 (file)
@@ -40,7 +40,8 @@
     (let ((arg (car args))
          (var (car vars)))
       (cond ((leaf-refs var)
-            (assert-continuation-type arg (leaf-type var)))
+            (assert-continuation-type arg (leaf-type var)
+                                       (lexenv-policy (node-lexenv call))))
            (t
             (flush-dest arg)
             (setf (car args) nil)))))
          (declare (type index ,n-supplied))
          ,(if (policy *lexenv* (zerop safety))
               `(declare (ignore ,n-supplied))
-              `(%verify-argument-count ,n-supplied ,nargs))
-         (%funcall ,fun ,@temps))))
+              `(%verify-arg-count ,n-supplied ,nargs))
+         (locally
+           ;; KLUDGE: The intent here is to enable tail recursion
+           ;; optimization, since leaving frames for wrapper
+           ;; functions like this on the stack is actually more
+           ;; annoying than helpful for debugging. Unfortunately
+           ;; trying to express this by messing with the
+           ;; ANSI-standard declarations is a little awkward, since
+           ;; no matter how we do it we'll tend to have side-effects
+           ;; on things like SPEED-vs.-SAFETY comparisons. Perhaps
+           ;; it'd be better to define a new SB-EXT:TAIL-RECURSIVELY
+           ;; declaration and use that? -- WHN 2002-07-08
+           (declare (optimize (speed 2) (debug 1)))
+           (%funcall ,fun ,@temps)))))
     (optional-dispatch
      (let* ((min (optional-dispatch-min-args fun))
            (max (optional-dispatch-max-args fun))
                           (n-count (gensym)))
                       `(multiple-value-bind (,n-context ,n-count)
                            (%more-arg-context ,n-supplied ,max)
-                         (%funcall ,more ,@temps ,n-context ,n-count))))))
+                         (locally
+                           ;; KLUDGE: As above, we're trying to
+                           ;; enable tail recursion optimization and
+                           ;; any other effects of this declaration
+                           ;; are accidental. -- WHN 2002-07-08
+                           (declare (optimize (speed 2) (debug 1)))
+                           (%funcall ,more ,@temps ,n-context ,n-count)))))))
             (t
-             (%argument-count-error ,n-supplied)))))))))
+             (%arg-count-error ,n-supplied)))))))))
 
 ;;; Make an external entry point (XEP) for FUN and return it. We
 ;;; convert the result of MAKE-XEP-LAMBDA in the correct environment,
 ;;; discover an XEP after the initial local call analyze pass.
 (defun make-xep (fun)
   (declare (type functional fun))
-  (aver (not (functional-entry-fun fun)))
+  (aver (null (functional-entry-fun fun)))
   (with-ir1-environment-from-node (lambda-bind (main-entry fun))
     (let ((res (ir1-convert-lambda (make-xep-lambda-expression fun)
                                   :debug-name (debug-namify
            (component-reanalyze *current-component*) t
            (component-reoptimize *current-component*) t)
       (etypecase fun
-       (clambda (locall-analyze-fun-1 fun))
+       (clambda
+        (locall-analyze-fun-1 fun))
        (optional-dispatch
         (dolist (ep (optional-dispatch-entry-points fun))
           (locall-analyze-fun-1 ep))
 
   (values))
 
-;;; We examine all NEW-FUNS in COMPONENT, attempting to convert calls
-;;; into local calls when it is legal. We also attempt to convert each
-;;; LAMBDA to a LET. LET conversion is also triggered by deletion of a
-;;; function reference, but functions that start out eligible for
-;;; conversion must be noticed sometime.
+;;; We examine all NEW-FUNCTIONALS in COMPONENT, attempting to convert
+;;; calls into local calls when it is legal. We also attempt to
+;;; convert each LAMBDA to a LET. LET conversion is also triggered by
+;;; deletion of a function reference, but functions that start out
+;;; eligible for conversion must be noticed sometime.
 ;;;
 ;;; Note that there is a lot of action going on behind the scenes
 ;;; here, triggered by reference deletion. In particular, the
 ;;; COMPONENT-LAMBDAS are being hacked to remove newly deleted and LET
 ;;; converted LAMBDAs, so it is important that the LAMBDA is added to
-;;; the COMPONENT-LAMBDAS when it is. Also, the COMPONENT-NEW-FUNS may
-;;; contain all sorts of drivel, since it is not updated when we
-;;; delete functions, etc. Only COMPONENT-LAMBDAS is updated.
+;;; the COMPONENT-LAMBDAS when it is. Also, the
+;;; COMPONENT-NEW-FUNCTIONALS may contain all sorts of drivel, since
+;;; it is not updated when we delete functions, etc. Only
+;;; COMPONENT-LAMBDAS is updated.
 ;;;
-;;; COMPONENT-REANALYZE-FUNS is treated similarly to
-;;; NEW-FUNS, but we don't add lambdas to the LAMBDAS.
+;;; COMPONENT-REANALYZE-FUNCTIONALS is treated similarly to
+;;; COMPONENT-NEW-FUNCTIONALS, but we don't add lambdas to the
+;;; LAMBDAS.
 (defun locall-analyze-component (component)
   (declare (type component component))
   (aver-live-component component)
   (loop
-    (let* ((new-fun (pop (component-new-funs component)))
-          (fun (or new-fun (pop (component-reanalyze-funs component)))))
-      (unless fun (return))
-      (let ((kind (functional-kind fun)))
-       (cond ((member kind '(:deleted :let :mv-let :assignment)))
-             ((and (null (leaf-refs fun)) (eq kind nil)
-                   (not (functional-entry-fun fun)))
-              (delete-functional fun))
+    (let* ((new-functional (pop (component-new-functionals component)))
+          (functional (or new-functional
+                          (pop (component-reanalyze-functionals component)))))
+      (unless functional
+       (return))
+      (let ((kind (functional-kind functional)))
+       (cond ((or (functional-somewhat-letlike-p functional)
+                  (eql kind :deleted))
+              (values)) ; nothing to do
+             ((and (null (leaf-refs functional)) (eq kind nil)
+                   (not (functional-entry-fun functional)))
+              (delete-functional functional))
              (t
-              ;; Fix/check FUN's relationship to COMPONENT-LAMDBAS.
-              (cond ((not (lambda-p fun))
-                     ;; Since FUN isn't a LAMBDA, this doesn't apply: no-op.
+              ;; Fix/check FUNCTIONAL's relationship to COMPONENT-LAMDBAS.
+              (cond ((not (lambda-p functional))
+                     ;; Since FUNCTIONAL isn't a LAMBDA, this doesn't
+                     ;; apply: no-op.
                      (values))
-                    (new-fun ; FUN came from NEW-FUNS, hence is new.
-                     ;; FUN becomes part of COMPONENT-LAMBDAS now.
-                     (aver (not (member fun (component-lambdas component))))
-                     (push fun (component-lambdas component)))
-                    ((eql (lambda-inlinep fun) :inline)
-                     ;; FUNs marked :INLINE are sometimes in
-                     ;; COMPONENT-LAMBDAS and sometimes not. I (WHN
-                     ;; 2002-01-01) haven't figured this one out yet,
-                     ;; so don't assert anything.
-                     ;;
-                     ;; (One possibility: LAMBDAs to represent the
-                     ;; inline expansions of things which are defined
-                     ;; elsewhere might not be in COMPONENT-LAMBDAS,
-                     ;; which LAMBDAs to represent the inline
-                     ;; expansions of local functions might in
-                     ;; COMPONENT-LAMBDAS?)
-                     (values))
-                    (t ; FUN is old.
-                     ;; FUN should be in COMPONENT-LAMBDAS already.
-                     (aver (member fun (component-lambdas component)))))
-              (locall-analyze-fun-1 fun)
-              (when (lambda-p fun)
-                (maybe-let-convert fun)))))))
+                    (new-functional ; FUNCTIONAL came from
+                                    ; NEW-FUNCTIONALS, hence is new.
+                     ;; FUNCTIONAL becomes part of COMPONENT-LAMBDAS now.
+                     (aver (not (member functional
+                                        (component-lambdas component))))
+                     (push functional (component-lambdas component)))
+                    (t ; FUNCTIONAL is old.
+                     ;; FUNCTIONAL should be in COMPONENT-LAMBDAS already.
+                     (aver (member functional (component-lambdas
+                                               component)))))
+              (locall-analyze-fun-1 functional)
+              (when (lambda-p functional)
+                (maybe-let-convert functional)))))))
   (values))
 
 (defun locall-analyze-clambdas-until-done (clambdas)
         ;; COMPONENT is the only one here. Let's make that explicit.
         (aver (= 1 (length (functional-components clambda))))
         (aver (eql component (first (functional-components clambda))))
-        (when (component-new-funs component)
+        (when (component-new-functionals component)
           (setf did-something t)
           (locall-analyze-component component))))
      (unless did-something
 ;;; to be in an infinite recursive loop, then change the reference to
 ;;; reference a fresh copy. We return whichever function we decide to
 ;;; reference.
-(defun maybe-expand-local-inline (fun ref call)
+(defun maybe-expand-local-inline (original-functional ref call)
   (if (and (policy call
-                  (and (>= speed space) (>= speed compilation-speed)))
+                  (and (>= speed space)
+                       (>= speed compilation-speed)))
           (not (eq (functional-kind (node-home-lambda call)) :external))
           (inline-expansion-ok call))
-      (with-ir1-environment-from-node call
-       (let* ((*lexenv* (functional-lexenv fun))
-              (won nil)
-              (res (catch 'local-call-lossage
-                     (prog1
-                         (ir1-convert-lambda
-                          (functional-inline-expansion fun)
-                          :debug-name (debug-namify "local inline ~A"
-                                                    (leaf-debug-name fun)))
-                       (setq won t)))))
-         (cond (won
-                (change-ref-leaf ref res)
-                res)
-               (t
-                (let ((*compiler-error-context* call))
-                  (compiler-note "couldn't inline expand because expansion ~
-                                  calls this LET-converted local function:~
-                                  ~%  ~S"
-                                 (leaf-debug-name res)))
-                fun))))
-      fun))
+      (multiple-value-bind (losing-local-functional converted-lambda)
+         (catch 'locall-already-let-converted
+           (with-ir1-environment-from-node call
+             (let ((*lexenv* (functional-lexenv original-functional)))
+               (values nil
+                       (ir1-convert-lambda
+                        (functional-inline-expansion original-functional)
+                        :debug-name (debug-namify
+                                     "local inline ~A"
+                                     (leaf-debug-name
+                                      original-functional)))))))
+       (cond (losing-local-functional
+              (let ((*compiler-error-context* call))
+                (compiler-note "couldn't inline expand because expansion ~
+                                calls this LET-converted local function:~
+                                ~%  ~S"
+                               (leaf-debug-name losing-local-functional)))
+              original-functional)
+             (t
+              (change-ref-leaf ref converted-lambda)
+              converted-lambda)))
+      original-functional))
 
 ;;; Dispatch to the appropriate function to attempt to convert a call.
 ;;; REF must be a reference to a FUNCTIONAL. This is called in IR1
-;;; optimize as well as in local call analysis. If the call is is
+;;; optimization as well as in local call analysis. If the call is is
 ;;; already :LOCAL, we do nothing. If the call is already scheduled
 ;;; for deletion, also do nothing (in addition to saving time, this
 ;;; also avoids some problems with optimizing collections of functions
   (values))
 
 ;;; Attempt to convert a multiple-value call. The only interesting
-;;; case is a call to a function that Looks-Like-An-MV-Bind, has
+;;; case is a call to a function that LOOKS-LIKE-AN-MV-BIND, has
 ;;; exactly one reference and no XEP, and is called with one values
 ;;; continuation.
 ;;;
       (assert-continuation-type
        (first (basic-combination-args call))
        (make-values-type :optional (mapcar #'leaf-type (lambda-vars ep))
-                        :rest *universal-type*))))
+                        :rest *universal-type*)
+       (lexenv-policy (node-lexenv call)))))
   (values))
 
 ;;; Attempt to convert a call to a lambda. If the number of args is
           ;; wrong. And we're in locall.lisp here, so it's probably
           ;; (haven't checked this..) a call to something in the same
           ;; file. So maybe it deserves a full warning anyway.
-          (compiler-warning
+          (compiler-warn
            "function called with ~R argument~:P, but wants exactly ~R"
            call-args nargs)
           (setf (basic-combination-kind call) :error)))))
     (cond ((< call-args min-args)
           ;; FIXME: See FIXME note at the previous
           ;; wrong-number-of-arguments warnings in this file.
-          (compiler-warning
+          (compiler-warn
            "function called with ~R argument~:P, but wants at least ~R"
            call-args min-args)
           (setf (basic-combination-kind call) :error))
          (t
           ;; FIXME: See FIXME note at the previous
           ;; wrong-number-of-arguments warnings in this file.
-          (compiler-warning
+          (compiler-warn
            "function called with ~R argument~:P, but wants at most ~R"
            call-args max-args)
           (setf (basic-combination-kind call) :error))))
         (flame (policy call (or (> speed inhibit-warnings)
                                 (> space inhibit-warnings))))
         (loser nil)
+         (allowp nil)
+         (allow-found nil)
         (temps (make-gensym-list max))
         (more-temps (make-gensym-list (length more))))
     (collect ((ignores)
               (key-vars var))
              ((:rest :optional))
              ((:more-context :more-count)
-              (compiler-warning "can't local-call functions with &MORE args")
+              (compiler-warn "can't local-call functions with &MORE args")
               (setf (basic-combination-kind call) :error)
               (return-from convert-more-call))))))
 
       (when (optional-dispatch-keyp fun)
        (when (oddp (length more))
-         (compiler-warning "function called with odd number of ~
-                            arguments in keyword portion")
+         (compiler-warn "function called with odd number of ~
+                         arguments in keyword portion")
 
          (setf (basic-combination-kind call) :error)
          (return-from convert-more-call))
            (let ((name (continuation-value cont))
                  (dummy (first temp))
                  (val (second temp)))
+              ;; FIXME: check whether KEY was supplied earlier
+              (when (and (eq name :allow-other-keys) (not allow-found))
+                (let ((val (second key)))
+                  (cond ((constant-continuation-p val)
+                         (setq allow-found t
+                               allowp (continuation-value val)))
+                        (t (when flame
+                             (compiler-note "non-constant :ALLOW-OTHER-KEYS value"))
+                           (setf (basic-combination-kind call) :error)
+                           (return-from convert-more-call)))))
              (dolist (var (key-vars)
                           (progn
                             (ignores dummy val)
-                            (setq loser name)))
+                             (unless (eq name :allow-other-keys)
+                               (setq loser name))))
                (let ((info (lambda-var-arg-info var)))
                  (when (eq (arg-info-key info) name)
                    (ignores dummy)
                    (supplied (cons var val))
                    (return)))))))
 
-       (when (and loser (not (optional-dispatch-allowp fun)))
-         (compiler-warning "function called with unknown argument keyword ~S"
-                           loser)
+       (when (and loser (not (optional-dispatch-allowp fun)) (not allowp))
+         (compiler-warn "function called with unknown argument keyword ~S"
+                        loser)
          (setf (basic-combination-kind call) :error)
          (return-from convert-more-call)))
 
        (join-components component clambda-component)))
     (let ((*current-component* component))
       (node-ends-block call))
-    ;; FIXME: Use PROPER-LIST-OF-LENGTH-P here, and look for other
+    ;; FIXME: Use DESTRUCTURING-BIND here, and grep for other 
     ;; uses of '=.*length' which could also be converted to use
-    ;; PROPER-LIST-OF-LENGTH-P.
+    ;; DESTRUCTURING-BIND or PROPER-LIST-OF-LENGTH-P.
     (aver (= (length (block-succ call-block)) 1))
     (let ((next-block (first (block-succ call-block))))
       (unlink-blocks call-block next-block)
 
   (declare (type clambda clambda) (type basic-combination call))
 
-  (let ((component (block-component (node-block call))))
+  (let ((component (node-component call)))
     (unlink-blocks (component-head component) (lambda-block clambda))
     (setf (component-lambdas component)
          (delete clambda (component-lambdas component)))
   (let* ((home (node-home-lambda call))
         (home-env (lambda-physenv home)))
 
+    (aver (not (eq home clambda)))
+
     ;; CLAMBDA belongs to HOME now.
     (push clambda (lambda-lets home))
     (setf (lambda-home clambda) home)
     ;; All of CLAMBDA's LETs belong to HOME now.
     (let ((lets (lambda-lets clambda)))
       (dolist (let lets)
-       (setf (lambda-home let) home)
-       (setf (lambda-physenv let) home-env))
+        (setf (lambda-home let) home)
+        (setf (lambda-physenv let) home-env))
       (setf (lambda-lets home) (nconc lets (lambda-lets home))))
     ;; CLAMBDA no longer has an independent existence as an entity
     ;; which has LETs.
     ;; HOME no longer calls CLAMBDA, and owns all of CLAMBDA's old
     ;; DFO dependencies.
     (setf (lambda-calls-or-closes home)
-         (delete clambda
-                 (nunion (lambda-calls-or-closes clambda)
-                         (lambda-calls-or-closes home))))
+          (delete clambda
+                  (nunion (lambda-calls-or-closes clambda)
+                          (lambda-calls-or-closes home))))
     ;; CLAMBDA no longer has an independent existence as an entity
     ;; which calls things or has DFO dependencies.
     (setf (lambda-calls-or-closes clambda) nil)
 
     ;; All of CLAMBDA's ENTRIES belong to HOME now.
     (setf (lambda-entries home)
-         (nconc (lambda-entries clambda)
-                (lambda-entries home)))
+          (nconc (lambda-entries clambda)
+                 (lambda-entries home)))
     ;; CLAMBDA no longer has an independent existence as an entity
     ;; with ENTRIES.
     (setf (lambda-entries clambda) nil))
          (cont (node-cont call))
          (call-type (node-derived-type call)))
       (when (eq (continuation-use cont) call)
-       (assert-continuation-type cont (continuation-asserted-type result)))
+        (set-continuation-type-assertion
+         cont
+         (continuation-asserted-type result)
+         (continuation-type-to-check result)))
       (unless (eq call-type *wild-type*)
        (do-uses (use result)
          (derive-node-type use call-type)))
 ;;; work. We change the CALL's CONT to be the continuation heading the
 ;;; BIND block, and also do REOPTIMIZE-CONTINUATION on the args and
 ;;; CONT so that LET-specific IR1 optimizations get a chance. We blow
-;;; away any entry for the function in *FREE-FUNCTIONS* so that nobody
+;;; away any entry for the function in *FREE-FUNS* so that nobody
 ;;; will create new references to it.
 (defun let-convert (fun call)
   (declare (type clambda fun) (type basic-combination call))
   (reoptimize-continuation (node-cont call))
   (values))
 
+;;; Are there any declarations in force to say CLAMBDA shouldn't be
+;;; LET converted?
+(defun declarations-suppress-let-conversion-p (clambda)
+  ;; From the user's point of view, LET-converting something that
+  ;; has a name is inlining it. (The user can't see what we're doing
+  ;; with anonymous things, and suppressing inlining
+  ;; for such things can easily give Python acute indigestion, so 
+  ;; we don't.)
+  (when (leaf-has-source-name-p clambda)
+    ;; ANSI requires that explicit NOTINLINE be respected.
+    (or (eq (lambda-inlinep clambda) :notinline)
+       ;; If (> DEBUG SPEED) we can guess that inlining generally
+       ;; won't be appreciated, but if the user specifically requests
+       ;; inlining, that takes precedence over our general guess.
+       (and (policy clambda (> debug speed))
+            (not (eq (lambda-inlinep clambda) :inline))))))
+
 ;;; We also don't convert calls to named functions which appear in the
 ;;; initial component, delaying this until optimization. This
 ;;; minimizes the likelihood that we will LET-convert a function which
 ;;; may have references added due to later local inline expansion.
 (defun ok-initial-convert-p (fun)
   (not (and (leaf-has-source-name-p fun)
-           (eq (component-kind (lambda-component fun))
-               :initial))))
+           (or (declarations-suppress-let-conversion-p fun)
+               (eq (component-kind (lambda-component fun))
+                   :initial)))))
 
 ;;; This function is called when there is some reason to believe that
 ;;; CLAMBDA might be converted into a LET. This is done after local
-;;; call analysis, and also when a reference is deleted. We only
-;;; convert to a let when the function is a normal local function, has
-;;; no XEP, and is referenced in exactly one local call. Conversion is
-;;; also inhibited if the only reference is in a block about to be
-;;; deleted. We return true if we converted.
-;;;
-;;; These rules may seem unnecessarily restrictive, since there are
-;;; some cases where we could do the return with a jump that don't
-;;; satisfy these requirements. The reason for doing things this way
-;;; is that it makes the concept of a LET much more useful at the
-;;; level of IR1 semantics. The :ASSIGNMENT function kind provides
-;;; another way to optimize calls to single-return/multiple call
-;;; functions.
-;;;
-;;; We don't attempt to convert calls to functions that have an XEP,
-;;; since we might be embarrassed later when we want to convert a
-;;; newly discovered local call. Also, see OK-INITIAL-CONVERT-P.
+;;; call analysis, and also when a reference is deleted. We return
+;;; true if we converted.
 (defun maybe-let-convert (clambda)
   (declare (type clambda clambda))
-  (let ((refs (leaf-refs clambda)))
-    (when (and refs
-              (null (rest refs))
-              (member (functional-kind clambda) '(nil :assignment))
-              (not (functional-entry-fun clambda)))
-      (let* ((ref-cont (node-cont (first refs)))
-            (dest (continuation-dest ref-cont)))
-       (when (and dest
-                   (basic-combination-p dest)
-                  (eq (basic-combination-fun dest) ref-cont)
-                  (eq (basic-combination-kind dest) :local)
-                  (not (block-delete-p (node-block dest)))
-                  (cond ((ok-initial-convert-p clambda) t)
-                        (t
-                         (reoptimize-continuation ref-cont)
-                         nil)))
-         (unless (eq (functional-kind clambda) :assignment)
-           (let-convert clambda dest))
-         (reoptimize-call dest)
-         (setf (functional-kind clambda)
-               (if (mv-combination-p dest) :mv-let :let))))
-      t)))
+  (unless (declarations-suppress-let-conversion-p clambda)
+    ;; We only convert to a LET when the function is a normal local
+    ;; function, has no XEP, and is referenced in exactly one local
+    ;; call. Conversion is also inhibited if the only reference is in
+    ;; a block about to be deleted.
+    ;;
+    ;; These rules limiting LET conversion may seem unnecessarily
+    ;; restrictive, since there are some cases where we could do the
+    ;; return with a jump that don't satisfy these requirements. The
+    ;; reason for doing things this way is that it makes the concept
+    ;; of a LET much more useful at the level of IR1 semantics. The
+    ;; :ASSIGNMENT function kind provides another way to optimize
+    ;; calls to single-return/multiple call functions.
+    ;;
+    ;; We don't attempt to convert calls to functions that have an
+    ;; XEP, since we might be embarrassed later when we want to
+    ;; convert a newly discovered local call. Also, see
+    ;; OK-INITIAL-CONVERT-P.
+    (let ((refs (leaf-refs clambda)))
+      (when (and refs
+                (null (rest refs))
+                (member (functional-kind clambda) '(nil :assignment))
+                (not (functional-entry-fun clambda)))
+       (let* ((ref (first refs))
+               (ref-cont (node-cont ref))
+              (dest (continuation-dest ref-cont)))
+         (when (and dest
+                    (basic-combination-p dest)
+                    (eq (basic-combination-fun dest) ref-cont)
+                    (eq (basic-combination-kind dest) :local)
+                    (not (block-delete-p (node-block dest)))
+                    (cond ((ok-initial-convert-p clambda) t)
+                          (t
+                           (reoptimize-continuation ref-cont)
+                           nil)))
+            (when (eq clambda (node-home-lambda dest))
+              (delete-lambda clambda)
+              (return-from maybe-let-convert nil))
+           (unless (eq (functional-kind clambda) :assignment)
+              (let-convert clambda dest))
+           (reoptimize-call dest)
+           (setf (functional-kind clambda)
+                 (if (mv-combination-p dest) :mv-let :let))))
+       t))))
 \f
 ;;;; tail local calls and assignments
 
        (link-blocks block (lambda-block fun))
        (values t (maybe-convert-to-assignment fun))))))
 
-;;; This is called when we believe it might make sense to convert Fun
-;;; to an assignment. All this function really does is determine when
-;;; a function with more than one call can still be combined with the
-;;; calling function's environment. We can convert when:
+;;; This is called when we believe it might make sense to convert
+;;; CLAMBDA to an assignment. All this function really does is
+;;; determine when a function with more than one call can still be
+;;; combined with the calling function's environment. We can convert
+;;; when:
 ;;; -- The function is a normal, non-entry function, and
 ;;; -- Except for one call, all calls must be tail recursive calls 
 ;;;    in the called function (i.e. are self-recursive tail calls)
 ;;; calls as long as they all return to the same place (i.e. have the
 ;;; same conceptual continuation.) A special case of this would be
 ;;; when all of the outside calls are tail recursive.
-(defun maybe-convert-to-assignment (fun)
-  (declare (type clambda fun))
-  (when (and (not (functional-kind fun))
-            (not (functional-entry-fun fun)))
-    (let ((non-tail nil)
-         (call-fun nil))
-      (when (and (dolist (ref (leaf-refs fun) t)
+(defun maybe-convert-to-assignment (clambda)
+  (declare (type clambda clambda))
+  (when (and (not (functional-kind clambda))
+            (not (functional-entry-fun clambda)))
+    (let ((outside-non-tail-call nil)
+         (outside-call nil))
+      (when (and (dolist (ref (leaf-refs clambda) t)
                   (let ((dest (continuation-dest (node-cont ref))))
                     (when (or (not dest)
                                (block-delete-p (node-block dest)))
                        (return nil))
                     (let ((home (node-home-lambda ref)))
-                      (unless (eq home fun)
-                        (when call-fun (return nil))
-                        (setq call-fun home))
+                      (unless (eq home clambda)
+                        (when outside-call
+                          (return nil))
+                        (setq outside-call dest))
                       (unless (node-tail-p dest)
-                        (when (or non-tail (eq home fun)) (return nil))
-                        (setq non-tail dest)))))
-                (ok-initial-convert-p fun))
-       (setf (functional-kind fun) :assignment)
-       (let-convert fun (or non-tail
-                            (continuation-dest
-                             (node-cont (first (leaf-refs fun))))))
-       (when non-tail (reoptimize-call non-tail))
-       t))))
+                        (when (or outside-non-tail-call (eq home clambda))
+                          (return nil))
+                        (setq outside-non-tail-call dest)))))
+                (ok-initial-convert-p clambda))
+        (cond (outside-call (setf (functional-kind clambda) :assignment)
+                            (let-convert clambda outside-call)
+                            (when outside-non-tail-call
+                              (reoptimize-call outside-non-tail-call))
+                            t)
+              (t (delete-lambda clambda)
+                 nil))))))