-
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
Don't bail out before linting in generic contexts. #70566
Conversation
d355157
to
40eae5b
Compare
(Let's make sure we perf test this before landing.) |
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 |
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 |
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 |
c3eaa3e
to
19a665b
Compare
@bors try |
⌛ Trying commit 19a665b1640d614ee7f76533f83e4743866060b5 with merge 3b9ef566a383a39e6b30850d4c26521d95e051fc... |
@rust-timer build |
☀️ Try build successful - checks-azure |
@jumbatm you used to have some new tests for arithmetic in associated consts in an earlier version of this PR. Those tests already pass, but having the order of operators swapped might still be interesting just in case their order in the MIR ever matters. Care to submit a separate PR that extends the existing |
We could start a |
Sure thing!
In my opinion, a I'd say because we're checking lints get emitted properly (ie, a ui test), it makes sense to me to put it under the existing |
That's a good place indeed |
There's also |
19a665b
to
4e7b6b6
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 |
4e7b6b6
to
69d0267
Compare
This is good for a perf run if there's no further comments. Can someone run rust-timer for me? Thanks! |
@bors try |
…op, r=RalfJung Don't bail out before linting in generic contexts. Fixes rust-lang#69021. cc rust-lang#70017 r? @RalfJung
This failed the rollup in #71143 (comment). @bors r- @jumbatm could you please rebase and re-bless? rustc output changed a bit so the reference files need updating. |
- Change to warnings so that all lints are emitted - Expect build-pass - Change placeholder FIXME to WARN.
This ensures we get still get lints for generic contexts.
a017a5b
to
ed73023
Compare
This is good to go again. |
@bors r=RalfJung |
📌 Commit ed73023 has been approved by |
…op, r=RalfJung Don't bail out before linting in generic contexts. Fixes rust-lang#69021. cc rust-lang#70017 r? @RalfJung
@jumbatm thanks! Please always leave a message after a force-push as GitHub only unreliably sends email notifications for them. |
Rollup of 5 pull requests Successful merges: - rust-lang#70566 (Don't bail out before linting in generic contexts.) - rust-lang#71141 (Provide better compiler output when using `?` on `Option` in fn returning `Result` and vice-versa) - rust-lang#71149 (remove an impossible branch from check_consts) - rust-lang#71179 (fix more clippy warnings) - rust-lang#71191 (Clean up E0520 explanation) Failed merges: r? @ghost
…=Mark-Simulacrum [beta] fix failing const validation This is the **beta branch fix** for rust-lang#71353, by reverting rust-lang#70566. r? @oli-obk Not sure if there is any extra process for the beta part. This is not a backport; we intend to "properly" fix this on master but for beta a revert is faster and less risky.
Fixes #69021.
cc #70017
r? @RalfJung