-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Remove special case for simd_shuffle
arg promotion
#69280
Remove special case for simd_shuffle
arg promotion
#69280
Conversation
After rust-lang/stdarch#825, these intrinsics are now defined with `#[rustc_args_required_const(2)]`, so the special-case is no longer necessary.
(rust_highfive has picked a reviewer for you, use r? to override) |
@bors r+ |
📌 Commit f581b55 has been approved by |
FWIW, we have function parameter attributes now, so |
Ooh, this would be much better. I can open the issue if you like? |
Yeah, that would be nice. |
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 |
@bors r- |
063186f
to
b43dc80
Compare
Some tests declare their own @bors r=petrochenkov |
📌 Commit b43dc80 has been approved by |
…pecial-case, r=petrochenkov Remove special case for `simd_shuffle` arg promotion After rust-lang/stdarch#825, these intrinsics are now defined with `#[rustc_args_required_const(2)]`, so the special-case is no longer necessary.
Rollup of 5 pull requests Successful merges: - #68863 (ci: switch macOS builders to 10.15) - #69142 (Add shared script for linkchecking books.) - #69248 (Don't eliminate frame pointers on thumb targets) - #69280 (Remove special case for `simd_shuffle` arg promotion) - #69284 (Reword OpenOptions::{create, create_new} doc.) Failed merges: r? @ghost
This change was made in `stdarch` but not `packed_simd`. See rust-lang/rust#69280 for background.
…nkov Revert #69280 Resolves #69313 by reverting #69280. After #69280, `#[rustc_args_required_const(2)]` is required on the declaration of `simd_shuffle` intrinsics. This is allowed breakage, since you can't define platform intrinsics on stable. However, the latest release of the widely used `packed_simd` crate defines these intrinsics without the requisite attribute. Since there's no urgency to merge #69280, let's revert it. We can reconsider when rust-lang/packed_simd#278 is included in a point release of `packed_simd`. r? @petrochenkov
This change was made in `stdarch` but not `packed_simd`. See rust-lang/rust#69280 for background.
This change was made in `stdarch` but not `packed_simd`. See rust-lang/rust#69280 for background.
After rust-lang/stdarch#825, these intrinsics are now defined with
#[rustc_args_required_const(2)]
, so the special-case is no longer necessary.