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

Initial dead code elimination work #41

Merged
merged 2 commits into from Jul 19, 2017
Merged

Initial dead code elimination work #41

merged 2 commits into from Jul 19, 2017

Conversation

phated
Copy link
Contributor

@phated phated commented Jul 17, 2017

This is my initial work on dead code elimination for #22 - There's still tons of work/cleanup do be done on this logic.

@ian-harshman-ck
Copy link
Contributor

Could add a hasFields property to the struct node to aid in this sort of logic

@phated
Copy link
Contributor Author

phated commented Jul 19, 2017

I really like that idea. I could add a size property (as a getter) to each Node that has fields to know how big each one is. Thanks @ian-harshman-ck!

@phated phated merged commit f62255b into master Jul 19, 2017
@phated phated deleted the dead-code-elimination2 branch August 8, 2017 19:21
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.

None yet

2 participants