Merged
Conversation
Ok, I can't use `std` in doc-links, only in doc-tests. Try 7...
…ding on Windows." This reverts commit 74a39a3.
When `rustc` is invoked with the `--print` argument, we don't actually generate any code (unless it's the `native-static-libs` option). So we don't need to error our in this case since there's no risk of generating either LLVM assertions or corrupted binaries.
Change some tests to `check-pass` that are only testing name resolution.
`normalize-stdout-test` removes the need for Make, and it can be updated with `--bless` this way
Per rust-lang#59356 it is expected that the interpretation of the bitmask depends on target endianness. Closes rust-lang#59356
…acrum Fix hyperlinks in From impls between Vec and VecDeque I'd been trying to link them, but apparently actually just added brackets: <https://doc.rust-lang.org/nightly/std/collections/struct.VecDeque.html#impl-From%3CVec%3CT%3E%3E> ~~This reverts commit 5168f5d.~~ ~~(I'd previously tried to make relative links, but those failed linkcheck because the types are exported at different levels. So just skip the links -- they're already linked in the function signature anyway.)~~ This makes the links now work.
Only error about MSVC + PGO + unwind if we're generating code In rust-lang#61853, we changed the error when using PGO & MSVC toolchain & panic=unwind into a warning. However, in the compiler team meeting on 2019-07-11, we found that not everybody was in favor of that change because of the possibility of broken binaries. This PR reverts that change so this is again an error. However, to work around an [issue the Firefox team is having](rust-lang#61002 (comment)), we will only emit the error if we're actually supposed to generate a binary. If the `rustc` is invoked with `--print` arguments (which means that no binary will actually be emitted), then we won't emit the error because there is not a possibility of the issue occurring. cc @EricRahm @nikomatsakis @pnkfelix @Centril
Check that trait is exported or public before adding hint Closes rust-lang#62194. This PR checks the `AccessLevels` of a trait to check whether adding the intercrate ambiguity hint is valid or not. I am unsure of both the use of `.unwrap()` as well as removing hints for [downstream *and* [upstream](https://github.com/rust-lang/rust/blob/92b0f52584c9375505ecdefdd7855b93a5919d51/src/librustc/traits/select.rs#L112-L142).
Update the help message on error for self type Fix rust-lang#62609
…nas-schievink Fix repeated wording in slice documentation Changes `of the slice of the slice` to `of the slice` in the chunk- and friends documentation of slices
…ochenkov Consolidate hygiene tests Some general clean-up extracted out from another PR
Remove last use of mem::uninitialized from std::io::util Addresses rust-lang#62397 for std::io::util
Add missing link to Infallible in TryFrom doc
update data_layout and features for armv7-wrs-vxworks
Document link_section arbitrary bytes Closes rust-lang#60633.
Disable Z3 in LLVM build Avoid building LLVM with Z3 if it happens to be installed. Fixes rust-lang#62750. r? @alexcrichton
…_compiler_messages_p7, r=alexreg normalize use of backticks in compiler messages for librustc/lint rust-lang#60532
Disable simd_select_bitmask test on big endian Per rust-lang#59356 it is expected that the interpretation of the bitmask depends on target endianness. Closes rust-lang#59356
…sion, r=eddyb Self-referencial type now called a recursive type As per Boat's suggestion - rust-lang#62539, this makes the error message clearer.
Emit artifact notifications for dependency files Emit `dep-info` artifact notifications for `.d` files.
Member
Author
|
@bors r+ p=1 |
Collaborator
|
📌 Commit 396903b has been approved by |
Member
Author
|
@bors p=10 |
Collaborator
bors
added a commit
that referenced
this pull request
Jul 18, 2019
Rollup of 15 pull requests Successful merges: - #61926 (Fix hyperlinks in From impls between Vec and VecDeque) - #62615 ( Only error about MSVC + PGO + unwind if we're generating code) - #62696 (Check that trait is exported or public before adding hint) - #62712 (Update the help message on error for self type) - #62728 (Fix repeated wording in slice documentation) - #62730 (Consolidate hygiene tests) - #62732 (Remove last use of mem::uninitialized from std::io::util) - #62740 (Add missing link to Infallible in TryFrom doc) - #62745 (update data_layout and features for armv7-wrs-vxworks) - #62749 (Document link_section arbitrary bytes) - #62752 (Disable Z3 in LLVM build) - #62764 (normalize use of backticks in compiler messages for librustc/lint) - #62774 (Disable simd_select_bitmask test on big endian) - #62777 (Self-referencial type now called a recursive type) - #62778 (Emit artifact notifications for dependency files) Failed merges: - #62746 ( do not use mem::uninitialized in std::io) r? @ghost
Collaborator
|
☀️ Test successful - checks-azure |
Contributor
|
📣 Toolstate changed by #62782! Tested on commit 311376d. 💔 clippy-driver on windows: test-pass → test-fail (cc @Manishearth @llogiq @mcarton @oli-obk @phansch, @rust-lang/infra). |
rust-highfive
added a commit
to rust-lang-nursery/rust-toolstate
that referenced
this pull request
Jul 18, 2019
Tested on commit rust-lang/rust@311376d. Direct link to PR: <rust-lang/rust#62782> 💔 clippy-driver on windows: test-pass → test-fail (cc @Manishearth @llogiq @mcarton @oli-obk @phansch, @rust-lang/infra). 💔 clippy-driver on linux: test-pass → test-fail (cc @Manishearth @llogiq @mcarton @oli-obk @phansch, @rust-lang/infra).
matthiaskrgr
added a commit
to matthiaskrgr/rust-clippy
that referenced
this pull request
Jul 18, 2019
(was merged as part of rust-lang/rust#62782 )
bors
added a commit
to rust-lang/rust-clippy
that referenced
this pull request
Jul 19, 2019
rustup rust-lang/rust#62764 (was merged as part of rust-lang/rust#62782 ) changelog: none
This file contains hidden or 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
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:
Failed merges:
r? @ghost