[libc] Fix fpbits test running 80bit ld everywhere (#115937)

michaelrj-google · web-flow · commit 6aa74038588e · 2024-11-12T12:52:08.000-08:00
After #115084 the 80 bit long double tests error if sizeof(long double) isn't 96 or 128 bits. This caused failures in long double is double systems (since long double is 64 bits) so I've disabled the 80 bit long double tests on systems that don't use them.
diff --git a/libc/test/src/__support/FPUtil/fpbits_test.cpp b/libc/test/src/__support/FPUtil/fpbits_test.cpp
@@ -124,6 +124,7 @@ TEST(LlvmLibcFPBitsTest, FPType_IEEE754_Binary128) {
       UInt128(Rep::quiet_nan()));
 }
 
+#ifdef LIBC_TYPES_LONG_DOUBLE_IS_X86_FLOAT80
 TEST(LlvmLibcFPBitsTest, FPType_X86_Binary80) {
   using Rep = FPRep<FPType::X86_Binary80>;
 
@@ -269,6 +270,7 @@ TEST(LlvmLibcFPBitsTest, FPType_X86_Binary80_IsNan) {
 #error "unhandled long double type"
 #endif
 }
+#endif // LIBC_TYPES_LONG_DOUBLE_IS_X86_FLOAT80
 
 enum class FP {
   ZERO,

Original file line number	Diff line number	Diff line change
`@@ -124,6 +124,7 @@ TEST(LlvmLibcFPBitsTest, FPType_IEEE754_Binary128) {`
`124`	`124`	`UInt128(Rep::quiet_nan()));`
`125`	`125`	`}`
`126`	`126`
	`127`	`+#ifdef LIBC_TYPES_LONG_DOUBLE_IS_X86_FLOAT80`
`127`	`128`	`TEST(LlvmLibcFPBitsTest, FPType_X86_Binary80) {`
`128`	`129`	`using Rep = FPRep<FPType::X86_Binary80>;`
`129`	`130`
`@@ -269,6 +270,7 @@ TEST(LlvmLibcFPBitsTest, FPType_X86_Binary80_IsNan) {`
`269`	`270`	`#error "unhandled long double type"`
`270`	`271`	`#endif`
`271`	`272`	`}`
	`273`	`+#endif // LIBC_TYPES_LONG_DOUBLE_IS_X86_FLOAT80`
`272`	`274`
`273`	`275`	`enum class FP {`
`274`	`276`	`ZERO,`