-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Remove avx512dq
and avx512vl
implication for avx512fp16
#140389
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
base: master
Are you sure you want to change the base?
Conversation
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
cc @a4lg there seems to be some doc errors in the PS: I am quite confused how they weren't caught in the stdarch CI |
On hindsight, this seems like a bug in rustdoc with cc @rust-lang/rustdoc @rust-lang/wg-gcc-backend edit: not spurious |
This comment has been minimized.
This comment has been minimized.
@sayantn It seems the Linkcheck tool on the CI does its own job and the error itself seems valid. However, multiple references to a single footnote is mandatory for simplicity of the new macro documentation with the platform guide. If you prefer the short term solution to pass CI, you may revert rust-lang/stdarch#1779. |
I created a rustdoc PR #140434 (implementing it was easier than I thought) to enable adopting rust-lang/stdarch#1779 in the future. I'm not sure whether adoption of this PR will result in immediate success (considering the default stage for building docs (stage 0), it might not be immediate) but I hope this is accepted. |
The job Click to see the possible cause of the failure (guessed by this bot)
|
I opened a PR rust-lang/stdarch#1792 that will revert rust-lang/stdarch#1779 for now. If this PR is applied, stdarch should be safely merged into the master branch. Even if #140434 is merged now, because the CI failure says this is caused by the stage0 compiler, we have to wait the next beta to merge rust-lang/stdarch#1779. |
According to Intel,
avx512fp16
requires onlyavx512bw
, but LLVM also enablesavx512vl
andavx512dq
whenavx512fp16
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 AVX512Related:
fma
andf16c
foravx512f
instd_detect
stdarch#1781@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