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

Revisit MIR opt levels #579

Open
wesleywiser opened this issue Jan 13, 2023 · 1 comment
Open

Revisit MIR opt levels #579

wesleywiser opened this issue Jan 13, 2023 · 1 comment

Comments

@wesleywiser
Copy link
Member

Meeting proposal info

  • Title: Revisit MIR opt levels
  • Type: technical

Summary

The current system of MIR opt levels 0-4 is suboptimal, in part because there is confusion what the individual levels mean. As we continue to add new MIR optimizations and revise existing ones, we should take the chance to improve the opt level system.

One potential option is to remove the levels entirely and derive what passes and (and to what strength or complexity) from other options the user has provided such as -Copt-level and -Cdebuginfo. If there are additional axis that are important to capture, other flags could be introduced for those.

Another option is to redefine the opt levels. Having four levels is confusing as it doesn't intuitively map to the 0-2 levels provided by -Copt-level (although that flag also has s and z opt levels which are not present in MIR opt levels at all). We could decide to redefine MIR opt levels to be 0-2 based. One challenge here is defining what the levels themselves mean (when does an optimization fall into opt level 1 or opt level 2?).

Prior discussions:

About this issue

This issue corresponds to a meeting proposal for the compiler team
steering meeting. It corresponds to a possible topic of
discussion. You can read more about the steering meeting procedure
here
.

Comment policy

These issues are meant to be used as an "announcements channel"
regarding the proposal, and not as a place to discuss the technical
details. Feel free to subscribe to updates. We'll post comments when
reviewing the proposal in meetings or making a scheduling decision.
In the meantime, if you have questions or ideas, ping the proposers
on Zulip (or elsewhere).

@scottmcm
Copy link
Member

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