Re: [PATCH] Fix PR80539

2017-04-27 Thread Sebastian Pop
The patch looks good to me. Thanks, Sebastian On Thu, Apr 27, 2017 at 4:25 AM, Richard Biener wrote: > > SCEV analysis requires us to be in loop-closed SSA form to be able > to compute overall effects of inner loops when required. Unfortunately > we have too many places it is used where we are

Re: [PATCH] Fix PR80539

2017-04-27 Thread Richard Biener
On Thu, 27 Apr 2017, Bin.Cheng wrote: > On Thu, Apr 27, 2017 at 10:25 AM, Richard Biener wrote: > > > > SCEV analysis requires us to be in loop-closed SSA form to be able > > to compute overall effects of inner loops when required. Unfortunately > > we have too many places it is used where we ar

Re: [PATCH] Fix PR80539

2017-04-27 Thread Bin.Cheng
On Thu, Apr 27, 2017 at 10:25 AM, Richard Biener wrote: > > SCEV analysis requires us to be in loop-closed SSA form to be able > to compute overall effects of inner loops when required. Unfortunately > we have too many places it is used where we are not in loop-closed SSA > form. The following p

[PATCH] Fix PR80539

2017-04-27 Thread Richard Biener
SCEV analysis requires us to be in loop-closed SSA form to be able to compute overall effects of inner loops when required. Unfortunately we have too many places it is used where we are not in loop-closed SSA form. The following patch makes us more conservative in two places where we previously