This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch, arm] fix gcc.target/arm/pr45094.c options
- From: Richard Earnshaw <rearnsha at arm dot com>
- To: Sandra Loosemore <sandra at codesourcery dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Julian Brown <julian at codesourcery dot com>
- Date: Thu, 05 Jun 2014 10:40:47 +0100
- Subject: Re: [patch, arm] fix gcc.target/arm/pr45094.c options
- Authentication-results: sourceware.org; auth=none
- References: <5388D158 dot 6030201 at codesourcery dot com>
On 30/05/14 19:43, Sandra Loosemore wrote:
> 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.
OK
R.
>
>
> pr45094.patch
>
>
> 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>
>