<div dir="ltr">Hi @gyiu<div><br></div><div>Sorry about the build break. I have reverted that part of the code in r315032. The patch which this assert comes with is a fix for an existing bug in MBP. I will add a test case when I do more investigation, hopefully from what you provided to me. </div><div><br></div><div>Thanks </div><div>-Xin</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 6, 2017 at 7:57 AM, Graham Yiu via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">gyiu added a comment.<br>
<br>
@trentxintong Hello, this change is actually causing the assert to trigger in 15 SPEC2006 benchmarks with '-O3 -mcpu=pwr8 -m64 -fprofile-use -flto=thin'.  Failures are during the link step.<br>
<br>
  lib/CodeGen/<wbr>MachineBlockPlacement.cpp:<wbr>2246: void {anonymous}::<wbr>MachineBlockPlacement::<wbr>buildLoopChains(const llvm::MachineLoop&): Assertion `L.isLoopExiting(<wbr>PreferredLoopExit) && "not an exiting block of current loop"' failed.<br>
<br>
Benchmarks: 400.perlbench 401.bzip2 403.gcc 433.milc 445.gobmk 447.dealII 450.soplex 453.povray 456.hmmer 458.sjeng 464.h264ref 471.omnetpp 473.astar 482.sphinx3 483.xalancbmk<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D38563" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D38563</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Software Engineer - Compiler Toolchain<br>Employee of Facebook Inc.</div>
</div>