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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 17 08:30:00 PDT 2021


RKSimon added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp:1223
+    if (!Mask.empty()) {
+      std::pair<int, MVT> LT = TLI->getTypeLegalizationCost(DL, Tp);
+      if (isVREVMask(Mask, LT.second, 16) || isVREVMask(Mask, LT.second, 32) ||
----------------
Add an assert that Mask is the correct width?


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

https://reviews.llvm.org/D98210



More information about the llvm-commits mailing list