From: Jakub Jelinek <jakub@redhat.com>
Date: Mon, 27 Feb 2012 15:53:15 +0000 (+0100)
Subject: re PR target/52375 (internal compiler error: in extract_insn, at recog.c:2123 at... 
X-Git-Tag: releases/gcc-4.7.0~190
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=56b1509941fb4f73c7cdc81e541acba0d2d38d2a;p=thirdparty%2Fgcc.git

re PR target/52375 (internal compiler error: in extract_insn, at recog.c:2123 at -O3 -mfpu=neon)

	PR target/52375
	* config/arm/neon.md (vashr<mode>3, vlshr<mode>3): Use
	s_register_operand in the test instead of REG_P.  Don't call
	gen_reg_rtx if it won't be used.

	* gcc.target/arm/pr52375.c: New test.
	* gcc.c-torture/compile/pr52375.c: New test.

From-SVN: r184603
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 1a9e0f988cf4..0406ed24da0d 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,10 @@
 2012-02-27  Jakub Jelinek  <jakub@redhat.com>
 
+	PR target/52375
+	* config/arm/neon.md (vashr<mode>3, vlshr<mode>3): Use
+	s_register_operand in the test instead of REG_P.  Don't call
+	gen_reg_rtx if it won't be used.
+
 	PR tree-optimization/52376
 	* ipa-split.c (split_function): Ignore CLOBBER stmts.
 
diff --git a/gcc/config/arm/neon.md b/gcc/config/arm/neon.md
index a6c891dafd62..e0d6a0cc10c6 100644
--- a/gcc/config/arm/neon.md
+++ b/gcc/config/arm/neon.md
@@ -1064,9 +1064,9 @@
 			(match_operand:VDQIW 2 "imm_rshift_or_reg_neon" "")))]
   "TARGET_NEON"
 {
-  rtx neg = gen_reg_rtx (<MODE>mode);
-  if (REG_P (operands[2]))
+  if (s_register_operand (operands[2], <MODE>mode))
     {
+      rtx neg = gen_reg_rtx (<MODE>mode);
       emit_insn (gen_neg<mode>2 (neg, operands[2]));
       emit_insn (gen_ashl<mode>3_signed (operands[0], operands[1], neg));
     }
@@ -1081,9 +1081,9 @@
 			(match_operand:VDQIW 2 "imm_rshift_or_reg_neon" "")))]
   "TARGET_NEON"
 {
-  rtx neg = gen_reg_rtx (<MODE>mode);
-  if (REG_P (operands[2]))
+  if (s_register_operand (operands[2], <MODE>mode))
     {
+      rtx neg = gen_reg_rtx (<MODE>mode);
       emit_insn (gen_neg<mode>2 (neg, operands[2]));
       emit_insn (gen_ashl<mode>3_unsigned (operands[0], operands[1], neg));
     }
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 0af8d11ad80a..5c9f70cf3c2d 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2012-02-27  Jakub Jelinek  <jakub@redhat.com>
+
+	PR target/52375
+	* gcc.target/arm/pr52375.c: New test.
+	* gcc.c-torture/compile/pr52375.c: New test.
+
 2012-02-27  Ulrich Weigand  <Ulrich.Weigand@de.ibm.com>
 
 	* gfortran.dg/typebound_operator_9.f03: Skip on SPU.
diff --git a/gcc/testsuite/gcc.c-torture/compile/pr52375.c b/gcc/testsuite/gcc.c-torture/compile/pr52375.c
new file mode 100644
index 000000000000..e6877bb11faa
--- /dev/null
+++ b/gcc/testsuite/gcc.c-torture/compile/pr52375.c
@@ -0,0 +1,12 @@
+/* PR target/52375 */
+
+struct C { int c, d; };
+
+unsigned
+foo (struct C *p)
+{
+  unsigned int b = 0, i;
+  for (i = 0; i < 64; i++)
+    b |= 0x80000000U >> p[i].c;
+  return b;
+}
diff --git a/gcc/testsuite/gcc.target/arm/pr52375.c b/gcc/testsuite/gcc.target/arm/pr52375.c
new file mode 100644
index 000000000000..0405c6685f62
--- /dev/null
+++ b/gcc/testsuite/gcc.target/arm/pr52375.c
@@ -0,0 +1,15 @@
+/* PR target/52375 */
+/* { dg-do compile } */
+/* { dg-require-effective-target arm_neon_ok } */
+/* { dg-options "-march=armv7-a -mfloat-abi=hard -mfpu=neon -O -ftree-vectorize" } */
+
+struct C { int c, d; };
+
+unsigned
+foo (struct C *p)
+{
+  unsigned int b = 0, i;
+  for (i = 0; i < 64; i++)
+    b |= 0x80000000U >> p[i].c;
+  return b;
+}