ci: test ZDebug on the PHP 8.6 beta via z-engine 8.6.x-dev - #24
Draft
lisachenko wants to merge 4 commits into
Draft
ci: test ZDebug on the PHP 8.6 beta via z-engine 8.6.x-dev#24lisachenko wants to merge 4 commits into
lisachenko wants to merge 4 commits into
Conversation
z-engine master supports PHP 8.6 (branch-aliased 8.6.x-dev) with linux and darwin definitions committed, so ZDebug can ride it the same way it rides 8.4.x-dev and 8.5.x-dev: - composer: accept lisachenko/z-engine 8.6.x-dev; Composer picks the matching line per running PHP minor as before - ci: add PHP 8.6 legs on ubuntu-latest, macos-latest (arm64) and macos-15-intel (x64) to both the test and PHPStan matrices - docs: extend the supported-version wording to 8.4 / 8.5 / 8.6 (beta) Verified locally on PHP 8.6.0beta2 (NTS): full suite (291 tests) and the integration gate (41 tests, fail-on-skipped) green, PHPStan level max clean, cs check clean. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U4cKi87mVQ896uSrw592fG
The Tests (PHP 8.6, macos-latest) leg fails with child crashes (exit 11) and corrupted function dispatch while the identical PHP build passes on macos-15-intel, and z-engine's own arm64 8.6 suite is green. This manually-dispatched workflow layers the boot sequence (Core::init -> EXT_STMT compile -> opcode handlers -> module registration -> full zdebug boot) on both 8.5 (control) and 8.6, and captures an lldb backtrace of the crash. Will be removed once the failure is root-caused. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U4cKi87mVQ896uSrw592fG
workflow_dispatch workflows on a non-default branch cannot be dispatched, so the temporary diagnostic job runs with the PR's CI instead. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U4cKi87mVQ896uSrw592fG
…tics The layered diagnostic run (see the removed diagnose job in the previous commits) isolated the arm64 failure to z-engine itself: with COMPILE_EXTENDED_STMT on, the first EXT_STMT dispatch through a user opcode handler already corrupts VM state (a by-ref closure counter reads back as a ZEngine\Core object), and instrumented debuggees segfault in ZEND_DO_UCALL_SPEC_RETVAL_USED_TAILCALL_HANDLER. Apple Silicon clang builds of PHP 8.6 select the new tail-call VM (ZEND_VM_KIND_TAILCALL); x64 clang and gcc keep the hybrid VM, which is why linux and macOS x64 are green on the same PHP source. PHP 8.5 on the same arm64 runner passes every probe. Reported upstream as lisachenko/z-engine#280. The arm64 8.6 leg keeps running as continue-on-error for visibility and becomes required again once the upstream issue is resolved; README/AGENTS note the platform exception. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U4cKi87mVQ896uSrw592fG
Owner
Author
|
Root cause of the
The Since no fix exists yet, this PR now runs the arm64+8.6 leg as Generated by Claude Code |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
z-engine's
masternow supports PHP 8.6 (branch-aliased8.6.x-dev, with linux and darwin x64/arm64 definitions committed — lisachenko/z-engine#58, lisachenko/z-engine#278), so ZDebug can be tested on the 8.6 beta the same way it rides the 8.4 and 8.5 lines.8.4.x-dev || 8.5.x-dev || 8.6.x-dev. The platformphprequirement of each z-engine line still makes Composer resolve the matching branch per running PHP minor (verified with dry runs on 8.4 and 8.5 — they keep picking8.4.x-dev/8.5.x-dev).ubuntu-latest,macos-latest(arm64) andmacos-15-intel(x64), and to the PHPStan matrix.Platform exception: macOS arm64 + PHP 8.6 (experimental leg)
Apple Silicon clang builds of PHP 8.6 select the new tail-call VM (
ZEND_VM_KIND_TAILCALL), under which z-engine's user opcode handlers corrupt VM state — every instrumented debuggee crashes. This reproduces with pure z-engine code (no zdebug) and is filed upstream as lisachenko/z-engine#280; the same probes pass on 8.5/arm64 and on 8.6/x64. Until it is resolved theTests (PHP 8.6, macos-latest)leg runs withcontinue-on-error: truefor visibility. See the PR comment for the full diagnosis.Verification
Local (PHP 8.6.0beta2 NTS linux-x64, z-engine
8.6.x-dev):composer test— OK, 291 tests, 1916 assertionscomposer test:integration(--fail-on-skipped) — OK, 41 tests, 782 assertions, none skippedcomposer phpstan(level max) — no errorscomposer cs:check— cleanCI: full matrix green on ubuntu (8.4/8.5/8.6), macOS x64 (8.4/8.5/8.6), macOS arm64 (8.4/8.5), PHPStan (8.4/8.5/8.6); macOS arm64 8.6 experimental per above.