On Fri, 10 Jul 2026, Jacek Caban via Mingw-w64-public wrote:
---
mingw-w64-crt/math/fabsf.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/mingw-w64-crt/math/fabsf.c b/mingw-w64-crt/math/fabsf.c
index e7e9d8fa7..702b9d02c 100644
--- a/mingw-w64-crt/math/fabsf.c
+++ b/mingw-w64-crt/math/fabsf.c
@@ -8,11 +8,11 @@ float fabsf (float x);
float
fabsf (float x)
{
-#if defined(__x86_64__) || defined(_AMD64_) || defined(__arm__) ||
defined(_ARM_) || defined(__aarch64__) || defined(_ARM64_)
- return __builtin_fabsf (x);
-#elif defined(__i386__) || defined(_X86_)
+#if defined(__i386__) || defined(_X86_)
float res = 0.0F;
asm volatile ("fabs;" : "=t" (res) : "0" (x));
return res;
-#endif /* defined(__x86_64__) || defined(_AMD64_) || defined(__arm__) ||
defined(_ARM_) || defined(__aarch64__) || defined(_ARM64_) */
+#else
+ return __builtin_fabsf(x);
+#endif
}
This change does look ok from a technical point of view, as it simplifies
the list of ifdefs. But is this a non-functional change, or is there any
build configuration where it makes a difference? Even for arm64ec, it
would hit the __x86_64__ case before. It would be good to clarify this,
either what it changes, or that it indeed is only a cosmetic change.
// Martin
_______________________________________________
Mingw-w64-public mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
