From: Vladimir Makarov <vmakarov@redhat.com>
Date: Thu, 13 Mar 2014 15:52:50 +0000 (+0000)
Subject: re PR rtl-optimization/57189 (Vector register is spilled for vector extract pattern)
X-Git-Tag: releases/gcc-4.9.0~466
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=7891065aa73da81e05d93711637c737fccd34678;p=thirdparty%2Fgcc.git

re PR rtl-optimization/57189 (Vector register is spilled for vector extract pattern)

2014-03-13  Vladimir Makarov  <vmakarov@redhat.com>

	PR rtl-optimization/57189
	* lra-constraints.c (process_alt_operands): Disfavor spilling
	vector pseudos.

2014-03-13  Vladimir Makarov  <vmakarov@redhat.com>

	PR rtl-optimization/57189
	* gcc.target/i386/pr57189.c: New.

From-SVN: r208549
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index a32702af42aa..91b1e41559ef 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2014-03-13  Vladimir Makarov  <vmakarov@redhat.com>
+
+	PR rtl-optimization/57189
+	* lra-constraints.c (process_alt_operands): Disfavor spilling
+	vector pseudos.
+
 2014-03-13  Cesar Philippidis  <cesar@codesourcery.com>
 
 	* lto-wrapper.c (maybe_unlink_file): Suppress diagnostic
diff --git a/gcc/lra-constraints.c b/gcc/lra-constraints.c
index a38b555b2ab0..288e24b4fe15 100644
--- a/gcc/lra-constraints.c
+++ b/gcc/lra-constraints.c
@@ -2302,9 +2302,20 @@ process_alt_operands (int only_alternative)
 		  if (lra_dump_file != NULL)
 		    fprintf
 		      (lra_dump_file,
-		       "            %d Spill pseudo in memory: reject+=3\n",
+		       "            %d Spill pseudo into memory: reject+=3\n",
 		       nop);
 		  reject += 3;
+		  if (VECTOR_MODE_P (mode))
+		    {
+		      /* Spilling vectors into memory is usually more
+			 costly as they contain big values.  */
+		      if (lra_dump_file != NULL)
+			fprintf
+			  (lra_dump_file,
+			   "            %d Spill vector pseudo: reject+=2\n",
+			   nop);
+		      reject += 2;
+		    }
 		}
 
 #ifdef SECONDARY_MEMORY_NEEDED
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 9cdf30dad15a..fd7e9795a289 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2014-03-13  Vladimir Makarov  <vmakarov@redhat.com>
+
+	PR rtl-optimization/57189
+	* gcc.target/i386/pr57189.c: New.
+
 2014-03-13  Paolo Carlini  <paolo.carlini@oracle.com>
 
 	PR c++/60254
diff --git a/gcc/testsuite/gcc.target/i386/pr57189.c b/gcc/testsuite/gcc.target/i386/pr57189.c
new file mode 100644
index 000000000000..389052cd66b0
--- /dev/null
+++ b/gcc/testsuite/gcc.target/i386/pr57189.c
@@ -0,0 +1,10 @@
+/* { dg-do compile } */
+/* { dg-options "-O2 -msse2 -march=k8" } */
+/* { dg-final { scan-assembler-not "movaps" } } */
+
+typedef int __v4si __attribute__ ((__vector_size__ (16)));
+
+int test (__v4si __A)
+{
+  return __builtin_ia32_vec_ext_v4si (__A, 0);
+}