[PATCH 04/13] powerpc: Use sqrt{f} builtin

Paul A. Clarke pc@us.ibm.com
Wed Jun 10 14:22:48 GMT 2020


On Tue, Jun 09, 2020 at 06:32:52PM -0300, Adhemerval Zanella via Libc-alpha wrote:
> The powerpc sqrt implementation is also simplified:
> 
>   - the static constants are open coded within the implementation.
>   - for !USE_SQRT_BUILTIN the function is implemented directly on
>     __ieee754_sqrt (it avoid an superflous extra jump).
> 
> Checked on powerpc-linux-gnu and powerpc64le-linux-gnu.
> ---
>  sysdeps/powerpc/fpu/e_sqrt.c                 | 57 ++++++--------------
>  sysdeps/powerpc/fpu/e_sqrtf.c                | 56 ++++++-------------
>  sysdeps/powerpc/fpu/math-use-builtins-sqrt.h |  9 ++++
>  3 files changed, 42 insertions(+), 80 deletions(-)
>  create mode 100644 sysdeps/powerpc/fpu/math-use-builtins-sqrt.h

LGTM.

Reviewed-by: Paul A. Clarke <pc@us.ibm.com>

PC


More information about the Libc-alpha mailing list