-
Notifications
You must be signed in to change notification settings - Fork 13.3k
[experiment] Try using a very simple heuristic for the inliner #68795
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
[experiment] Try using a very simple heuristic for the inliner #68795
Conversation
@bors try |
Awaiting bors try build completion |
⌛ Trying commit 6261869 with merge 80ce9f1ae6fa204a416bef15e14657568c0e26cf... |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
☀️ Try build successful - checks-azure |
Queued 80ce9f1ae6fa204a416bef15e14657568c0e26cf with parent 0d34a87, future comparison URL. |
Finished benchmarking try commit 80ce9f1ae6fa204a416bef15e14657568c0e26cf, comparison URL. |
@wesleywiser Do you know why check builds get optimized MIR? |
According to the dep graph, it's coming from
|
Well |
I didn't know we did that either. I guess for consts coming from other crates, the only MIR we have is from the |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@bors try |
Awaiting bors try build completion |
⌛ Trying commit facb659eec2640cd71edfb9c94fb0d3706b1abca with merge 61fc21524391da43bfc56aa003624ebec2d96776... |
facb659
to
bd1728b
Compare
bd1728b
to
cf0876f
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
^ Is it possible that inlining doesn't take opaque return types into account? |
@bors try |
Awaiting bors try build completion |
⌛ Trying commit cf0876f with merge 0d96668e5d8c52d98ea13b30a512b086f988b305... |
☀️ Try build successful - checks-azure |
Queued 0d96668e5d8c52d98ea13b30a512b086f988b305 with parent bae3d0d, future comparison URL. |
Finished benchmarking try commit 0d96668e5d8c52d98ea13b30a512b086f988b305, comparison URL. |
@eddyb FYI, results seem to be very similar to the last run. |
r? @ghost