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

Pflua compile ir #227

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Pflua compile ir #227

wants to merge 1 commit into from

Conversation

kbara
Copy link
Contributor

@kbara kbara commented Jun 13, 2015

Add the ability to dump the AST, ANF, and SSA IRs to pflua-compile, with and without optimization.
Obsolete pflua-expand.
Update other tools/internals to reflect convert_anf and convert_ssa requiring an optimize argument.

@kbara
Copy link
Contributor Author

kbara commented Jun 13, 2015

@andywingo Are you happy with convert_ssa and convert_ast taking an extra boolean parameter for whether to apply optimizations?

@wingo
Copy link
Contributor

wingo commented Jul 1, 2015

I can see us wanting more than one option, for example to turn specific optimizations on or off. Actually the printing could be done by e.g. ssa.lua itself if the debug_ssa option is set. Dunno yo. I'm kinda negative on adding boolean options. Boolean options, especially optional booleans, are an antipattern :) If you want to go ahead please use option dicts and utils.parse_opts but let's treat this as low-priority for now

Marked pflua-expand as obsolete; replaced by shell script.
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