Remove special case for simd_shuffle arg promotion#69280
Remove special case for simd_shuffle arg promotion#69280bors merged 3 commits intorust-lang:masterfrom
simd_shuffle arg promotion#69280Conversation
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.