Skip to content

Commit

Permalink
Enable reliable_f16_math on x86
Browse files Browse the repository at this point in the history
This has been disabled due to an LLVM misoptimization with `powi.f16`
[1]. This was fixed upstream and the fix is included in LLVM20, so tests
no longer need to be disabled.

`f16` still remains disabled on MinGW due to the ABI issue.

[1]: llvm/llvm-project#98665
  • Loading branch information
tgross35 committed Mar 11, 2025
1 parent 374ce1f commit 2b97977
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions library/std/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -154,8 +154,6 @@ fn main() {
&& match (target_arch.as_str(), target_os.as_str()) {
// FIXME: Disabled on Miri as the intrinsics are not implemented yet.
_ if is_miri => false,
// x86 has a crash for `powi`: <https://github.com/llvm/llvm-project/issues/105747>
("x86" | "x86_64", _) => false,
// Assume that working `f16` means working `f16` math for most platforms, since
// operations just go through `f32`.
_ => true,
Expand Down

0 comments on commit 2b97977

Please sign in to comment.