This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH]: Machine independent patch, was: Update SSE5 vector multiplication, shift, rotate
- From: "Andrew Pinski" <pinskia at gmail dot com>
- To: "Paolo Bonzini" <bonzini at gnu dot org>
- Cc: "Richard Guenther" <rguenther at suse dot de>, "Michael Meissner" <michael dot meissner at amd dot com>, "Uros Bizjak" <ubizjak at gmail dot com>, gcc-patches at gcc dot gnu dot org, dwarak dot rajagopal at amd dot com, christophe dot harle at amd dot com, hongjiu dot lu at intel dot com
- Date: Tue, 22 Apr 2008 11:27:47 -0700
- Subject: Re: [PATCH]: Machine independent patch, was: Update SSE5 vector multiplication, shift, rotate
- References: <20080417185036.GA15776@mmeissner-gold.amd.com> <20080418212901.GA30924@mmeissner-gold.amd.com> <Pine.LNX.4.64.0804190007230.4180@zhemvz.fhfr.qr> <20080418223845.GA13657@mmeissner-gold.amd.com> <Pine.LNX.4.64.0804191357020.4180@zhemvz.fhfr.qr> <20080421225247.GA10310@mmeissner-gold.amd.com> <Pine.LNX.4.64.0804221008180.4130@zhemvz.fhfr.qr> <20080422150738.GA12807@mmeissner-gold.amd.com> <Pine.LNX.4.64.0804221724390.4130@zhemvz.fhfr.qr> <480E06BB.7060704@gnu.org>
On Tue, Apr 22, 2008 at 8:39 AM, Paolo Bonzini <bonzini@gnu.org> wrote:
> I agree a lot with this (Michael's). OTOH if you want to go the other way,
> I attach a patch to convert the SPU port (mostly untested); this goes with
> the patch I already posted for AltiVec.
Actually is there a reason why we need an extra vector << scalar ?
Can't we have it match with vec_duplicate and have the tree level
always accept vector shift vector including the tree vectorizer
producing that? The SPU part already does this for constants really
with the 'W' constraint.
Thanks,
Andrew Pinski
- References:
- [PATCH]: Update SSE5 vector multiplication, shift, rotate
- Re: [PATCH]: Machine independent patch, was: Update SSE5 vector multiplication, shift, rotate
- Re: [PATCH]: Machine independent patch, was: Update SSE5 vector multiplication, shift, rotate
- Re: [PATCH]: Machine independent patch, was: Update SSE5 vector multiplication, shift, rotate
- Re: [PATCH]: Machine independent patch, was: Update SSE5 vector multiplication, shift, rotate
- Re: [PATCH]: Machine independent patch, was: Update SSE5 vector multiplication, shift, rotate
- Re: [PATCH]: Machine independent patch, was: Update SSE5 vector multiplication, shift, rotate
- Re: [PATCH]: Machine independent patch, was: Update SSE5 vector multiplication, shift, rotate
- Re: [PATCH]: Machine independent patch, was: Update SSE5 vector multiplication, shift, rotate
- Re: [PATCH]: Machine independent patch, was: Update SSE5 vector multiplication, shift, rotate