This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH/AARCH64] make bswap vector consistent with scalar name
- From: Marcus Shawcroft <marcus dot shawcroft at gmail dot com>
- To: Andrew Pinski <pinskia at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 5 Dec 2014 14:03:04 +0000
- Subject: Re: [PATCH/AARCH64] make bswap vector consistent with scalar name
- Authentication-results: sourceware.org; auth=none
- References: <CA+=Sn1n_TWd9mxgESKgx8KAzgmyEVbVRZ5+14dtqrO1x+pkmBg at mail dot gmail dot com>
On 24 November 2014 at 17:49, Andrew Pinski <pinskia@gmail.com> wrote:
> I had some local patches in my tree which adds a bswap tree code.
> This breaks the aarch64 back-end vectorizing of byteswaps as we use
> the standard mechanism to see if a tree code vectorizes (optabs).
> Since it make sense to have consistent of the pattern names between
> the vector version and the scalar version, I am proposing this patch
> to make them consistent.
>
> OK? Build and tested on aarch64-elf with no regressions.
>
> Thanks,
> Andrew Pinski
>
> ChangeLog:
> * config/aarch64/aarch64-simd-builtins.def (bswap): Use CF2 rather
> than CF10 so 2 is appended on the code.
> * config/aarch64/aarch64-simd.md (bswap<mode>): Rename to ...
> (bswap<mode>2): This so it matches for the optabs.
OK /Marcus