RE: Gimple loop splitting v2

2016-10-25 Thread Tamar Christina
gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Michael Matz > Sent: 24 October 2016 10:02 > To: Bin.Cheng > Cc: Jeff Law; gcc-patches List > Subject: Re: Gimple loop splitting v2 > > Hi, > > On Mon, 24 Oct 2016, Bin.Cheng wrote: > > > Unfortunately I didn't r

Re: Gimple loop splitting v2

2016-10-24 Thread Michael Matz
Hi, On Mon, 24 Oct 2016, Bin.Cheng wrote: > Unfortunately I didn't reproduce the improvement in my run on AArch64, I > will double check if I made some mistakes. Yeah, our regular testers also didn't pick up these kinds of improvements. As I said, the machine was quite jumpy (though not loade

Re: Gimple loop splitting v2

2016-10-24 Thread Bin.Cheng
On Thu, Oct 20, 2016 at 3:55 PM, Bin.Cheng wrote: > On Thu, Oct 20, 2016 at 3:43 PM, Michael Matz wrote: >> Hi, >> >> On Sat, 5 Dec 2015, Jeff Law wrote: >> >>> Nit. I don't think you want a comma after "so". And it looks like your >>> comment got truncated as well. >>> >>> With the comment abo

Re: Gimple loop splitting v2

2016-10-20 Thread Jeff Law
On 10/20/2016 08:43 AM, Michael Matz wrote: Hi, On Sat, 5 Dec 2015, Jeff Law wrote: Nit. I don't think you want a comma after "so". And it looks like your comment got truncated as well. With the comment above fixed, this is fine for the trunk. I'm terribly sorry to have dropped the ball h

Re: Gimple loop splitting v2

2016-10-20 Thread Bin.Cheng
On Thu, Oct 20, 2016 at 3:43 PM, Michael Matz wrote: > Hi, > > On Sat, 5 Dec 2015, Jeff Law wrote: > >> Nit. I don't think you want a comma after "so". And it looks like your >> comment got truncated as well. >> >> With the comment above fixed, this is fine for the trunk. > > I'm terribly sorry

Re: Gimple loop splitting v2

2016-10-20 Thread Michael Matz
Hi, On Sat, 5 Dec 2015, Jeff Law wrote: > Nit. I don't think you want a comma after "so". And it looks like your > comment got truncated as well. > > With the comment above fixed, this is fine for the trunk. I'm terribly sorry to have dropped the ball here, but I've committed this now after

Re: Gimple loop splitting v2

2016-07-27 Thread Richard Biener
On Wed, Jul 27, 2016 at 8:17 AM, Andrew Pinski wrote: > On Tue, Jul 26, 2016 at 4:32 AM, Richard Biener > wrote: >> On Mon, Jul 25, 2016 at 10:57 PM, Andrew Pinski wrote: >>> On Wed, Dec 2, 2015 at 5:23 AM, Michael Matz wrote: Hi, On Tue, 1 Dec 2015, Jeff Law wrote: > >

Re: Gimple loop splitting v2

2016-07-26 Thread Andrew Pinski
On Tue, Jul 26, 2016 at 4:32 AM, Richard Biener wrote: > On Mon, Jul 25, 2016 at 10:57 PM, Andrew Pinski wrote: >> On Wed, Dec 2, 2015 at 5:23 AM, Michael Matz wrote: >>> Hi, >>> >>> On Tue, 1 Dec 2015, Jeff Law wrote: >>> > So, okay for trunk? -ENOPATCH >>> >>> Sigh :) >>> Here it is.

Re: Gimple loop splitting v2

2016-07-26 Thread Richard Biener
On Mon, Jul 25, 2016 at 10:57 PM, Andrew Pinski wrote: > On Wed, Dec 2, 2015 at 5:23 AM, Michael Matz wrote: >> Hi, >> >> On Tue, 1 Dec 2015, Jeff Law wrote: >> >>> > So, okay for trunk? >>> -ENOPATCH >> >> Sigh :) >> Here it is. > > > I found one problem with it. > Take: > void f(int *a, int M,

Re: Gimple loop splitting v2

2016-07-25 Thread Andrew Pinski
On Wed, Dec 2, 2015 at 5:23 AM, Michael Matz wrote: > Hi, > > On Tue, 1 Dec 2015, Jeff Law wrote: > >> > So, okay for trunk? >> -ENOPATCH > > Sigh :) > Here it is. I found one problem with it. Take: void f(int *a, int M, int *b) { for(int i = 0; i <= M; i++) { if (i < M) a[i

Re: Gimple loop splitting v2

2015-12-04 Thread Jeff Law
On 12/02/2015 06:23 AM, Michael Matz wrote: Hi, On Tue, 1 Dec 2015, Jeff Law wrote: So, okay for trunk? -ENOPATCH Sigh :) Here it is. Ciao, Michael. * common.opt (-fsplit-loops): New flag. * passes.def (pass_loop_split): Add. * opts.c (default_options_table): Add O

Re: Gimple loop splitting v2

2015-12-02 Thread Michael Matz
Hi, On Tue, 1 Dec 2015, Jeff Law wrote: > > So, okay for trunk? > -ENOPATCH Sigh :) Here it is. Ciao, Michael. * common.opt (-fsplit-loops): New flag. * passes.def (pass_loop_split): Add. * opts.c (default_options_table): Add OPT_fsplit_loops entry at -O3. (enab

Re: Gimple loop splitting v2

2015-12-01 Thread Jeff Law
On 12/01/2015 09:46 AM, Michael Matz wrote: Hi, So, okay for trunk? -ENOPATCH Jeff