Skip to content

Rewrite core transformer#1336

Merged
b-studios merged 18 commits intomainfrom
rewrite-core-transformer
Mar 25, 2026
Merged

Rewrite core transformer#1336
b-studios merged 18 commits intomainfrom
rewrite-core-transformer

Conversation

@phischu
Copy link
Collaborator

@phischu phischu commented Mar 23, 2026

This should fix #1158

@b-studios
Copy link
Collaborator

I tried to reduce as many eta-redexes as possible without breaking backwards compatiblity. Some eta-redexes are purposefully not covered in this PR:

  • eta redexes around resume (which might require a redesign)
  • eta redexes around pattern matching cases (where the right hand sides are lifted to local definitions)

All others should have been eliminated now. If you still find some, please let me know.

@b-studios
Copy link
Collaborator

@phischu once CI passes this is ready to merge

@b-studios b-studios merged commit 471d634 into main Mar 25, 2026
7 checks passed
@b-studios b-studios deleted the rewrite-core-transformer branch March 25, 2026 13:30
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.

Suboptimal core due to eta-expanded returns

3 participants