Re: SLP for vectors

2013-04-02 Thread Richard Biener
On Sat, Mar 30, 2013 at 5:14 PM, Marc Glisse 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 properly done

Re: SLP for vectors

2013-04-02 Thread Richard Biener
On Mon, Apr 1, 2013 at 5:52 PM, Marc Glisse 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 commit separat

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 = TREE_INT

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 someth

Re: SLP for vectors

2013-01-31 Thread Richard Biener
On Wed, Jan 30, 2013 at 9:45 PM, Marc Glisse wrote: > On Tue, 29 Jan 2013, Richard Biener wrote: > >> On Sun, Jan 27, 2013 at 4:28 PM, Marc Glisse wrote: >>> >>> Hello, >>> >>> this message is to check that I am not doing something absurd and ask for >>> a >>> bit of advice. >>> >>> In the attach

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 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

Re: SLP for vectors

2013-01-29 Thread Richard Biener
On Sun, Jan 27, 2013 at 4:28 PM, Marc Glisse 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 issues: the cos

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 objec