Skip to content
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

vget_low_s8 behind feature flag on beta #94083

Closed
dragostis opened this issue Feb 17, 2022 · 5 comments
Closed

vget_low_s8 behind feature flag on beta #94083

dragostis opened this issue Feb 17, 2022 · 5 comments
Labels
C-bug Category: This is a bug. O-AArch64 Armv8-A or later processors in AArch64 mode T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.

Comments

@dragostis
Copy link

While using the aarch64 intrinsics that are stable on beta I've noticed that vget_low_s8 still complains about not being given the stdsimd feature.

use std::arch::aarch64::*;

fn main() {
    [
        unsafe { vmovl_s8(vget_low_s8(val.0)) },
        unsafe { vmovl_s8(vget_high_s8(val.0)) },
    ];
}

Meta

rustc --version --verbose:

rustc 1.59.0-beta.8 (1945ce657 2022-02-12)
binary: rustc
commit-hash: 1945ce6579506787e0b18f0a2ea03fdb4dfc81c7
commit-date: 2022-02-12
host: x86_64-apple-darwin
release: 1.59.0-beta.8
LLVM version: 13.0.0
Backtrace

error[E0658]: use of unstable library feature 'stdsimd'

@dragostis dragostis added the C-bug Category: This is a bug. label Feb 17, 2022
@dragostis
Copy link
Author

cc #91951

@Mark-Simulacrum
Copy link
Member

cc @Amanieu

It looks like this may have gotten overlooked during stabilization? It likely does not block a release, though, and I'm not inclined to do fairly last minute backports of some subset of intrinsics

@dragostis
Copy link
Author

dragostis commented Feb 18, 2022

Most likely overlooked since its cognate, vget_high_s8, was stabilized.

@Amanieu
Copy link
Member

Amanieu commented Feb 18, 2022

Yes, it seems this was overlooked in the stabilization. It's an easy fix, but probably not worth backporting at the last minute. In the meantime it should be possible to work around using some of the other intrinsics.

Amanieu added a commit to Amanieu/rust that referenced this issue Feb 19, 2022
@Enselic Enselic added the O-AArch64 Armv8-A or later processors in AArch64 mode label Aug 9, 2024
@jieyouxu
Copy link
Member

Triage: stabilization fixed in stdarch submodule bump inside #94529.

@jieyouxu jieyouxu added T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. and removed needs-triage-legacy labels Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug. O-AArch64 Armv8-A or later processors in AArch64 mode T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants