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

Rename [Part of] Optimizer #2461

Open
zoffixznet opened this issue Nov 2, 2018 · 2 comments
Open

Rename [Part of] Optimizer #2461

zoffixznet opened this issue Nov 2, 2018 · 2 comments
Labels
meta Repo organization, code style, etc. optimizer

Comments

@zoffixznet
Copy link
Contributor

zoffixznet commented Nov 2, 2018

Filing mostly to have a single reference point for it, as I saw it brought up a couple of times.

Core point: "optimizer" means entirely optional things executed to gain better speed/memory. However, our static optimizer does things that go beyond that (e.g. compile-time signature checking or literal-to-native handling).

So, we need to create a name to a part of the stuff optimizer does that isn't optional. I don't think it has to be related to any phaser, but we probably should make --optimize=off switch to NOT disable that part.

Off the hip, I propose the name Finalizer for the thing and the Finalizer finalizes things.

IRC refs:

@zoffixznet zoffixznet added the meta Repo organization, code style, etc. label Nov 2, 2018
@lizmat
Copy link
Contributor

lizmat commented Nov 2, 2018 via email

@zoffixznet
Copy link
Contributor Author

👍 on icing 😸

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta Repo organization, code style, etc. optimizer
Projects
None yet
Development

No branches or pull requests

3 participants