From: Rich Felker Date: Thu, 23 Apr 2015 10:21:49 +0000 (-0400) Subject: fix regression in x86_64 math asm with old binutils X-Git-Tag: v1.1.9~25 X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=18938c299c0923e7034e8bfc5b250ded3b0ef78a;p=oweals%2Fmusl.git fix regression in x86_64 math asm with old binutils the implicit-operand form of fucomip is rejected by binutils 2.19 and perhaps other versions still in use. writing both operands explicitly fixes the issue. there is no change to the resulting output. commit a732e80d33b4fd6f510f7cec4f5573ef5d89bc4e was the source of this regression. --- diff --git a/src/math/x32/exp2l.s b/src/math/x32/exp2l.s index dfb2bc7c..e9edb96f 100644 --- a/src/math/x32/exp2l.s +++ b/src/math/x32/exp2l.s @@ -6,7 +6,7 @@ expm1l: fmulp movl $0xc2820000,-4(%esp) flds -4(%esp) - fucomip %st(1) + fucomip %st(1),%st fld1 jb 1f # x*log2e <= -65, return -1 without underflow @@ -15,7 +15,7 @@ expm1l: ret 1: fld %st(1) fabs - fucomip %st(1) + fucomip %st(1),%st fstp %st(0) ja 1f f2xm1 @@ -48,7 +48,7 @@ exp2l: fld %st(1) fsub %st(1) faddp - fucomip %st(1) + fucomip %st(1),%st je 2f # x - 0x1p63 + 0x1p63 == x movl $1,(%esp) flds (%esp) # 0x1p-149 diff --git a/src/math/x86_64/exp2l.s b/src/math/x86_64/exp2l.s index 0e9bdf9f..effab2bd 100644 --- a/src/math/x86_64/exp2l.s +++ b/src/math/x86_64/exp2l.s @@ -6,7 +6,7 @@ expm1l: fmulp movl $0xc2820000,-4(%rsp) flds -4(%rsp) - fucomip %st(1) + fucomip %st(1),%st fld1 jb 1f # x*log2e <= -65, return -1 without underflow @@ -15,7 +15,7 @@ expm1l: ret 1: fld %st(1) fabs - fucomip %st(1) + fucomip %st(1),%st fstp %st(0) ja 1f f2xm1 @@ -48,7 +48,7 @@ exp2l: fld %st(1) fsub %st(1) faddp - fucomip %st(1) + fucomip %st(1),%st je 2f # x - 0x1p63 + 0x1p63 == x movl $1,(%rsp) flds (%rsp) # 0x1p-149