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

[Aurelien Jarno <aurelien@aurel32.net>]

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