On 18.03.2025 10:44, Martin Storsjö wrote:
On Mon, 17 Mar 2025, Jacek Caban wrote:

---
mingw-w64-crt/def-include/crt-aliases.def.in  |  30 ++-
.../api-ms-win-crt-convert-l1-1-0.def.in      |   8 +-
.../api-ms-win-crt-math-l1-1-0.def.in         | 148 ++++++-------
.../lib-common/ucrtbase-common.def.in         | 194 +++++++++---------
4 files changed, 189 insertions(+), 191 deletions(-)

diff --git a/mingw-w64-crt/def-include/crt-aliases.def.in b/mingw-w64-crt/def-include/crt-aliases.def.in
index 06526dde5..e6953714b 100644
--- a/mingw-w64-crt/def-include/crt-aliases.def.in
+++ b/mingw-w64-crt/def-include/crt-aliases.def.in
@@ -251,27 +251,25 @@ wcstok == wcstok_s
ADD_UNDERSCORE(nextafter)
#endif

-#if defined(__arm__) || defined(__aarch64__)
; This is list of symbol aliases for C99 ARM long double functions
; They are defined as aliases to regular double symbols as on ARM, long double is equal to regular double
-acosl == acos
-asinl == asin
-atan2l == atan2
-atanl == atan
-ceill == ceil
-cosl == cos
-expl == exp
-floorl == floor
-fmodl == fmod
-log10l == log10
-logl == log
+F_LD64(acosl == acos)
+F_LD64(asinl == asin)
+F_LD64(atan2l == atan2)
+F_LD64(atanl == atan)
+F_LD64(ceill == ceil)
+F_LD64(cosl == cos)
+F_LD64(expl == exp)
+F_LD64(floorl == floor)
+F_LD64(fmodl == fmod)
+F_LD64(log10l == log10)
+F_LD64(logl == log)
; FIXME: Why is powl alias defined only for UCRT?
#ifdef UCRTBASE
-powl == pow
-#endif
-sinl == sin
-tanl == tan
+F_LD64(powl == pow)
#endif
+F_LD64(sinl == sin)
+F_LD64(tanl == tan)

; This is list of symbol aliases for C11 functions
#ifdef UCRTBASE
diff --git a/mingw-w64-crt/lib-common/api-ms-win-crt-convert-l1-1-0.def.in b/mingw-w64-crt/lib-common/api-ms-win-crt-convert-l1-1-0.def.in
index 651b81c2a..f48deaae4 100644
--- a/mingw-w64-crt/lib-common/api-ms-win-crt-convert-l1-1-0.def.in
+++ b/mingw-w64-crt/lib-common/api-ms-win-crt-convert-l1-1-0.def.in
@@ -47,7 +47,7 @@ _strtoi64
_strtoi64_l
_strtoimax_l
_strtol_l
-F_ARM_ANY(_strtold_l) ; Can't use long double functions from the CRT on x86
+F_LD64(_strtold_l)

The patch in itself looks good to me. I guess one can debate whether these comments should be kept or not (to indicate places where we've applied manual modifications, as opposed to just arch specific differences between architectures in what gendef outputs) though.


I assumed that F_LD64 already conveys that. This comment would apply to all uses of F_LD64 and, unlike arch-specific macros, it explains the reason for conditional inclusion. That said, I don’t mind restoring the comments if you still find them useful.


Thanks,

Jacek



_______________________________________________
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public

Reply via email to