This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH AArch64 2/2] Replace temporary inline assembler for vget_high
- From: Marcus Shawcroft <marcus dot shawcroft at gmail dot com>
- To: Alan Lawrence <alan dot lawrence at arm dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 4 Sep 2014 14:29:07 +0100
- Subject: Re: [PATCH AArch64 2/2] Replace temporary inline assembler for vget_high
- Authentication-results: sourceware.org; auth=none
- References: <53E9E499 dot 4050806 at arm dot com> <53E9E87B dot 4080700 at arm dot com> <CAFqB+PwY-3kd42K67iftsgOqkTX9+6sTWGU8F=TjTNS81RQa=w at mail dot gmail dot com> <5408675A dot 4060506 at arm dot com>
On 4 September 2014 14:21, Alan Lawrence <alan.lawrence@arm.com> wrote:
> Ah, right you are. (Curiously I still can't find Richard's previous message
> in Thunderbird!)
>
> for first patch, gcc/testsuite/ChangeLog:
>
> * gcc.target/aarch64/vget_high_1.c: New test.
> * gcc.target/aarch64/vget_low_1.c: Likewise.
>
> for second patch, gcc/ChangeLog:
>
> * config/aarch64/arm_neon.h (__GET_HIGH): New macro.
> (vget_high_f32, vget_high_f64, vget_high_p8, vget_high_p16,
> vget_high_s8, vget_high_s16, vget_high_s32, vget_high_s64,
> vget_high_u8, vget_high_u16, vget_high_u32, vget_high_u64):
> Remove temporary __asm__ and reimplement.
OK both.
/Marcus