[libc-commits] [libc] [libc][NFC] Rename SPECIAL_X86_LONG_DOUBLE in LIBC_LONG_DOUBLE_IS_X86_FLOAT80 (PR #73950)

via libc-commits libc-commits at lists.llvm.org
Fri Dec 1 05:07:13 PST 2023


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff f1d0276e4c42301155e900424ea734aca7ec97a8 c808f544a43b58c04b0449882556021577ffe5bd -- libc/src/__support/FPUtil/FPBits.h libc/src/__support/FPUtil/ManipulationFunctions.h libc/src/__support/FPUtil/NormalFloat.h libc/src/__support/FPUtil/generic/sqrt.h libc/src/__support/FPUtil/generic/sqrt_80_bit_long_double.h libc/src/__support/macros/properties/float.h libc/src/__support/str_to_float.h libc/test/src/__support/str_to_float_test.cpp libc/test/src/stdio/sprintf_test.cpp libc/test/src/stdlib/strtold_test.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/libc/test/src/stdio/sprintf_test.cpp b/libc/test/src/stdio/sprintf_test.cpp
index 9d82a9d925..344853beaf 100644
--- a/libc/test/src/stdio/sprintf_test.cpp
+++ b/libc/test/src/stdio/sprintf_test.cpp
@@ -1024,7 +1024,8 @@ TEST_F(LlvmLibcSPrintfTest, FloatDecimalConv) {
 
 // Some float128 systems (specifically the ones used for aarch64 buildbots)
 // don't respect signs for long double NaNs.
-#if defined(LIBC_LONG_DOUBLE_IS_X86_FLOAT80) || defined(LIBC_LONG_DOUBLE_IS_FLOAT64)
+#if defined(LIBC_LONG_DOUBLE_IS_X86_FLOAT80) ||                                \
+    defined(LIBC_LONG_DOUBLE_IS_FLOAT64)
   written = LIBC_NAMESPACE::sprintf(buff, "%LF", -ld_nan);
   ASSERT_STREQ_LEN(written, buff, "-NAN");
 #endif

``````````

</details>


https://github.com/llvm/llvm-project/pull/73950


More information about the libc-commits mailing list