Remove now redundant checks in the VM

Author: iluuu1994

Zend/zend_compile.c

@@ -535,6 +535,7 @@ static zend_always_inline uint32_t get_temporary_variable(void) /* {{{ */
 }
 /* }}} */
 
+// FIXME: Check all uses of lookup_cv() to see whether they need CHECK_VAR.
 static int lookup_cv(zend_string *name) /* {{{ */{
 	zend_op_array *op_array = CG(active_op_array);
 	int i = 0;

Zend/zend_compile.h

@@ -434,6 +434,7 @@ static zend_always_inline uint32_t zend_visibility_to_set_visibility(uint32_t vi
 #define ZEND_SHORT_CIRCUITING_CHAIN_EMPTY 2
 
 // Must not clash with ZEND_SHORT_CIRCUITING_CHAIN_MASK
+// FIXME: Can we get rid of this? CHECK_VAR instruction should be inserted now instead.
 #define ZEND_JMP_NULL_BP_VAR_IS 4
 
 char *zend_visibility_string(uint32_t fn_flags);