Skip to content

Commit ec0220a

Browse files
committed
Auto merge of rust-lang#132434 - tgross35:f128-tests, r=<try>
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]: rust-lang#132433 try-job: arm-android try-job: armhf-gnu try-job: i686-gnu try-job: x86_64-apple-1 try-job: i686-mingw try-job: x86_64-msvc-ext
2 parents a0d98ff + 15075bf commit ec0220a

File tree

5 files changed

+9
-8
lines changed

5 files changed

+9
-8
lines changed

library/Cargo.lock

+2-2
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,9 @@ dependencies = [
6161

6262
[[package]]
6363
name = "compiler_builtins"
64-
version = "0.1.134"
64+
version = "0.1.137"
6565
source = "registry+https://github.com/rust-lang/crates.io-index"
66-
checksum = "2f743e6f7410a78c261505c729f389583de40eec62332cc8cdf2c8b9bf73049a"
66+
checksum = "12ca24dd460d0e3d28cc55e2d6a1bfe84e03577bc9e43b2d96d86ccd26ab7b91"
6767
dependencies = [
6868
"cc",
6969
"rustc-std-workspace-core",

library/alloc/Cargo.toml

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ edition = "2021"
1010

1111
[dependencies]
1212
core = { path = "../core" }
13-
compiler_builtins = { version = "0.1.134", features = ['rustc-dep-of-std'] }
13+
compiler_builtins = { version = "=0.1.137", features = ['rustc-dep-of-std'] }
1414

1515
[dev-dependencies]
1616
rand = { version = "0.8.5", default-features = false, features = ["alloc"] }

library/std/Cargo.toml

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ cfg-if = { version = "1.0", features = ['rustc-dep-of-std'] }
1717
panic_unwind = { path = "../panic_unwind", optional = true }
1818
panic_abort = { path = "../panic_abort" }
1919
core = { path = "../core", public = true }
20-
compiler_builtins = { version = "0.1.134" }
20+
compiler_builtins = { version = "=0.1.137" }
2121
unwind = { path = "../unwind" }
2222
hashbrown = { version = "0.15", default-features = false, features = [
2323
'rustc-dep-of-std',

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.

src/tools/tidy/src/deps.rs

+1
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ const LICENSES: &[&str] = &[
2424
"BSD-2-Clause OR Apache-2.0 OR MIT", // zerocopy
2525
"ISC",
2626
"MIT / Apache-2.0",
27+
"MIT AND Apache-2.0 WITH LLVM-exception AND (MIT OR Apache-2.0)", // compiler-builtins
2728
"MIT OR Apache-2.0 OR LGPL-2.1-or-later", // r-efi, r-efi-alloc
2829
"MIT OR Apache-2.0 OR Zlib", // tinyvec_macros
2930
"MIT OR Apache-2.0",

0 commit comments

Comments
 (0)