Re: [PATCH] Fix PR91790 by considering different first_stmt_info for realign

2019-11-26 Thread Richard Biener
On Wed, 27 Nov 2019, Kewen.Lin wrote: > Hi, > > As PR91790 exposed, when we have one slp node whose first_stmt_info_for_drptr > is different from first_stmt_info, it's possible that the first_stmt DR isn't > initialized yet before stmt SLP_TREE_SCALAR_STMTS[0] of slp node. So we > shouldn't use f

Re: [PATCH] Fix PR91790

2019-09-20 Thread Richard Biener
On Thu, 19 Sep 2019, Bill Schmidt wrote: > > On 9/19/19 1:34 PM, Segher Boessenkool wrote: > > Hi! > > > > On Tue, Sep 17, 2019 at 09:45:54AM +0200, Richard Biener wrote: > >> The following fixes an old vectorizer issue with realignment support > >> (thus only powerpc is affected) and BB vectoriz

Re: [PATCH] Fix PR91790

2019-09-19 Thread Bill Schmidt
On 9/19/19 1:34 PM, Segher Boessenkool wrote: Hi! On Tue, Sep 17, 2019 at 09:45:54AM +0200, Richard Biener wrote: The following fixes an old vectorizer issue with realignment support (thus only powerpc is affected) and BB vectorization. The realignment token is set up from the wrong data-ref

Re: [PATCH] Fix PR91790

2019-09-19 Thread Segher Boessenkool
Hi! On Tue, Sep 17, 2019 at 09:45:54AM +0200, Richard Biener wrote: > The following fixes an old vectorizer issue with realignment support > (thus only powerpc is affected) and BB vectorization. The realignment > token is set up from the wrong data-ref which causes an SSA verification > failure b