Require fma and f16c for avx512f in std_detect by sayantn · Pull Request #1781 · rust-lang/stdarch (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation1 Commits1 Checks58 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

@sayantn

@rustbot

r? @Amanieu

rustbot has assigned @Amanieu.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

This was referenced

Apr 24, 2025

@sayantn

@Amanieu Amanieu added this pull request to the merge queue

May 1, 2025

Merged via the queue into rust-lang:master with commit f1c1839

May 1, 2025

58 checks passed

VlaDexa added a commit to VlaDexa/rust that referenced this pull request

May 2, 2025

@VlaDexa

Remove avx512dq and avx512vl implication for avx512fp16

According to Intel, avx512fp16 requires only avx512bw, but LLVM also enables avx512vl and avx512dq when avx512fp16 is active. This is relic code, and will be fixed in LLVM soon. We should remove this from Rust too asap, especially before the stabilization of AVX512

Related:

@rustbot label O-x86_64 O-x86_32 A-SIMD A-target-feature T-compiler -T-libs r? @Amanieu

Update: the LLVM fix has been merged

cc @rust-lang/wg-llvm will it be possible to update the rustc llvm version to something after llvm/llvm-project#137450

Zalathar added a commit to Zalathar/rust that referenced this pull request

May 2, 2025

@Zalathar

Remove avx512dq and avx512vl implication for avx512fp16

According to Intel, avx512fp16 requires only avx512bw, but LLVM also enables avx512vl and avx512dq when avx512fp16 is active. This is relic code, and will be fixed in LLVM soon. We should remove this from Rust too asap, especially before the stabilization of AVX512

Related:

@rustbot label O-x86_64 O-x86_32 A-SIMD A-target-feature T-compiler -T-libs r? @Amanieu

Update: the LLVM fix has been merged

cc @rust-lang/wg-llvm will it be possible to update the rustc llvm version to something after llvm/llvm-project#137450

rust-timer added a commit to rust-lang-ci/rust that referenced this pull request

May 2, 2025

@rust-timer

Rollup merge of rust-lang#140389 - sayantn:avx512fp16, r=Amanieu

Remove avx512dq and avx512vl implication for avx512fp16

According to Intel, avx512fp16 requires only avx512bw, but LLVM also enables avx512vl and avx512dq when avx512fp16 is active. This is relic code, and will be fixed in LLVM soon. We should remove this from Rust too asap, especially before the stabilization of AVX512

Related:

@rustbot label O-x86_64 O-x86_32 A-SIMD A-target-feature T-compiler -T-libs r? @Amanieu

Update: the LLVM fix has been merged

cc @rust-lang/wg-llvm will it be possible to update the rustc llvm version to something after llvm/llvm-project#137450

github-actions bot pushed a commit to model-checking/verify-rust-std that referenced this pull request

May 9, 2025

@Zalathar

Remove avx512dq and avx512vl implication for avx512fp16

According to Intel, avx512fp16 requires only avx512bw, but LLVM also enables avx512vl and avx512dq when avx512fp16 is active. This is relic code, and will be fixed in LLVM soon. We should remove this from Rust too asap, especially before the stabilization of AVX512

Related:

@rustbot label O-x86_64 O-x86_32 A-SIMD A-target-feature T-compiler -T-libs r? @Amanieu

Update: the LLVM fix has been merged

cc @rust-lang/wg-llvm will it be possible to update the rustc llvm version to something after llvm/llvm-project#137450