diff mbox

[rs6000] Cost model adjustment

Message ID 0a8ea7d7-4fcb-39ed-d872-49cb8e9247aa@linux.vnet.ibm.com
State New
Headers show

Commit Message

Bill Schmidt Dec. 13, 2016, 5:13 p.m. UTC 
Hi,

This patch makes a slight adjustment to the vectorization cost model that
was previously overlooked.

Bootstrapped and tested on powerpc64le-unknown-linux gnu with no regressions.
Ok for trunk?

Thanks,
Bill


2016-12-13  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>

	* config/rs6000/rs600.c (rs6000_builtin_vectorization_cost):
	Adjust unaligned load cost.

Comments

Segher Boessenkool Dec. 13, 2016, 6:26 p.m. UTC | #1 
Hi Bill,

On Tue, Dec 13, 2016 at 11:13:08AM -0600, Bill Schmidt wrote:
> This patch makes a slight adjustment to the vectorization cost model that
> was previously overlooked.
> 
> Bootstrapped and tested on powerpc64le-unknown-linux gnu with no regressions.
> Ok for trunk?

Sure, thanks!


> 2016-12-13  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>
> 
> 	* config/rs6000/rs600.c (rs6000_builtin_vectorization_cost):
> 	Adjust unaligned load cost.
diff mbox

Patch

Index: gcc/config/rs6000/rs6000.c
===================================================================
--- gcc/config/rs6000/rs6000.c	(revision 243578)
+++ gcc/config/rs6000/rs6000.c	(working copy)
@@ -5358,6 +5358,9 @@  rs6000_builtin_vectorization_cost (enum vect_cost_
         return 3;
 
       case unaligned_load:
+	if (TARGET_P9_VECTOR)
+	  return 3;
+
 	if (TARGET_EFFICIENT_UNALIGNED_VSX)
 	  return 1;