Hi Tamar, Thanks for your patch.
Just one comment about your ChangeLog entry for the testsuiet change: shouldn't it mention that it is a new testcase? The patch you attached seems to create the file. Best regards, Thomas On Mon, 13 Aug 2018 at 10:33, Tamar Christina <tamar.christ...@arm.com> wrote: > Hi All, > > On AArch64 we have integer modes larger than TImode, and while we can > generate > moves for these they're not as efficient. > > So instead make sure we limit the maximum we can copy to TImode. This > means > copying a 16 byte struct will issue 1 TImode copy, which will be done > using a > single STP as we expect but an CImode sized copy won't issue CImode > operations. > > Bootstrapped and regtested on aarch4-none-linux-gnu and no issues. > Crosstested aarch4_be-none-elf and no issues. > > Ok for trunk? > > Thanks, > Tamar > > gcc/ > 2018-08-13 Tamar Christina <tamar.christ...@arm.com> > > * config/aarch64/aarch64.c (aarch64_expand_movmem): Set TImode max. > > gcc/testsuite/ > 2018-08-13 Tamar Christina <tamar.christ...@arm.com> > > * gcc.target/aarch64/large_struct_copy_2.c: Add assembler scan. > > -- >
