From 0a1f1e78fbdfaf2c01e9c2368023b2533e7136cf Mon Sep 17 00:00:00 2001 From: Anton Blanchard Date: Mon, 4 Jan 2016 13:28:52 -0200 Subject: Eliminate redundant sign extensions in pow() When looking at the code generated for pow() on ppc64 I noticed quite a few sign extensions. Making the array indices unsigned reduces the number of sign extensions from 24 to 7. Tested for powerpc64le and x86_64. --- ChangeLog | 6 ++++++ sysdeps/ieee754/dbl-64/e_pow.c | 6 ++++-- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index bea73e3582..de8a1bcb65 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2016-01-04 Anton Blanchard + + * sysdeps/ieee754/dbl-64/e_pow.c (log1): Use unsigned int + for array indices. + * sysdeps/ieee754/dbl-64/e_pow.c (my_log2): Likewise. + 2016-01-04 Joseph Myers * NEWS: Update copyright dates. diff --git a/sysdeps/ieee754/dbl-64/e_pow.c b/sysdeps/ieee754/dbl-64/e_pow.c index 2ff2bb6e56..663fa392c2 100644 --- a/sysdeps/ieee754/dbl-64/e_pow.c +++ b/sysdeps/ieee754/dbl-64/e_pow.c @@ -245,7 +245,8 @@ static double SECTION log1 (double x, double *delta, double *error) { - int i, j, m; + unsigned int i, j; + int m; double uu, vv, eps, nx, e, e1, e2, t, t1, t2, res, add = 0; mynumber u, v; #ifdef BIG_ENDI @@ -344,7 +345,8 @@ static double SECTION my_log2 (double x, double *delta, double *error) { - int i, j, m; + unsigned int i, j; + int m; double uu, vv, eps, nx, e, e1, e2, t, t1, t2, res, add = 0; double ou1, ou2, lu1, lu2, ov, lv1, lv2, a, a1, a2; double y, yy, z, zz, j1, j2, j7, j8; -- cgit 1.4.1