[PATCH] D64316: [ARM] MVE vector of 64bit types

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 9 03:07:41 PDT 2019


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


================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:337
   }
+  // We can do bitwise operations on v2i64 vectors
+  setOperationAction(ISD::AND, MVT::v2i64, Legal);
----------------
SjoerdMeijer wrote:
> Do we need to predicate this on MVE?
Yep, we only call this if we have MVEIntegerOps. Neon is handled elsewhere and I think will promote all and/or/xors to v4i32.


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

https://reviews.llvm.org/D64316





More information about the llvm-commits mailing list