[llvm-commits] vector widening patch

[Chris Lattner]

On Oct 18, 2008, at 12:45 AM, Duncan Sands wrote:

> Hi,
>
>> Here is a patch for vector widen.  In certain cases, it is more
>> profitable to widen a vector from an illegal type to a legal type
>> (e.g., v7i8 to v8i8) instead of scalarzing the vector.  This patch
>> implements that.  The default implementation is to widen if there  
>> is a
>> legal type that is wider than the illegal type and the vector has  
>> more
>> elements.  Any target can override the default logic and implement it
>> own policy.  Please let me know if you have any comments or concerns.
>
> how about adding Widen to LegalizeAction, to make things uniform with
> expand and promote.  Also, I think it would be better to do this in
> LegalizeTypes [*] rather than in LegalizeDAG - it's the future, and
> it's also much cleaner.

Wouldn't it make sense to use 'expand' to mean "scalarize" and  
'promote' to mean "widen"?

-Chris