[PATCH] D64665: [MachinePipeliner] Refactor schedule emission logic

[Jinsong Ji via Phabricator via llvm-commits]

jsji added a comment.

Gave it a try on test-suite, with `-O3 -mcpu=pwr9 --ppc-enable-pipeliner`

The following testcases failed with experimental codegen, while passed with origianl codegen.

  test-suite :: MultiSource/Benchmarks/MallocBench/espresso/espresso.test
  test-suite :: MultiSource/Benchmarks/MiBench/security-sha/security-sha.test



================
Comment at: include/llvm/CodeGen/MachinePipeliner.h:361
   void setMAX_II();
+
 };
----------------
Extra new line


================
Comment at: lib/CodeGen/MachineLoopUtils.cpp:1
+//=- MachineLoopUtils.cpp - Functions for manipulating loops ----------------=//
+//
----------------
jsji wrote:
> New file, you need to update `llvm/lib/CodeGen/CMakeLists.txt` as well.
Still missing patch for `llvm/lib/CodeGen/CMakeLists.txt`.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64665/new/

https://reviews.llvm.org/D64665