push in Void Context

The attached patch avoids pushing an IV onto the stack for a push op
in void context.  The "Is there attached magic" branch in the op checks
for void context as does the non-magical branch in the unshift op.

-- c

[Edited version of patch after review by Yitzchak Scott-Thoennes]

1 files changed, 3 insertions, 1 deletions

diff --git a/pp.c b/pp.c
index e3b7798065..fae2d6db9b 100644
--- a/pp.c
+++ b/pp.c
@@ -4738,7 +4738,9 @@ PP(pp_push)
 
 	PL_delaymagic = 0;
 	SP = ORIGMARK;
-	PUSHi( AvFILLp(ary) + 1 );
+	if (OP_GIMME(PL_op, 0) != G_VOID) {
+	    PUSHi( AvFILL(ary) + 1 );
+	}
     }
     RETURN;
 }