diff options
| author | jvoisin | 2026-05-10 00:09:44 +0200 |
|---|---|---|
| committer | jvoisin | 2026-05-10 01:06:20 +0200 |
| commit | 7d8180a29b2ac45ef1814a7a2cad8e4da937ac76 (patch) | |
| tree | 6fdc236f5ded2320ba004118e6fcbf43ea84c91c /src/tests/disable_function/disabled_functions_ret_user_used.phpt | |
| parent | bcec0cafc9edbf1a563f184debf01169aed64c85 (diff) | |
PHP 8.5's opcache optimizer can inline trivial user functions (constant
return values), completely eliminating the DO_UCALL opcode. When this
happens, zend_execute_ex is never invoked and snuffleupagus's
return-value monitoring hooks never fire.
Fix this by setting ZEND_ACC_HAS_TYPE_HINTS on monitored functions'
op_arrays during compilation (via sp_op_array_handler). This flag is
checked by opcache's zend_try_inline_call() and prevents inlining.
For 0-arg functions — the only ones eligible for inlining — there are
no RECV opcodes, so the runtime impact is zero.
To enable sp_op_array_handler when return-value rules are configured,
the extension now registers itself as a zend extension and sets
ZEND_COMPILE_HANDLE_OP_ARRAY (previously only done for global_strict).
The disabled_function_echo_2 test is updated to use separate echo
statements and opcache.optimization_level=0, since opcache's echo
merging is a compile-time string concatenation that cannot be prevented
per-function.
This is a bit ugly, but it's the less awful solution to be able to hook return
values.
Diffstat (limited to 'src/tests/disable_function/disabled_functions_ret_user_used.phpt')
0 files changed, 0 insertions, 0 deletions
