[PATCH] D94444: [RFC][Scalable] Add scalable shuffle intrinsic to extract evens from a pair of vectors

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added inline comments.


================
Comment at: llvm/docs/LangRef.rst:16206
+
+The arguments to this intrinsic must be vectors of the same type.
+
----------------
AIUI, for scalable vectors the minimum number of elements must be a power of two in order to be able to legalize this operation without widening (not just an even number as I suggested earlier). Can you add that as a restriction?


================
Comment at: llvm/lib/IR/Verifier.cpp:5176
+    break;
+  }
   };
----------------
Can you also check the constraint that the minimum number of elements must be a power of two for scalable vectors?


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

https://reviews.llvm.org/D94444