<div dir="ltr">I added comments to your review. I'm not sure why we are trying to make jump threading do this, and the testcases added are already taken care of by other passes ...<div><br></div><div>(those passes are also faster at it)</div><div><br></div><div>(So either we should have testcases that only jump threading can get, or we shouldn't make jump threading more expensive to do this)</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 31, 2017 at 9:12 AM, Renato Golin via Phabricator via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rengolin accepted this revision.<br>
rengolin added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM, thanks!<br>
<span class=""><br>
<br>
<br>
================<br>
Comment at: test/Transforms/JumpThreading/<wbr>thread-loads.ll:326<br>
+cond2:<br>
+  %l2 = load i64, i64* %P<br>
+  call void @fn2(i64 %l2)<br>
----------------<br>
</span><span class="">junbuml wrote:<br>
> rengolin wrote:<br>
> > Isn't there just one hop here? `%l2 -> %l1`?<br>
> ><br>
> > I thought you were testing multiple predecessors.<br>
> It's one hop from cond2 to entry, but cond2 -> cond1 -> entry is two hop.<br>
> I also added another test which has three hop.<br>
</span>Right, now it makes more sense. Thanks!<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="https://reviews.llvm.org/D29200" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D29200</a><br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">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/<wbr>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>