[PATCH] D62164: [PowerPC] Enable MachinePipeliner for P9 with -ppc-enable-pipeliner

[Jinsong Ji via Phabricator via llvm-commits]

jsji marked an inline comment as done.
jsji added a comment.

Thanks for catching this.



================
Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.cpp:3978
+      LoopCount->getOperand(1).setImm(Offset - 1);
+    return Offset - 1;
+  }
----------------
hfinkel wrote:
> This return value is incorrect if Offset was zero?
Good catch. Although we shouldn't call `reduceLoopCount` with trip count == 0, this should be fixed.


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

https://reviews.llvm.org/D62164