This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [AArch64] Implement some vca*_f[32,64] intrinsics
- From: Christophe Lyon <christophe dot lyon at linaro dot org>
- To: Marcus Shawcroft <marcus dot shawcroft at gmail dot com>
- Cc: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 2 Jul 2014 09:59:59 +0200
- Subject: Re: [AArch64] Implement some vca*_f[32,64] intrinsics
- Authentication-results: sourceware.org; auth=none
- References: <53A839EB dot 3090603 at arm dot com> <CAFqB+PzfamtfspZa_xKS9ht9XtgGC4JBM7kp5v2jgCTjBL01vQ at mail dot gmail dot com>
Hi,
It seems some of the scan-assembler directives fail:
http://cbuild.validation.linaro.org/build/cross-validation/gcc/trunk/212196/aarch64-none-elf/diff-gcc-rh50-aarch64-none-elf-default-default-default.txt
Christophe.
On 1 July 2014 14:13, Marcus Shawcroft <marcus.shawcroft@gmail.com> wrote:
> On 23 June 2014 15:30, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
>> Hi all,
>>
>> This patch implements some absolute compare intrinsics in arm_neon.h.
>>
>> Execution tests are added.
>> Tested aarch64-none-elf, aarch64_be-none-elf, bootstrapped on aarch64 linux
>
>
> +/* { dg-do run } */
> +/* { dg-options "-save-temps -O3" } */
> +
> +#include "arm_neon.h"
> +#include <stdio.h>
> +
>
> Drop the dependence on stdio.h please.
>
> + fprintf (stderr, "Expected: %ld, got %ld\n", expected, actual);
>
> No need to print the expected value, abort( i alone is fine. Dropping
> the printf will make the #include above go away.
>
> Ok with those changes.
>
> /Marcus