[PATCH] D94958: [ARM] Add MVE add.sat costs

[Sjoerd Meijer via Phabricator via llvm-commits]

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

Looks reasonable.



================
Comment at: llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp:1532
+  case Intrinsic::usub_sat:
+    if (ST->hasMVEIntegerOps()) {
+      std::pair<int, MVT> LT =
----------------
Nit: early break:

  if (!ST->hasMVEIntegerOps())
    break;


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

https://reviews.llvm.org/D94958