Skip to content

feat: Stage 35a optimizer contracts, dispatch, and escape audit - #191

Merged
amasiye merged 2 commits into
mainfrom
develop
Oct 2, 2026
Merged

amasiye merged 2 commits into
mainfrom
develop

Conversation

@amasiye

@amasiye amasiye commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Derive native pointer contracts and bounded class stack placement from shared validated MIR proofs.
  • Preserve initialization, reverse destruction, and checked cleanup; reject forged escaping stack environments and classify taking closure arguments correctly.
  • Test optimized direct/virtual/interface dispatch, unboxed specialization, metadata negatives, and native/PHP ownership parity.
  • Add object/code-size and optimizer metrics to opt-in standalone and multi-file build-plan reports; pin the merged benchmark matrix.
  • Remove the job-wide raw runtime archive override from CI so both native profiles exercise compiler-selected, provenance-verified runtimes. Keep explicit override coverage isolated in runtime-artifact tests.

Validation

  • Full canonical php scripts/validate_work_unit.php --llvm passed, including default workspace checks and all selected LLVM suites. Durable interpreter/Cranelift/LLVM parity: 11 passed; no failures.
  • Focused LLVM optimizer (10), performance-report (12), and PHP (107) tests passed. Stronger runtime identity assertions also passed in the final focused report run.
  • Four macOS native cleanup/closure leak probes reported zero leaks.
  • Formatting, documentation, diagnostic, artifact-policy, and published benchmark-pin guards passed.
  • One bounded review and one fix-verification round completed. No further automated review requests planned; the GitHub automated reviewer reported its usage limit, not a successful review.
  • Compiler validation cache: 12.67 GiB on the approved secondary volume; no broad cache cleanup.

Delivery Gates

All delivery gates passed. Final companion pins are published (tooling 88a44b9, website f9c5040), both with green CI. Installed doriac and doria-lsp report compiler 7bf2e1376c39f60d73f3809ee876406da74e9280; the installed compiler passes its native smoke after the installation cache is removed. All 10 corrected-head compiler CI checks are green, including Linux/macOS/Windows backend parity and Linux leak checks; there are no unresolved review threads. Final installed benchmark verification passed 76 harness tests, 10 native smoke targets, five interpreter cases, and four million-iteration dispatch checks, with verified debug/release runtime provenance. Reports remain baseline-ineligible. Controlled timing remains Measurement Status: Pending Available Runner, not a performance pass. Stage 36 is next, not started.

Invalidated Elsewhere

Language-server hover wording distinguishes source dispatch from optimized call shape. Benchmarks own separate specialized/erased and code-size evidence. The website compiler pin follows the delivered revision; no public API or playground example rewrite is required.

…rage

Share native effect and storage proofs, preserve cleanup, and validate closure escape through aggregate and call paths. Taking closure arguments now receive owned environments; malformed escaping stack environments are rejected.

Add optimized dispatch and pointer-contract regressions, durable ownership parity, and multi-file performance reports. Pin the published Stage 35a benchmark matrix and remove the duplicate historical benchmark SHA assertion while retaining canonical pin validation. Cross-repository delivery remains in progress.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

Use compiler-selected, provenance-verified runtime profiles in canonical CI instead of a job-wide raw release archive override. Strengthen report assertions and record the optimizer, parity, and companion closure; controlled timing remains pending an eligible runner.
@amasiye
amasiye merged commit b468df6 into main Oct 2, 2026
10 checks passed
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.

1 participant