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