Stdlib: locale_lookup() JIT path throws instead of matching php-src locale lookup (ext/intl/locale/locale_methods.c)

[PurHur]

Category

bug · php-src-strict

Problem

locale_lookup() already has a PHP implementation in ext/intl/VmLocale.php, and the VM path returns the php-src result for ordinary lookup cases. The JIT entrypoint in ext/intl/locale_lookup.php still hard-fails with RuntimeException: locale_lookup() JIT lowering not implemented; use VM, so the same builtin diverges by backend instead of matching Zend/php-src.

Repro	Zend / php-src expected	JIT (2026-08-18)
locale_lookup([\"de-DE\",\"de\"], \"de-CH\", true, \"en\")	\"de\"	RuntimeException: locale_lookup() JIT lowering not implemented; use VM

VM/reference confirmation in this tree today:

• ./script/docker-exec.sh -- bash -lc 'php test/repro/issue_25198_locale_lookup_named.php' prints lookup_pos=de and lookup_named=de
• ./script/docker-exec.sh -- bash -lc 'php -r '\''require \"vendor/autoload.php\"; var_dump(PHPCompiler\\ext\\intl\\VmLocale::lookup([\"de-DE\",\"de\"], \"de-CH\", true, \"en\"));'\''' prints string(2) \"de\"

php-src reference

• php/php-src ext/intl/locale/locale_methods.c — locale_lookup() / RFC 4647 lookup semantics
• php/php-src ext/intl/php_intl.stub.php — procedural arginfo / named-parameter surface

PHP implementation target

• ext/intl/locale_lookup.php — replace the hardcoded JIT exception with lowering that forwards to the existing PHP/intl helper path
• ext/intl/VmLocale.php — keep the lookup semantics SSOT; JIT should reuse this behavior rather than adding new C/runtime logic
• lib/JIT/ helper if needed, but keep semantics in PHP-owned intl helpers

Repro

./script/docker-exec.sh -- bash -lc 'php test/repro/issue_25198_locale_lookup_named.php'
./script/docker-exec.sh -- bash -lc 'php -r '\''require "vendor/autoload.php"; $ctx=(new ReflectionClass("PHPCompiler\\JIT\\Context"))->newInstanceWithoutConstructor(); try { (new PHPCompiler\ext\intl\locale_lookup())->call($ctx); } catch (Throwable $e) { echo get_class($e),": ",$e->getMessage(),"\n"; }'\'''

Done when

• locale_lookup() returns the same result on JIT as php-src/VM for the lookup repro above instead of throwing
• Named/optional args from test/repro/issue_25198_locale_lookup_named.php behave identically on VM and JIT
• Compliance .phpt guard under test/compliance/cases/intl/ covers the JIT result path, not only reflection metadata
• php-src-strict; no php-compiler-strict shortcut

[PurHur]

claim: PHP Compiler STD Lib Advanced — batch (issues #32118, #32119, #32120)

[PurHur]

claim: PHP Compiler STD Lib Advanced — continuing #32118 (locale_lookup JIT); #32119/#32120 queued after merge.

[PurHur]

Merged in #32174 (2c92f6bdb6).

Verification (Docker ./script/phpunit.sh / docker-exec.sh):

• --filter 'LocaleLookupJitShrinkTest|LocaleLookupJitCompileTest|LocaleLookupJit32118' → OK (7 tests, 24 assertions). JIT + VM .phpt result path (lookup_pos=de / named / vars / fallback) plus compile-time fold.
• php test/repro/issue_32118_locale_lookup_jit.php → lookup_pos=de, lookup_named=de, lookup_vars=de, lookup_fallback=en_US.
• php script/bootstrap-inventory.php --check → OK 7583/7583 after merge.

php-src-strict: ext/intl/JitLocaleLookup.php + LocaleLookupJitHelper → VmLocale::lookup() (php-src ext/intl/locale/locale_methods.c). Next in batch: #32119.