This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[patch, arm] fix gcc.target/arm/pr45094.c options
- From: Sandra Loosemore <sandra at codesourcery dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Cc: Julian Brown <julian at CODESOURCERY dot COM>
- Date: Fri, 30 May 2014 12:43:36 -0600
- Subject: [patch, arm] fix gcc.target/arm/pr45094.c options
- Authentication-results: sourceware.org; auth=none
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>