Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Eta-expansion at the top of each Core function definition #1481

Closed
lukaszcz opened this issue Aug 25, 2022 · 0 comments · Fixed by #1571
Closed

Eta-expansion at the top of each Core function definition #1481

lukaszcz opened this issue Aug 25, 2022 · 0 comments · Fixed by #1571
Assignees
@janmasrovira
Labels
core Related to JuvixCore enhancement New feature or request
Milestone
0.3 - Valencia

Comments

@lukaszcz
Copy link
Collaborator

lukaszcz commented Aug 25, 2022 •
edited
Loading

Depends on PR #1421

Notes

  • Eta-expand only at the top of the Nodes stored in InfoTable/IdentContext.
  • Use type information associated with function definitions to determine eta-expansion.
  • The newly created Lambdas should have type information in BinderInfo, determined from the function type.
  • For testing, it may be useful to close Parse optional type information in JVC files #1509 first.

Starting point

  • Core/Transformation/Base.hs defines the transformation framework.
  • Core/Transformation/Eta.hs already contains eta-expansion for constructors and builtins.
@lukaszcz lukaszcz added enhancement New feature or request pending-review core Related to JuvixCore labels Aug 25, 2022
@lukaszcz lukaszcz added this to the 0.3 milestone Aug 25, 2022
@lukaszcz lukaszcz mentioned this issue Sep 6, 2022
Closed
@lukaszcz lukaszcz changed the title JuvixCore transformation: Eta-expansion at the top of each function definition Eta-expansion at the top of each Core function definition Sep 23, 2022
@janmasrovira janmasrovira mentioned this issue Oct 3, 2022
Merged
@janmasrovira janmasrovira linked a pull request Oct 3, 2022 that will close this issue
Eta expansion at the top of each core function definition (#1481) #1571
Merged
lukaszcz pushed a commit that referenced this issue Nov 14, 2022
@janmasrovira
Eta expansion at the top of each core function definition (#1481) (#1571
1691556 
)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@janmasrovira janmasrovira

Labels
core Related to JuvixCore enhancement New feature or request
Projects
None yet
Milestone
0.3 - Valencia
Development

Successfully merging a pull request may close this issue.

Eta expansion at the top of each core function definition (#1481) anoma/juvix
3 participants
@lukaszcz @jonaprieto @janmasrovira