[Mlir-commits] [mlir] [mlir][vector] Add tests for scalable vectors in bufferize-invalid.mlir (PR #102361)

[Andrzej Warzyński]

================
@@ -6,3 +6,12 @@ func.func @mask(%t0: tensor<?xf32>, %val: vector<16xf32>, %idx: index, %m0: vect
   %0 = vector.mask %m0 { vector.transfer_write %val, %t0[%idx] : vector<16xf32>, tensor<?xf32> } : vector<16xi1> -> tensor<?xf32>
   return %0 : tensor<?xf32>
 }
+
+// -----
+
+// CHECK-LABEL: func @mask_scalable(
+func.func @mask_scalable(%t0: tensor<?xf32>, %val: vector<[16]xf32>, %idx: index, %m0: vector<[16]xi1>) -> tensor<?xf32> {
+  // expected-error @+1 {{'vector.mask' op body must bufferize in-place}}
----------------
banach-space wrote:

Yeah, there isn't anything specific about scalable vectors here. However, we want to avoid a situation where scalable vectors are "missed" when implementing this at some later point in time (we wouldn't know without a test).

I guess we can achieve this by simply updating the existing example to use scalable vectors. WDYT?

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