Hi Tom,<br><br>For mips, i64's action should be Expand too.<br><br><div class="gmail_quote">On Wed, Feb 27, 2013 at 3: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><font color="#888888"><br>
-Tom<br>
</font></span><div><div><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" 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/mailman/listinfo/llvm-commits</a><br>
<br>
</div></div><br>_______________________________________________<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/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br>