Skip to content

Commit

Permalink
Enable f16 tests on x86 Apple platforms
Browse files Browse the repository at this point in the history
These were disabled because Apple uses a special ABI for `f16`.
`compiler-builtins` merged a fix for this in [1], which has since
propagated to rust-lang/rust. Enable tests since there should be no
remaining issues on these platforms.

[1]: rust-lang/compiler-builtins#675
  • Loading branch information
tgross35 committed Sep 28, 2024
1 parent 3aae770 commit c3d3d1a
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions std/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -102,9 +102,6 @@ fn main() {
("arm64ec", _) => false,
// MinGW ABI bugs <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115054>
("x86_64", "windows") => false,
// Apple has a special ABI for `f16` that we do not yet support
// FIXME(builtins): fixed by <https://github.com/rust-lang/compiler-builtins/pull/675>
("x86" | "x86_64", _) if target_vendor == "apple" => false,
// Infinite recursion <https://github.com/llvm/llvm-project/issues/97981>
("csky", _) => false,
("hexagon", _) => false,
Expand Down

0 comments on commit c3d3d1a

Please sign in to comment.