<div dir="ltr">Sorry I was thinking about static MBPI. Yes, using branch weight metadata means we can change it. Your original number looks good then :]<div><br></div><div>LGTM.</div><div><br></div><div>Thanks,</div><div>Manman</div>
<div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 11, 2014 at 12:26 PM, Akira Hatanaka <span dir="ltr"><<a href="mailto:ahatanak@gmail.com" target="_blank">ahatanak@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">Yes, I changed both the weights in the metadata and the string that checks the weights in the machine IR dump.<div>
<br></div><div>Did that answer your question?</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">
On Tue, Feb 11, 2014 at 12:20 PM, 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 class="gmail_extra"><br><br><div class="gmail_quote"><div>On Tue, Feb 11, 2014 at 12:15 PM, Akira Hatanaka <span dir="ltr"><<a href="mailto:ahatanak@gmail.com" target="_blank">ahatanak@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">Updated patch attached. I added FileCheck to the command line and changed the weight to something that looks more normal.</div>


</blockquote><div><br></div></div><div>Are the weights fixed with the testing case? I don't get why you can just change the weights to something.</div><div>LGTM otherwise,</div><div><br></div><div>Manman</div><div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div><br></div><div><br></div><div>Please review. Thank you.</div></div><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>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><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>
</div></div></blockquote></div></div><br></div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>