Add comment to explaining the new check

laruence · laruence · commit 249915f938f6 · 2015-07-08T17:44:54.000+08:00
diff --git a/ext/opcache/Optimizer/optimize_temp_vars_5.c b/ext/opcache/Optimizer/optimize_temp_vars_5.c
@@ -152,6 +152,9 @@ void optimize_temporary_variables(zend_op_array *op_array, zend_optimizer_ctx *c
 			currT = VAR_NUM(ZEND_RESULT(opline).var) - offset;
 			if (valid_T[currT]) {
 				if (start_of_T[currT] == opline) {
+					/* ZEND_FAST_CALL can not share temporary var with others
+					 * since the fast_var could also be set by ZEND_HANDLE_EXCEPTION
+					 * which could be ahead of it */
 					if (opline->opcode != ZEND_FAST_CALL) {
 						taken_T[map_T[currT]] = 0;
 					}
