refine tests

Author: tgross35

testcrate/build.rs

@@ -22,16 +22,16 @@ fn main() {
         // GCC and LLVM disagree on the ABI of `f16` and `f128` with MinGW. See
         // <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115054>.
         || target.contains("windows-gnu")
-    // // FIXME(llvm): There is an ABI incompatibility between GCC and Clang on 32-bit x86.
-    // // See <https://github.com/llvm/llvm-project/issues/77401>.
-    // || target.starts_with("i686")
+        // // FIXME(llvm): There is an ABI incompatibility between GCC and Clang on 32-bit x86.
+        // // See <https://github.com/llvm/llvm-project/issues/77401>.
+        // || target.starts_with("i686")
         // 32-bit PowerPC and 64-bit LE gets code generated that Qemu cannot handle. See
         // <https://github.com/rust-lang/compiler-builtins/pull/606#issuecomment-2105635926>.
         || target.starts_with("powerpc-")
         || target.starts_with("powerpc64le-")
-    // FIXME: We get different results from the builtin functions. See
-    // <https://github.com/rust-lang/compiler-builtins/pull/606#issuecomment-2105657287>.
-    // || target.starts_with("powerpc64-")
+        // FIXME: We get different results from the builtin functions. See
+        // <https://github.com/rust-lang/compiler-builtins/pull/606#issuecomment-2105657287>.
+        || target.starts_with("powerpc64-")
     {
         features.insert(Feature::NoSysF128);
         features.insert(Feature::NoSysF128IntConvert);
@@ -43,11 +43,6 @@ fn main() {
         features.insert(Feature::NoSysF128IntConvert);
     }
 
-    // if target.contains("-unknown-linux-") {
-    //     // No `__extendhftf2` on x86, no `__trunctfhf2` on aarch64
-    //     features.insert(Feature::NoSysF16F128Convert);
-    // }
-
     if target.starts_with("wasm32-") {
         // Linking says "error: function signature mismatch: __extendhfsf2" and seems to
         // think the signature is either `(i32) -> f32` or `(f32) -> f32`