Revert "Properly release zend_ast_op_array in OPcache"

This reverts commit 345834b.

Author: TimWolla

ext/opcache/zend_persist.c

@@ -190,9 +190,6 @@ static zend_ast *zend_persist_ast(zend_ast *ast)
 		node = (zend_ast *) copy;
 	} else if (ast->kind == ZEND_AST_OP_ARRAY) {
 		zend_ast_op_array *copy = zend_shared_memdup(ast, sizeof(zend_ast_op_array));
-		/* We're holding a separate reference to the op_array in the AST. Release it
-		 * early because zend_persist_op_array() is destructive. */
-		destroy_op_array(copy->op_array);
 		zval z;
 		ZVAL_PTR(&z, copy->op_array);
 		zend_persist_op_array(&z);

ext/opcache/zend_persist_calc.c

@@ -91,14 +91,6 @@ static void zend_persist_ast_calc(zend_ast *ast)
 		zval z;
 		ZVAL_PTR(&z, zend_ast_get_op_array(ast)->op_array);
 		zend_persist_op_array_calc(&z);
-
-		/* If op_array is shared, the function name refcount is still incremented for each use,
-		 * so we need to release it here. We remembered the original function name in xlat. */
-		zend_string *old_function_name =
-			zend_shared_alloc_get_xlat_entry(&zend_ast_get_op_array(ast)->op_array->function_name);
-		if (old_function_name) {
-			zend_string_release_ex(old_function_name, 0);
-		}
 	} else if (ast->kind == ZEND_AST_CALLABLE_CONVERT) {
 		ADD_SIZE(sizeof(zend_ast_fcc));
 	} else if (zend_ast_is_decl(ast)) {