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.

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

Jump to bottom

Add a MIR pass manager #77665

Closed
wants to merge 5 commits into from
Closed

Add a MIR pass manager #77665

wants to merge 5 commits into from

Commits on Oct 11, 2020

  1. Define a PassManager for managing MIR passes

    @ecstatic-morse
    ecstatic-morse committed Oct 11, 2020
    Configuration menu
    Copy the full SHA
    da4cb6c View commit details
    Browse the repository at this point in the history

  2. Declare opt-level for each MIR transformation

    @ecstatic-morse
    ecstatic-morse committed Oct 11, 2020
    Configuration menu
    Copy the full SHA
    5f98540 View commit details
    Browse the repository at this point in the history

  3. Validate is not a MIR transformation 

    In general, read-only passes should not be declared as a `MirPass`,
since they always run and their MIR dump provides no additional
information.
    @ecstatic-morse
    ecstatic-morse committed Oct 11, 2020
    Configuration menu
    Copy the full SHA
    5b46f91 View commit details
    Browse the repository at this point in the history

  4. Add mechanism for skipping MIR passes

    @ecstatic-morse
    ecstatic-morse committed Oct 11, 2020
    Configuration menu
    Copy the full SHA
    51da28a View commit details
    Browse the repository at this point in the history

  5. Add missing flags for mir-opt tests of unsound passes 

    These don't actually test anything since whoever implemented
`-Zunsound-mir-opts` forgot to set it when unsound passes were checked.
They break now that the passes are skipped entirely without
`-Zunsound-mir-opts` instead of just being no-ops.
    @ecstatic-morse
    ecstatic-morse committed Oct 11, 2020
    Configuration menu
    Copy the full SHA
    d325564 View commit details
    Browse the repository at this point in the history