Re: [PATCH] Fix BZ #29463 in the ibm128 implementation of y1l too

[Aurelien Jarno <aurelien@aurel32.net>]

Ping. Tulio (as powerpc machine maintainer) or Joseph (as math component
maintainer), could you please have a look at this patch?

On 2022-10-10 00:09, Aurelien Jarno wrote:
> On 2022-08-22 14:05, Michael Hudson-Doyle via Libc-alpha wrote:
> > ---
> >  sysdeps/ieee754/ldbl-128ibm/e_j1l.c | 3 +++
> >  1 file changed, 3 insertions(+)
> > 
> > diff --git a/sysdeps/ieee754/ldbl-128ibm/e_j1l.c b/sysdeps/ieee754/ldbl-128ibm/e_j1l.c
> > index f85ba94466..0a5fe68342 100644
> > --- a/sysdeps/ieee754/ldbl-128ibm/e_j1l.c
> > +++ b/sysdeps/ieee754/ldbl-128ibm/e_j1l.c
> > @@ -792,10 +792,13 @@ __ieee754_y1l (long double x)
> >      {
> >        /* 0 <= x <= 2 */
> >        SET_RESTORE_ROUNDL (FE_TONEAREST);
> > +      xx = math_opt_barrier (xx);
> > +      x = math_opt_barrier (x);
> >        z = xx * xx;
> >        p = xx * neval (z, Y0_2N, NY0_2N) / deval (z, Y0_2D, NY0_2D);
> >        p = -TWOOPI / xx + p;
> >        p = TWOOPI * __ieee754_logl (x) * __ieee754_j1l (x) + p;
> > +      math_force_eval (p);
> >        return p;
> >      }
> >  
> 
> Tested-by: Aurelien Jarno <aurelien@aurel32.net>
> Reviewed-by: Aurelien Jarno <aurelien@aurel32.net>
> 
> Small nitpick, you might want a similar commit message than for the
> ldbl-128, especially now that they will be far from another in the git
> log. Note however that for that one the issue is reproducible with GCC
> 12 even in -O2.
> 
> -- 
> Aurelien Jarno                          GPG: 4096R/1DDD8C9B
> aurelien@aurel32.net                 http://www.aurel32.net
> 

-- 
Aurelien Jarno                          GPG: 4096R/1DDD8C9B
aurelien@aurel32.net                 http://www.aurel32.net