[PATCH] D68461: [ARM][MVE] Enable truncating masked stores

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 7 07:19:49 PDT 2019


samparker marked an inline comment as done.
samparker added inline comments.


================
Comment at: lib/Target/ARM/ARMTargetTransformInfo.cpp:498
 
-  if (DataTy->isVectorTy()) {
-    // We don't yet support narrowing or widening masked loads/stores. Expand
-    // them for the moment.
-    unsigned VecWidth = DataTy->getPrimitiveSizeInBits();
-    if (VecWidth != 128)
+  unsigned EltWidth = DataTy->getScalarSizeInBits();
+  if (auto *VecTy = dyn_cast<VectorType>(DataTy)) {
----------------
dmgreen wrote:
> This is the same as in the load patch?
Yes, sorry. I've got two separate downstream branches and forgot to keep this part off this patch.


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

https://reviews.llvm.org/D68461





More information about the llvm-commits mailing list