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.