[PATCH 0/1] MR8347: musl: Don't use __builtin_rint in clang builds.

[Piotr Caban (＠piotr)]

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58381

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8347

[PATCH 1/1] musl: Don't use __builtin_rint in clang builds.

[Piotr Caban]

From: Piotr Caban <piotr(a)codeweavers.com>

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58381
---
 libs/musl/src/math/rint.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libs/musl/src/math/rint.c b/libs/musl/src/math/rint.c
index 61df07485b8..89ec41b2202 100644
--- a/libs/musl/src/math/rint.c
+++ b/libs/musl/src/math/rint.c
@@ -7,7 +7,7 @@ static const float_t toint = 1 / FLT_EPSILON;
 
 double __cdecl rint(double x)
 {
-#if defined(__GNUC__) || defined(__clang__)
+#if defined(__GNUC__)
     return __builtin_rint(x);
 #else
     union {double f; uint64_t i;} u = {x};
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/8347

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about libs/musl/src/math/rint.c:
> double __cdecl rint(double x)
>  {
> -#if defined(__GNUC__) || defined(__clang__)
> +#if defined(__GNUC__)

`__GNUC__` is defined in Clang mingw mode, something like `&& !defined(__clang__)` would catch it.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8347#note_106870