Hi Thomas, Thanks for the review.
I’ll correct the typo before committing if I have no other changes required by a maintainer. Regards, Tamar. From: Thomas Preudhomme <[email protected]> Sent: Monday, August 13, 2018 14:37 To: Tamar Christina <[email protected]> Cc: [email protected]; nd <[email protected]>; James Greenhalgh <[email protected]>; Richard Earnshaw <[email protected]>; Marcus Shawcroft <[email protected]> Subject: Re: [PATCH][GCC][AArch64] Limit movmem copies to TImode copies. 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 <[email protected]<mailto:[email protected]>> 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 <[email protected]<mailto:[email protected]>> * config/aarch64/aarch64.c (aarch64_expand_movmem): Set TImode max. gcc/testsuite/ 2018-08-13 Tamar Christina <[email protected]<mailto:[email protected]>> * gcc.target/aarch64/large_struct_copy_2.c: Add assembler scan. --
