Don't swap operands of ZEND_MUL

If this is used with operator overloading, then the operation does
not necessarily commute (for example, matrix multiplication).

diff --git a/ext/opcache/Optimizer/pass3.c b/ext/opcache/Optimizer/pass3.c
index fc2cae10d8d705168136e035c5c09226cab72681..fecadbcec587c1d0543e7e60edfe4fa88cb066e1 100644 (file)
--- a/ext/opcache/Optimizer/pass3.c
+++ b/ext/opcache/Optimizer/pass3.c
@@ -93,7 +93,6 @@ void zend_optimizer_pass3(zend_op_array *op_array)
                                        if ((opline->op2_type & (IS_VAR | IS_CV))
                                                && opline->op2.var == next_opline->op1.var &&
                                                (opline->opcode == ZEND_ADD ||
-                                                opline->opcode == ZEND_MUL ||
                                                 opline->opcode == ZEND_BW_OR ||
                                                 opline->opcode == ZEND_BW_AND ||
                                                 opline->opcode == ZEND_BW_XOR)) {