-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Make some lints incremental #57293
Make some lints incremental #57293
Conversation
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 |
⌛ Trying commit 9566a3fa5b92ab30ec65fe7d08ca6cb8d9f22d45 with merge 3328a32075ca09fd6e1c680549c865165add5641... |
☀️ Test successful - status-travis |
@rust-timer build 3328a32075ca09fd6e1c680549c865165add5641 |
Success: Queued 3328a32075ca09fd6e1c680549c865165add5641 with parent 6b2c311, comparison URL. |
Finished benchmarking try commit 3328a32075ca09fd6e1c680549c865165add5641 |
The performance improvements look pretty great! I hope to getting to review this and the related PRs later today or tomorrow. cc @rust-lang/wg-compiler-performance |
What's up with |
This is great! =) |
@bors try |
[WIP] Make some lints incremental Blocked on #57253 r? @michaelwoerister
☀️ Test successful - checks-travis |
@rust-timer build bc1de3e |
Success: Queued bc1de3e with parent 1bc6bae, comparison URL. |
Finished benchmarking try commit bc1de3e |
@bors try |
Make some lints incremental Blocked on #57253 r? @michaelwoerister
☀️ Test successful - checks-travis |
@rust-timer build 1caecbc |
Success: Queued 1caecbc with parent d30b99f, comparison URL. |
Make some lints incremental Blocked on #57253 r? @michaelwoerister
☀️ Try build successful - checks-travis |
@rust-timer build ca674ea |
Success: Queued ca674ea with parent 54479c6, comparison URL. |
Finished benchmarking try commit ca674ea |
That looks much better @bors r+ |
📌 Commit 4093bec has been approved by |
Thanks @nnethercote for the analysis and @Zoxc for fixing the regression! |
Make some lints incremental Blocked on rust-lang#57253 r? @michaelwoerister
Make some lints incremental Blocked on rust-lang#57253 r? @michaelwoerister
Make some lints incremental Blocked on rust-lang#57253 r? @michaelwoerister
Make some lints incremental Blocked on rust-lang#57253 r? @michaelwoerister
Rollup of 18 pull requests Successful merges: - #57293 (Make some lints incremental) - #57565 (syntax: Remove warning for unnecessary path disambiguators) - #58253 (librustc_driver => 2018) - #58837 (librustc_interface => 2018) - #59268 (Add suggestion to use `&*var` when `&str: From<String>` is expected) - #59283 (Make ASCII case conversions more than 4× faster) - #59284 (adjust MaybeUninit API to discussions) - #59372 (add rustfix-able suggestions to trim_{left,right} deprecations) - #59390 (Make `ptr::eq` documentation mention fat-pointer behavior) - #59393 (Refactor tuple comparison tests) - #59420 ([CI] record docker image info for reuse) - #59421 (Reject integer suffix when tuple indexing) - #59430 (Renames `EvalContext` to `InterpretCx`) - #59439 (Generalize diagnostic for `x = y` where `bool` is the expected type) - #59449 (fix: Make incremental artifact deletion more robust) - #59451 (Add `Default` to `std::alloc::System`) - #59459 (Add some tests) - #59460 (Include id in Thread's Debug implementation) Failed merges: r? @ghost
Combine all builtin late lints and make lint checking parallel Blocked on rust-lang#57293. Cuts runtime of late lint checking from 3.222s to 0.546s with 8 threads on `winapi` (@retep998 ;) ) r? @estebank
Clean up query cache code There are a couple of queries for which we do not promote saved results, which have been removed from caching here. This was likely the cause of the regression in #57293 (comment). r? @michaelwoerister
Blocked on #57253
r? @michaelwoerister