<div dir="ltr">For NVPTX, the Expand action should be set on all of {i1, i8, i16, i32, i64, f32, f64}</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 27, 2013 at 6:14 PM, Tom Stellard <span dir="ltr"><<a href="mailto:tom@stellard.net" target="_blank">tom@stellard.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Here is an updated patch.  The previous version had some unrelated<br>
changes included in it.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Tom<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Wed, Feb 27, 2013 at 05:21:59PM -0500, Tom Stellard wrote:<br>
> Forgot the patch...<br>
><br>
> On Wed, Feb 27, 2013 at 05:17:32PM -0500, Tom Stellard wrote:<br>
> > Hi,<br>
> ><br>
> > The attached patch fixes a bug where the DAGCombiner was passing the<br>
> > MVT::Other value type to TLI.isOperationLegalOrCustom() for a BR_CC node.<br>
> > LegalizeDAG uses the value type of the comparison operands, and I think<br>
> > this is what the DAGCombiner should be using as well.<br>
> ><br>
> > I've updated the *ISelLowering files for the affected targets, but<br>
> > backend code owners should double check my work.<br>
> ><br>
> > Thanks,<br>
> > Tom<br>
> > _______________________________________________<br>
> > llvm-commits mailing list<br>
> > <a href="mailto:llvm-commits@cs.uiuc.edu">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/mailman/listinfo/llvm-commits</a><br>
<br>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">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/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><br><div>Thanks,</div><div><br></div><div>Justin Holewinski</div>
</div>