Delay IS_UNDEF check for FETCH_DIM_R and family

Reuse zend_fetch_dimension_address_read() for FETCH_LIST

Optimize ASSIGN_DIM (separate common unspecializeble code into helper functions).

Separate unspecializeble code of INIT_DYNAMIC_CALL into separate helper functions.

IS_CONST operands don't have to be separated. Use reference-counting instead of duplication.

- with opcache all IS_CONST operands are not refcounted (scalars, interned strings or immutable a

IS_CONST operands don't have to be separated. Use reference-counting instead of duplication.

- with opcache all IS_CONST operands are not refcounted (scalars, interned strings or immutable arrays)
- without opcache IS_CONST operands are not shared between processes or threads and may use common reference counters

show more ...

Initialize op_array->run_time_cache on slow path of INIT_FCALL opcode and omit corresponding check on fast path of DO_FCALL.

Merge zend_execute_data->called_scope into zend_execute_data->This.
"called_scope" made sense only for static method calls, for dynamic calls it was always equal to the class of $this.
Now EG

Merge zend_execute_data->called_scope into zend_execute_data->This.
"called_scope" made sense only for static method calls, for dynamic calls it was always equal to the class of $this.
Now EG(This) may store IS_OBJECT + $this or IS_UNUSED + "called_scope" (of course, "called_scope" may be NULL).
Some code might need to be adopted to support this change.
Checks (Z_OBJ(EX(This))) might need to be converted into (Z_TYPE(EX(This)) == IS_OBJECT).

show more ...

Flags ZEND_CALL_RELEASE_THIS and ZEND_CALL_CLOSURE should never be set together.
Closures keep responsibility for releasing $this theirselves.

Drop some dead defines

Squashed commit of the following:

commit 98471821a89d7106de508fc544504ba674025abe
Author: Dmitry Stogov <dmitry@zend.com>
Date: Thu Mar 17 21:56:04 2016 +0300

Fixed wr

Squashed commit of the following:

commit 98471821a89d7106de508fc544504ba674025abe
Author: Dmitry Stogov <dmitry@zend.com>
Date: Thu Mar 17 21:56:04 2016 +0300

Fixed wrong constant usage

commit 8183b811e74c2989bcd8ab9d870fc3adbe973980
Author: Dmitry Stogov <dmitry@zend.com>
Date: Thu Mar 17 21:10:23 2016 +0300

Added ability to serialize and serialize opcode handlers for file-cache

commit 3516b261de7694bb1f34c5ae1adfb2f29dfab7bf
Author: Dmitry Stogov <dmitry@zend.com>
Date: Thu Mar 17 10:11:59 2016 +0300

Added missed file

commit f4475a23608623ccbf11b00c1c74df1db55dffa2
Author: Dmitry Stogov <dmitry@zend.com>
Date: Thu Mar 17 10:00:45 2016 +0300

Transparently introduce type-specialized opcode handlers.
This affects only PHP VM, and doesn't change anything else.

show more ...

Sorry for that... Merged by accident....

Revert "Transparently introduce type-specialized opcode handlers."

This reverts commit 59d00b8bcc05f72b29c3c768542b2bb6e69e1af4.

Transparently introduce type-specialized opcode handlers.
This affects only PHP VM, and doesn't change anything else.

Added possibility for extra specialization for SMART_BRANCH (e.g. IS_EQUAL+JMPZ superinstruction).

More effecient fix for bug #71756

Improve the codes (FETCH_DIM_R/IS is read context, we should not expose IS_REFERENCE)

Bug #71756 (Call-by-reference widens scope to uninvolved functions when used in switch)

Add support for negative string offsets (syntax)

Use quick check for first arguments

Fix bug #71572

Discussed on the internals list as part of
http://markmail.org/message/fa625yc2eegk2dio.

Remove ZEND_CALL_CTOR_RESULT_UNUSED

Instead emit a FREE on the result of NEW.

Remove EXT_TYPE_UNUSED in favor of IS_UNUSED

This means we no longer allocate an unused VAR for the retval of
instructions that support unused results.

Nearly all instructions a

Remove EXT_TYPE_UNUSED in favor of IS_UNUSED

This means we no longer allocate an unused VAR for the retval of
instructions that support unused results.

Nearly all instructions already used the result variable only if
it was used. The only exception to this was the return value
variable for internal function call results. I've adjusted the code
to use a stack zval for the unused return case now. As we have
retval specialization now, we know that it doesn't matter.

show more ...

Add extra VM operand specialization

For now RETVAL and OP_DATA= are supported

Rename IS_ERROR into _IS_ERROR (to prevent redefinition on Windows).

Use special type IS_ERROR instread of EG(error_zval). (we still need EG(error_zval) for SPL support).

Fixed bug #71474 (Crash because of VM stack corruption on Magento2).