This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, ARM] Skip pr59858.c test for -mfloat-abi=hard
- From: Richard Earnshaw <rearnsha at arm dot com>
- To: Ian Bolton <ian dot bolton at arm dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 14 Feb 2014 13:46:35 +0000
- Subject: Re: [PATCH, ARM] Skip pr59858.c test for -mfloat-abi=hard
- Authentication-results: sourceware.org; auth=none
- References: <000001cf28c8$7b2f2050$718d60f0$ at bolton@arm.com>
On 13/02/14 14:32, Ian Bolton wrote:
> Hi,
>
> The pr59858.c testcase explicitly sets -msoft-float which is incompatible
> with our -mfloat-abi=hard variant.
>
> This patch therefore should not be run if you have -mfloat-abi=hard.
>
> Tested with both variations for arm-none-eabi build.
>
> OK for commit?
>
> Cheers,
> Ian
>
>
> 2014-02-13 Ian Bolton <ian.bolton@arm.com>
>
> testsuite/
> * gcc.target/arm/pr59858.c: Skip test if -mfloat-abi=hard.
>
>
> pr59858-skip-if-hard-float-patch-v2.txt
>
>
> diff --git a/gcc/testsuite/gcc.target/arm/pr59858.c b/gcc/testsuite/gcc.target/arm/pr59858.c
> index 463bd38..1e03203 100644
> --- a/gcc/testsuite/gcc.target/arm/pr59858.c
> +++ b/gcc/testsuite/gcc.target/arm/pr59858.c
> @@ -1,5 +1,6 @@
> /* { dg-do compile } */
> /* { dg-options "-march=armv5te -marm -mthumb-interwork -Wall -Wstrict-prototypes -Wstrict-aliasing -funsigned-char -fno-builtin -fno-asm -msoft-float -std=gnu99 -mlittle-endian -mthumb -fno-stack-protector -Os -g -feliminate-unused-debug-types -funit-at-a-time -fmerge-all-constants -fstrict-aliasing -fno-tree-loop-optimize -fno-tree-dominator-opts -fno-strength-reduce -fPIC -w" } */
> +/* { dg-skip-if "Test is not compatible with hard-float" { *-*-* } { "-mfloat-abi=hard" } { "" } } */
>
> typedef enum {
> REG_ENOSYS = -1,
>
This won't work if hard-float is the default. Take a look at the way
other tests check for this.