Skip to content

Commit 15075bf

Browse files
committed
Enable f128 tests on all non-buggy platforms 🎉
With the `compiler-builtins` update to 0.1.137 [1], we now provide symbols necessary to work with `f128` everywhere. This means that we are no longer restricted to 64-bit linux, and can enable tests by default. There are still a handful of platforms that need to remain disabled because of bugs. Math support is still off by default since those symbols are not yet available. [1]: #132433
1 parent b868454 commit 15075bf

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

Diff for: library/std/build.rs

+4-4
Original file line numberDiff line numberDiff line change
@@ -131,10 +131,10 @@ fn main() {
131131
("sparc", _) => false,
132132
// MinGW ABI bugs <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115054>
133133
("x86_64", "windows") if target_env == "gnu" && target_abi != "llvm" => false,
134-
// 64-bit Linux is about the only platform to have f128 symbols by default
135-
(_, "linux") if target_pointer_width == 64 => true,
136-
// Almost all OSs are missing symbol. compiler-builtins will have to add them.
137-
_ => false,
134+
// There are no known problems on other platforms, so the only requirement is that symbols
135+
// are available. `compiler-builtins` provides all symbols required for core `f128`
136+
// support, so this should work for everything else.
137+
_ => true,
138138
};
139139

140140
// Configure platforms that have reliable basics but may have unreliable math.

0 commit comments

Comments
 (0)