Merge lp:~yao-codesourcery/gcc-linaro/4.4-lp-612011 into lp:gcc-linaro/4.4

[Yao Qi]

This patch is to fix GCC PR45094, and has been submitted to gcc-patches http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00249.html

Ramana Radhakrishnan from ARM reviewed it, and OK with it. Still waiting for arm maintainer's comments. In order to catch up the linaro monthly release, I rise merge request to linaro gcc 4.4 before this is approved on upstreams.

[Andrew Stubbs (ams-codesourcery)]

This is OK.

I'd prefer to wait until it's finally accepted upstream, but we want it in the release.

review: Approve

=== modified file 'ChangeLog.linaro'
--- ChangeLog.linaro	2010-08-05 05:56:23 +0000
+++ ChangeLog.linaro	2010-08-06 05:13:12 +0000
@@ -1,3 +1,13 @@
[phone]  Yao Qi  <[e-mail]>
+
+	LP: #612011
+	gcc/
+	* config/arm/arm.c (output_move_double): Fix typo generating 
+	instructions ('ldr'->'str').
+
+	gcc/testsuite/
+	* gcc.target/arm/pr45094.c: New test.
+
 2010-08-05  Yao Qi  <[e-mail]>
 
 	gcc/testsuite

=== modified file 'gcc/config/arm/arm.c'
--- gcc/config/arm/arm.c	2010-08-02 17:08:15 +0000
+++ gcc/config/arm/arm.c	2010-08-06 05:13:12 +0000
@@ -12758,13 +12758,13 @@
 	    {
 	      if (GET_CODE (XEXP (operands[0], 0)) == PRE_MODIFY)
 		{
-		  output_asm_insn ("ldr%?\t%0, [%1, %2]!", otherops);
-		  output_asm_insn ("ldr%?\t%H0, [%1, #4]", otherops);
+		  output_asm_insn ("str%?\t%0, [%1, %2]!", otherops);
+		  output_asm_insn ("str%?\t%H0, [%1, #4]", otherops);
 		}
 	      else
 		{
-		  output_asm_insn ("ldr%?\t%H0, [%1, #4]", otherops);
-		  output_asm_insn ("ldr%?\t%0, [%1], %2", otherops);
+		  output_asm_insn ("str%?\t%H0, [%1, #4]", otherops);
+		  output_asm_insn ("str%?\t%0, [%1], %2", otherops);
 		}
 	    }
 	  else if (GET_CODE (XEXP (operands[0], 0)) == PRE_MODIFY)

=== added file 'gcc/testsuite/gcc.target/arm/pr45094.c'
--- gcc/testsuite/gcc.target/arm/pr45094.c	1970-01-01 00:00:00 +0000
+++ gcc/testsuite/gcc.target/arm/pr45094.c	2010-08-06 05:13:12 +0000
@@ -0,0 +1,27 @@
+/* { dg-do run } */
+/* { dg-require-effective-target arm_neon_hw } */
+/* { dg-options "-O2 -mcpu=cortex-a8" } */
+/* { dg-add-options arm_neon } */
+
+#include <stdlib.h>
+
+long long buffer[32];
+
+void __attribute__((noinline)) f(long long *p, int n)
+{
+  while (--n >= 0)
+    {
+      *p = 1;
+      p += 32;
+    }
+}
+
+int main(void)
+{
+  f(buffer, 1);
+  
+  if (!buffer[0])
+    abort();
+
+  return 0;
+}