[llvm-commits] [llvm] r75308 - in /llvm/trunk: include/llvm/CodeGen/ValueTypes.h include/llvm/CodeGen/ValueTypes.td lib/VMCore/ValueTypes.cpp utils/TableGen/CodeGenTarget.cpp

[Chris Lattner]

On Jul 12, 2009, at 4:53 PM, Bob Wilson wrote:

>>
>> Right, just don't declare the regpair regclass as "legal" at isel
>> lowering time.
>
> I could definitely go that route, i.e., define one MVT for the new
> Neon regclasses instead of adding all these new vector types.  I don't
> think it would make much difference either way in terms of getting
> Neon to work.
>
> (I had mentioned in an email yesterday that an alternative to the new
> types would be defining a separate intrinsic for each type.  I
> realized later that would be silly.  We could just use arguments to
> the intrinsics to specify the element size and vector length.)
>
> It sounds like you would prefer to avoid adding all the new vector
> types.  Is that right?

Yes, I'd prefer to avoid adding all the types.  It doesn't feel  
"right".  After talking with Evan about it, please find a different  
approach to solve the problem.  Thanks Bob!

-Chris