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

Enable MIR inlining #91743

Merged
merged 12 commits into from
Jul 2, 2022
Merged

Enable MIR inlining #91743

merged 12 commits into from
Jul 2, 2022
+252 −170

Commits on Jun 30, 2022

  1. Don't assert polymorphization has taken effect in const eval 

    Const eval no longer runs MIR optimizations so unless this is getting
run as part of a MIR optimization like const-prop, there can be unused
type parameters even if polymorphization is enabled.
    @wesleywiser @cjgillot
    wesleywiser authored and cjgillot committed Jun 30, 2022
    Configuration menu
    Copy the full SHA
    5999f34 View commit details
    Browse the repository at this point in the history

  2. Check history earlier.

    @cjgillot
    cjgillot committed Jun 30, 2022
    Configuration menu
    Copy the full SHA
    114c928 View commit details
    Browse the repository at this point in the history

  3. Do not ICE when failing to resolve.

    @cjgillot
    cjgillot committed Jun 30, 2022
    Configuration menu
    Copy the full SHA
    2e45cd4 View commit details
    Browse the repository at this point in the history

  4. Recover when failing to normalize closure signature.

    @cjgillot
    cjgillot committed Jun 30, 2022
    Configuration menu
    Copy the full SHA
    0161ecd View commit details
    Browse the repository at this point in the history

  5. Skip inlining if there are normalization issues.

    @cjgillot
    cjgillot committed Jun 30, 2022
    Configuration menu
    Copy the full SHA
    ce4b5c8 View commit details
    Browse the repository at this point in the history

  6. Do not error on the param_env during inlining. 

    This created unexpected diagnostics while compiling alga:
    cannot satisfy `<Self as Module>::Ring == _`

Turns out that we don't need this diagnostic as we disable inlining when
it would trigger an ICE.
    @cjgillot
    cjgillot committed Jun 30, 2022
    Configuration menu
    Copy the full SHA
    b33c6e1 View commit details
    Browse the repository at this point in the history

  7. Test enabling MIR inliner

    @wesleywiser @cjgillot
    wesleywiser authored and cjgillot committed Jun 30, 2022
    Configuration menu
    Copy the full SHA
    d1d9092 View commit details
    Browse the repository at this point in the history

  8. Allow inlining #[inline] functions.

    @cjgillot
    cjgillot committed Jun 30, 2022
    Configuration menu
    Copy the full SHA
    6bb549f View commit details
    Browse the repository at this point in the history

  9. Specialize a few tests depending on opt-level.

    @cjgillot
    cjgillot committed Jun 30, 2022
    Configuration menu
    Copy the full SHA
    6a99461 View commit details
    Browse the repository at this point in the history

Commits on Jul 1, 2022

  1. Ignore test with panic=abort.

    @cjgillot
    cjgillot committed Jul 1, 2022
    Configuration menu
    Copy the full SHA
    2f35392 View commit details
    Browse the repository at this point in the history

  2. Bless recursion test.

    @cjgillot
    cjgillot committed Jul 1, 2022
    Configuration menu
    Copy the full SHA
    a3e1a2b View commit details
    Browse the repository at this point in the history

  3. Amend codegen test.

    @cjgillot
    cjgillot committed Jul 1, 2022
    3 Configuration menu
    Copy the full SHA
    cbbf06b View commit details
    Browse the repository at this point in the history