[PATCH] tree-optimization/96163 - fix placement issue with SLP and vectors

2020-07-13 Thread Richard Biener
This avoids placing stmts beyond the vectorizer region begin which confuses vect_stmt_dominates_stmt_p. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2020-07-13 Richard Biener PR tree-optimization/96163 * tree-vect-slp.c (vect_schedule_slp_instance):

Re: SLP for vectors

2013-04-02 Thread Richard Biener
On Mon, Apr 1, 2013 at 5:52 PM, Marc Glisse marc.gli...@inria.fr wrote: On Sat, 30 Mar 2013, Marc Glisse wrote: * tree-flow-inline.h (get_addr_base_and_unit_offset_1): Handle BIT_FIELD_REF. I wrote a safer version of this for PR52436: That variant is ok - please test and

Re: SLP for vectors

2013-04-02 Thread Richard Biener
On Sat, Mar 30, 2013 at 5:14 PM, Marc Glisse marc.gli...@inria.fr wrote: On Tue, 29 Jan 2013, Richard Biener wrote: So yes, handling BIT_FIELD_REF in the vectorizer looks like the correct way to do - but mind that you should constrain the BIT_FIELD_REFs you allow (I suppose in the end that's

Re: SLP for vectors

2013-04-01 Thread Marc Glisse
On Sat, 30 Mar 2013, Marc Glisse wrote: * tree-flow-inline.h (get_addr_base_and_unit_offset_1): Handle BIT_FIELD_REF. I wrote a safer version of this for PR52436: case BIT_FIELD_REF: - return NULL_TREE; + { + HOST_WIDE_INT this_off =

Re: SLP for vectors

2013-03-30 Thread Marc Glisse
On Tue, 29 Jan 2013, Richard Biener wrote: So yes, handling BIT_FIELD_REF in the vectorizer looks like the correct way to do - but mind that you should constrain the BIT_FIELD_REFs you allow (I suppose in the end that's properly done by other part of the analysis). Does that mean adding

Re: SLP for vectors

2013-01-30 Thread Marc Glisse
On Tue, 29 Jan 2013, Richard Biener wrote: On Sun, Jan 27, 2013 at 4:28 PM, Marc Glisse marc.gli...@inria.fr wrote: Hello, this message is to check that I am not doing something absurd and ask for a bit of advice. In the attached patch, I let SLP recognize vector loads/stores just like it

Re: SLP for vectors

2013-01-29 Thread Richard Biener
On Sun, Jan 27, 2013 at 4:28 PM, Marc Glisse marc.gli...@inria.fr wrote: Hello, this message is to check that I am not doing something absurd and ask for a bit of advice. In the attached patch, I let SLP recognize vector loads/stores just like it recognizes those in an array. It has a few

SLP for vectors

2013-01-27 Thread Marc Glisse
Hello, this message is to check that I am not doing something absurd and ask for a bit of advice. In the attached patch, I let SLP recognize vector loads/stores just like it recognizes those in an array. It has a few issues: the cost of the vectorized version is overestimated, the base