[Mlir-commits] [mlir] Add Vector-dialect interleave-to-shuffle pattern (PR #91800)

[Andrzej Warzyński]

================
@@ -77,9 +78,33 @@ class UnrollInterleaveOp : public OpRewritePattern<vector::InterleaveOp> {
   int64_t targetRank = 1;
 };
 
+class InterleaveToShuffle : public OpRewritePattern<vector::InterleaveOp> {
----------------
banach-space wrote:

Note, `vector.shuffle` doesn't support scalable vectors:
* https://github.com/llvm/llvm-project/pull/88733

You can just disable this pattern when `sourceType.isScalable()` returns `true`. It would be nice to have a test to verity that as well.

Also, would you mind adding a comment with IR `BEFORE` and `AFTER` (similar to what's been added to `UnrollInterleaveOp`)? I know that right now this is a small file and the pattern is self-explanatory, but these files tend to grow a lot at which point comments can be really helpful.

https://github.com/llvm/llvm-project/pull/91800