[PATCH] D98210: [ARM] Add VREV MVE shuffle costs

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 9 00:51:45 PST 2021


RKSimon added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp:1222
+
+    if (!Mask.empty()) {
+      std::pair<int, MVT> LT = TLI->getTypeLegalizationCost(DL, Tp);
----------------
Why aren't you just using TTI::SK_Reverse?


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

https://reviews.llvm.org/D98210



More information about the llvm-commits mailing list