Rollup of 10 pull requests #109205
Closed
Rollup of 10 pull requests #109205
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…scaping bound vars
…r=GuillaumeGomez rustdoc: fix type search for `Option` combinators
…s, r=WaffleLapkin error-msg: impl better suggestion for `E0532` Fixes rust-lang#106862 No test as there is already a test which is nearly identical to the example in the linked issue.
…t-for-write, r=notriddle rustdoc: DocFS: Replace rayon with threadpool and enable it for all targets Fixes rust-lang#109060. Switching to `threadpool` makes it a bit simpler for us to wait for all tasks in `DocFS` directly in the `Drop` implementation. I'm also curious if making all the writes into a thread pool could improve run time for rustdoc on all other platforms than Windows as well. I'll run a perf check to see. cc ``@ehuss`` r? ``@notriddle``
…, r=WaffleLapkin Assert def-kind is correct for alias types Make sure we're not constructing alias types for the wrong def-kind, at least for debug cases 😅
… r=oli-obk make `define_opaque_types` fully explicit based on the idea of rust-lang#108389. Moved `define_opaque_types` into the actual operations, e.g. `eq`, instead of `infcx.at` because normalization doesn't use `define_opaque_types` and even creates it's own `At` with a different `define_opaque_types` internally. Somewhat surprisingly, coherence actually relies on `DefineOpaqueTypes::Yes` for soundness which was revealed because I've incorrectly used `DefineOpaqueTypes::No` in `equate_impl_headers`. It feels concerning that even though this is the case, we still sometimes use `DefineOpaqueTypes::No` in coherence. I did not look into this as part of this PR as it is purely changing the structure of the code without changing behavior in any way. r? ``@oli-obk``
…ompiler-errors Some cleanups in our normalization logic Changed a match to be exhaustive and deduplicated some code. r? ``@compiler-errors`` this pulls out the uncontroversial part of rust-lang#108860
…, r=WaffleLapkin mv tests/codegen/issue-* tests/codegen/issues/ No changes to the contents; just a move. Like how there's a <https://github.com/rust-lang/rust/tree/master/tests/ui/issues> folder.
Unequal → Not equal Fixes rust-lang#109168
…tip, r=jsha rustdoc: remove `std::` from primitive intra-doc link tooltips Take the intra-doc link to the method `iter` from https://doc.rust-lang.org/nightly/std/slice/struct.Iter.html Before: `method std::slice::iter` After: `method slice::iter`
…iplett Mention UEFI target promotion in release notes for 1.67.0 The promotion happened in rust-lang#103933, but it was forgotten in the release notes for 1.67.0. See also discussion in https://rust-lang.zulipchat.com/#narrow/stream/241545-t-release/topic/UEFI.20targets.20relnotes
rustbot
added
S-waiting-on-review
|
@bors r+ rollup=never p=10 |
bors
added
S-waiting-on-bors
|
⌛ Testing commit 1326ced with merge e4f0ab8a18bef339b1536644bb3b7ab13ddc07da... |
|
💔 Test failed - checks-actions |
bors
added
S-waiting-on-review
|
The job Click to see the possible cause of the failure (guessed by this bot) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Successful merges:
Optioncombinators #108875 (rustdoc: fix type search forOptioncombinators)E0532#108971 (error-msg: impl better suggestion forE0532)define_opaque_typesfully explicit #109166 (makedefine_opaque_typesfully explicit)std::from primitive intra-doc link tooltips #109185 (rustdoc: removestd::from primitive intra-doc link tooltips)Failed merges:
r? @ghost
@rustbot modify labels: rollup
Create a similar rollup