Re: [PATCH] [4.3 projects] Verctorizer - versioning for alias

[Dorit Nuzman <DORIT@il.ibm.com>]

Victor Kaplansky/Haifa/IBM wrote on 16/08/2007 16:48:11:

> Dorit Nuzman/Haifa/IBM wrote on 16/08/2007 04:11:46 PM:
> >
...
> > I think you can use TYPE_VECTOR_SUBPARTS() instead of
> > GET_MODE_SIZE (TYPE_MODE ()) (just a tiny bit more compact)
>
> I think that it slightly different.  For example TYPE_VECTOR_SUBPARTS()
> returns 4 for a vector of integers on ppc, while
> GET_MODE_SIZE (TYPE_MODE ()) returns 16 - size of vector in bytes.
>

in that case you can use TYPE_SIZE_UNIT ()

dorit

> >
> > Also, for more compact code, you could compute the vf*step once and
> > reuse it for both then and else. For your consideration.
> >
> > +     (vinfo_for_stmt (DR_STMT (dr))))));
> > +
> > +      segment_length =
> > + fold_convert (sizetype,
> > +   fold_build2 (PLUS_EXPR, integer_type_node,
> > +     fold_build2 (MULT_EXPR, integer_type_node, DR_STEP (dr),
> > +    vect_factor),
> > +     vector_size));
> > +
> > +
> > +    }
> > +  else
> > +    {
> > +      segment_length =
> > + fold_convert (sizetype,
> > +   fold_build2 (MULT_EXPR, integer_type_node, DR_STEP (dr),
> > +         vect_factor));
> > +    }
> > +
> > +    return segment_length;
> > +}
>
> OK, I'll address this in next patch.