ping?<br><br><div class="gmail_quote">On Tue Jul 29 2014 at 6:24:28 PM Jingyue Wu <<a href="mailto:jingyue@google.com">jingyue@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Can anyone take a look at the latest diff?<br>
<br>
In the latest diff, I use SimplifyInstruction to fold both PHI nodes and select instructions. Doing so requires a minor fix in the logic of clobbering dead uses in select instructions: to clobber x/y in "select undef, x, y", we should change the condition from "undef" to 0/1. Otherwise, "select undef, undef, y" or "select undef, x, undef" can introduce more undefined behavior. This wasn't an issue in the old code, because the old code does not fold a select with undef condition.<br>

<br>
<a href="http://reviews.llvm.org/D4659" target="_blank">http://reviews.llvm.org/D4659</a><br>
<br>
<br>
</blockquote></div>