Re: [PATCH][GCC][AArch64] Limit movmem copies to TImode copies.

2018-08-29 Thread James Greenhalgh
On Wed, Aug 15, 2018 at 07:55:18AM -0500, Tamar Christina wrote: > Hi All, > > I'm updating the patch with the suggested changes and also fixing a bug with > a boundary condition. > > On AArch64 we have integer modes larger than TImode, and while we can generate > moves for these they're not as

Re: [PATCH][GCC][AArch64] Limit movmem copies to TImode copies.

2018-08-15 Thread Tamar Christina
comes out what you intend it to here: > https://gcc.gnu.org/ml/gcc-patches/2018-08/msg00233.html > > I would just ask you to change the data types accordingly and test it > with -mabi=ilp32. > > Thanks > Sudi > > > From: Thomas Preudhomme > > Sent: Monday, Au

Re: [PATCH][GCC][AArch64] Limit movmem copies to TImode copies.

2018-08-14 Thread Sudakshina Das
data types accordingly and test it with -mabi=ilp32. Thanks Sudi From: Thomas Preudhomme Sent: Monday, August 13, 2018 14:37 To: Tamar Christina Cc: gcc-patches@gcc.gnu.org; nd ; James Greenhalgh ; Richard Earnshaw ; Marcus Shawcroft Subject: Re: [PATCH][GCC][AArch64] Limit movmem copies to TI

RE: [PATCH][GCC][AArch64] Limit movmem copies to TImode copies.

2018-08-13 Thread Tamar Christina
Earnshaw ; Marcus Shawcroft 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

Re: [PATCH][GCC][AArch64] Limit movmem copies to TImode copies.

2018-08-13 Thread Thomas Preudhomme
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 wrote: > Hi All, > > On A

[PATCH][GCC][AArch64] Limit movmem copies to TImode copies.

2018-08-13 Thread Tamar Christina
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