[PATCH] D66339: [SVE] Fixed-length vector MVT ranges

David Greene via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 16 10:26:38 PDT 2019


greened added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:312
+    if (VT.isScalableVector())
+      continue;
     setOperationAction(ISD::ROTL, VT, Expand);
----------------
Should we replace this with a concat/filter iterator on the vector value types in the future?  Not suggesting this needs to be done now.  Obviously this would apply to the many similar patterns below.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66339





More information about the llvm-commits mailing list