From 8cf1b11d372cb4d6a4ec129311cc2541aca0529b Mon Sep 17 00:00:00 2001 From: Brendon Cahoon Date: Thu, 27 Oct 2022 11:30:20 -0500 Subject: [PATCH] Fix sinH to return the valid result intead of nan A previous change swapped the return values so that the function return nan when BUILTIN_ISFINITE_F16 is true. Change-Id: Ie37ced76c5b53787016bf1437a00cec3c2e34191 (cherry picked from commit 05637f62cc0b062f09f40d0816dc8fcf37180936) --- ocml/src/sinH.cl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ocml/src/sinH.cl b/ocml/src/sinH.cl index 83ba1cab..527a8e71 100644 --- a/ocml/src/sinH.cl +++ b/ocml/src/sinH.cl @@ -21,7 +21,7 @@ MATH_MANGLE(sin)(half x) s ^= (r.i > (short)1 ? (short)0x8000 : (short)0) ^ (AS_SHORT(x) & (short)0x8000); if (!FINITE_ONLY_OPT()) { - s = BUILTIN_ISFINITE_F16(ax) ?(short)QNANBITPATT_HP16 : s; + s = BUILTIN_ISFINITE_F16(ax) ? s : (short)QNANBITPATT_HP16; } return AS_HALF(s);