On Wed, Aug 07, 2019 at 08:23:48PM +0100, Richard Sandiford wrote:
> The Advanced SIMD and SVE permute patterns both split the permute
> operation into a base name and a hilo suffix.  That works well, but it
> means that for "@" patterns, we need to pass the permute code twice,
> once for the base name and once for the suffix.
> 
> Having a unified name avoids that and also makes the definitions
> slightly simpler.
> 
> Tested on aarch64-linux-gnu (with and without SVE) and aarch64_be-elf.
> OK to install?

OK.

Thanks,
James

> 
> 2019-08-07  Richard Sandiford  <richard.sandif...@arm.com>
> 
> gcc/
>       * config/aarch64/iterators.md (perm_insn): Include the "1"/"2" suffix.
>       (perm_hilo): Remove UNSPEC_ZIP*, UNSEPC_TRN* and UNSPEC_UZP*.
>       * config/aarch64/aarch64-simd.md
>       (aarch64_<PERMUTE:perm_insn><PERMUTE:perm_hilo><mode>): Rename to..
>       (aarch64_<PERMUTE:perm_insn><mode>): ...this and remove perm_hilo
>       from the asm template.
>       * config/aarch64/aarch64-sve.md
>       (aarch64_<perm_insn><perm_hilo><PRED_ALL:mode>): Rename to..
>       (aarch64_<perm_insn><PRED_ALL:mode>): ...this and remove perm_hilo
>       from the asm template.
>       (aarch64_<perm_insn><perm_hilo><SVE_ALL:mode>): Rename to..
>       (aarch64_<perm_insn><SVE_ALL:mode>): ...this and remove perm_hilo
>       from the asm template.
>       * config/aarch64/aarch64-simd-builtins.def: Update comment.

Reply via email to