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