| Message ID | 0f58ee81-3c83-4885-886f-58ffb7f768ec@linux.ibm.com |
|---|---|
| State | New |
| Headers | show |
| Series | rs6000, built-in cleanup patch series | expand |
Hi, on 2024/4/20 05:18, Carl Love wrote: > rs6000, remove __builtin_vsx_xvnegdp and __builtin_vsx_xvnegsp built-ins > > The undocumented __builtin_vsx_xvnegdp and __builtin_vsx_xvnegsp are > redundant. The overloaded vec_neg built-in provides the same > functionality. The two buit-ins are not documented nor are there any > test cases for them. > > Remove the definitions so users will use the overloaded vec_neg built-in > which is documented in the PVIPR. OK, thanks! BR, Kewen > > gcc/ChangeLog: > * config/rs6000/rs6000-builtins.def (__builtin_vsx_xvnegdp, > __builtin_vsx_xvnegsp): Remove built-in definitions. > --- > gcc/config/rs6000/rs6000-builtins.def | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/gcc/config/rs6000/rs6000-builtins.def b/gcc/config/rs6000/rs6000-builtins.def > index f33564d3d9c..d65c858ac0c 100644 > --- a/gcc/config/rs6000/rs6000-builtins.def > +++ b/gcc/config/rs6000/rs6000-builtins.def > @@ -1763,12 +1763,6 @@ > const vf __builtin_vsx_xvnabssp (vf); > XVNABSSP vsx_nabsv4sf2 {} > > - const vd __builtin_vsx_xvnegdp (vd); > - XVNEGDP negv2df2 {} > - > - const vf __builtin_vsx_xvnegsp (vf); > - XVNEGSP negv4sf2 {} > - > const vd __builtin_vsx_xvnmadddp (vd, vd, vd); > XVNMADDDP nfmav2df4 {} >
diff --git a/gcc/config/rs6000/rs6000-builtins.def b/gcc/config/rs6000/rs6000-builtins.def index f33564d3d9c..d65c858ac0c 100644 --- a/gcc/config/rs6000/rs6000-builtins.def +++ b/gcc/config/rs6000/rs6000-builtins.def @@ -1763,12 +1763,6 @@ const vf __builtin_vsx_xvnabssp (vf); XVNABSSP vsx_nabsv4sf2 {} - const vd __builtin_vsx_xvnegdp (vd); - XVNEGDP negv2df2 {} - - const vf __builtin_vsx_xvnegsp (vf); - XVNEGSP negv4sf2 {} - const vd __builtin_vsx_xvnmadddp (vd, vd, vd); XVNMADDDP nfmav2df4 {}
rs6000, remove __builtin_vsx_xvnegdp and __builtin_vsx_xvnegsp built-ins The undocumented __builtin_vsx_xvnegdp and __builtin_vsx_xvnegsp are redundant. The overloaded vec_neg built-in provides the same functionality. The two buit-ins are not documented nor are there any test cases for them. Remove the definitions so users will use the overloaded vec_neg built-in which is documented in the PVIPR. gcc/ChangeLog: * config/rs6000/rs6000-builtins.def (__builtin_vsx_xvnegdp, __builtin_vsx_xvnegsp): Remove built-in definitions. --- gcc/config/rs6000/rs6000-builtins.def | 6 ------ 1 file changed, 6 deletions(-)