Hi,
on 2024/5/30 00:00, Carl Love wrote:
> This is a new patch to removed the built-ins that were inadvertently missing
> in the previous series.
>
> Carl
> --------------------------------------------------------------
>
> rs6000, Remove redundant float/double type conversions
Nit: s! float/double type conversions! vector float/double conversion builtins!
OK for trunk with this subject tweaked.
BR,
Kewen
>
> The following built-ins are redundant as they are covered by another
> overloaded built-in.
>
> __builtin_vsx_xvcvspdp covered by vec_double{e,o}
> __builtin_vsx_xvcvdpsp covered by vec_float{e,o}
> __builtin_vsx_xvcvsxwdp covered by vec_double{e,o}
> __builtin_vsx_xvcvuxddp_uns covered by vec_double
>
> Remove the redundant built-ins. They are not documented nor do they have
> test cases.
>
> gcc/ChangeLog:
> * config/rs6000/rs6000-builtins.def (__builtin_vsx_xvcvspdp,
> __builtin_vsx_xvcvdpsp, __builtin_vsx_xvcvsxwdp,
> __builtin_vsx_xvcvuxddp_uns): Remove.
> ---
> gcc/config/rs6000/rs6000-builtins.def | 12 ------------
> 1 file changed, 12 deletions(-)
>
> diff --git a/gcc/config/rs6000/rs6000-builtins.def
> b/gcc/config/rs6000/rs6000-builtins.def
> index cea2649b86c..6049f3a4599 100644
> --- a/gcc/config/rs6000/rs6000-builtins.def
> +++ b/gcc/config/rs6000/rs6000-builtins.def
> @@ -1679,9 +1679,6 @@
> const signed int __builtin_vsx_xvcmpgtsp_p (signed int, vf, vf);
> XVCMPGTSP_P vector_gt_v4sf_p {pred}
>
> - const vf __builtin_vsx_xvcvdpsp (vd);
> - XVCVDPSP vsx_xvcvdpsp {}
> -
> const vsll __builtin_vsx_xvcvdpsxds (vd);
> XVCVDPSXDS vsx_fix_truncv2dfv2di2 {}
>
> @@ -1691,9 +1688,6 @@
> const vsll __builtin_vsx_xvcvdpuxds_scale (vd, const int);
> XVCVDPUXDS_SCALE vsx_xvcvdpuxds_scale {}
>
> - const vd __builtin_vsx_xvcvspdp (vf);
> - XVCVSPDP vsx_xvcvspdp {}
> -
> const vsll __builtin_vsx_xvcvspsxds (vf);
> VEC_VSIGNEDE_V4SF vsignede_v4sf {}
>
> @@ -1715,9 +1709,6 @@
> const vf __builtin_vsx_xvcvsxdsp (vsll);
> XVCVSXDSP vsx_xvcvsxdsp {}
>
> - const vd __builtin_vsx_xvcvsxwdp (vsi);
> - XVCVSXWDP vsx_xvcvsxwdp {}
> -
> const vf __builtin_vsx_xvcvsxwsp (vsi);
> XVCVSXWSP vsx_floatv4siv4sf2 {}
>
> @@ -1727,9 +1718,6 @@
> const vd __builtin_vsx_xvcvuxddp_scale (vsll, const int<5>);
> XVCVUXDDP_SCALE vsx_xvcvuxddp_scale {}
>
> - const vd __builtin_vsx_xvcvuxddp_uns (vull);
> - XVCVUXDDP_UNS vsx_floatunsv2div2df2 {}
> -
> const vf __builtin_vsx_xvcvuxdsp (vull);
> XVCVUXDSP vsx_xvcvuxdsp {}
>