https://github.com/arsenm created
https://github.com/llvm/llvm-project/pull/128023
I'm guessing the only reason the __make_mantissa* functions
exist were to support this, so maybe these can be deleted now.
This is broken in the non-constant string case, since it ends
up emitting a call to the libm function
>From 88b441975bc452c5b19d30b4d534fbce0b16dc0b Mon Sep 17 00:00:00 2001
From: Matt Arsenault <matthew.arsena...@amd.com>
Date: Sun, 20 Nov 2022 08:51:42 -0800
Subject: [PATCH] HIP: Use builtin_nan instead of manual expansion
I'm guessing the only reason the __make_mantissa* functions
exist were to support this, so maybe these can be deleted now.
This is broken in the non-constant string case, since it ends
up emitting a call to the libm function
---
clang/lib/Headers/__clang_hip_math.h | 45 ++--------------------------
1 file changed, 3 insertions(+), 42 deletions(-)
diff --git a/clang/lib/Headers/__clang_hip_math.h
b/clang/lib/Headers/__clang_hip_math.h
index 79cb7906852c4..8c21f5d882181 100644
--- a/clang/lib/Headers/__clang_hip_math.h
+++ b/clang/lib/Headers/__clang_hip_math.h
@@ -519,24 +519,8 @@ float modff(float __x, float *__iptr) {
}
__DEVICE__
-float nanf(const char *__tagp __attribute__((nonnull))) {
- union {
- float val;
- struct ieee_float {
- unsigned int mantissa : 22;
- unsigned int quiet : 1;
- unsigned int exponent : 8;
- unsigned int sign : 1;
- } bits;
- } __tmp;
- __static_assert_type_size_equal(sizeof(__tmp.val), sizeof(__tmp.bits));
-
- __tmp.bits.sign = 0u;
- __tmp.bits.exponent = ~0u;
- __tmp.bits.quiet = 1u;
- __tmp.bits.mantissa = __make_mantissa(__tagp);
-
- return __tmp.val;
+float nanf(const char *__tagp) {
+ return __builtin_nanf(__tagp);
}
__DEVICE__
@@ -1072,30 +1056,7 @@ double modf(double __x, double *__iptr) {
__DEVICE__
double nan(const char *__tagp) {
-#if !_WIN32
- union {
- double val;
- struct ieee_double {
- uint64_t mantissa : 51;
- uint32_t quiet : 1;
- uint32_t exponent : 11;
- uint32_t sign : 1;
- } bits;
- } __tmp;
- __static_assert_type_size_equal(sizeof(__tmp.val), sizeof(__tmp.bits));
-
- __tmp.bits.sign = 0u;
- __tmp.bits.exponent = ~0u;
- __tmp.bits.quiet = 1u;
- __tmp.bits.mantissa = __make_mantissa(__tagp);
-
- return __tmp.val;
-#else
- __static_assert_type_size_equal(sizeof(uint64_t), sizeof(double));
- uint64_t __val = __make_mantissa(__tagp);
- __val |= 0xFFF << 51;
- return *reinterpret_cast<double *>(&__val);
-#endif
+ return __builtin_nan(__tagp);
}
__DEVICE__
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
