This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][PING] Vectorize conversions directly
- From: Richard Earnshaw <rearnsha at arm dot com>
- To: Dmitry Plotnikov <dplotnikov at ispras dot ru>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Ira Rosen <IRAR at il dot ibm dot com>, Ramana Radhakrishnan <ramana dot radhakrishnan at linaro dot org>, "Joseph S. Myers" <joseph at codesourcery dot com>, Richard Henderson <rth at redhat dot com>, "dm at ispras dot ru" <dm at ispras dot ru>
- Date: Thu, 22 Dec 2011 13:43:43 +0000
- Subject: Re: [PATCH][PING] Vectorize conversions directly
- References: <4EA04B20.1090009@ispras.ru> <4EA51EBB.9010506@ispras.ru> <Pine.LNX.4.64.1110241359470.6925@digraph.polyomino.org.uk> <CACUk7=UEPqYkg2Z-kH6EuSK13LGvPfLK-_u26X9NgyOOuw=rsw@mail.gmail.com> <Pine.LNX.4.64.1110241643180.6925@digraph.polyomino.org.uk> <4EAA6637.4010009@ispras.ru> <CACUk7=UVEhVJipTsnpjcabeh8Yohi3ugJhvWUy2XEWRFTiHSww@mail.gmail.com> <OF2B4D420F.F2DBD362-ONC2257954.002B76DF-C2257954.002C013B@il.ibm.com> <4EF32D34.90704@ispras.ru>
On 22/12/11 13:14, Dmitry Plotnikov wrote:
> gcc/config/arm/
> * neon.md (float<mode><V_CVTTOF>2): New.
> (floatuns<mode><V_CVTTOF>2): New.
> (fix_trunc<mode><V_CVTTOI>2): New.
> (fix_truncuns<mode><V_CVTTOI>2): New.
> * iterators.md (V_CVTTOF): New iterator.
> (V_CVTTOI): New iterator.
>
>
> diff --git a/gcc/config/arm/iterators.md b/gcc/config/arm/iterators.md
> index 85dd641..de4340c 100644
> --- a/gcc/config/arm/iterators.md
> +++ b/gcc/config/arm/iterators.md
> @@ -197,6 +197,10 @@
> (define_mode_attr V_CVTTO [(V2SI "V2SF") (V2SF "V2SI")
> (V4SI "V4SF") (V4SF "V4SI")])
>
> +(define_mode_attr V_CVTTOF [(V2SI "v2sf") (V4SI "v4sf")])
> +
> +(define_mode_attr V_CVTTOI [(V2SF "v2si") (V4SF "v4si")])
> +
attributes can be any superset of the iterator, so you don't need two
separate attributes here.
;; As above but in lower case.
(define_mode_attr V_cvtto [V2SI "v2sf") (V2SF "v2si")
(V4SI "v4sf") (V4SF "v4si")])
is perfectly adequate and matches other attributes in the ARM back-end.
The ARM bits are OK with that change.
R.