[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