This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch, ARM] Implement support for NEON vmovn.
- From: Richard Earnshaw <rearnsha at arm dot com>
- To: Tejas Belagod <tejas dot belagod at arm dot com>
- Cc: "Joseph S. Myers" <joseph at codesourcery dot com>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 01 Sep 2010 15:55:07 +0100
- Subject: Re: [Patch, ARM] Implement support for NEON vmovn.
- References: <1283247561.30429.112.camel@e102484-lin.cambridge.arm.com> <Pine.LNX.4.64.1008311150520.1899@digraph.polyomino.org.uk> <1283264810.30429.128.camel@e102484-lin.cambridge.arm.com>
On Tue, 2010-08-31 at 15:26 +0100, Tejas Belagod wrote:
> OK. Attached is a new patch that changes the vect_pack_trunc flag in
> target-supports.exp. There are tests already in gcc.dg/vect (eg.
> vect-multitypes-14.c) that check for vect_pack_trunc support and pass if
> loops are vectorized. These tests pass with this patch. But they are not
> written to xfail if vect_pack_trunc is not supported - should they have
> been?
>
> --
> Tejas Belagod
> ARM.
>
> New Changelog:
>
> gcc/
>
> 2010-08-31 Tejas Belagod <tejas.belagod@arm.com>
>
> * config/arm/neon.md (vec_pack_trunc_<mode>): Instruction
> pattern for vmovn. Expansion in case of non
> -mvectorize-with-neon-quad.
> (neon_vec_pack_trunc_<mode>): Instruction pattern for vmovn for
> non- -mvectorize-with-neon-quad case.
> (move_lo_quad_<mode>): New expansion to vmov into low part.
> (move_hi_quad_<mode>): New expansion to vmov into high part.
> (move_lo_quad_v4si): Refactor to move_lo_quad_<mode> expansion.
> (move_lo_quad_v4sf): Likewise.
> (move_lo_quad_v8hi): Likewise.
> (neon_move_lo_quad_<mode>): Instruction pattern for vmov into
> low part.
> (neon_move_hi_quad_<mode>): Instruction pattern for vmov into
> high part.
> * config/arm/iterators.md (ANY128): New mode iterator.
> (V_narrow_pack): New mode attribute.
> (V_HALF): Add attribute.
> (V_DOUBLE): Add attribute.
> (V_mode_nunits): Add attribute.
>
> gcc/testsuite
>
> 2010-08-31 Tejas Belagod <tejas.belagod@arm.com>
>
> * lib/target-supports.exp
> (check_effective_target_vect_pack_trunc): Set vect_pack_trunc
> supported flag to true for neon.
>
This is OK.
R.