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

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

[PATCH] Fix PR91790 by considering different first_stmt_info for realign

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 first_stmt_info for vect_setup_realignment, instead

Re: [PATCH] Fix PR91790

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

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

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

[PATCH] Fix PR91790

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 but in other circumstances might simply generate wrong code. Bootstrap runn