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

Enable mir inlining by default #805

Closed
5 of 6 tasks
bjorn3 opened this issue Nov 23, 2019 · 2 comments
Closed
5 of 6 tasks

Enable mir inlining by default #805

bjorn3 opened this issue Nov 23, 2019 · 2 comments
Labels
optimize-speed The speed of the generated executable

Comments

@bjorn3
Copy link
Member

bjorn3 commented Nov 23, 2019

Blocked on:

Cranelift doesn't have any inlining support itself.

Edit(2019-12-23): Added more blocking issues
Edit(2021-01-31): Added upstream tracking issue

@bjorn3 bjorn3 added the optimize-speed The speed of the generated executable label Nov 23, 2019
@bjorn3
Copy link
Member Author

bjorn3 commented Jan 31, 2021

bjorn3@84f24e5 re-enables mir inlining again for libstd. This resulted in a perf improvement of about 30% on simple-raytracer.

@bjorn3
Copy link
Member Author

bjorn3 commented Jul 18, 2022

Since rust-lang/rust#91743 the MIR inliner is enabled by default in release mode. Given that the optimized MIR is stored in the crate metadata, having the standard library compiled with MIR inlining enabled benefits even debug mode user programs.

@bjorn3 bjorn3 closed this as completed Jul 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
optimize-speed The speed of the generated executable
Projects
None yet
Development

No branches or pull requests

1 participant