On Tue, Oct 25, 2011 at 10:18:26AM -0400, David Edelsohn wrote: > On Mon, Oct 24, 2011 at 8:17 PM, Richard Henderson <r...@redhat.com> wrote: > > The ones that expand to VPERM can be handled by generic code. > > The even v4si and v4sf expanders remain until vector.md can be > > updated to not invoke them directly. > > > +;; ??? This is still used directly by vector.md > > (define_expand "vec_extract_evenv4si" > > > +;; ??? This is still used directly by vector.md > > (define_expand "vec_extract_evenv4sf" > > > -(define_expand "vec_extract_evenv8hi" > > > -(define_expand "vec_extract_evenv16qi" > > > -(define_expand "vec_extract_oddv4si" > > > -(define_expand "vec_extract_oddv4sf" > > Okay. > > Any comments, Mike?
No, we should clean up the vec_extract_even/odd stuff eventually, but assuming the generic code handles the cases, the patches are fine. -- Michael Meissner, IBM 5 Technology Place Drive, M/S 2757, Westford, MA 01886-3141, USA meiss...@linux.vnet.ibm.com fax +1 (978) 399-6899
