Fix float range reduction problems (bug 14283).

This commit is contained in:
Joseph Myers 2012-07-03 17:10:42 +00:00
parent e2283f38fe
commit 7a845b2c23
5 changed files with 46 additions and 22 deletions

View File

@ -1,5 +1,14 @@
2012-07-03 Joseph Myers <joseph@codesourcery.com> 2012-07-03 Joseph Myers <joseph@codesourcery.com>
[BZ #14283]
* sysdeps/ieee754/flt-32/k_rem_pio2f.c (__kernel_rem_pio2f): Shift
by 7 not 8 to examine high bit of fractional part. Use volatile
variables when splitting into final array of floats if
__FLT_EVAL_METHOD__ != 0.
* math/libm-test.inc (cos_test): Add another test.
(sin_test): Likewise.
* sysdeps/i386/fpu/libm-test-ulps: Update.
[BZ #14273] [BZ #14273]
* math/libm-test.inc (cosh_test): Add more tests. * math/libm-test.inc (cosh_test): Add more tests.

2
NEWS
View File

@ -9,6 +9,8 @@ Version 2.17
* The following bugs are resolved with this release: * The following bugs are resolved with this release:
14283
Version 2.16 Version 2.16

View File

@ -2589,6 +2589,8 @@ cos_test (void)
TEST_f_f (cos, 0.80190127184058835, 0.69534156199418473); TEST_f_f (cos, 0.80190127184058835, 0.69534156199418473);
#endif #endif
TEST_f_f (cos, 0x1.442f74p+15, 2.4407839902314016628485779006274989801517e-06L);
#ifndef TEST_FLOAT #ifndef TEST_FLOAT
TEST_f_f (cos, 1e22, 0.5232147853951389454975944733847094921409L); TEST_f_f (cos, 1e22, 0.5232147853951389454975944733847094921409L);
TEST_f_f (cos, 0x1p1023, -0.826369834614147994500785680811743734805L); TEST_f_f (cos, 0x1p1023, -0.826369834614147994500785680811743734805L);
@ -7692,6 +7694,8 @@ sin_test (void)
TEST_f_f (sin, 0x1p65, -0.047183876212354673805106149805700013943218L); TEST_f_f (sin, 0x1p65, -0.047183876212354673805106149805700013943218L);
TEST_f_f (sin, -0x1p65, 0.047183876212354673805106149805700013943218L); TEST_f_f (sin, -0x1p65, 0.047183876212354673805106149805700013943218L);
TEST_f_f (sin, 0x1.7f4134p+103, -6.6703229329788657073304190650534846045235e-08L);
#ifdef TEST_DOUBLE #ifdef TEST_DOUBLE
TEST_f_f (sin, 0.80190127184058835, 0.71867942238767868); TEST_f_f (sin, 0.80190127184058835, 0.71867942238767868);
TEST_f_f (sin, 2.522464e-1, 2.4957989804940911e-1); TEST_f_f (sin, 2.522464e-1, 2.4957989804940911e-1);

View File

@ -1656,8 +1656,8 @@ Test "j1 (0.75) == 0.349243602174862192523281016426251335":
double: 1 double: 1
idouble: 1 idouble: 1
Test "j1 (0x1.3ffp+74) == 1.818984347516051243459364437186082741567e-12": Test "j1 (0x1.3ffp+74) == 1.818984347516051243459364437186082741567e-12":
float: 1 float: 2
ifloat: 1 ifloat: 2
ildouble: 1 ildouble: 1
ldouble: 1 ldouble: 1
Test "j1 (0x1.ff00000000002p+840) == 1.846591691699331493194965158699937660696e-127": Test "j1 (0x1.ff00000000002p+840) == 1.846591691699331493194965158699937660696e-127":
@ -2359,8 +2359,8 @@ float: 1
idouble: 1 idouble: 1
ifloat: 1 ifloat: 1
Test "y0 (0x1.3ffp+74) == 1.818984347516051243459467456433028748678e-12": Test "y0 (0x1.3ffp+74) == 1.818984347516051243459467456433028748678e-12":
float: 1 float: 2
ifloat: 1 ifloat: 2
ildouble: 1 ildouble: 1
ldouble: 1 ldouble: 1
Test "y0 (0x1.ff00000000002p+840) == 1.846591691699331493194965158699937660696e-127": Test "y0 (0x1.ff00000000002p+840) == 1.846591691699331493194965158699937660696e-127":
@ -2956,9 +2956,9 @@ ldouble: 2
Function: "j1": Function: "j1":
double: 2 double: 2
float: 1 float: 2
idouble: 2 idouble: 2
ifloat: 1 ifloat: 2
ildouble: 1 ildouble: 1
ldouble: 1 ldouble: 1
@ -3128,9 +3128,9 @@ ldouble: 1
Function: "y0": Function: "y0":
double: 2 double: 2
float: 1 float: 2
idouble: 2 idouble: 2
ifloat: 1 ifloat: 2
ildouble: 1 ildouble: 1
ldouble: 1 ldouble: 1

View File

@ -88,7 +88,7 @@ recompute:
iq[jz-1] -= i<<(8-q0); iq[jz-1] -= i<<(8-q0);
ih = iq[jz-1]>>(7-q0); ih = iq[jz-1]>>(7-q0);
} }
else if(q0==0) ih = iq[jz-1]>>8; else if(q0==0) ih = iq[jz-1]>>7;
else if(z>=(float)0.5) ih=2; else if(z>=(float)0.5) ih=2;
if(ih>0) { /* q > 0.5 */ if(ih>0) { /* q > 0.5 */
@ -166,24 +166,33 @@ recompute:
y[0] = (ih==0)? fw: -fw; y[0] = (ih==0)? fw: -fw;
break; break;
case 1: case 1:
case 2: case 2:;
fw = 0.0; #if __FLT_EVAL_METHOD__ != 0
for (i=jz;i>=0;i--) fw += fq[i]; volatile
y[0] = (ih==0)? fw: -fw; #endif
fw = fq[0]-fw; float fv = 0.0;
for (i=1;i<=jz;i++) fw += fq[i]; for (i=jz;i>=0;i--) fv += fq[i];
y[1] = (ih==0)? fw: -fw; y[0] = (ih==0)? fv: -fv;
fv = fq[0]-fv;
for (i=1;i<=jz;i++) fv += fq[i];
y[1] = (ih==0)? fv: -fv;
break; break;
case 3: /* painful */ case 3: /* painful */
for (i=jz;i>0;i--) { for (i=jz;i>0;i--) {
fw = fq[i-1]+fq[i]; #if __FLT_EVAL_METHOD__ != 0
fq[i] += fq[i-1]-fw; volatile
fq[i-1] = fw; #endif
float fv = fq[i-1]+fq[i];
fq[i] += fq[i-1]-fv;
fq[i-1] = fv;
} }
for (i=jz;i>1;i--) { for (i=jz;i>1;i--) {
fw = fq[i-1]+fq[i]; #if __FLT_EVAL_METHOD__ != 0
fq[i] += fq[i-1]-fw; volatile
fq[i-1] = fw; #endif
float fv = fq[i-1]+fq[i];
fq[i] += fq[i-1]-fv;
fq[i-1] = fv;
} }
for (fw=0.0,i=jz;i>=2;i--) fw += fq[i]; for (fw=0.0,i=jz;i>=2;i--) fw += fq[i];
if(ih==0) { if(ih==0) {