Re: [PATCH] Fix PR59822

2014-01-16 Thread Richard Biener
Jakub Jelinek wrote: >On Thu, Jan 16, 2014 at 09:40:24AM +0100, Richard Biener wrote: >> That should be enough I think (we can't hoist PHIs anyway). > >Ok, here is what I've finally successfully bootstrapped/regtested >on x86_64-linux and i686-linux, ok for trunk? Ok. Thanks, Richard. >2014-01-

Re: [PATCH] Fix PR59822

2014-01-16 Thread Jakub Jelinek
On Thu, Jan 16, 2014 at 09:40:24AM +0100, Richard Biener wrote: > That should be enough I think (we can't hoist PHIs anyway). Ok, here is what I've finally successfully bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2014-01-16 Jakub Jelinek * tree-vectorizer.h (s

Re: [PATCH] Fix PR59822

2014-01-16 Thread Richard Biener
On Wed, 15 Jan 2014, Jakub Jelinek wrote: > On Wed, Jan 15, 2014 at 02:06:24PM +0100, Richard Biener wrote: > > > > This should fix PR59822, us creating in valid SSA form when > > hoisting a[i_2] out of a loop where i_2 is defined inside the loop. > > > > Bootstrap / regtest running on x86_64-un

Re: [PATCH] Fix PR59822

2014-01-15 Thread Jakub Jelinek
On Wed, Jan 15, 2014 at 02:06:24PM +0100, Richard Biener wrote: > > This should fix PR59822, us creating in valid SSA form when > hoisting a[i_2] out of a loop where i_2 is defined inside the loop. > > Bootstrap / regtest running on x86_64-unknown-linux-gnu. Unfortunately with my patch on top of