Skip to content

php-in-php: OpensslEncryptRuntime NestedJIT via JitVmHelperLink (#22683)#22684

Merged
PurHur merged 1 commit into
masterfrom
agent/runtime-php-openssl-encrypt-helperlink
Jul 23, 2026
Merged

php-in-php: OpensslEncryptRuntime NestedJIT via JitVmHelperLink (#22683)#22684
PurHur merged 1 commit into
masterfrom
agent/runtime-php-openssl-encrypt-helperlink

Conversation

@PurHur

@PurHur PurHur commented Jul 23, 2026

Copy link
Copy Markdown
Owner

Summary

Net shrink: OpensslEncryptRuntime.php −34 lines of NestedJIT boilerplate (shared path in JitVmHelperLink). No new C.

Closes #22683

Verification (exact)

$ php script/bootstrap-inventory.php --check
OK 6455/6455

$ ./script/docker-exec.sh -- bash -lc 'make bootstrap-selfhost-link'
bootstrap-selfhost-link: OK /compiler/build/selfhost
EXIT:0

$ ./script/docker-exec.sh -- bash -lc 'source script/php-env.sh && vendor/bin/phpunit test/unit/OpensslEncryptRuntimeShrinkTest.php'
OK (2 tests, 10 assertions)

$ ./script/docker-exec.sh -- bash -lc 'source script/php-env.sh && vendor/bin/phpunit test/unit/JIT/OpensslEncryptRuntimeStandaloneTest.php'
OK (2 tests, 13 assertions)   # includes ensureLinked ABI registration

$ make bootstrap-selfhost-vm-driver-execute-probe
bootstrap-selfhost-vm-driver-execute-probe: OK /compiler/build/selfhost-lib-spine-smoke
PROBE_EXIT:0

Test plan

  • Shrink + standalone unit tests (Docker)
  • make bootstrap-selfhost-vm-driver-execute-probe
  • inventory --check OK 6455/6455
  • make bootstrap-selfhost-link OK (pre-claim pillar 1)

Route OpensslEncryptJitHelper compile through JitVmHelperLink::ensureCompiled
(peer OpensslDigest #22554) so user-script AOT gets env clear + helper cache.

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur
PurHur merged commit fc10221 into master Jul 23, 2026
@PurHur
PurHur deleted the agent/runtime-php-openssl-encrypt-helperlink branch July 23, 2026 15:33
@PurHur

PurHur commented Jul 23, 2026

Copy link
Copy Markdown
Owner Author

Cleanup hygiene

Deleting remote branch agent/runtime-php-openssl-encrypt-helperlink.

Reason: PR merged into master (#22684 / fc102217e); issue #22683 closed; no open PR on this head.

Open PR #22717 (agent/issue-22642-gen0-refresh-oom) and release/v1.1.0-dev left untouched.

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: OpensslEncryptRuntime NestedJIT → JitVmHelperLink::ensureCompiled (#1492)

1 participant