diff mbox

[rs6000] Remove orphaned VMUL*_UNS built-ins

Message ID 6589aabe-b709-98ed-1117-8ea241becd13@linux.vnet.ibm.com
State New
Headers show

Commit Message

Bill Schmidt March 10, 2017, 6:16 p.m. UTC
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.

Comments

Segher Boessenkool March 10, 2017, 7:14 p.m. UTC | #1
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.
Bill Schmidt March 10, 2017, 7:16 p.m. UTC | #2
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.
>
diff mbox

Patch

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: