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

Make unused function pass optional #409

Open

3 tasks done

bryanjos opened this issue Sep 12, 2017 · 1 comment · Fixed by #434

Assignees

Labels

level:intermediate

Collaborator

bryanjos commented Sep 12, 2017 •

edited

Make a new compiler option, remove_unused_functions. By default it should be set to Mix.env == :prod. Meaning by default it would be true only for production builds.
In compiler, if remove_unused_functions is true, skip the FindUsedFunctions phase.
In the Translate phase, when remove_unused_functions is false, set the used field equal to the definitions field. All other compilation should happen as normal

The text was updated successfully, but these errors were encountered:

bryanjos added the level:intermediate label

bryanjos added this to the 0.32.0 milestone

bryanjos self-assigned this

bryanjos mentioned this issue

Remove unused functions option #434

Merged

bryanjos closed this as completed in #434

bryanjos mentioned this issue

Test file line numbers and compiler fixes #441

Merged

Collaborator Author

bryanjos commented Dec 10, 2017

Having some issues with this still so going to open it back up

bryanjos reopened this

bryanjos removed this from the 0.32.0 milestone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment