Skip to content

php-in-php: drop JitEnv putenv thin libc fork — always GetenvJitHelper (#21023) - #21042

Merged
PurHur merged 1 commit into
masterfrom
agent/runtime-php-putenv-21023
Jul 19, 2026
Merged

PurHur merged 1 commit into
masterfrom
agent/runtime-php-putenv-21023

Conversation

@PurHur

@PurHur PurHur commented Jul 19, 2026

Copy link
Copy Markdown
Owner

Summary

C runtime: no new runtime/*.c; net shrink is the thin LLVM early-return (~15 LOC of forked emit path).

Closes #21023

Test plan

  • vendor/bin/phpunit test/unit/PutenvRuntimeShrinkTest.php → OK (2 tests, 11 assertions)
  • ./phpc build -o /tmp/putenv_smoke test/bootstrap-aot/putenv_smoke.php && /tmp/putenv_smokeputenv ok
  • make bootstrap-selfhost-vm-driver-execute-probe → OK

Verification transcript

OK (2 tests, 11 assertions)
putenv ok
smoke_ok
bootstrap-selfhost-vm-driver-execute-probe: OK

Made with Cursor

#21023)

Route thin standalone AOT putenv through GetenvJitHelper overlay + libc
setenv mirror (peer getenv #20644). Return bare i1 from literal putenv so
Internal::call type inference succeeds for putenv_smoke.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

php-in-php: drop isThinStandaloneAotMain libc fork from JitEnv putenv — always GetenvJitHelper + libc mirror (re-#20499)

1 participant