Skip to content

php-in-php: delete JitSoundex LLVM — route soundex through SoundexJitHelper PHP (#1492) #13448

Closed

Description

@PurHur

Category

php-in-php · php-src-strict

Problem

soundex() JIT still emits ~177 LOC LLVM in JitSoundex.php. VM SSOT is VmString::soundex().

php-src reference

  • ext/standard/string.c — PHP_FUNCTION(soundex)

Repro

wc -l ext/standard/JitSoundex.php
./script/docker-exec.sh -- bash -lc 'source script/php-env.sh && vendor/bin/phpunit test/unit/SoundexBuiltinTest.php'

Done when

  • __compiler_soundex thin bridge calls SoundexJitHelper compiled PHP
  • JitSoundex.php deleted
  • SoundexRuntimeShrinkTest + soundex compliance green

Refs #2416, #1492

Activity

  1. added
    enhancementNew feature or request
    implementation-readySpec complete: repro, php-src ref, done-when — safe for workers to claim
    on Jun 29, 2026
  2. PurHur commented on Jun 29, 2026

    @PurHur
    OwnerAuthor

    claim: PHP Runtime → PHP — batch (#13447 merged, #13448)

  3. PurHur commented on Jun 29, 2026

    @PurHur
    OwnerAuthor

    Merged via PR #13453.

    Verification (Docker):

    • vendor/bin/phpunit --filter SoundexRuntimeShrink — OK (3 tests)
    • vendor/bin/phpunit --filter Soundex — VM/JIT compliance green (7 tests)
    • make bootstrap-selfhost-vm-driver-execute-probe — OK

    Deleted JitSoundex.php (~177 LLVM lines).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestimplementation-readySpec complete: repro, php-src ref, done-when — safe for workers to claim

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions