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

[Chris Lattner]

On Aug 3, 2012, at 2:03 PM, Hal Finkel <hfinkel at anl.gov> wrote:

> On Fri, 03 Aug 2012 13:50:26 -0700
> Chris Lattner <clattner at apple.com> wrote:
> 
>> 
>> On Aug 3, 2012, at 12:45 PM, Hal Finkel <hfinkel at anl.gov> wrote:
>> 
>>> 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?
>> 
>> I'm not opposed to this, but please only add the specific VTs you
>> need.  We don't want to preemptively add dead code.
> 
> Fair enough.
> 
> Heikki had specifically mentioned wanting v2 through v16i1. I can
> certainly stick to those (or I can just add v4i1, which I need, and
> let the TCE folks propose their own patch).

Sounds good.

-Chris