Skip to content

Walk closure by-ref convergence passes in deep statement context - #6208

Merged
ondrejmirtes merged 1 commit into
2.2.xfrom
byref-closure-deep-convergence
Aug 11, 2026
Merged

Walk closure by-ref convergence passes in deep statement context#6208
ondrejmirtes merged 1 commit into
2.2.xfrom
byref-closure-deep-convergence

Conversation

@ondrejmirtes

Copy link
Copy Markdown
Member

The closure by-ref fixpoint walked its intermediate passes with a top-level statement context, so every inner loop re-converged from scratch on every pass — convergence within convergence, which the loop handlers themselves avoid by walking their intermediate passes deep. A by-ref closure with nested loops re-created the innermost expressions' results many times over; deep-context passes bring it in line with the loop handlers' discipline, and the final top-level walk keeps full precision.

Zero expectation churn: full test suite green in both turbo modes, self-analysis and CS clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01DaBZjgksga4c5s6Q9FniY7

The closure by-ref fixpoint walked its intermediate passes with a top-level
statement context, so every inner loop re-converged from scratch on every
pass - convergence within convergence, which the loop handlers themselves
avoid by walking their passes with enterDeep(). A recursive by-ref closure
with nested loops re-created the innermost expressions' results many times
over; deep-context passes bring it in line with the loop handlers'
discipline, and the final top-level walk keeps full precision.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DaBZjgksga4c5s6Q9FniY7
@ondrejmirtes
ondrejmirtes merged commit 8147843 into 2.2.x Aug 11, 2026
427 checks passed
@ondrejmirtes
ondrejmirtes deleted the byref-closure-deep-convergence branch August 11, 2026 16:02
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