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

Document what exactly --backend-aggressive-opts does #1016

Closed
KOLANICH opened this issue Sep 10, 2021 · 2 comments
Closed

Document what exactly --backend-aggressive-opts does #1016

KOLANICH opened this issue Sep 10, 2021 · 2 comments

Comments

@KOLANICH
Copy link

KOLANICH commented Sep 10, 2021

When I have it enabled sometimes I see a lot of garbage (well, it may be not garbage, but when the flag is not set, these are not emitted) in the beginnings of functions (instead of reduction of amount of garbage that I expect).

Also it seems to be eliminating some expressions, leaving behind an emty uninitialized variable (which was initialized by the eliminated expression), that is used later.

In other words, currently this option seems to be completely useless and harmful.

@PeterMatula
Copy link
Collaborator

It took some investigation, but it looks like it enables any optimization that is prefixed with Aggressive (in its name in source codes). Currently, it means these two:

Looking at those comments, they look like antiquated ideas that we implemented a long time ago and never properly used. Also looking at them, they don't look like a good idea - both are indeed harmful and we should not do them if we don't want to damage the original semantics.

I will remove them and all the related code momentarily.

@KOLANICH
Copy link
Author

KOLANICH commented Oct 1, 2021

Thanks.

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

No branches or pull requests

2 participants