From: Ira Rosen <irar@il.ibm.com>
Date: Tue, 30 Jun 2009 11:33:27 +0000 (+0000)
Subject: re PR tree-optimization/40542 (vectorizes access to volatile array)
X-Git-Tag: releases/gcc-4.5.0~4859
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=1825a1f3cab46c55e7377ed50ab5bd1358b2b0ee;p=thirdparty%2Fgcc.git

re PR tree-optimization/40542 (vectorizes access to volatile array)


	PR tree-optimization/40542
	* tree-vect-stmts.c (vect_analyze_stmt): Don't vectorize volatile
	types.

From-SVN: r149090
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 79637f070fdd..644ef85c09b0 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2009-06-30  Ira Rosen  <irar@il.ibm.com>
+
+	PR tree-optimization/40542
+	* tree-vect-stmts.c (vect_analyze_stmt): Don't vectorize volatile 
+	types.
+
 2009-06-30  Martin Jambor  <mjambor@suse.cz>
 
 	PR tree-optimization/40582
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 49ca9c1ba60e..7ab502e74404 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2009-06-30  Ira Rosen  <irar@il.ibm.com>
+
+	PR tree-optimization/40542
+	* gcc.dg/vect/pr40542.c: New test.
+
 2009-06-30  Martin Jambor  <mjambor@suse.cz>
 
 	PR tree-optimization/40582
diff --git a/gcc/testsuite/gcc.dg/vect/pr40542.c b/gcc/testsuite/gcc.dg/vect/pr40542.c
new file mode 100644
index 000000000000..0a827724dc8f
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/vect/pr40542.c
@@ -0,0 +1,14 @@
+/* { dg-do compile } */
+
+void
+volarr_cpy(char *d, volatile char *s)
+{
+  int i;
+  
+  for (i = 0; i < 16; i++)
+    d[i] = s[i];
+}
+
+/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" { xfail *-*-* } } } */
+/* { dg-final { cleanup-tree-dump "vect" } } */
+
diff --git a/gcc/tree-vect-stmts.c b/gcc/tree-vect-stmts.c
index 586e906b1dea..1c9415b70315 100644
--- a/gcc/tree-vect-stmts.c
+++ b/gcc/tree-vect-stmts.c
@@ -3910,6 +3910,14 @@ vect_analyze_stmt (gimple stmt, bool *need_to_vectorize, slp_tree node)
       print_gimple_stmt (vect_dump, stmt, 0, TDF_SLIM);
     }
 
+  if (gimple_has_volatile_ops (stmt))
+    {    
+      if (vect_print_dump_info (REPORT_UNVECTORIZED_LOCATIONS))
+        fprintf (vect_dump, "not vectorized: stmt has volatile operands");
+
+      return false;
+    }
+ 
   /* Skip stmts that do not need to be vectorized. In loops this is expected 
      to include:
      - the COND_EXPR which is the loop exit condition