This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][ARM] Fix names of some rounding intrinsics, impement vrndx_f32 and vrndxq_f32
- From: Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>
- To: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Ramana Radhakrishnan <ramana dot radhakrishnan at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>
- Date: Thu, 11 Dec 2014 12:26:22 +0000
- Subject: Re: [PATCH][ARM] Fix names of some rounding intrinsics, impement vrndx_f32 and vrndxq_f32
- Authentication-results: sourceware.org; auth=none
- References: <54218CCB dot 3030101 at arm dot com>
- Reply-to: ramrad01 at arm dot com
On Tue, Sep 23, 2014 at 4:07 PM, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
> Hi all,
>
> Some intrinsics had the wrong name (inconsistent with the NEON intrinsics
> spec). This patch fixes that and adds the vrndx_f32 and vrndxq_f32
> intrinsics that were missing.
> These map down to vrintx.f32 NEON instructions (d and q forms). We already
> had builtins defined for them, just the intrinsics were not wired up to them
> properly.
>
> Tested arm-none-eabi
>
> Ok for trunk?
This is OK if no regressions.
Ramana
>
> 2014-09-23 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
>
> * config/arm/arm_neon.h (vrndqn_f32): Rename to...
> (vrndnq_f32): ... this.
> (vrndqa_f32): Rename to...
> (vrndaq_f32): ... this.
> (vrndqp_f32): Rename to...
> (vrndpq_f32): ... this.
> (vrndqm_f32): Rename to...
> (vrndmq_f32): ... this.
> (vrndx_f32): New intrinsic.
> (vrndxq_f32): Likewise.
>
> 2014-09-23 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
>
> * gcc.target/arm/simd/neon-vrndx_f32_1.c: New test.
> * gcc.target/arm/simd/neon-vrndxq_f32_1.c: Likewise.
> * gcc.target/arm/neon/vrndqaf32.c: Rename to...
> * gcc.target/arm/neon/vrndaqf32.c: ... This. Update intrinsic names.
> * gcc.target/arm/neon/vrndqmf32.c: Rename to...
> * gcc.target/arm/neon/vrndmqf32.c: ... This. Update intrinsic names.
> * gcc.target/arm/neon/vrndqnf32.c: Rename to...
> * gcc.target/arm/neon/vrndnqf32.c: ... This. Update intrinsic names.
> * gcc.target/arm/neon/vrndqpf32.c: Rename to...
> * gcc.target/arm/neon/vrndpqf32.c: ... This. Update intrinsic names.