diff mbox

Add check for aarch64 in vect_cmdline_needed

Message ID 527AD1E8.2040003@codesourcery.com
State New
Headers show

Commit Message

Cesar Philippidis Nov. 6, 2013, 11:34 p.m. UTC 
The following tests were failing on Aarch64 because the vectorizer was
successfully able to vectorize their loop nests:

FAIL: gcc.dg/tree-ssa/gen-vect-11b.c scan-tree-dump-times vect
"vectorized 0 loops" 1
FAIL: gcc.dg/tree-ssa/gen-vect-11c.c scan-tree-dump-times vect
"vectorized 0 loops" 1

Apparently the intent of these particular tests is to ensure that those
loops do not get vectorized when the target does not support SIMD.

Is this OK for trunk? If so, please commit it since I do not have a SVN
account.

Thanks,
Cesar
2013-11-05  Cesar Philippidis  <cesar@codesourcery.com>

	gcc/testsuite/
	* lib/target-supports.exp 
	(check_effective_target_vect_cmdline_neeed): Add aarch64 to the list 
	of targets that do not need command line argument to enable SIMD.

Comments

Joseph Myers Nov. 7, 2013, 1:06 a.m. UTC | #1 
You should be testing aarch64*-*-* so as to match aarch64_be targets.
diff mbox

Patch

Index: gcc/testsuite/lib/target-supports.exp
===================================================================
--- gcc/testsuite/lib/target-supports.exp	(revision 423006)
+++ gcc/testsuite/lib/target-supports.exp	(working copy)
@@ -1920,7 +1920,8 @@  proc check_effective_target_vect_cmdline_needed {
 		     || [check_effective_target_powerpc_altivec]))
 	     || ([istarget sparc*-*-*] && [check_effective_target_sparc_vis])
              || [istarget spu-*-*]
-	     || ([istarget arm*-*-*] && [check_effective_target_arm_neon]) } {
+	     || ([istarget arm*-*-*] && [check_effective_target_arm_neon]) 
+	     || [istarget aarch64-*-*] } {
 	   set et_vect_cmdline_needed_saved 0
 	}
     }