On Fri, May 27, 2016 at 05:57:17PM +0100, Kyrill Tkachov wrote: > Hi all, > > I notice that we can do without aarch64_simd_attr_length_move. The move > alternatives for > the OI,CI,XImode modes that involve memory operands all use a single > load/store so are always > length 4, whereas the register-to-register moves have a statically-known > length of > (GET_MODE_BITSIZE (mode) / 128) * 4, i.e. 4 bytes for every 128-bit SIMD move > instruction. > This is already encoded in the insn_count mode attribute so use that when > needed. > > That way we avoid a call to recog and a switch statement just to get the > length of an insn. > > Bootstrapped and tested on aarch64. > > Ok for trunk?
OK. Thanks, James > > Thanks, > Kyrill > > 2016-05-27 Kyrylo Tkachov <kyrylo.tkac...@arm.com> > > * config/aarch64/aarch64.c (aarch64_simd_attr_length_move): Delete. > * config/aarch64/aarch64-protos.h (aarch64_simd_attr_length_move): > Delete prototype. > * config/aarch64/iterators.md (insn_count): Add descriptive comment. > * config/aarch64/aarch64-simd.md (*aarch64_mov<mode>, VSTRUCT modes): > Remove use of aarch64_simd_attr_length_move, set length attribute > directly. > (*aarch64_be_movoi): Likewise. > (*aarch64_be_movci): Likewise. > (*aarch64_be_movxi): Likewise.
