This is an automated email from the ASF dual-hosted git repository.
acassis pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git
The following commit(s) were added to refs/heads/master by this push:
new 3ccc5f0f76 libm/newlib: fix newlib build error
3ccc5f0f76 is described below
commit 3ccc5f0f76cb0ab4195d69ee3d81a5b4c9d5c5f0
Author: yanghuatao <[email protected]>
AuthorDate: Mon Aug 7 17:20:19 2023 +0800
libm/newlib: fix newlib build error
Fix error :
nuttx/libs/libm/newlib/include/math.h:89:8: error: ‘_Thread_local’ does not
name a type; did you mean ‘thread_local’?
89 | extern _Thread_local int _tls_gamma_signgam;
| ^~~~~~~~~~~~~
| thread_local
In file included from /mnt/nuttx/include/libcxx/math.h:300,
from /mnt/nuttx/include/libcxx/cmath:308,
from /mnt/nuttx/include/libcxx/charconv:79,
from libcxx/src/string.cpp:10:
/mnt/nuttx/libs/libm/newlib/include/math.h:89:8: error: ‘_Thread_local’
does not name a type; did you mean ‘thread_local’?
89 | extern _Thread_local int _tls_gamma_signgam;
Signed-off-by: yanghuatao <[email protected]>
---
libs/libm/newlib/include/math.h | 21 +++++++++++++++++++++
1 file changed, 21 insertions(+)
diff --git a/libs/libm/newlib/include/math.h b/libs/libm/newlib/include/math.h
index be308410ca..1fcc8c497c 100644
--- a/libs/libm/newlib/include/math.h
+++ b/libs/libm/newlib/include/math.h
@@ -44,6 +44,27 @@ extern "C"
# endif
#endif /* __GNUC_PREREQ */
+#ifndef __has_extension
+# ifdef __has_feature
+# define __has_extension __has_feature
+# else
+# define __has_extension(x) 0
+# endif
+#endif
+
+#if !__has_extension(c_thread_local)
+/* XXX: Some compilers (Clang 3.3, GCC 4.7) falsely announce C++11 mode
+ * without actually supporting the thread_local keyword. Don't check for
+ * the presence of C++11 when defining _Thread_local.
+ */
+# if /* (defined(__cplusplus) && __cplusplus >= 201103L) || */ \
+ __has_extension(cxx_thread_local)
+# define _Thread_local thread_local
+# else
+# define _Thread_local __thread
+# endif
+#endif
+
/* Newlib doesn't fully support long double math functions so far.
* On platforms where long double equals double the long double functions
* let's define _LDBL_EQ_DBL to simply call the double functions.
