[libc-commits] [libc] [libc][NFC] Rename `LIBC_LONG_DOUBLE_IS_` macros (PR #83399)
via libc-commits
libc-commits at lists.llvm.org
Thu Feb 29 02:23:34 PST 2024
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 3196005f6bedbed61a86626a9e4f8fee7437a914 670035e7641b6710597a2156f9a2dbf5a6ab15d3 -- 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/FPUtil/x86_64/sqrt.h libc/src/__support/float_to_string.h libc/src/__support/macros/properties/types.h libc/src/__support/str_to_float.h libc/test/src/__support/FPUtil/fpbits_test.cpp libc/test/src/__support/str_to_long_double_test.cpp libc/test/src/math/smoke/nanl_test.cpp libc/test/src/stdio/sprintf_test.cpp libc/test/src/stdio/sscanf_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/src/__support/float_to_string.h b/libc/src/__support/float_to_string.h
index 5ba65de8c2..744842ced8 100644
--- a/libc/src/__support/float_to_string.h
+++ b/libc/src/__support/float_to_string.h
@@ -615,7 +615,7 @@ public:
}
};
-#if !defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT64) && \
+#if !defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT64) && \
!defined(LIBC_COPT_FLOAT_TO_STR_NO_SPECIALIZE_LD)
// --------------------------- LONG DOUBLE FUNCTIONS ---------------------------
diff --git a/libc/test/src/stdio/sprintf_test.cpp b/libc/test/src/stdio/sprintf_test.cpp
index 460a2122cb..ab8bdb23b1 100644
--- a/libc/test/src/stdio/sprintf_test.cpp
+++ b/libc/test/src/stdio/sprintf_test.cpp
@@ -1158,7 +1158,7 @@ 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_TYPES_LONG_DOUBLE_IS_X86_FLOAT80) || \
+#if defined(LIBC_TYPES_LONG_DOUBLE_IS_X86_FLOAT80) || \
defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT64)
written = LIBC_NAMESPACE::sprintf(buff, "%LF", -ld_nan);
ASSERT_STREQ_LEN(written, buff, "-NAN");
@@ -1363,9 +1363,9 @@ TEST_F(LlvmLibcSPrintfTest, FloatDecimalConv) {
#endif
written = LIBC_NAMESPACE::sprintf(buff, "%.1La",
- 0xf.fffffffffffffffp16380L); #if defined(LIBC_TYPES_LONG_DOUBLE_IS_X86_FLOAT80)
- ASSERT_STREQ_LEN(written, buff, "0x1.0p+16384");
- #elif defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT64)
+ 0xf.fffffffffffffffp16380L); #if
+ defined(LIBC_TYPES_LONG_DOUBLE_IS_X86_FLOAT80) ASSERT_STREQ_LEN(written, buff,
+ "0x1.0p+16384"); #elif defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT64)
ASSERT_STREQ_LEN(written, buff, "inf");
#elif defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT128)
ASSERT_STREQ_LEN(written, buff, "0x2.0p+16383");
@@ -1603,8 +1603,8 @@ TEST_F(LlvmLibcSPrintfTest, FloatDecimalLongDoubleConv) {
// Length Modifier Tests.
- // TODO(michaelrj): Add tests for LIBC_TYPES_LONG_DOUBLE_IS_FLOAT64 and 128 bit long
- // double systems.
+ // TODO(michaelrj): Add tests for LIBC_TYPES_LONG_DOUBLE_IS_FLOAT64 and 128
+ // bit long double systems.
// TODO(michaelrj): Fix the tests to only depend on the digits the long double
// is accurate for.
@@ -2180,9 +2180,9 @@ TEST_F(LlvmLibcSPrintfTest, FloatExponentConv) {
#endif
written = LIBC_NAMESPACE::sprintf(buff, "%.1La",
- 0xf.fffffffffffffffp16380L); #if defined(LIBC_TYPES_LONG_DOUBLE_IS_X86_FLOAT80)
- ASSERT_STREQ_LEN(written, buff, "0x1.0p+16384");
- #elif defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT64)
+ 0xf.fffffffffffffffp16380L); #if
+ defined(LIBC_TYPES_LONG_DOUBLE_IS_X86_FLOAT80) ASSERT_STREQ_LEN(written, buff,
+ "0x1.0p+16384"); #elif defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT64)
ASSERT_STREQ_LEN(written, buff, "inf");
#elif defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT128)
ASSERT_STREQ_LEN(written, buff, "0x2.0p+16383");
@@ -2806,9 +2806,9 @@ TEST_F(LlvmLibcSPrintfTest, FloatAutoConv) {
#endif
written = LIBC_NAMESPACE::sprintf(buff, "%.1La",
- 0xf.fffffffffffffffp16380L); #if defined(LIBC_TYPES_LONG_DOUBLE_IS_X86_FLOAT80)
- ASSERT_STREQ_LEN(written, buff, "0x1.0p+16384");
- #elif defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT64)
+ 0xf.fffffffffffffffp16380L); #if
+ defined(LIBC_TYPES_LONG_DOUBLE_IS_X86_FLOAT80) ASSERT_STREQ_LEN(written, buff,
+ "0x1.0p+16384"); #elif defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT64)
ASSERT_STREQ_LEN(written, buff, "inf");
#elif defined(LIBC_TYPES_LONG_DOUBLE_IS_FLOAT128)
ASSERT_STREQ_LEN(written, buff, "0x2.0p+16383");
``````````
</details>
https://github.com/llvm/llvm-project/pull/83399
More information about the libc-commits
mailing list