[llvm-commits] Buggy SelectionDAG binop vector widening

[Duncan Sands]

Hi Dale,

>> Thanks for the advice. I changed all three occurances of isTypeLegal to
>> isTypeSynthesizable in DAGTypeLegalizer::WidenVecRes_Binary. There are
>> still six other isTypeLegal call sites in LegalizeVectorTypes.cpp that
>> probably should be updated, but I didn't touch those for now. The
>> updated patch is attached.
>
> It looks OK to me but I am not expert in this code. Duncan, you wrote
> this didn't you? Can you look?

Mon Ping wrote the vector widening stuff.  I will try to take a look too.

Ciao,

Duncan.