[PATCH] D68337: [ARM][MVE] Enable extending masked loads

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 16 05:49:34 PDT 2019


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

LGTM. If Craig is happy with the rest.



================
Comment at: lib/Target/ARM/ARMTargetTransformInfo.cpp:510
+  // ABI alignment of 8 or 16.
+  if (!Alignment.hasValue())
+    return true;
----------------
We still probably want to stop i64's and other types.

Maybe do it like "(EltWidth == 32 && (!Alignment || Alignment >= 4)) || ..."


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

https://reviews.llvm.org/D68337





More information about the llvm-commits mailing list