<div dir="ltr">Hi Hal,<div><br></div><div>I'm keen to get this committed so I can work on the follow-on suggestion I made myself.</div><div><br></div><div>Will you get time to commit this soon? or is it OK if I commit it on your behalf instead?</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, 14 Oct 2015 at 18:49 Philip Reames 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">reames accepted this revision.<br>
reames added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM w/minor comment addressed<br>
<br>
Are you planning on implementing the follow on suggestions?  If not, we should make sure they get tracked either as TODOs or bugs.<br>
<br>
<br>
================<br>
Comment at: lib/Analysis/ValueTracking.cpp:990<br>
@@ +989,3 @@<br>
+<br>
+  computeKnownBits(I->getOperand(1), KnownZero, KnownOne, DL, Depth + 1, Q);<br>
+<br>
----------------<br>
It would be more clearly correct to use the two temporaries for this calculation.  The current code is correct, but slightly confusing.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D12706" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12706</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>