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 {} >