<div dir="ltr">Updated patch attached. I added FileCheck to the command line and changed the weight to something that looks more normal.<div><br></div><div><br></div><div>Please review. Thank you.</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Feb 11, 2014 at 11:20 AM, Manman Ren <span dir="ltr"><<a href="mailto:manman.ren@gmail.com" target="_blank">manman.ren@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><br><div>The testing case seems to miss FileCheck.</div><div>+; RUN: llc -march=arm -print-machineinstrs=tailduplication -tail-dup-size=100 -enable-tail-merge=false -disable-cgp < %s<br></div><div><br></div>

<div>+; CHECK: Successors according to CFG: BB#1(1234) BB#2(5678)<br></div><div>+; CHECK: Successors according to CFG: BB#1(4321) BB#2(8765)<br></div><div><br></div><div>The above numbers look weird 1234 5678 :]</div><div>

<br></div><div>Thanks for working on this.</div><div>Manman</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="">On Tue, Feb 11, 2014 at 10:36 AM, Akira Hatanaka <span dir="ltr"><<a href="mailto:ahatanak@gmail.com" target="_blank">ahatanak@gmail.com</a>></span> wrote:<br>

</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><div dir="ltr">There are two lines in TailDuplication.cpp where edge weights are not passed to MachineBasicBlock::addSuccessor, which results in the weights being dropped from MachineBasicBlock. The attached patch fixes this bug.</div>


<br></div>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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>
</blockquote></div><br></div>