<div dir="ltr">LGTM, too.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 7 February 2013 01:13, Nadav Rotem <span dir="ltr"><<a href="mailto:nrotem@apple.com" target="_blank">nrotem@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It's odd that in the ARM backend SELECTS are marked as expand, but there is a pattern to match the expanded form and turn it into vbsel.<br>

<br>
In any case, this change  LGTM.<br>
<br>
<br>
On Feb 6, 2013, at 4:41 PM, Arnold Schwaighofer <<a href="mailto:aschwaighofer@apple.com">aschwaighofer@apple.com</a>> wrote:<br>
<br>
> Oh, yes.<br>
><br>
> <0001-ARM-cost-model-Add-costs-for-vector-selects.patch><br>
><br>
> Thanks,<br>
> Arnold<br>
<div class="HOEnZb"><div class="h5">><br>
> On Feb 6, 2013, at 6:18 PM, Nadav Rotem <<a href="mailto:nrotem@apple.com">nrotem@apple.com</a>> wrote:<br>
><br>
>> +  static const CostTblEntry<MVT> NEONSelectTbl[] = {<br>
>> +    { ISD::SELECT,   MVT::v8i8,   1},<br>
>> +    { ISD::SELECT,   MVT::v16i8,  1},<br>
>> +    { ISD::SELECT,   MVT::v4i16,  1},<br>
>> +    { ISD::SELECT,   MVT::v8i16,  1},<br>
>> +    { ISD::SELECT,   MVT::v2i32,  1},<br>
>> +    { ISD::SELECT,   MVT::v4i32,  1},<br>
>> +    { ISD::SELECT,   MVT::v2i64,  1},<br>
>> +    { ISD::SELECT,   MVT::v2f32,  1},<br>
>> +    { ISD::SELECT,   MVT::v4f32,  1},<br>
>> +    { ISD::SELECT,   MVT::v2f64,  1},<br>
>> +  };<br>
>> +<br>
>><br>
>> You don't need to use a table if you know that vector selects are free for all legal types.<br>
>><br>
>> Thanks,<br>
>> Nadav<br>
<br>
</div></div></blockquote></div><br></div>