Skip to content

[pull] master from php:master#832

Merged
pull[bot] merged 10 commits intoturkdevops:masterfrom
php:master
Apr 2, 2026
Merged

[pull] master from php:master#832
pull[bot] merged 10 commits intoturkdevops:masterfrom
php:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Apr 2, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

iluuu1994 and others added 4 commits April 2, 2026 11:33
When a smart branch and jump live in separate basic blocks, the JIT can't skip
the jitting of the jump, as it may be reachable through another predecessor.
When the smart branch is executed using zend_jit_handler(), we're manually
writing the result of the branch to the given temporary so that the jump will
work as expected. That happens in zend_jit_set_cond().

However, this was only correctly handled for JMPZ branches. The current opline
was compared to opline following the jump, which would set the var to 1 if
equal, i.e. the branch was not taken, meaning var was not zero. For JMPNZ we
need to do the opposite.

Fixes GH-21593
* PHP-8.4:
  Fix function JIT JMPNZ smart branch
* PHP-8.5:
  Fix function JIT JMPNZ smart branch
@pull pull bot locked and limited conversation to collaborators Apr 2, 2026
@pull pull bot added the ⤵️ pull label Apr 2, 2026
jordikroon and others added 6 commits April 2, 2026 17:04
RFC: https://wiki.php.net/rfc/parameter-doccomments

---------

Co-authored-by: Christian Schneider <schneider@search.ch>
Co-authored-by: Tim Düsterhus <tim@tideways-gmbh.com>
Co-authored-by: Daniel Scherzer <daniel.e.scherzer@gmail.com>
PQresultMemorySize is available since libpq 12.
On Windows libpq 16.2 is used at time of writing:
https://github.com/winlibs/postgresql
FreeBSD builds without DQS (double-quote support). Transform tests so we can
re-eanble them for FreeBSD. Add separate tests for DQS.

Closes GH-21495
@pull pull bot merged commit 6609966 into turkdevops:master Apr 2, 2026
3 of 4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants