-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
check missing docs for reexported macros as well #56407
check missing docs for reexported macros as well #56407
Conversation
r? @zackmdavis (rust_highfive has picked a reviewer for you, use r? to override) |
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 |
bec43d8
to
4c0981b
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 |
I don't understand how the failure can be linked to this PR... |
The failure is because it's now checking for the However as can be seen from the added ui test this hasn't actually worked. It's only giving a |
Thanks for the explanation @ollie27! I'll fix this in the next days. |
4c0981b
to
8bb752f
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 |
8bb752f
to
801ebc5
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 |
Arf, I need to make a PR to |
e8ed50d
to
68830a1
Compare
Re-assigning: r? @varkor |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me if the avx512_target_feature
feature is intentional (or without it if unnecessary).
@@ -0,0 +1,13 @@ | |||
// Copyright 2018 The Rust Project Developers. See the COPYRIGHT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: copyright is no longer needed.
src/libcore/lib.rs
Outdated
@@ -129,6 +129,7 @@ | |||
#![feature(reverse_bits)] | |||
#![feature(non_exhaustive)] | |||
#![feature(structural_match)] | |||
#![feature(avx512_target_feature)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The stdsimd update is adding documentation so that this error isn't raised, but why do you need this feature?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without it we get:
error[E0658]: the target feature `avx512f` is currently unstable (see issue #44839)e
--> src/libcore/../stdsimd/coresimd/x86/avx512f.rs:13:18
|
13 | #[target_feature(enable = "avx512f")]
| ^^^^^^^^^^^^^^^^^^
|
= help: add #![feature(avx512_target_feature)] to the crate attributes to enable
error[E0658]: the target feature `avx512f` is currently unstable (see issue #44839)
--> src/libcore/../stdsimd/coresimd/x86/avx512f.rs:30:18
|
30 | #[target_feature(enable = "avx512f")]
| ^^^^^^^^^^^^^^^^^^
|
= help: add #![feature(avx512_target_feature)] to the crate attributes to enable
error[E0658]: the target feature `avx512f` is currently unstable (see issue #44839)
--> src/libcore/../stdsimd/coresimd/x86/avx512f.rs:43:18
|
43 | #[target_feature(enable = "avx512f")]
| ^^^^^^^^^^^^^^^^^^
|
= help: add #![feature(avx512_target_feature)] to the crate attributes to enable
error[E0658]: the target feature `avx512f` is currently unstable (see issue #44839)
--> src/libcore/../stdsimd/coresimd/x86/avx512f.rs:55:18
|
55 | #[target_feature(enable = "avx512f")]
| ^^^^^^^^^^^^^^^^^^
|
= help: add #![feature(avx512_target_feature)] to the crate attributes to enable
error[E0658]: the target feature `avx512f` is currently unstable (see issue #44839)
--> src/libcore/../stdsimd/coresimd/x86/avx512f.rs:65:18
|
65 | #[target_feature(enable = "avx512f")]
| ^^^^^^^^^^^^^^^^^^
|
= help: add #![feature(avx512_target_feature)] to the crate attributes to enable
error: aborting due to 5 previous errors
☔ The latest upstream changes (presumably #56926) made this pull request unmergeable. Please resolve the merge conflicts. |
0e10b09
to
ed9cc00
Compare
ed9cc00
to
47e294e
Compare
@bors: r=varkor |
📌 Commit 6fb9e2f224b4a85490a33cd1e60c97ad9983b98e has been approved by |
⌛ Testing commit 6fb9e2f224b4a85490a33cd1e60c97ad9983b98e with merge 63c79e99dfa3cb80b47f1b5d000ab9c0b4ff9c1d... |
💔 Test failed - status-travis |
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 |
805ca68
to
ad525db
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 |
ad525db
to
5576925
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 |
5576925
to
0b272e7
Compare
Let's give it another try... @bors: r=varkor p=10 |
📌 Commit 0b272e7 has been approved by |
… r=varkor check missing docs for reexported macros as well Fixes #56334.
☀️ Test successful - status-appveyor, status-travis |
Fixes #56334.