<div dir="ltr">NP. Done.<div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">thanks,<br>Cong</div></div>
<br><div class="gmail_quote">On Wed, Jan 27, 2016 at 5:10 PM, Xinliang David Li <span dir="ltr"><<a href="mailto:davidxl@google.com" target="_blank">davidxl@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">np.<br>
<br>
Cong, can you add a comment for the new test case?<br>
<span class="HOEnZb"><font color="#888888"><br>
David<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Wed, Jan 27, 2016 at 5:09 PM, David Kreitzer<br>
<<a href="mailto:david.l.kreitzer@intel.com">david.l.kreitzer@intel.com</a>> wrote:<br>
> DavidKreitzer added inline comments.<br>
><br>
> ================<br>
> Comment at: llvm/trunk/test/CodeGen/X86/fp-une-cmp.ll:51<br>
> @@ +50,3 @@<br>
> +; CHECK:       %bb1<br>
> +; CHECK:       jmp<br>
> +;<br>
> ----------------<br>
> davidxl wrote:<br>
>> DavidKreitzer wrote:<br>
>> > If we invert the compound branch at the end of the entry block and place bb1 before bb2, we can eliminate the jmp at the end of bb1. Do you know why that isn't happening?<br>
>> ><br>
>> The test case is explicitly added to test the ability for MachineBlockPlacement to break the topological order and reorder BB2 ahead of BB1 (BB1 is ahead of BB2 in source order) -- look at the profile annotation that BB1 is a really cold block -- this reordering is not possible without this patch.<br>
>><br>
>>  See also discussions in <a href="http://reviews.llvm.org/D11393?vs=on&id=45764&whitespace=ignore-most#toc" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11393?vs=on&id=45764&whitespace=ignore-most#toc</a><br>
> Thanks for the explanation!  I missed the profile annotation - Please ignore my comment on this one.<br>
><br>
><br>
> Repository:<br>
>   rL LLVM<br>
><br>
> <a href="http://reviews.llvm.org/D11393" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11393</a><br>
><br>
><br>
><br>
</div></div></blockquote></div><br></div></div>