<div dir="ltr">Hi Hal,<div><br></div><div>Firstly, sorry about the conflicting message asking Philip to review the patch - I sent it from Phab before I'd seen you'd replied 15 minutes before!</div><div><br></div><div>I went to write a better testcase, and actually found that you're right, -O3 does indeed manage to constant propagate correctly. This is strange, and I'm confused as to why I didn't see this in any of my testing. I suspect that I may have over tuned my testcase, or only been testing specific passes.</div><div><br></div><div>Either way, SCCP is the pass that does the magic. It works fine on both my reduced and my original testcase, so the only thing for me to do is to add SCCP to the list of passes run late in the LTO pass manager, which I'll do in another review.</div><div><br></div><div>Thankyou for the extra prod. It has avoided a bunch of useless code being committed!</div><div><br></div><div>Cheers,</div><div><br></div><div>James<br><br><div class="gmail_quote"><div dir="ltr">On Thu, 10 Dec 2015 at 15:37 James Molloy via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jmolloy added a comment.<br>
<br>
Hi Philip,<br>
<br>
Would you feel comfortable reviewing the rest of the code? I'm aware that David is rather swamped with reviews and am trying to find alternative reviewers.<br>
<br>
Cheers,<br>
<br>
James<br>
<br>
<br>
<a href="http://reviews.llvm.org/D15232" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15232</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div></div>