Skip to content

Split-TU: fold emitCfgReturnOperand into TernaryJumpIfEchoMerge (#36387) - #36752

Closed
PurHur wants to merge 1 commit into
masterfrom
fix/36387-ternary-jumpif-echo-merge-concern
Closed

PurHur wants to merge 1 commit into
masterfrom
fix/36387-ternary-jumpif-echo-merge-concern

Conversation

@PurHur

@PurHur PurHur commented Sep 4, 2026 •

Copy link
Copy Markdown
Owner

Summary

Part of #36387 / Part of #36403.

Test plan

  • ./script/aot-smoke.sh → 9/9 + size gate OK
  • php script/check-size-budgets.php → OK
  • ./script/spine-sync.sh --no-link → coverage/count OK 8198/8198 (stamp synced)
  • make north-star5-verify-fast → OK (spine 8198/8198)

Closes nothing (epic continues) — Part of #36387.

Move JUMPIF echo-merge / ?: return-phi helpers (~1055 lines) out of
JIT.php toward host-CFG split-TU emit (gen-0 <30m). Ratchet size
budgets and sync spine inventory / footnotes / sidecar stamp.

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur

PurHur commented Sep 4, 2026

Copy link
Copy Markdown
Owner Author

Superseded by merged #36751 (TernaryJumpIfEchoMerge on master a0f6cc350b) — same JUMPIF/?: extract under a shorter Concern name. Closing this duplicate; continuing with the next #36387 / #36386 slice.

@PurHur

PurHur commented Sep 4, 2026

Copy link
Copy Markdown
Owner Author

Duplicate of #36751 — closing.

@PurHur

PurHur commented Sep 4, 2026

Copy link
Copy Markdown
Owner Author

Superseded by clean delta on tip after #36751 conflict: #36755

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