<div dir="ltr">This was the same approach I was taking. Approved unless others have an objection.<div><br></div><div>LGTM.<div><br></div><div>-bw</div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Dec 6, 2013 at 9:43 AM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This patch fixes PR18143.<br>
<br>
InstCombiner::FoldOpIntoPhi assumed that Constant::isNullValue can check for false, but this isn’t true for ConstantExpr.  It appears that this bug was latent until some time after 3.2, when the ConstantExpr in question stopped being constant-folded.<br>

<br>
<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></div>