[PATCH] D62907: [ARM] Implement TTI::isHardwareLoopProfitable

[Dave Green via Phabricator via llvm-commits]

dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

LGTM. With one minor point



================
Comment at: lib/Target/ARM/ARMTargetTransformInfo.cpp:706
+  // For now, for simplicity, only support loops with one exit block.
+  if (!L->getExitBlock() || !SE.getBackedgeTakenCount(L))
+    return false;
----------------
Can we remove this !SE.getBackedgeTakenCount(L) check now?


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

https://reviews.llvm.org/D62907