From: Xinchen Hui <laruence@gmail.com>
Date: Tue, 11 Mar 2014 06:23:14 +0000 (+0800)
Subject: Review ended
X-Git-Tag: POST_PHPNG_MERGE~412^2~343
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=53e312c66342c5bf307aff9fca939c749bdafe49;p=php

Review ended

Revert "An demo(for review) to show how to fix the problem(symbol table resize)"

This reverts commit 60c354510b6731ec8348af9c5e230a4859c7c383.
---

diff --git a/Zend/zend_execute.c b/Zend/zend_execute.c
index b9f710cce3..7977c873b3 100644
--- a/Zend/zend_execute.c
+++ b/Zend/zend_execute.c
@@ -214,9 +214,8 @@ static zend_never_inline zval *_get_zval_cv_lookup(zval *ptr, zend_uint var, int
 	if (EG(active_symbol_table)) {
 		ret = zend_hash_find(EG(active_symbol_table), cv);
 		if (ret) {
-			ZEND_ASSERT(Z_TYPE_P(ret) == IS_INDIRECT);
-			ZVAL_INDIRECT(ptr, Z_INDIRECT_P(ret));
-			return Z_INDIRECT_P(ptr);
+			ZVAL_INDIRECT(ptr, ret);
+			return ret;
 		}
 	}
 
@@ -232,14 +231,12 @@ static zend_never_inline zval *_get_zval_cv_lookup(zval *ptr, zend_uint var, int
 			/* break missing intentionally */
 		case BP_VAR_W:
 			if (EG(active_symbol_table)) {
-				zval zv;
-				ZVAL_COPY_VALUE(ptr, &EG(uninitialized_zval));
-				ZVAL_INDIRECT(&zv, ptr);
-				zend_hash_update(EG(active_symbol_table), cv, &zv);
+				ret = zend_hash_update(EG(active_symbol_table), cv, ret);
+				ZVAL_INDIRECT(ptr, ret);
 			} else {
 				ZVAL_NULL(ptr);
+				ret = ptr;
 			}
-			ret = ptr;
 			break;
 	}
 	return ret;
@@ -253,9 +250,8 @@ static zend_never_inline zval *_get_zval_cv_lookup_BP_VAR_R(zval *ptr, zend_uint
 	if (EG(active_symbol_table)) {
 		ret = zend_hash_find(EG(active_symbol_table), cv);
 		if (ret) {
-			ZEND_ASSERT(Z_TYPE_P(ret) == IS_INDIRECT);
-			ZVAL_INDIRECT(ptr, Z_INDIRECT_P(ret));
-			return Z_INDIRECT_P(ptr);
+			ZVAL_INDIRECT(ptr, ret);
+			return ret;
 		}
 	}
 
@@ -271,9 +267,8 @@ static zend_never_inline zval *_get_zval_cv_lookup_BP_VAR_UNSET(zval *ptr, zend_
 	if (EG(active_symbol_table)) {
 		ret = zend_hash_find(EG(active_symbol_table), cv);
 		if (ret) {
-			ZEND_ASSERT(Z_TYPE_P(ret) == IS_INDIRECT);
-			ZVAL_INDIRECT(ptr, Z_INDIRECT_P(ret));
-			return Z_INDIRECT_P(ptr);
+			ZVAL_INDIRECT(ptr, ret);
+			return ret;
 		}
 	}
 
@@ -289,9 +284,8 @@ static zend_never_inline zval *_get_zval_cv_lookup_BP_VAR_IS(zval *ptr, zend_uin
 	if (EG(active_symbol_table)) {
 		ret = zend_hash_find(EG(active_symbol_table), cv);
 		if (ret) {
-			ZEND_ASSERT(Z_TYPE_P(ret) == IS_INDIRECT);
-			ZVAL_INDIRECT(ptr, Z_INDIRECT_P(ret));
-			return Z_INDIRECT_P(ptr);
+			ZVAL_INDIRECT(ptr, ret);
+			return ret;
 		}
 	}
 
@@ -306,17 +300,13 @@ static zend_never_inline zval *_get_zval_cv_lookup_BP_VAR_RW(zval *ptr, zend_uin
 	if (EG(active_symbol_table)) {
 		ret = zend_hash_find(EG(active_symbol_table), cv);
 		if (ret) {
-			ZEND_ASSERT(Z_TYPE_P(ret) == IS_INDIRECT);
-			ZVAL_INDIRECT(ptr, Z_INDIRECT_P(ret));
-			return Z_INDIRECT_P(ptr);
-		} else {
-			zval zv;
-			ZVAL_COPY_VALUE(ptr, &EG(uninitialized_zval));
-			ZVAL_INDIRECT(&zv, ptr);
-			zend_hash_update(EG(active_symbol_table), cv, &zv);
-			zend_error(E_NOTICE, "Undefined variable: %s", cv->val);
-			return ptr;
+			ZVAL_INDIRECT(ptr, ret);
+			return ret;
 		}
+		ret = zend_hash_update(EG(active_symbol_table), cv, &EG(uninitialized_zval));
+		ZVAL_INDIRECT(ptr, ret);
+		zend_error(E_NOTICE, "Undefined variable: %s", cv->val);
+		return ret;
 	} else {
 		ZVAL_NULL(ptr);
 		zend_error(E_NOTICE, "Undefined variable: %s", cv->val);
@@ -332,16 +322,12 @@ static zend_never_inline zval *_get_zval_cv_lookup_BP_VAR_W(zval *ptr, zend_uint
 	if (EG(active_symbol_table)) {
 		ret = zend_hash_find(EG(active_symbol_table), cv);
 		if (ret) {
-			ZEND_ASSERT(Z_TYPE_P(ret) == IS_INDIRECT);
-			ZVAL_INDIRECT(ptr, Z_INDIRECT_P(ret));
-			return Z_INDIRECT_P(ptr);
-		} else {
-			zval zv;
-			ZVAL_COPY_VALUE(ptr, &EG(uninitialized_zval));
-			ZVAL_INDIRECT(&zv, ptr);
-			zend_hash_update(EG(active_symbol_table), cv, &zv);
-			return ptr;
+			ZVAL_INDIRECT(ptr, ret);
+			return ret;
 		}
+		ret = zend_hash_update(EG(active_symbol_table), cv, &EG(uninitialized_zval));
+		ZVAL_INDIRECT(ptr, ret);
+		return ret;
 	} else {
 		ZVAL_NULL(ptr);
 		return ptr;
@@ -1539,20 +1525,20 @@ static inline zend_brk_cont_element* zend_brk_cont(int nest_levels, int array_of
 		zend_hash_apply(EG(active_symbol_table), (apply_func_t) zend_check_symbol TSRMLS_CC);	\
 	}
 
-static int zend_check_symbol(zval *zv TSRMLS_DC)
+static int zend_check_symbol(zval **pz TSRMLS_DC)
 {
-	if (Z_TYPE_P(zv) > 17) {
-		fprintf(stderr, "Warning!  %x has invalid type!\n", *zv);
+	if (Z_TYPE_PP(pz) > 9) {
+		fprintf(stderr, "Warning!  %x has invalid type!\n", *pz);
 /* See http://support.microsoft.com/kb/190351 */
 #ifdef PHP_WIN32
 		fflush(stderr);
 #endif
-	} else if (Z_TYPE_P(zv) == IS_ARRAY) {
-		zend_hash_apply(Z_ARRVAL_P(zv), (apply_func_t) zend_check_symbol TSRMLS_CC);
-	} else if (Z_TYPE_P(zv) == IS_OBJECT) {
+	} else if (Z_TYPE_PP(pz) == IS_ARRAY) {
+		zend_hash_apply(Z_ARRVAL_PP(pz), (apply_func_t) zend_check_symbol TSRMLS_CC);
+	} else if (Z_TYPE_PP(pz) == IS_OBJECT) {
 
 		/* OBJ-TBI - doesn't support new object model! */
-		zend_hash_apply(Z_OBJPROP_P(zv), (apply_func_t) zend_check_symbol TSRMLS_CC);
+		zend_hash_apply(Z_OBJPROP_PP(pz), (apply_func_t) zend_check_symbol TSRMLS_CC);
 	}
 
 	return 0;
@@ -1742,10 +1728,11 @@ static zend_always_inline zend_execute_data *i_create_execute_data_from_op_array
 		if (!EG(active_symbol_table)) {
 			ZVAL_COPY(EX_VAR_NUM(op_array->this_var), &EG(This));
 		} else {
-			zval zv;
 			ZVAL_COPY(EX_VAR_NUM(op_array->this_var), &EG(This));
-			ZVAL_INDIRECT(&zv, EX_VAR_NUM(op_array->this_var));
-			zend_hash_str_add(EG(active_symbol_table), "this", sizeof("this")-1, &zv);
+			zval *zv = zend_hash_str_add(EG(active_symbol_table), "this", sizeof("this")-1, EX_VAR(op_array->this_var));
+			if (zv) {
+				ZVAL_INDIRECT(EX_VAR_NUM(op_array->this_var), zv);
+			}
 		}
 	}
 
diff --git a/Zend/zend_execute_API.c b/Zend/zend_execute_API.c
index 2d6cc38d92..8faba97fc7 100644
--- a/Zend/zend_execute_API.c
+++ b/Zend/zend_execute_API.c
@@ -156,7 +156,7 @@ void init_executor(TSRMLS_D) /* {{{ */
 	ZVAL_LONG(&tmp, 0);
 	zend_vm_stack_push(&tmp TSRMLS_CC);
 
-	zend_hash_init(&EG(symbol_table).ht, 50, NULL, ZVAL_INDIRECT_PTR_DTOR, 0);
+	zend_hash_init(&EG(symbol_table).ht, 50, NULL, ZVAL_PTR_DTOR, 0);
 	EG(active_symbol_table) = &EG(symbol_table).ht;
 
 	zend_llist_apply(&zend_extensions, (llist_apply_func_t) zend_extension_activator TSRMLS_CC);
@@ -426,16 +426,6 @@ ZEND_API void _zval_ptr_dtor(zval *zval_ptr ZEND_FILE_LINE_DC) /* {{{ */
 }
 /* }}} */
 
-ZEND_API void zval_indirect_ptr_dtor(zval *zval_ptr) /* {{{ */
-{
-	if (Z_TYPE_P(zval_ptr) == IS_INDIRECT) {
-		zval_ptr = Z_INDIRECT_P(zval_ptr);
-	}
-	zval_ptr_dtor(zval_ptr);
-}
-
-/* }}} */
-
 ZEND_API void _zval_internal_ptr_dtor(zval *zval_ptr ZEND_FILE_LINE_DC) /* {{{ */
 {
 	if (Z_REFCOUNTED_P(zval_ptr)) {
@@ -1726,7 +1716,7 @@ ZEND_API void zend_rebuild_symbol_table(TSRMLS_D) /* {{{ */
 				EG(active_symbol_table) = *(EG(symtable_cache_ptr)--);
 			} else {
 				ALLOC_HASHTABLE(EG(active_symbol_table));
-				zend_hash_init(EG(active_symbol_table), ex->op_array->last_var, NULL, ZVAL_INDIRECT_PTR_DTOR, 0);
+				zend_hash_init(EG(active_symbol_table), ex->op_array->last_var, NULL, ZVAL_PTR_DTOR, 0);
 				/*printf("Cache miss!  Initialized %x\n", EG(active_symbol_table));*/
 			}
 			ex->symbol_table = EG(active_symbol_table);
@@ -1738,9 +1728,10 @@ ZEND_API void zend_rebuild_symbol_table(TSRMLS_D) /* {{{ */
  			}
 			for (i = 0; i < ex->op_array->last_var; i++) {
 				if (Z_TYPE_P(EX_VAR_NUM_2(ex, i)) != IS_UNDEF) {
-					zval zv;
-					ZVAL_INDIRECT(&zv, EX_VAR_NUM_2(ex, i));
-					zend_hash_update(EG(active_symbol_table), ex->op_array->vars[i], &zv);
+					zval *zv = zend_hash_update(EG(active_symbol_table),
+						ex->op_array->vars[i],
+						EX_VAR_NUM_2(ex, i));
+					ZVAL_INDIRECT(EX_VAR_NUM_2(ex, i), zv);
 				}
 			}
 		}
diff --git a/Zend/zend_variables.c b/Zend/zend_variables.c
index 538dbc22cd..47b07eec79 100644
--- a/Zend/zend_variables.c
+++ b/Zend/zend_variables.c
@@ -287,6 +287,7 @@ ZEND_API void _zval_ptr_dtor_wrapper(zval *zval_ptr)
 	zval_ptr_dtor(zval_ptr);
 }
 
+
 ZEND_API void _zval_internal_ptr_dtor_wrapper(zval *zval_ptr)
 {
 	zval_internal_ptr_dtor(zval_ptr);
diff --git a/Zend/zend_variables.h b/Zend/zend_variables.h
index 377e9f07d2..2e2b183d76 100644
--- a/Zend/zend_variables.h
+++ b/Zend/zend_variables.h
@@ -54,7 +54,6 @@ ZEND_API void _zval_internal_dtor_for_ptr(zval *zvalue ZEND_FILE_LINE_DC);
 ZEND_API void _zval_internal_dtor(zval *zvalue ZEND_FILE_LINE_DC);
 ZEND_API void _zval_internal_ptr_dtor(zval *zvalue ZEND_FILE_LINE_DC);
 ZEND_API void _zval_dtor_wrapper(zval *zvalue);
-ZEND_API void zval_indirect_ptr_dtor(zval *zval_ptr);
 #define zval_copy_ctor(zvalue) _zval_copy_ctor((zvalue) ZEND_FILE_LINE_CC)
 #define zval_dtor(zvalue) _zval_dtor((zvalue) ZEND_FILE_LINE_CC)
 #define zval_ptr_dtor(zval_ptr) _zval_ptr_dtor((zval_ptr) ZEND_FILE_LINE_CC)
@@ -71,13 +70,11 @@ ZEND_API void _zval_internal_ptr_dtor_wrapper(zval *zvalue);
 #define zval_ptr_dtor_wrapper _zval_ptr_dtor_wrapper
 #define zval_internal_dtor_wrapper _zval_internal_dtor_wrapper
 #define zval_internal_ptr_dtor_wrapper _zval_internal_ptr_dtor_wrapper
-#define zval_indirect_ptr_dtor_wrapper _zval_indirect_ptr_wrapper
 #else
 #define zval_copy_ctor_wrapper _zval_copy_ctor_func
 #define zval_ptr_dtor_wrapper _zval_ptr_dtor
 #define zval_internal_dtor_wrapper _zval_internal_dtor
 #define zval_internal_ptr_dtor_wrapper _zval_internal_ptr_dtor
-#define zval_indirect_ptr_dtor_wrapper _zval_indirect_ptr_dtor
 #endif
 
 ZEND_API void zval_add_ref(zval *p);
@@ -91,7 +88,6 @@ END_EXTERN_C()
 #define ZVAL_INTERNAL_DTOR zval_internal_dtor_wrapper
 #define ZVAL_INTERNAL_PTR_DTOR zval_internal_ptr_dtor_wrapper
 #define ZVAL_COPY_CTOR zval_copy_ctor_wrapper
-#define ZVAL_INDIRECT_PTR_DTOR zval_indirect_ptr_dtor
 
 #endif
 
diff --git a/Zend/zend_vm_def.h b/Zend/zend_vm_def.h
index 262b65d023..6049f658f0 100644
--- a/Zend/zend_vm_def.h
+++ b/Zend/zend_vm_def.h
@@ -1069,16 +1069,30 @@ ZEND_VM_HELPER_EX(zend_fetch_var_address_helper, CONST|TMP|VAR|CV, UNUSED|CONST|
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (OP1_TYPE != IS_TMP_VAR) {
+					FREE_OP1();
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+				FREE_OP1();
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (OP1_TYPE == IS_VAR && !OP1_FREE) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -1086,35 +1100,11 @@ ZEND_VM_HELPER_EX(zend_fetch_var_address_helper, CONST|TMP|VAR|CV, UNUSED|CONST|
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (OP1_TYPE != IS_TMP_VAR) {
-				FREE_OP1();
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-			FREE_OP1();
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (OP1_TYPE == IS_VAR && !OP1_FREE) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -3017,10 +3007,9 @@ ZEND_VM_HANDLER(107, ZEND_CATCH, CONST, CV)
 	}
 	ZVAL_OBJ(EX_VAR_NUM(opline->op2.var), EG(exception));
 	if (EG(active_symbol_table)) {
-		zval zv;
 		zend_string *cv = CV_DEF_OF(opline->op2.var);
-		ZVAL_INDIRECT(&zv, EX_VAR_NUM(opline->op2.var));
-		zend_hash_update(EG(active_symbol_table), cv, &zv);
+		zval *zv = zend_hash_update(EG(active_symbol_table), cv, EX_VAR_NUM(opline->op2.var));
+		ZVAL_INDIRECT(EX_VAR_NUM(opline->op2.var), zv);
 	}
 	if (UNEXPECTED(EG(exception) != exception)) {
 		EG(exception)->gc.refcount++;
@@ -4631,9 +4620,6 @@ ZEND_VM_HANDLER(114, ZEND_ISSET_ISEMPTY_VAR, CONST|TMP|VAR|CV, UNUSED|CONST|VAR)
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (OP1_TYPE != IS_CONST && varname == &tmp) {
diff --git a/Zend/zend_vm_execute.h b/Zend/zend_vm_execute.h
index 25ee31d73e..32042fc548 100644
--- a/Zend/zend_vm_execute.h
+++ b/Zend/zend_vm_execute.h
@@ -3622,16 +3622,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CONST_CONST(int type
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_CONST != IS_TMP_VAR) {
+
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_CONST == IS_VAR && !0) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -3639,35 +3653,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CONST_CONST(int type
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_CONST != IS_TMP_VAR) {
-
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_CONST == IS_VAR && !0) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -4223,9 +4213,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_CONST_CONST_HANDLER(ZEND_O
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_CONST != IS_CONST && varname == &tmp) {
@@ -5417,16 +5404,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CONST_VAR(int type,
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_CONST != IS_TMP_VAR) {
+
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_CONST == IS_VAR && !0) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -5434,35 +5435,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CONST_VAR(int type,
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_CONST != IS_TMP_VAR) {
-
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_CONST == IS_VAR && !0) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -5904,9 +5881,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_CONST_VAR_HANDLER(ZEND_OPC
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_CONST != IS_CONST && varname == &tmp) {
@@ -6139,16 +6113,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CONST_UNUSED(int typ
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_CONST != IS_TMP_VAR) {
+
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_CONST == IS_VAR && !0) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -6156,35 +6144,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CONST_UNUSED(int typ
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_CONST != IS_TMP_VAR) {
-
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_CONST == IS_VAR && !0) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -6594,9 +6558,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_CONST_UNUSED_HANDLER(ZEND_
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_CONST != IS_CONST && varname == &tmp) {
@@ -7230,10 +7191,9 @@ static int ZEND_FASTCALL  ZEND_CATCH_SPEC_CONST_CV_HANDLER(ZEND_OPCODE_HANDLER_A
 	}
 	ZVAL_OBJ(EX_VAR_NUM(opline->op2.var), EG(exception));
 	if (EG(active_symbol_table)) {
-		zval zv;
 		zend_string *cv = CV_DEF_OF(opline->op2.var);
-		ZVAL_INDIRECT(&zv, EX_VAR_NUM(opline->op2.var));
-		zend_hash_update(EG(active_symbol_table), cv, &zv);
+		zval *zv = zend_hash_update(EG(active_symbol_table), cv, EX_VAR_NUM(opline->op2.var));
+		ZVAL_INDIRECT(EX_VAR_NUM(opline->op2.var), zv);
 	}
 	if (UNEXPECTED(EG(exception) != exception)) {
 		EG(exception)->gc.refcount++;
@@ -8811,16 +8771,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_TMP_CONST(int type,
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_TMP_VAR != IS_TMP_VAR) {
+					zval_dtor(free_op1.var);
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+				zval_dtor(free_op1.var);
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_TMP_VAR == IS_VAR && !1) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -8828,35 +8802,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_TMP_CONST(int type,
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_TMP_VAR != IS_TMP_VAR) {
-				zval_dtor(free_op1.var);
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-			zval_dtor(free_op1.var);
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_TMP_VAR == IS_VAR && !1) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -9320,9 +9270,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_TMP_CONST_HANDLER(ZEND_OPC
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_TMP_VAR != IS_CONST && varname == &tmp) {
@@ -10491,16 +10438,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_TMP_VAR(int type, ZE
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_TMP_VAR != IS_TMP_VAR) {
+					zval_dtor(free_op1.var);
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+				zval_dtor(free_op1.var);
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_TMP_VAR == IS_VAR && !1) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -10508,35 +10469,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_TMP_VAR(int type, ZE
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_TMP_VAR != IS_TMP_VAR) {
-				zval_dtor(free_op1.var);
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-			zval_dtor(free_op1.var);
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_TMP_VAR == IS_VAR && !1) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -10988,9 +10925,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_TMP_VAR_HANDLER(ZEND_OPCOD
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_TMP_VAR != IS_CONST && varname == &tmp) {
@@ -11223,16 +11157,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_TMP_UNUSED(int type,
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_TMP_VAR != IS_TMP_VAR) {
+					zval_dtor(free_op1.var);
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+				zval_dtor(free_op1.var);
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_TMP_VAR == IS_VAR && !1) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -11240,35 +11188,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_TMP_UNUSED(int type,
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_TMP_VAR != IS_TMP_VAR) {
-				zval_dtor(free_op1.var);
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-			zval_dtor(free_op1.var);
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_TMP_VAR == IS_VAR && !1) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -11567,9 +11491,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_TMP_UNUSED_HANDLER(ZEND_OP
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_TMP_VAR != IS_CONST && varname == &tmp) {
@@ -14573,16 +14494,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_VAR_CONST(int type,
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_VAR != IS_TMP_VAR) {
+					zval_ptr_dtor_nogc(free_op1.var);
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+				zval_ptr_dtor_nogc(free_op1.var);
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_VAR == IS_VAR && !(free_op1.var != NULL)) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -14590,35 +14525,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_VAR_CONST(int type,
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_VAR != IS_TMP_VAR) {
-				zval_ptr_dtor_nogc(free_op1.var);
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-			zval_ptr_dtor_nogc(free_op1.var);
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_VAR == IS_VAR && !(free_op1.var != NULL)) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -15951,9 +15862,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_VAR_CONST_HANDLER(ZEND_OPC
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_VAR != IS_CONST && varname == &tmp) {
@@ -19121,16 +19029,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_VAR_VAR(int type, ZE
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_VAR != IS_TMP_VAR) {
+					zval_ptr_dtor_nogc(free_op1.var);
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+				zval_ptr_dtor_nogc(free_op1.var);
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_VAR == IS_VAR && !(free_op1.var != NULL)) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -19138,35 +19060,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_VAR_VAR(int type, ZE
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_VAR != IS_TMP_VAR) {
-				zval_ptr_dtor_nogc(free_op1.var);
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-			zval_ptr_dtor_nogc(free_op1.var);
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_VAR == IS_VAR && !(free_op1.var != NULL)) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -20463,9 +20361,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_VAR_VAR_HANDLER(ZEND_OPCOD
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_VAR != IS_CONST && varname == &tmp) {
@@ -21105,16 +21000,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_VAR_UNUSED(int type,
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_VAR != IS_TMP_VAR) {
+					zval_ptr_dtor_nogc(free_op1.var);
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+				zval_ptr_dtor_nogc(free_op1.var);
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_VAR == IS_VAR && !(free_op1.var != NULL)) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -21122,35 +21031,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_VAR_UNUSED(int type,
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_VAR != IS_TMP_VAR) {
-				zval_ptr_dtor_nogc(free_op1.var);
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-			zval_ptr_dtor_nogc(free_op1.var);
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_VAR == IS_VAR && !(free_op1.var != NULL)) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -21738,9 +21623,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_VAR_UNUSED_HANDLER(ZEND_OP
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_VAR != IS_CONST && varname == &tmp) {
@@ -31846,16 +31728,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CV_CONST(int type, Z
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_CV != IS_TMP_VAR) {
+
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_CV == IS_VAR && !0) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -31863,35 +31759,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CV_CONST(int type, Z
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_CV != IS_TMP_VAR) {
-
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_CV == IS_VAR && !0) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -33011,9 +32883,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_CV_CONST_HANDLER(ZEND_OPCO
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_CV != IS_CONST && varname == &tmp) {
@@ -36051,16 +35920,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CV_VAR(int type, ZEN
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_CV != IS_TMP_VAR) {
+
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_CV == IS_VAR && !0) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -36068,35 +35951,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CV_VAR(int type, ZEN
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_CV != IS_TMP_VAR) {
-
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_CV == IS_VAR && !0) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -37272,9 +37131,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_CV_VAR_HANDLER(ZEND_OPCODE
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_CV != IS_CONST && varname == &tmp) {
@@ -37909,16 +37765,30 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CV_UNUSED(int type,
 				case BP_VAR_RW:
 					zend_error(E_NOTICE,"Undefined variable: %s", Z_STRVAL_P(varname));
 					/* break missing intentionally */
-				case BP_VAR_W: {
-					   zval zv;
-					   ZVAL_COPY_VALUE(EX_VAR(opline->result.var), &EG(uninitialized_zval));
-					   ZVAL_INDIRECT(&zv, EX_VAR(opline->result.var));
-					   zend_hash_update(target_symbol_table, Z_STR_P(varname), &zv);
-				   	}
+				case BP_VAR_W:
+					retval = zend_hash_update(target_symbol_table, Z_STR_P(varname), &EG(uninitialized_zval));
 					break;
 				EMPTY_SWITCH_DEFAULT_CASE()
 			}
 		}
+		switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
+			case ZEND_FETCH_GLOBAL:
+				if (IS_CV != IS_TMP_VAR) {
+
+				}
+				break;
+			case ZEND_FETCH_LOCAL:
+
+				break;
+			case ZEND_FETCH_STATIC:
+				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
+				break;
+			case ZEND_FETCH_GLOBAL_LOCK:
+				if (IS_CV == IS_VAR && !0) {
+					Z_ADDREF_P(EX_VAR(opline->op1.var));
+				}
+				break;
+		}
 	}
 
 
@@ -37926,35 +37796,11 @@ static int ZEND_FASTCALL zend_fetch_var_address_helper_SPEC_CV_UNUSED(int type,
 		zval_dtor(&tmp_varname);
 	}
 
-	if (retval && EXPECTED(Z_TYPE_P(retval) == IS_INDIRECT)) {
-		retval = Z_INDIRECT_P(retval);
-	}
-
-	switch (opline->extended_value & ZEND_FETCH_TYPE_MASK) {
-		case ZEND_FETCH_GLOBAL:
-			if (IS_CV != IS_TMP_VAR) {
-
-			}
-			break;
-		case ZEND_FETCH_LOCAL:
-
-			break;
-		case ZEND_FETCH_STATIC:
-			if (retval) {
-				zval_update_constant(retval, (void*) 1 TSRMLS_CC);
-			}
-			break;
-		case ZEND_FETCH_GLOBAL_LOCK:
-			if (IS_CV == IS_VAR && !0) {
-				Z_ADDREF_P(EX_VAR(opline->op1.var));
-			}
-			break;
+	if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
+		SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
 	}
 
 	if (EXPECTED(retval != NULL)) {
-		if (opline->extended_value & ZEND_FETCH_MAKE_REF) {
-			SEPARATE_ZVAL_TO_MAKE_IS_REF(retval);
-		}
 		if (Z_REFCOUNTED_P(retval)) Z_ADDREF_P(retval);
 		switch (type) {
 			case BP_VAR_R:
@@ -38429,9 +38275,6 @@ static int ZEND_FASTCALL  ZEND_ISSET_ISEMPTY_VAR_SPEC_CV_UNUSED_HANDLER(ZEND_OPC
 			if ((value = zend_hash_find(target_symbol_table, Z_STR_P(varname))) == NULL) {
 				isset = 0;
 			}
-			if (UNEXPECTED(Z_TYPE_P(value) == IS_INDIRECT)) {
-				value = Z_INDIRECT_P(value);
-			}
 		}
 
 		if (IS_CV != IS_CONST && varname == &tmp) {