This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[patch, arm] fix gcc.target/arm/pr45094.c options


This execution test specifies -mcpu=cortex-a8 but there is no corresponding check to make sure that the hardware/simulator being used to run the test can run cortex-a8 code. (The specific case we tripped over was in combination with a -mbig-endian multilib; the combination of the two options results in BE8 code rather than BE32.) It seems simplest just to remove the specific -mcpu option and rely on the multilib options to supply appropriate test flags for the execution environment.

OK to commit?

-Sandra


2014-05-30  Julian Brown  <julian@codesourcery.com>
	    Sandra Loosemore  <sandra@codesourcery.com>

	gcc/testsuite/
	* gcc.target/arm/pr45094.c: Remove -mcpu=cortex-a8, dg-skip-if
	options.

Index: gcc/testsuite/gcc.target/arm/pr45094.c
===================================================================
--- gcc/testsuite/gcc.target/arm/pr45094.c	(revision 211087)
+++ gcc/testsuite/gcc.target/arm/pr45094.c	(working copy)
@@ -1,7 +1,6 @@
 /* { dg-do run } */
-/* { dg-skip-if "incompatible options" { arm*-*-* } { "-march=*" } { "-march=armv7-a" } } */
 /* { dg-require-effective-target arm_neon_hw } */
-/* { dg-options "-O2 -mcpu=cortex-a8" } */
+/* { dg-options "-O2" } */
 /* { dg-add-options arm_neon } */
 
 #include <stdlib.h>

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]