Language: continue/break 0 is Zend positive-integer compile fatal (#32155)

[PurHur]

Summary

• continue 0 / break 0 now compile-abort with php-src's 'continue' operator accepts only positive integers (same for break), instead of leaking php-parser Too high of a count for Stmt_Continue.
• PHP SSOT in lib/Ast/BreakContinueOperandCompileCheck.php (AST visitor before php-cfg LoopResolver); JIT stdout stays empty — fatal is on stderr as Zend PHP Fatal error.
• Vendor LoopResolver distinguishes non-positive vs too-deep so a missed visitor still cannot emit Stmt_Continue.

php-src: Zend/zend_compile.c (zend_compile_break_continue()).
PHP: lib/Ast/BreakContinueOperandCompileCheck.php, lib/Runtime.php.
C runtime lines removed: none.

Closes #32155

Test plan

• Issue repro VM + JIT (stderr/stdout split)
• Dedicated PHPUnit ContinueZero32155VMTest / JITTest
• Unit BreakContinueOperandCompileCheckTest + GotoStatementTest

Verification

./script/docker-exec.sh -- bash -lc 'php bin/vm.php test/repro/maintainer_gap_continue_zero.php 2>&1; echo EXIT:$?'

PHP Fatal error:  'continue' operator accepts only positive integers in /compiler/test/repro/maintainer_gap_continue_zero.php on line 3
EXIT:255

./script/docker-exec.sh -- bash -lc 'php bin/jit.php test/repro/maintainer_gap_continue_zero.php > /tmp/c0-out.txt 2> /tmp/c0-err.txt; echo EXIT:$?; echo "---stdout---"; cat /tmp/c0-out.txt; echo "---stderr---"; cat /tmp/c0-err.txt'

EXIT:255
---stdout---
---stderr---
PHP Fatal error:  'continue' operator accepts only positive integers in /compiler/test/repro/maintainer_gap_continue_zero.php on line 3

./script/phpunit.sh --filter 'ContinueZero32155|BreakContinueOperandCompileCheck|GotoStatementTest'

OK (16 tests, 35 assertions)

./script/docker-exec.sh -- bash -lc 'php script/bootstrap-inventory.php --check'

OK 7579/7579

Not covered: full VMTest/JITTest suites; AOT phpc build of this repro; ci-fast.sh; check-selfhost-spine-coverage-sync.php is red on this tree for 6 pre-existing inventory files not in spine (JitLdapResult.php, StreamIncludeOpen*, …) — our new unit is required from the spine.

Made with Cursor