diff mbox series

[uclibc-ng-devel] nan.h: define NAN only if not already defined

Message ID 20250203144327.13657-1-marcus.haehnel@kernkonzept.com
State Accepted
Headers show
Series [uclibc-ng-devel] nan.h: define NAN only if not already defined | expand

Commit Message

Marcus Haehnel Feb. 3, 2025, 2:43 p.m. UTC 
From: Frank Mehnert <frank.mehnert@kernkonzept.com>

Clang-19 defines NAN in <float.h>. Check if NAN is already defined to
prevent a compiler warning.

Change-Id: I29ebcb2a7a0c1eade46a73074a1701ea67869128
Signed-off-by: Marcus Haehnel <marcus.haehnel@kernkonzept.com>
---
 libc/sysdeps/linux/common/bits/nan.h | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Waldemar Brodkorb Feb. 10, 2025, 8:44 a.m. UTC | #1 
Hi Marcus,
Marcus Haehnel wrote,

> From: Frank Mehnert <frank.mehnert@kernkonzept.com>
> 
> Clang-19 defines NAN in <float.h>. Check if NAN is already defined to
> prevent a compiler warning.
> 
> Change-Id: I29ebcb2a7a0c1eade46a73074a1701ea67869128
> Signed-off-by: Marcus Haehnel <marcus.haehnel@kernkonzept.com>
> ---
>  libc/sysdeps/linux/common/bits/nan.h | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/libc/sysdeps/linux/common/bits/nan.h b/libc/sysdeps/linux/common/bits/nan.h
> index 00cb405f1..46cfb613f 100644
> --- a/libc/sysdeps/linux/common/bits/nan.h
> +++ b/libc/sysdeps/linux/common/bits/nan.h
> @@ -25,7 +25,9 @@
>  
>  #if __GNUC_PREREQ(3,3)
>  
> -# define NAN	(__builtin_nanf (""))
> +# ifndef NAN
> +#  define NAN	(__builtin_nanf (""))
> +# endif
>  
>  #elif defined __GNUC__
>  
> -- 
> 2.47.1

Patch applied and pushed, thanks
 Waldemar
diff mbox series

Patch

diff --git a/libc/sysdeps/linux/common/bits/nan.h b/libc/sysdeps/linux/common/bits/nan.h
index 00cb405f1..46cfb613f 100644
--- a/libc/sysdeps/linux/common/bits/nan.h
+++ b/libc/sysdeps/linux/common/bits/nan.h
@@ -25,7 +25,9 @@ 
 
 #if __GNUC_PREREQ(3,3)
 
-# define NAN	(__builtin_nanf (""))
+# ifndef NAN
+#  define NAN	(__builtin_nanf (""))
+# endif
 
 #elif defined __GNUC__