Skip to content

unimplemented instrinsic in hyper (_mm256_lddqu_si256) #1408

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

Closed
y21 opened this issue Oct 31, 2023 · 2 comments · Fixed by #1417
Closed

unimplemented instrinsic in hyper (_mm256_lddqu_si256) #1408

y21 opened this issue Oct 31, 2023 · 2 comments · Fixed by #1417
Labels
A-core-arch Area: Necessary for full core::arch support

Comments

@y21
Copy link
Member

y21 commented Oct 31, 2023

Noticed this in a project using reqwest, but it also seems to reproduce using just hyper and tokio:

# Cargo.toml
[package]
name = "repro"
version = "0.1.0"
edition = "2021"

[dependencies]
hyper = { version = "0.14.27", features = ["client", "http1", "tcp"] }
tokio = { version = "1.33.0", features = ["full"] }
// main.rs
#[tokio::main]
async fn main() {
    let client = hyper::Client::new();
    client
        .request(
            hyper::Request::builder()
                .uri("http://httpbin.org/get")
                .body(hyper::Body::empty())
                .unwrap(),
        )
        .await
        .unwrap();
}
trap at Instance { def: Item(DefId(2:14310 ~ core[53bd]::core_arch::x86::avx::_mm256_lddqu_si256)), args: [] } (_ZN4core9core_arch3x863avx18_mm256_lddqu_si25617h4bd75b5668ffeef1E): llvm.x86.avx.ldu.dq.256

Awesome project btw! It's great to see that this is available as a rustup component now

@bjorn3
Copy link
Member

bjorn3 commented Oct 31, 2023

Turns out cg_clif miscompiled is_x86_feature_detected!() such that AVX is never detected: #1405 (comment) This has resulted in me not noticing all the AVX intrinsics that crates rely on. When using the rustup component, an LLVM compiled libstd will be used which has a correct is_x86_feature_detected!() which does detect AVX. I will work on implementing this AVX vendor intrinsic tomorrow.

@bjorn3 bjorn3 added the A-core-arch Area: Necessary for full core::arch support label Oct 31, 2023
@bjorn3
Copy link
Member

bjorn3 commented Nov 11, 2023

Should be fixed in the latest nightly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-core-arch Area: Necessary for full core::arch support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants