-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Rollup of 8 pull requests #68503
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
Rollup of 8 pull requests #68503
Conversation
Partially address rust-lang#64167.
Fixes rust-lang#68368 Previously, I assumed that the substs contained only types, which caused the computed index number to be wrong.
The call to `count` on the inner iterator can overflow even if `Skip` itself would return less that `usize::max_value()` items.
This change adds the x86_64-fuchsia and aarch64-fuchsia LLVM targets to those allowed to invoke -Zsanitizer. Currently, the only overlap between compiler_rt sanitizers supported by both rustc and Fuchsia is ASan.
This was a typo that slipped in, and meant that we were still not properly clearing out std.
Address inconsistency in using "is" with "declared here" "is" was generally used for NLL diagnostics, but not other diagnostics. Using "is" makes the diagnostics sound more natural and readable, so it seems sensible to commit to them throughout. r? @Centril
…y-vec, r=davidtwco Suggest borrowing `Vec<NonCopy>` in for loop Partially address rust-lang#64167.
…estebank Account for non-types in substs for opaque type error messages Fixes rust-lang#68368 Previously, I assumed that the substs contained only types, which caused the computed index number to be wrong.
Avoid overflow in `std::iter::Skip::count` The call to `count` on the inner iterator can overflow even if `Skip` itself would return less that `usize::max_value()` items. Fixes rust-lang#68139
…r=alexcrichton Enable ASan on Fuchsia This change adds the x86_64-fuchsia and aarch64-fuchsia LLVM targets to those allowed to invoke -Zsanitizer. Currently, the only overlap between compiler_rt sanitizers supported by both rustc and Fuchsia is ASan.
Implement `unused_parens` for block return values Fixes rust-lang#68386 r? @Centril
Add my (@flip1995) name to .mailmap
…ing, r=alexcrichton Clear out std, not std tools This was a typo that slipped in, and meant that we were still not properly clearing out std. This is basically rust-lang#67760 but actually correct...
@bors r+ p=8 rollup=never |
📌 Commit ac737d6 has been approved by |
⌛ Testing commit ac737d6 with merge 6bd75e836bc3e417ffe6c4ad1169c0db30feb23b... |
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 |
💔 Test failed - checks-azure |
Successful merges:
Vec<NonCopy>
in for loop #68424 (Suggest borrowingVec<NonCopy>
in for loop)std::iter::Skip::count
#68469 (Avoid overflow instd::iter::Skip::count
)unused_parens
for block return values #68479 (Implementunused_parens
for block return values)Failed merges:
r? @ghost