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.

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

Use deferred reification during reachability analysis #2449

Merged
merged 1 commit into from
Jan 9, 2018

Conversation

Praetonus
Copy link
Member

This change is a followup to abdcf2e. Instead of fully reifying functions during reachability analysis, the pass now uses deferred reification as needed.

Functions are now fully reified one at a time during code generation, instead of all at once during reachability analysis. In the future, code generation will use deferred reification as well.

This change results in a huge improvement in the memory usage of the compiler.

Copy link
Member

@jemc jemc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exciting!

But watch out for the CI failure.

@jemc
Copy link
Member

jemc commented Jan 1, 2018

Looks like the CI has a segfault in the codegen tests.

This change is a followup to abdcf2e. Instead of fully reifying
functions during reachability analysis, the pass now uses deferred
reification as needed.

Functions are now fully reified one at a time during code generation,
instead of all at once during reachability analysis. In the future,
code generation will use deferred reification as well.

This change results in a huge improvement in the memory usage of the
compiler.
@Praetonus
Copy link
Member Author

CI passed, merging.

@Praetonus Praetonus merged commit 884c62d into ponylang:master Jan 9, 2018
@Praetonus Praetonus deleted the deferred-reify-2 branch January 9, 2018 12:47
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.

2 participants