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  <jul...@codesourcery.com>
            Sandra Loosemore  <san...@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>

Reply via email to