-
Notifications
You must be signed in to change notification settings - Fork 13k
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
travis: print some statistics from "top". #48841
Conversation
d0f30d4
to
02ab63c
Compare
First results: https://gist.github.com/matthiaskrgr/7ab3c5998dc6943069927683e3a26a2e |
The thing you mentioned is caused by large, build-blocking crates like libsyntax_ext or librustc. Most of the time is spent doing codegen, and as long as we don't use codegen-units for release build, there is not really a way to resolve this issue. |
If the purpose of stage0 is solely to bootstrap stage1, maybe we can compile stage0 with several codegen units? |
Maybe it works. The best way is to open a PoC PR and check the time difference. |
I tried to make this work in #48843 , let's see if it works... |
… ci. Currently commented out, can be commented in in .travis.yml if needed.
I updated the commit so that the .travis.yml line that enables the logging is commented out. |
I think this wants to be merged now... r? @kennytm |
cc @alexcrichton @rust-lang/infra #48828. This PR implements a very simple script which polls We've got some data out of it in #48841 (comment). Personally I incline to close this, because
(BTW,
) |
As discussed above, I'm closing this PR. Thanks for the exploration @matthiaskrgr! |
cc #48828