> On 28 Apr 2025, at 21:27, Ayan Shafqat <ayan.x.shaf...@gmail.com> wrote:
> 
> Rebased with gcc 15.1
> 
> This patch changes the `sqrt` builtin definition from `BUILTIN_VHSDF_DF`
> to `BUILTIN_VHSDF_HSDF` in `aarch64-simd-builtins.def`, ensuring the
> builtin covers half, single, and double precision variants. The redundant
> `VAR1 (UNOP, sqrt, 2, FP, hf)` lines are removed, as they are no longer
> needed now that `BUILTIN_VHSDF_HSDF` handles those cases.
> 

Ok.
Thanks,
Kyrill

> gcc/ChangeLog:
> 
> * config/aarch64/aarch64-simd-builtins.def: Change
> BUILTIN_VHSDF_DF to BUILTIN_VHSDF_HSDF
> 
> Signed-off-by: Ayan Shafqat <ayan.x.shaf...@gmail.com>
> Signed-off-by: Andrew Pinski <quic_apin...@quicinc.com>
> ---
> gcc/config/aarch64/aarch64-simd-builtins.def | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/gcc/config/aarch64/aarch64-simd-builtins.def 
> b/gcc/config/aarch64/aarch64-simd-builtins.def
> index 6cc45b18a72..685bf0dc408 100644
> --- a/gcc/config/aarch64/aarch64-simd-builtins.def
> +++ b/gcc/config/aarch64/aarch64-simd-builtins.def
> @@ -57,7 +57,7 @@
>   VAR1 (BINOPP, pmull, 0, DEFAULT, v8qi)
>   VAR1 (BINOPP, pmull_hi, 0, DEFAULT, v16qi)
>   BUILTIN_VHSDF_HSDF (BINOP, fmulx, 0, FP)
> -  BUILTIN_VHSDF_DF (UNOP, sqrt, 2, FP)
> +  BUILTIN_VHSDF_HSDF (UNOP, sqrt, 2, FP)
>   BUILTIN_VDQ_I (BINOP, addp, 0, DEFAULT)
>   BUILTIN_VDQ_I (BINOPU, addp, 0, DEFAULT)
>   BUILTIN_VDQ_BHSI (UNOP, clrsb, 2, DEFAULT)
> @@ -848,9 +848,6 @@
>   BUILTIN_VHSDF_HSDF (BINOP_USS, facgt, 0, FP)
>   BUILTIN_VHSDF_HSDF (BINOP_USS, facge, 0, FP)
> 
> -  /* Implemented by sqrt<mode>2.  */
> -  VAR1 (UNOP, sqrt, 2, FP, hf)
> -
>   /* Implemented by <optab><mode>hf2.  */
>   VAR1 (UNOP, floatdi, 2, FP, hf)
>   VAR1 (UNOP, floatsi, 2, FP, hf)
> -- 
> 2.43.0
> 

Reply via email to