Re: [PATCH PR69042/01]Add IV candidate for use with constant offset stripped in base.

2016-03-23 Thread Jeff Law
On 03/23/2016 09:29 AM, Bin.Cheng wrote: I think the patch is ok and eventually we need to revisit the outer loop handling with this as an excuse. Yes, currently GCC doesn't handle loop nest at all. Will revisit this part in coming stage1. Patch applied as revision 234429. So given there's f

Re: [PATCH PR69042/01]Add IV candidate for use with constant offset stripped in base.

2016-03-23 Thread Bin.Cheng
On Wed, Mar 23, 2016 at 2:18 PM, Richard Biener wrote: > On Wed, Mar 23, 2016 at 2:58 PM, Bin.Cheng wrote: >> On Tue, Mar 22, 2016 at 11:01 AM, Richard Biener >> wrote: >>> On Tue, Mar 22, 2016 at 11:22 AM, Bin.Cheng wrote: On Wed, Mar 16, 2016 at 10:06 AM, Richard Biener wrote:

Re: [PATCH PR69042/01]Add IV candidate for use with constant offset stripped in base.

2016-03-23 Thread Richard Biener
On Wed, Mar 23, 2016 at 2:58 PM, Bin.Cheng wrote: > On Tue, Mar 22, 2016 at 11:01 AM, Richard Biener > wrote: >> On Tue, Mar 22, 2016 at 11:22 AM, Bin.Cheng wrote: >>> On Wed, Mar 16, 2016 at 10:06 AM, Richard Biener >>> wrote: On Wed, Mar 16, 2016 at 10:48 AM, Bin Cheng wrote:

Re: [PATCH PR69042/01]Add IV candidate for use with constant offset stripped in base.

2016-03-23 Thread Bin.Cheng
On Tue, Mar 22, 2016 at 11:01 AM, Richard Biener wrote: > On Tue, Mar 22, 2016 at 11:22 AM, Bin.Cheng wrote: >> On Wed, Mar 16, 2016 at 10:06 AM, Richard Biener >> wrote: >>> >>> On Wed, Mar 16, 2016 at 10:48 AM, Bin Cheng wrote: >>> > Hi, >>> > When I tried to decrease # of IV candidates, I re

Re: [PATCH PR69042/01]Add IV candidate for use with constant offset stripped in base.

2016-03-22 Thread Richard Biener
On Tue, Mar 22, 2016 at 11:22 AM, Bin.Cheng wrote: > On Wed, Mar 16, 2016 at 10:06 AM, Richard Biener > wrote: >> >> On Wed, Mar 16, 2016 at 10:48 AM, Bin Cheng wrote: >> > Hi, >> > When I tried to decrease # of IV candidates, I removed code that adds IV >> > candidates for use with constant of

Re: [PATCH PR69042/01]Add IV candidate for use with constant offset stripped in base.

2016-03-22 Thread Bin.Cheng
On Wed, Mar 16, 2016 at 10:06 AM, Richard Biener wrote: > > On Wed, Mar 16, 2016 at 10:48 AM, Bin Cheng wrote: > > Hi, > > When I tried to decrease # of IV candidates, I removed code that adds IV > > candidates for use with constant offset stripped in use->base. This is > > kind of too aggress

Re: [PATCH PR69042/01]Add IV candidate for use with constant offset stripped in base.

2016-03-16 Thread Richard Biener
On Wed, Mar 16, 2016 at 10:48 AM, Bin Cheng wrote: > Hi, > When I tried to decrease # of IV candidates, I removed code that adds IV > candidates for use with constant offset stripped in use->base. This is kind > of too aggressive and triggers PR69042. So here is a patch adding back the > miss

[PATCH PR69042/01]Add IV candidate for use with constant offset stripped in base.

2016-03-16 Thread Bin Cheng
Hi, When I tried to decrease # of IV candidates, I removed code that adds IV candidates for use with constant offset stripped in use->base. This is kind of too aggressive and triggers PR69042. So here is a patch adding back the missing candidates. Honestly, this patch doesn't truly fix the is