X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=src%2Fcompiler%2Fmacros.lisp;h=e96a92bf134a903658a0a07ef2828f00a18e595a;hb=d131dfb49a3e6522d2358d14252f3f52cfcd202a;hp=98ab8fc99a6dc9180d42f7dfc6d44fbbadf21eb7;hpb=6053e7f804b430144bb09e2d107ad4ab3fb97db4;p=sbcl.git diff --git a/src/compiler/macros.lisp b/src/compiler/macros.lisp index 98ab8fc..e96a92b 100644 --- a/src/compiler/macros.lisp +++ b/src/compiler/macros.lisp @@ -619,30 +619,29 @@ (ctran-next it)) (t (return))))) ,@(when lvar-var - `((,lvar-var #1=(when (valued-node-p ,node-var) - (node-lvar ,node-var)) - #1#)))) + `((,lvar-var (when (valued-node-p ,node-var) + (node-lvar ,node-var)) + (when (valued-node-p ,node-var) + (node-lvar ,node-var)))))) (nil) ,@body ,@(when restart-p `((when (block-delete-p ,n-block) (return))))))) -;;; like DO-NODES, only iterating in reverse order +;;; Like DO-NODES, only iterating in reverse order. Should be careful +;;; with block being split under us. (defmacro do-nodes-backwards ((node-var lvar block) &body body) (let ((n-block (gensym)) - (n-start (gensym)) (n-prev (gensym))) - `(do* ((,n-block ,block) - (,n-start (block-start ,n-block)) - (,node-var (block-last ,n-block) (ctran-use ,n-prev)) - (,n-prev (node-prev ,node-var) (node-prev ,node-var)) - (,lvar #1=(when (valued-node-p ,node-var) (node-lvar ,node-var)) - #1#)) - (nil) - ,@body - (when (eq ,n-prev ,n-start) - (return nil))))) + `(loop with ,n-block = ,block + for ,node-var = (block-last ,n-block) then (ctran-use ,n-prev) + for ,n-prev = (when ,node-var (node-prev ,node-var)) + and ,lvar = (when (and ,node-var (valued-node-p ,node-var)) + (node-lvar ,node-var)) + while ,node-var + do (progn + ,@body)))) (defmacro do-nodes-carefully ((node-var block) &body body) (with-unique-names (n-block n-ctran)