<div dir="ltr">Thankyou Jinsong!<div><br></div><div>Two test failures in the test-suite is very promising!</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 15 Aug 2019 at 13:43, Jinsong Ji via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">jsji added a comment.<br>
<br>
Gave it a try on test-suite, with `-O3 -mcpu=pwr9 --ppc-enable-pipeliner`<br>
<br>
The following testcases failed with experimental codegen, while passed with origianl codegen.<br>
<br>
  test-suite :: MultiSource/Benchmarks/MallocBench/espresso/espresso.test<br>
  test-suite :: MultiSource/Benchmarks/MiBench/security-sha/security-sha.test<br>
<br>
<br>
<br>
================<br>
Comment at: include/llvm/CodeGen/MachinePipeliner.h:361<br>
   void setMAX_II();<br>
+<br>
 };<br>
----------------<br>
Extra new line<br>
<br>
<br>
================<br>
Comment at: lib/CodeGen/MachineLoopUtils.cpp:1<br>
+//=- MachineLoopUtils.cpp - Functions for manipulating loops ----------------=//<br>
+//<br>
----------------<br>
jsji wrote:<br>
> New file, you need to update `llvm/lib/CodeGen/CMakeLists.txt` as well.<br>
Still missing patch for `llvm/lib/CodeGen/CMakeLists.txt`.<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D64665/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D64665/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D64665" rel="noreferrer" target="_blank">https://reviews.llvm.org/D64665</a><br>
<br>
<br>
<br>
</blockquote></div>