[PATCH] D59787: [ARM] Implement TTI::getMemcpyCost

[Dave Green via Phabricator via llvm-commits]

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

Nice. LGTM



================
Comment at: lib/Target/ARM/ARMTargetTransformInfo.cpp:422
+  const Function *F = I->getParent()->getParent();
+  const unsigned Limit = TLI->getMaxStoresPerMemmove(F->optForMinSize());
+  std::vector<EVT> MemOps;
----------------
F->hasMinSize()


================
Comment at: test/Analysis/CostModel/ARM/memcpy.ll:180
+;
+; strict-align:
+;
----------------
This can be removed?


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

https://reviews.llvm.org/D59787