Changeset 13528

Timestamp:

08/21/11 20:03:44 (10 years ago)

Author:

ehuelsmann

Message:

Fix MULTIPLE-VALUE-PROG1.10 the right way.

File:

• trunk/abcl/src/org/armedbear/lisp/compiler-pass2.lisp (modified) (2 diffs)

trunk/abcl/src/org/armedbear/lisp/compiler-pass2.lisp

@@ -3006 +3006 @@
     (emit-getfield +lisp-thread+ "_values" +lisp-object-array+)
     (astore values-register)
-
-    ;; we need to clear the values again:
-    ;;  some parts will consider a non-null _values array
-    ;;  by itself a legitimate return value (multiple values)
-    ;;  however, if we have a non-local return after the previous form
-    ;;  set the values array, other code may pick up the values instead
-    ;;  of the actual return code. (Fixes MULTIPLE-VALUE-PROG1.10)
-    (emit-clear-values)
     (compile-progn-body subforms nil nil)
     ;; Restore multiple values returned by first subform.
@@ -3895 +3887 @@
         ;; inside the block we're returning from?
         (unless (enclosed-by-protected-block-p block)
-          (unless (compiland-single-valued-p *current-compiland*)
-            (emit-clear-values))
+          (emit-clear-values)
           (compile-form result-form (block-target block) nil)
           (when (and (block-needs-environment-restoration block)