[llvm-commits] [PATCH] Adding v*i1 MVTs

Hal Finkel hfinkel at anl.gov
Fri Aug 24 16:30:38 PDT 2012


On Fri, 3 Aug 2012 20:02:38 +0000
"Villmow, Micah" <Micah.Villmow at amd.com> wrote:

> Hal,
>  That sounds good. I won't be able to do any combining until next
> Tuesday, so if you want to merge them and submit together I have no
> issue with it.

Were you doing this, or was I? I think that based on the exchange with
Chris and Heikki we're sticking to just adding v2i1, v4i1, v8i1 for now.

Thanks again,
Hal

> 
> The patch itself looks fine to me.
> Micah
> 
> > -----Original Message-----
> > From: Hal Finkel [mailto:hfinkel at anl.gov]
> > Sent: Friday, August 03, 2012 12:46 PM
> > To: llvm-commits at cs.uiuc.edu; Villmow, Micah; Heikki Kultala
> > Subject: [PATCH] Adding v*i1 MVTs
> > 
> > Please review the attached patch which adds v*i1 MVTs. I need these
> > for properly representing the output of vectorized selects and
> > boolean operations on the BG/Q supercomputer (v4i1 specifically).
> > As others have also expressed an interest in these, and in the name
> > of limiting the number of renumbering patches, this patch adds v2i1
> > through v64i1.
> > 
> > Heikki, will this meet your needs as well?
> > 
> > Micah, this patch obviously affects the same renumbering as your <2
> > x [i8|i16]> patch. Maybe we should combine them?
> > 
> > Thanks again,
> > Hal
> > 
> > --
> > Hal Finkel
> > Postdoctoral Appointee
> > Leadership Computing Facility
> > Argonne National Laboratory
> 



-- 
Hal Finkel
Postdoctoral Appointee
Leadership Computing Facility
Argonne National Laboratory



More information about the llvm-commits mailing list