Alright, I'll make those changes.  Thanks for the review!<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 29, 2012 at 8:52 AM, Duncan Sands <span dir="ltr"><<a href="mailto:baldrick@free.fr" target="_blank">baldrick@free.fr</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Justin,<div class="im"><br>
<br>
On 29/11/12 14:07, Justin Holewinski wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The attached patch teaches DAGTypeLegalizer how to handle VSELECT operands.  We<br>
are hitting an assert in SplitVectorOperand() when it tries to visit the operand<br>
of a VSELECT, and this patch fixes that.<br>
</blockquote>
<br></div>
you need to split because the condition operand is not legal, right?<br>
And presumably that's because you want to split vectors of boolean<br>
(as in the patch you just sent in) rather than promoting them.<br>
It's impossible for the other operands to be illegal, since they must<br>
have the same type as the result type, and if the result type was illegal<br>
then it would have been split already during result legalization (result<br>
legalization happens before operand legalization).  So can you please assert<br>
that OpNo is zero.  In addition LoNumElts and HiNumElts must be equal since<br>
splitting is always in two, so how about an assertion that they are equal.<br>
Also, please add a testcase.  Otherwise LGTM.<br>
<br>
Ciao, Duncan.<br>
<br>
<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
<br>
We handle the splitting by splitting the entire VSELECT operation using<br>
EXTRACT_SUBVECTOR.<br>
<br>
--<br>
<br>
Thanks,<br>
<br>
Justin Holewinski<br>
<br>
<br>
<br></div>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
<br>
</blockquote>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><br><div>Thanks,</div><div><br></div><div>Justin Holewinski</div><br>
</div>