]> granicus.if.org Git - php/commitdiff
Remove dead code from ZEND_THROW
authorNikita Popov <nikic@php.net>
Tue, 6 May 2014 18:02:26 +0000 (20:02 +0200)
committerNikita Popov <nikic@php.net>
Tue, 6 May 2014 18:02:26 +0000 (20:02 +0200)
The IS_CONST case always throws an E_ERROR two lines above that,
so no need to handle it further.

Zend/zend_vm_def.h
Zend/zend_vm_execute.h

index 7657b6dcc1a5355649caf76b41046947ed05ca5f..afb569bdc26d8b96a50dda2c6dc14c95e557feb5 100644 (file)
@@ -2913,7 +2913,6 @@ ZEND_VM_HANDLER(108, ZEND_THROW, CONST|TMP|VAR|CV, ANY)
 {
        USE_OPLINE
        zval *value;
-       zval exception;
        zend_free_op free_op1;
 
        SAVE_OPLINE();
@@ -2927,10 +2926,7 @@ ZEND_VM_HANDLER(108, ZEND_THROW, CONST|TMP|VAR|CV, ANY)
        }
 
        zend_exception_save(TSRMLS_C);
-       if (OP1_TYPE == IS_CONST) {
-               ZVAL_DUP(&exception, value);
-               value = &exception;
-       } else if (OP1_TYPE != IS_TMP_VAR) {
+       if (OP1_TYPE != IS_TMP_VAR) {
                if (Z_REFCOUNTED_P(value)) Z_ADDREF_P(value);
        }
 
index e803acbdaa1346226117b84505410d86e6d79bc5..d4d58b1bddbe72dedbb973cd5fa81f18e1b3c624 100644 (file)
@@ -2663,7 +2663,6 @@ static int ZEND_FASTCALL  ZEND_THROW_SPEC_CONST_HANDLER(ZEND_OPCODE_HANDLER_ARGS
 {
        USE_OPLINE
        zval *value;
-       zval exception;
 
 
        SAVE_OPLINE();
@@ -2677,10 +2676,7 @@ static int ZEND_FASTCALL  ZEND_THROW_SPEC_CONST_HANDLER(ZEND_OPCODE_HANDLER_ARGS
        }
 
        zend_exception_save(TSRMLS_C);
-       if (IS_CONST == IS_CONST) {
-               ZVAL_DUP(&exception, value);
-               value = &exception;
-       } else if (IS_CONST != IS_TMP_VAR) {
+       if (IS_CONST != IS_TMP_VAR) {
                if (Z_REFCOUNTED_P(value)) Z_ADDREF_P(value);
        }
 
@@ -7774,7 +7770,6 @@ static int ZEND_FASTCALL  ZEND_THROW_SPEC_TMP_HANDLER(ZEND_OPCODE_HANDLER_ARGS)
 {
        USE_OPLINE
        zval *value;
-       zval exception;
        zend_free_op free_op1;
 
        SAVE_OPLINE();
@@ -7788,10 +7783,7 @@ static int ZEND_FASTCALL  ZEND_THROW_SPEC_TMP_HANDLER(ZEND_OPCODE_HANDLER_ARGS)
        }
 
        zend_exception_save(TSRMLS_C);
-       if (IS_TMP_VAR == IS_CONST) {
-               ZVAL_DUP(&exception, value);
-               value = &exception;
-       } else if (IS_TMP_VAR != IS_TMP_VAR) {
+       if (IS_TMP_VAR != IS_TMP_VAR) {
                if (Z_REFCOUNTED_P(value)) Z_ADDREF_P(value);
        }
 
@@ -12842,7 +12834,6 @@ static int ZEND_FASTCALL  ZEND_THROW_SPEC_VAR_HANDLER(ZEND_OPCODE_HANDLER_ARGS)
 {
        USE_OPLINE
        zval *value;
-       zval exception;
        zend_free_op free_op1;
 
        SAVE_OPLINE();
@@ -12856,10 +12847,7 @@ static int ZEND_FASTCALL  ZEND_THROW_SPEC_VAR_HANDLER(ZEND_OPCODE_HANDLER_ARGS)
        }
 
        zend_exception_save(TSRMLS_C);
-       if (IS_VAR == IS_CONST) {
-               ZVAL_DUP(&exception, value);
-               value = &exception;
-       } else if (IS_VAR != IS_TMP_VAR) {
+       if (IS_VAR != IS_TMP_VAR) {
                if (Z_REFCOUNTED_P(value)) Z_ADDREF_P(value);
        }
 
@@ -29905,7 +29893,6 @@ static int ZEND_FASTCALL  ZEND_THROW_SPEC_CV_HANDLER(ZEND_OPCODE_HANDLER_ARGS)
 {
        USE_OPLINE
        zval *value;
-       zval exception;
 
 
        SAVE_OPLINE();
@@ -29919,10 +29906,7 @@ static int ZEND_FASTCALL  ZEND_THROW_SPEC_CV_HANDLER(ZEND_OPCODE_HANDLER_ARGS)
        }
 
        zend_exception_save(TSRMLS_C);
-       if (IS_CV == IS_CONST) {
-               ZVAL_DUP(&exception, value);
-               value = &exception;
-       } else if (IS_CV != IS_TMP_VAR) {
+       if (IS_CV != IS_TMP_VAR) {
                if (Z_REFCOUNTED_P(value)) Z_ADDREF_P(value);
        }