Skip to content

php-in-php: StringVarDump NestedJIT → JitVmHelperLink::ensureCompiled (#1492) #23143

Description

@PurHur

Category

php-in-php · runtime-shrink (#1492, peer #23097 PowInt / #23116 ArrayDiff / #22575 MathFrexp)

Problem

lib/JIT/Builtin/StringVarDump.php still hand-rolls NestedJitCompileScope::run + parseAndCompile + new JIT() for VarDumpJitHelper.php. Peer string/math runtimes route helper compile through JitVmHelperLink::ensureCompiled for consistent user-script env clearing, insert-block restore, and helper-runtime cache.

php-src reference

PHP implementation target

  • lib/JIT/Builtin/StringVarDump.php — replace ensureJitHelperCompiled body with JitVmHelperLink::ensureCompiled + lookupCompiled; drop use PHPCompiler\JIT / hand-rolled NestedJitCompileScope::run (keep ::isActive early-return if still required)
  • test/unit/VarDumpRuntimeShrinkTest.php — guard JitVmHelperLink::ensureCompiled, no hand-rolled NestedJitCompileScope::run / parseAndCompile / new JIT(

Repro

./script/docker-exec.sh -- bash -lc 'source script/php-env.sh && vendor/bin/phpunit test/unit/VarDumpRuntimeShrinkTest.php'
make bootstrap-selfhost-vm-driver-execute-probe

Done when

  • StringVarDump uses JitVmHelperLink::ensureCompiled only for helper compile
  • Shrink unit test green
  • make bootstrap-selfhost-vm-driver-execute-probe green
  • PR body cites lines removed

Refs #1492, #9195, #13241, #16565, #23097

Metadata

Metadata

Assignees

No one assigned

    Labels

    area:compilerCompiler / CFG / JITenhancementNew feature or requestimplementation-readySpec complete: repro, php-src ref, done-when — safe for workers to claimphase-0:FoundationPhase 0 – foundation & DevEx

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions