* [PATCH, rs6000] Remove orphaned VMUL*_UNS built-ins
@ 2017-03-10 18:16 Bill Schmidt
2017-03-10 19:14 ` Segher Boessenkool
0 siblings, 1 reply; 3+ messages in thread
From: Bill Schmidt @ 2017-03-10 18:16 UTC (permalink / raw)
To: GCC Patches; +Cc: Segher Boessenkool, David Edelsohn, jakub
Hi,
Jakub observed that these built-ins are no longer reachable (and
haven't been for quite a while). Time to chuck them out.
Bootstrapped and tested on powerpc64le-unknown-linux-gnu with
(surprise!) no regressions. Is this ok for trunk?
Thanks,
Bill
2017-03-10 Bill Schmidt <wschmidt@linux.vnet.ibm.com>
* config/rs6000/rs6000-builtin.def (VMULEUB_UNS): Remove orphaned
built-in.
(VMULEUH_UNS): Likewise.
(VMULOUB_UNS): Likewise.
(VMULOUH_UNS): Likewise.
* config/rs6000/rs6000.c (builtin_function_type): Remove
references to ALTIVEC_BUILTIN_VMUL[EO]U[BH]_UNS.
Index: gcc/config/rs6000/rs6000-builtin.def
===================================================================
--- gcc/config/rs6000/rs6000-builtin.def (revision 246040)
+++ gcc/config/rs6000/rs6000-builtin.def (working copy)
@@ -1059,16 +1059,12 @@ BU_ALTIVEC_2 (VMINUW, "vminuw", CONST, umin
BU_ALTIVEC_2 (VMINSW, "vminsw", CONST, sminv4si3)
BU_ALTIVEC_2 (VMINFP, "vminfp", CONST, sminv4sf3)
BU_ALTIVEC_2 (VMULEUB, "vmuleub", CONST, vec_widen_umult_even_v16qi)
-BU_ALTIVEC_2 (VMULEUB_UNS, "vmuleub_uns", CONST, vec_widen_umult_even_v16qi)
BU_ALTIVEC_2 (VMULESB, "vmulesb", CONST, vec_widen_smult_even_v16qi)
BU_ALTIVEC_2 (VMULEUH, "vmuleuh", CONST, vec_widen_umult_even_v8hi)
-BU_ALTIVEC_2 (VMULEUH_UNS, "vmuleuh_uns", CONST, vec_widen_umult_even_v8hi)
BU_ALTIVEC_2 (VMULESH, "vmulesh", CONST, vec_widen_smult_even_v8hi)
BU_ALTIVEC_2 (VMULOUB, "vmuloub", CONST, vec_widen_umult_odd_v16qi)
-BU_ALTIVEC_2 (VMULOUB_UNS, "vmuloub_uns", CONST, vec_widen_umult_odd_v16qi)
BU_ALTIVEC_2 (VMULOSB, "vmulosb", CONST, vec_widen_smult_odd_v16qi)
BU_ALTIVEC_2 (VMULOUH, "vmulouh", CONST, vec_widen_umult_odd_v8hi)
-BU_ALTIVEC_2 (VMULOUH_UNS, "vmulouh_uns", CONST, vec_widen_umult_odd_v8hi)
BU_ALTIVEC_2 (VMULOSH, "vmulosh", CONST, vec_widen_smult_odd_v8hi)
BU_ALTIVEC_2 (VNOR, "vnor", CONST, norv4si3)
BU_ALTIVEC_2 (VOR, "vor", CONST, iorv4si3)
Index: gcc/config/rs6000/rs6000.c
===================================================================
--- gcc/config/rs6000/rs6000.c (revision 246040)
+++ gcc/config/rs6000/rs6000.c (working copy)
@@ -18526,10 +18526,6 @@ builtin_function_type (machine_mode mode_ret, mach
break;
/* unsigned 2 argument functions. */
- case ALTIVEC_BUILTIN_VMULEUB_UNS:
- case ALTIVEC_BUILTIN_VMULEUH_UNS:
- case ALTIVEC_BUILTIN_VMULOUB_UNS:
- case ALTIVEC_BUILTIN_VMULOUH_UNS:
case ALTIVEC_BUILTIN_VMULEUB:
case ALTIVEC_BUILTIN_VMULEUH:
case ALTIVEC_BUILTIN_VMULOUB:
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH, rs6000] Remove orphaned VMUL*_UNS built-ins
2017-03-10 18:16 [PATCH, rs6000] Remove orphaned VMUL*_UNS built-ins Bill Schmidt
@ 2017-03-10 19:14 ` Segher Boessenkool
2017-03-10 19:16 ` Bill Schmidt
0 siblings, 1 reply; 3+ messages in thread
From: Segher Boessenkool @ 2017-03-10 19:14 UTC (permalink / raw)
To: Bill Schmidt; +Cc: GCC Patches, David Edelsohn, jakub
On Fri, Mar 10, 2017 at 12:16:31PM -0600, Bill Schmidt wrote:
> Jakub observed that these built-ins are no longer reachable (and
> haven't been for quite a while). Time to chuck them out.
>
> Bootstrapped and tested on powerpc64le-unknown-linux-gnu with
> (surprise!) no regressions. Is this ok for trunk?
Yes please. Thanks!
Segher
> 2017-03-10 Bill Schmidt <wschmidt@linux.vnet.ibm.com>
>
> * config/rs6000/rs6000-builtin.def (VMULEUB_UNS): Remove orphaned
> built-in.
> (VMULEUH_UNS): Likewise.
> (VMULOUB_UNS): Likewise.
> (VMULOUH_UNS): Likewise.
> * config/rs6000/rs6000.c (builtin_function_type): Remove
> references to ALTIVEC_BUILTIN_VMUL[EO]U[BH]_UNS.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH, rs6000] Remove orphaned VMUL*_UNS built-ins
2017-03-10 19:14 ` Segher Boessenkool
@ 2017-03-10 19:16 ` Bill Schmidt
0 siblings, 0 replies; 3+ messages in thread
From: Bill Schmidt @ 2017-03-10 19:16 UTC (permalink / raw)
To: Segher Boessenkool; +Cc: GCC Patches, David Edelsohn, jakub
Thanks! Committed in revision 246046.
> On Mar 10, 2017, at 1:14 PM, Segher Boessenkool <segher@kernel.crashing.org> wrote:
>
> On Fri, Mar 10, 2017 at 12:16:31PM -0600, Bill Schmidt wrote:
>> Jakub observed that these built-ins are no longer reachable (and
>> haven't been for quite a while). Time to chuck them out.
>>
>> Bootstrapped and tested on powerpc64le-unknown-linux-gnu with
>> (surprise!) no regressions. Is this ok for trunk?
>
> Yes please. Thanks!
>
>
> Segher
>
>
>> 2017-03-10 Bill Schmidt <wschmidt@linux.vnet.ibm.com>
>>
>> * config/rs6000/rs6000-builtin.def (VMULEUB_UNS): Remove orphaned
>> built-in.
>> (VMULEUH_UNS): Likewise.
>> (VMULOUB_UNS): Likewise.
>> (VMULOUH_UNS): Likewise.
>> * config/rs6000/rs6000.c (builtin_function_type): Remove
>> references to ALTIVEC_BUILTIN_VMUL[EO]U[BH]_UNS.
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-03-10 19:16 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-10 18:16 [PATCH, rs6000] Remove orphaned VMUL*_UNS built-ins Bill Schmidt
2017-03-10 19:14 ` Segher Boessenkool
2017-03-10 19:16 ` Bill Schmidt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).