Patchwork Add check for aarch64 in vect_cmdline_needed

login
register
mail settings
Submitter Cesar Philippidis
Date Nov. 7, 2013, 3:20 p.m.
Message ID <527BAFD2.4030205@codesourcery.com>
Download mbox | patch
Permalink /patch/289391/
State New
Headers show

Comments

Cesar Philippidis - Nov. 7, 2013, 3:20 p.m.
On 11/6/13, 5:06 PM, Joseph S. Myers wrote:

> You should be testing aarch64*-*-* so as to match aarch64_be targets.

Thank you for catching that. Please commit this new patch if is OK. I
don't have SVN access.

Thanks,
Cesar
2013-11-06  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.
Marcus Shawcroft - Nov. 13, 2013, 2:20 p.m.
On 7 November 2013 15:20, Cesar Philippidis <cesar@codesourcery.com> wrote:
> On 11/6/13, 5:06 PM, Joseph S. Myers wrote:
>
>> You should be testing aarch64*-*-* so as to match aarch64_be targets.
>
> Thank you for catching that. Please commit this new patch if is OK. I
> don't have SVN access.

Applied as 204745 thanks.

/Marcus

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
 	}
     }