[Mlir-commits] [mlir] andrzej/update xfer perm tests 2 (PR #123237)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Thu Jan 16 12:21:56 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir

@llvm/pr-subscribers-mlir-vector

Author: Andrzej WarzyƄski (banach-space)

<details>
<summary>Changes</summary>

- **[mlir][vector] Update tests for xfer permutation lowering**
- **[mlir][vector] Update tests for xfer permutation lowering (2/N)**


---

Patch is 26.45 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/123237.diff


1 Files Affected:

- (modified) mlir/test/Dialect/Vector/vector-transfer-permutation-lowering.mlir (+131-141) 


``````````diff
diff --git a/mlir/test/Dialect/Vector/vector-transfer-permutation-lowering.mlir b/mlir/test/Dialect/Vector/vector-transfer-permutation-lowering.mlir
index 0feaf690af2510..6705905633e0fa 100644
--- a/mlir/test/Dialect/Vector/vector-transfer-permutation-lowering.mlir
+++ b/mlir/test/Dialect/Vector/vector-transfer-permutation-lowering.mlir
@@ -1,29 +1,27 @@
 // RUN: mlir-opt %s --transform-interpreter --split-input-file | FileCheck %s
 
+///  CHECK: #[[$MAP:.*]] = affine_map<(d0, d1, d2, d3) -> (d1, 0, d3)>
+
 ///----------------------------------------------------------------------------------------
-/// vector.transfer_write -> vector.transpose + vector.transfer_write
 /// [Pattern: TransferWritePermutationLowering]
+///
+/// IN: vector.transfer_write (_transposed_ minor identity permutation map)
+/// OUT: vector.transpose + vector.transfer_write (minor identity permutation map)
 ///----------------------------------------------------------------------------------------
-/// Input:
-///   * vector.transfer_write op with a permutation that under a transpose
-///     _would be_ a minor identity permutation map
-/// Output:
-///   * vector.transpose + vector.transfer_write with a permutation map which
-///     _is_ a minor identity
-
-// CHECK-LABEL:   func.func @xfer_write_transposing_permutation_map
+
+// CHECK-LABEL:   func.func @xfer_write_minor_identity_transposed
 // CHECK-SAME:      %[[VEC:.*]]: vector<4x8xi16>,
-// CHECK-SAME:      %[[MEM:.*]]: memref<2x2x8x4xi16>) {
+// CHECK-SAME:      %[[MEM:.*]]: memref<2x2x8x4xi16>
 // CHECK:           %[[TR:.*]] = vector.transpose %[[VEC]], [1, 0] : vector<4x8xi16> to vector<8x4xi16>
 // CHECK:           vector.transfer_write
 // CHECK-NOT:       permutation_map
 // CHECK-SAME:      %[[TR]], %[[MEM]]{{.*}} {in_bounds = [true, true]} : vector<8x4xi16>, memref<2x2x8x4xi16>
-func.func @xfer_write_transposing_permutation_map(
+func.func @xfer_write_minor_identity_transposed(
     %vec: vector<4x8xi16>,
-    %mem: memref<2x2x8x4xi16>) {
+    %mem: memref<2x2x8x4xi16>,
+    %idx: index) {
 
-  %c0 = arith.constant 0 : index
-  vector.transfer_write %vec, %mem[%c0, %c0, %c0, %c0] {
+  vector.transfer_write %vec, %mem[%idx, %idx, %idx, %idx] {
     in_bounds = [true, true],
     permutation_map = affine_map<(d0, d1, d2, d3) -> (d3, d2)>
   } : vector<4x8xi16>, memref<2x2x8x4xi16>
@@ -31,24 +29,25 @@ func.func @xfer_write_transposing_permutation_map(
   return
 }
 
-// Even with out-of-bounds, it is safe to apply this pattern
-// CHECK-LABEL:   func.func @xfer_write_transposing_permutation_map_out_of_bounds
+// Even with out-of-bounds accesses, it is safe to apply this pattern
+
+// CHECK-LABEL:   func.func @xfer_write_minor_identity_transposed_out_of_bounds
 // CHECK-SAME:      %[[VEC:.*]]: vector<4x8xi16>,
-// CHECK-SAME:      %[[MEM:.*]]: memref<2x2x?x?xi16>) {
-// CHECK:           %[[C0:.*]] = arith.constant 0 : index
+// CHECK-SAME:      %[[MEM:.*]]: memref<2x2x?x?xi16>,
+// CHECK-SAME:      %[[IDX:.*]]: index) {
 // CHECK:           %[[TR:.*]] = vector.transpose %[[VEC]], [1, 0] : vector<4x8xi16> to vector<8x4xi16>
 // Expect the in_bounds attribute to be preserved. Since we don't print it when
 // all flags are "false", it should not appear in the output.
 // CHECK-NOT:       in_bounds
 // CHECK:           vector.transfer_write
 // CHECK-NOT:       permutation_map
-// CHECK-SAME:      %[[TR]], %[[MEM]][%[[C0]], %[[C0]], %[[C0]], %[[C0]]] : vector<8x4xi16>, memref<2x2x?x?xi16>
-func.func @xfer_write_transposing_permutation_map_out_of_bounds(
+// CHECK-SAME:      %[[TR]], %[[MEM]][%[[IDX]], %[[IDX]], %[[IDX]], %[[IDX]]] : vector<8x4xi16>, memref<2x2x?x?xi16>
+func.func @xfer_write_minor_identity_transposed_out_of_bounds(
     %vec: vector<4x8xi16>,
-    %mem: memref<2x2x?x?xi16>) {
+    %mem: memref<2x2x?x?xi16>,
+    %idx: index) {
 
-  %c0 = arith.constant 0 : index
-  vector.transfer_write %vec, %mem[%c0, %c0, %c0, %c0] {
+  vector.transfer_write %vec, %mem[%idx, %idx, %idx, %idx] {
     in_bounds = [false, false],
     permutation_map = affine_map<(d0, d1, d2, d3) -> (d3, d2)>
   } : vector<4x8xi16>, memref<2x2x?x?xi16>
@@ -56,21 +55,22 @@ func.func @xfer_write_transposing_permutation_map_out_of_bounds(
   return
 }
 
-// CHECK-LABEL:   func.func @xfer_write_transposing_permutation_map_with_mask_scalable
+// CHECK-LABEL:   func.func @xfer_write_minor_identity_transposed_with_mask_scalable
 // CHECK-SAME:      %[[VEC:.*]]: vector<4x[8]xi16>,
 // CHECK-SAME:      %[[MEM:.*]]: memref<2x2x?x4xi16>,
-// CHECK-SAME:      %[[MASK:.*]]: vector<[8]x4xi1>) {
+// CHECK-SAME:      %[[MASK:.*]]: vector<[8]x4xi1>
 // CHECK:           %[[TR:.*]] = vector.transpose %[[VEC]], [1, 0] : vector<4x[8]xi16> to vector<[8]x4xi16>
 // CHECK:           vector.transfer_write
 // CHECK-NOT:       permutation_map
 // CHECK-SAME:      %[[TR]], %[[MEM]]{{.*}}, %[[MASK]] {in_bounds = [true, true]} : vector<[8]x4xi16>, memref<2x2x?x4xi16>
-func.func @xfer_write_transposing_permutation_map_with_mask_scalable(
+func.func @xfer_write_minor_identity_transposed_with_mask_scalable(
     %vec: vector<4x[8]xi16>,
     %mem: memref<2x2x?x4xi16>,
-    %mask: vector<[8]x4xi1>) {
+    %mask: vector<[8]x4xi1>,
+    %idx: index) {
 
   %c0 = arith.constant 0 : index
-  vector.transfer_write %vec, %mem[%c0, %c0, %c0, %c0], %mask {
+  vector.transfer_write %vec, %mem[%idx, %idx, %idx, %idx], %mask {
     in_bounds = [true, true],
     permutation_map = affine_map<(d0, d1, d2, d3) -> (d3, d2)>
   } : vector<4x[8]xi16>, memref<2x2x?x4xi16>
@@ -79,16 +79,18 @@ func.func @xfer_write_transposing_permutation_map_with_mask_scalable(
 }
 
 // Masked version is not supported
-// CHECK-LABEL:   func.func @xfer_write_transposing_permutation_map_masked
+
+// CHECK-LABEL:   func.func @xfer_write_minor_identity_transposed_map_masked
 // CHECK-NOT: vector.transpose
-func.func @xfer_write_transposing_permutation_map_masked(
+func.func @xfer_write_minor_identity_transposed_map_masked(
     %vec: vector<4x8xi16>,
     %mem: memref<2x2x8x4xi16>,
-    %mask: vector<8x4xi1>) {
+    %mask: vector<8x4xi1>,
+    %idx: index) {
 
   %c0 = arith.constant 0 : index
   vector.mask %mask {
-    vector.transfer_write %vec, %mem[%c0, %c0, %c0, %c0] {
+    vector.transfer_write %vec, %mem[%idx, %idx, %idx, %idx] {
       in_bounds = [true, true],
       permutation_map = affine_map<(d0, d1, d2, d3) -> (d3, d2)>
     } : vector<4x8xi16>, memref<2x2x8x4xi16>
@@ -98,24 +100,24 @@ func.func @xfer_write_transposing_permutation_map_masked(
 }
 
 ///----------------------------------------------------------------------------------------
-/// vector.transfer_write -> vector.broadcast + vector.transpose + vector.transfer_write
 /// [Patterns: TransferWriteNonPermutationLowering + TransferWritePermutationLowering]
+///
+/// IN: vector.transfer_write
+///         (neither a minor identity nor transposed minor identity map)
+/// OUT 1: vector.broadcast + vector.transfer_write
+///         (transposed minor identity)
+/// OUT 2: vector.transfer_write -> vector.broadcast + vector.transpose + vector.transfer_write
+///         (minor identity)
 ///----------------------------------------------------------------------------------------
-/// Input:
-///   * vector.transfer_write op with a map which _is not_ a permutation of a
-///     minor identity
-/// Output:
-///   * vector.broadcast + vector.transpose + vector.transfer_write with a map
-///     which _is_ a permutation of a minor identity
-
-// CHECK-LABEL:   func.func @xfer_write_non_transposing_permutation_map(
+
+// CHECK-LABEL:   func.func @xfer_write_non_minor_identity(
 // CHECK-SAME:      %[[MEM:.*]]: memref<?x?xf32>,
 // CHECK-SAME:      %[[VEC:.*]]: vector<7xf32>,
 // CHECK-SAME:      %[[IDX_1:.*]]: index, %[[IDX_2:.*]]: index) {
 // CHECK:           %[[BC:.*]] = vector.broadcast %[[VEC]] : vector<7xf32> to vector<1x7xf32>
 // CHECK:           %[[TR:.*]] = vector.transpose %[[BC]], [1, 0] : vector<1x7xf32> to vector<7x1xf32>
 // CHECK:           vector.transfer_write %[[TR]], %[[MEM]]{{\[}}%[[IDX_1]], %[[IDX_2]]] {in_bounds = [false, true]} : vector<7x1xf32>, memref<?x?xf32>
-func.func @xfer_write_non_transposing_permutation_map(
+func.func @xfer_write_non_minor_identity(
     %mem : memref<?x?xf32>,
     %vec : vector<7xf32>,
     %idx_1 : index,
@@ -128,8 +130,9 @@ func.func @xfer_write_non_transposing_permutation_map(
   return
 }
 
-// Even with out-of-bounds, it is safe to apply this pattern
-// CHECK-LABEL:   func.func @xfer_write_non_transposing_permutation_map_with_mask_out_of_bounds(
+// Even with out-of-bounds accesses, it is safe to apply this pattern
+
+// CHECK-LABEL:   func.func @xfer_write_non_minor_identity_with_mask_out_of_bounds(
 // CHECK-SAME:      %[[MEM:.*]]: memref<?x?xf32>,
 // CHECK-SAME:      %[[VEC:.*]]: vector<7xf32>,
 // CHECK-SAME:      %[[IDX_1:.*]]: index, %[[IDX_2:.*]]: index,
@@ -139,7 +142,7 @@ func.func @xfer_write_non_transposing_permutation_map(
 // CHECK:           %[[TR_MASK:.*]] = vector.transpose %[[BC_MASK]], [1, 0] : vector<1x7xi1> to vector<7x1xi1>
 // CHECK:           %[[TR_VEC:.*]] = vector.transpose %[[BC_VEC]], [1, 0] : vector<1x7xf32> to vector<7x1xf32>
 // CHECK:           vector.transfer_write %[[TR_VEC]], %[[MEM]]{{\[}}%[[IDX_1]], %[[IDX_2]]], %[[TR_MASK]] {in_bounds = [false, true]} : vector<7x1xf32>, memref<?x?xf32>
-func.func @xfer_write_non_transposing_permutation_map_with_mask_out_of_bounds(
+func.func @xfer_write_non_minor_identity_with_mask_out_of_bounds(
     %mem : memref<?x?xf32>,
     %vec : vector<7xf32>,
     %idx_1 : index,
@@ -154,23 +157,22 @@ func.func @xfer_write_non_transposing_permutation_map_with_mask_out_of_bounds(
   return
 }
 
-// CHECK:           func.func @permutation_with_mask_xfer_write_scalable(
+// CHECK-LABEL:     func.func @xfer_write_non_minor_identity_with_mask_scalable(
 // CHECK-SAME:        %[[VEC:.*]]: vector<4x[8]xi16>,
 // CHECK-SAME:        %[[MEM:.*]]: memref<1x4x?x1xi16>,
-// CHECK-SAME:        %[[MASK:.*]]: vector<4x[8]xi1>) {
-// CHECK:             %[[C0:.*]] = arith.constant 0 : index
+// CHECK-SAME:        %[[MASK:.*]]: vector<4x[8]xi1>
 // CHECK:             %[[BC_1:.*]] = vector.broadcast %[[VEC]] : vector<4x[8]xi16> to vector<1x4x[8]xi16>
 // CHECK:             %[[BC_2:.*]] = vector.broadcast %[[MASK]] : vector<4x[8]xi1> to vector<1x4x[8]xi1>
 // CHECK:             %[[TRANSPOSE_1:.*]] =  vector.transpose %[[BC_2]], [1, 2, 0] : vector<1x4x[8]xi1> to vector<4x[8]x1xi1>
 // CHECK:             %[[TRANSPOSE_2:.*]] =  vector.transpose %[[BC_1]], [1, 2, 0] : vector<1x4x[8]xi16> to vector<4x[8]x1xi16>
 // CHECK:             vector.transfer_write %[[TRANSPOSE_2]], %[[MEM]]{{.*}}, %[[TRANSPOSE_1]] {in_bounds = [true, true, true]} : vector<4x[8]x1xi16>, memref<1x4x?x1xi16>
-func.func @permutation_with_mask_xfer_write_scalable(
+func.func @xfer_write_non_minor_identity_with_mask_scalable(
     %vec: vector<4x[8]xi16>,
     %mem: memref<1x4x?x1xi16>,
-    %mask: vector<4x[8]xi1>){
+    %mask: vector<4x[8]xi1>,
+    %idx: index){
 
-  %c0 = arith.constant 0 : index
-  vector.transfer_write %vec, %mem[%c0, %c0, %c0, %c0], %mask {
+  vector.transfer_write %vec, %mem[%idx, %idx, %idx, %idx], %mask {
     in_bounds = [true, true],
     permutation_map = affine_map<(d0, d1, d2, d3) -> (d1, d2)>
   } : vector<4x[8]xi16>, memref<1x4x?x1xi16>
@@ -178,15 +180,16 @@ func.func @permutation_with_mask_xfer_write_scalable(
   return
 }
 
-// transfer_write in MaskOp case not supported.
-// CHECK-LABEL: func @masked_permutation_xfer_write_fixed_width
+// Masked version is not supported
+
+// CHECK-LABEL: func @xfer_write_non_minor_identity_masked
 //  CHECK-SAME:   %[[DEST:.*]]: tensor<?x?xf32>,
 //  CHECK-SAME:   %[[VEC:.*]]: vector<16xf32>,
 //  CHECK-SAME:   %[[IDX:.*]]: index,
 //  CHECK-SAME:   %[[MASK:.*]]: vector<16xi1>
 //   CHECK-NOT:   vector.transpose
 //       CHECK:   vector.mask %[[MASK]] { vector.transfer_write %[[VEC]], %[[DEST]]{{.*}} vector<16xf32>, tensor<?x?xf32> } : vector<16xi1> -> tensor<?x?xf32>
-func.func @masked_permutation_xfer_write_fixed_width(
+func.func @xfer_write_non_minor_identity_masked(
     %dest: tensor<?x?xf32>,
     %vec: vector<16xf32>,
     %idx: index,
@@ -201,21 +204,22 @@ func.func @masked_permutation_xfer_write_fixed_width(
   return %res : tensor<?x?xf32>
 }
 
-// CHECK-LABEL: func.func @masked_permutation_xfer_write_scalable(
+// CHECK-LABEL: func.func @xfer_write_non_minor_identity_masked_scalable
 //  CHECK-SAME:   %[[VEC:.*]]: vector<4x[8]xi16>,
 //  CHECK-SAME:   %[[DEST:.*]]: tensor<?x?x?x?xf32>,
-//  CHECK-SAME:   %[[MASK:.*]]: vector<4x[8]xi1>)
+//  CHECK-SAME:   %[[MASK:.*]]: vector<4x[8]xi1>
 //  CHECK-SAME:   -> tensor<?x?x?x?xf32> {
 //   CHECK-NOT:   vector.transpose
 //       CHECK:   vector.mask %[[MASK]] { vector.transfer_write %[[VEC]], %[[DEST]]{{.*}} : vector<4x[8]xi16>, tensor<?x?x?x?xf32> } : vector<4x[8]xi1> -> tensor<?x?x?x?xf32>
-func.func @masked_permutation_xfer_write_scalable(
+func.func @xfer_write_non_minor_identity_masked_scalable(
     %vec: vector<4x[8]xi16>,
     %dest: tensor<?x?x?x?xf32>,
-    %mask:  vector<4x[8]xi1>) -> tensor<?x?x?x?xf32> {
+    %mask:  vector<4x[8]xi1>,
+    %idx: index) -> tensor<?x?x?x?xf32> {
 
   %c0 = arith.constant 0 : index
   %res = vector.mask %mask {
-    vector.transfer_write %vec, %dest[%c0, %c0, %c0, %c0] {
+    vector.transfer_write %vec, %dest[%idx, %idx, %idx, %idx] {
       in_bounds = [true, true],
       permutation_map = affine_map<(d0, d1, d2, d3) -> (d1, d2)>
     } : vector<4x[8]xi16>, tensor<?x?x?x?xf32>
@@ -224,22 +228,23 @@ func.func @masked_permutation_xfer_write_scalable(
   return %res : tensor<?x?x?x?xf32>
 }
 
-// transfer_write in MaskOp case not supported.
-// CHECK-LABEL: func @masked_non_permutation_xfer_write_fixed_width
+// Masked version is not supported
+
+// CHECK-LABEL: func @xfer_write_non_minor_identity_masked_2
 //  CHECK-SAME:   %[[DEST:.*]]: tensor<?x?x?x?xf32>
 //  CHECK-SAME:   %[[VEC:.*]]: vector<14x8x16xf32>
-//  CHECK-SAME:   %[[IDX:.*]]: index) -> tensor<?x?x?x?xf32>
+//  CHECK-SAME:   %[[DIM:.*]]: index, %[[IDX:.*]]: index) -> tensor<?x?x?x?xf32>
 //   CHECK-NOT:   vector.broadcast
 //       CHECK:   vector.mask %0 { vector.transfer_write %[[VEC]], %[[DEST]]{{.*}} : vector<14x8x16xf32>, tensor<?x?x?x?xf32> } : vector<14x8x16xi1> -> tensor<?x?x?x?xf32>
-func.func @masked_non_permutation_xfer_write_fixed_width(
+func.func @xfer_write_non_minor_identity_masked_2(
     %dest : tensor<?x?x?x?xf32>,
     %vec : vector<14x8x16xf32>,
-    %dim : index) -> tensor<?x?x?x?xf32> {
+    %dim : index,
+    %idx: index) -> tensor<?x?x?x?xf32> {
 
-  %c0 = arith.constant 0 : index
   %mask = vector.create_mask %dim, %dim, %dim : vector<14x8x16xi1>
   %res = vector.mask %mask {
-    vector.transfer_write %vec, %dest[%c0, %c0, %c0, %c0] {
+    vector.transfer_write %vec, %dest[%idx, %idx, %idx, %idx] {
       in_bounds = [false, false, true],
       permutation_map = affine_map<(d0, d1, d2, d3) -> (d0, d1, d3)>
     } : vector<14x8x16xf32>, tensor<?x?x?x?xf32>
@@ -249,35 +254,35 @@ func.func @masked_non_permutation_xfer_write_fixed_width(
 }
 
 ///----------------------------------------------------------------------------------------
-/// vector.transfer_read
+/// [Pattern: TransferOpReduceRank (for leading 0 dim) + 
+///           TransferReadPermutationLowering (for transposed minor identity)]
+///
+/// IN: vector.transfer_read
+///     (_transposed_ minor identity permutation map, with 0 or more broadcast dims)
+/// OUT: vector.transpose + vector.transfer_write
+///     (minor identity permutation map with 0 or more leading broadcast dims)
 ///----------------------------------------------------------------------------------------
-/// Input:
-///   * vector.transfer_read op with a permutation map
-/// Output:
-///   * vector.transfer_read with a permutation map composed of leading zeros followed by a minor identiy +
-///     vector.transpose op
+/// TODO: Inner broadcast dim - see also the block at the bottom of this file
 
-// CHECK-LABEL:   func.func @permutation_with_mask_xfer_read_fixed_width(
+// CHECK-LABEL:   func.func @xfer_read_minor_identity_tranposed_with_mask
 // CHECK-SAME:      %[[MEM:.*]]: memref<?x?xf32>,
-// CHECK-SAME:      %[[IDX_1:.*]]: index,
-// CHECK-SAME:      %[[IDX_2:.*]]: index) -> vector<8x4x2xf32> {
-// CHECK:           %[[C0:.*]] = arith.constant 0 : index
+// CHECK-SAME:      %[[DIM_1:.*]]: index, %[[DIM_2:.*]]: index, %[[IDX:.*]]: index) -> vector<8x4x2xf32> {
 // CHECK:           %[[PASS_THROUGH:.*]] = arith.constant 0.000000e+00 : f32
-// CHECK:           %[[MASK:.*]] = vector.create_mask %[[IDX_2]], %[[IDX_1]] : vector<2x4xi1>
-// CHECK:           %[[T_READ:.*]] = vector.transfer_read %[[MEM]]{{\[}}%[[C0]], %[[C0]]], %[[PASS_THROUGH]], %[[MASK]] {in_bounds = [true, true]} : memref<?x?xf32>, vector<2x4xf32>
+// CHECK:           %[[MASK:.*]] = vector.create_mask %[[DIM_2]], %[[DIM_1]] : vector<2x4xi1>
+// CHECK:           %[[T_READ:.*]] = vector.transfer_read %[[MEM]]{{\[}}%[[IDX]], %[[IDX]]], %[[PASS_THROUGH]], %[[MASK]] {in_bounds = [true, true]} : memref<?x?xf32>, vector<2x4xf32>
 // CHECK:           %[[BCAST:.*]] = vector.broadcast %[[T_READ]] : vector<2x4xf32> to vector<8x2x4xf32>
 // CHECK:           %[[TRANSPOSE:.*]] = vector.transpose %[[BCAST]], [0, 2, 1] : vector<8x2x4xf32> to vector<8x4x2xf32>
 // CHECK:           return %[[TRANSPOSE]] : vector<8x4x2xf32>
-func.func @permutation_with_mask_xfer_read_fixed_width(
+func.func @xfer_read_minor_identity_tranposed_with_mask(
     %mem: memref<?x?xf32>,
     %dim_1: index,
-    %dim_2: index) -> (vector<8x4x2xf32>) {
+    %dim_2: index,
+    %idx: index) -> (vector<8x4x2xf32>) {
 
-  %c0 = arith.constant 0 : index
-  %cst_0 = arith.constant 0.000000e+00 : f32
+  %pad = arith.constant 0.000000e+00 : f32
 
   %mask = vector.create_mask %dim_2, %dim_1 : vector<2x4xi1>
-  %res = vector.transfer_read %mem[%c0, %c0], %cst_0, %mask {
+  %res = vector.transfer_read %mem[%idx, %idx], %pad, %mask {
     in_bounds = [true, true, true],
     permutation_map = affine_map<(d0, d1) -> (0, d1, d0)>
   } : memref<?x?xf32>, vector<8x4x2xf32>
@@ -285,27 +290,25 @@ func.func @permutation_with_mask_xfer_read_fixed_width(
   return %res : vector<8x4x2xf32>
 }
 
-// CHECK-LABEL:   func.func @permutation_with_mask_xfer_read_scalable(
+// CHECK-LABEL:   func.func @xfer_read_minor_identity_tranposed_with_mask_scalable(
 // CHECK-SAME:      %[[MEM:.*]]: memref<?x?xf32>,
-// CHECK-SAME:      %[[IDX_1:.*]]: index,
-// CHECK-SAME:      %[[IDX_2:.*]]: index) -> vector<8x[4]x2xf32> {
-// CHECK:           %[[C0:.*]] = arith.constant 0 : index
-// CHECK:           %[[PASS_THROUGH:.*]] = arith.constant 0.000000e+00 : f32
-// CHECK:           %[[MASK:.*]] = vector.create_mask %[[IDX_2]], %[[IDX_1]] : vector<2x[4]xi1>
-// CHECK:           %[[T_READ:.*]] = vector.transfer_read %[[MEM]]{{\[}}%[[C0]], %[[C0]]], %[[PASS_THROUGH]], %[[MASK]] {in_bounds = [true, true]} : memref<?x?xf32>, vector<2x[4]xf32>
+// CHECK-SAME:      %[[DIM_1:.*]]: index, %[[DIM_2:.*]]: index, %[[IDX:.*]]: index) -> vector<8x[4]x2xf32> {
+// CHECK:           %[[PAD:.*]] = arith.constant 0.000000e+00 : f32
+// CHECK:           %[[MASK:.*]] = vector.create_mask %[[DIM_2]], %[[DIM_1]] : vector<2x[4]xi1>
+// CHECK:           %[[T_READ:.*]] = vector.transfer_read %[[MEM]]{{\[}}%[[IDX]], %[[IDX]]], %[[PAD]], %[[MASK]] {in_bounds = [true, true]} : memref<?x?xf32>, vector<2x[4]xf32>
 // CHECK:           %[[BCAST:.*]] = vector.broadcast %[[T_READ]] : vector<2x[4]xf32> to vector<8x2x[4]xf32>
 // CHECK:           %[[TRANSPOSE:.*]] = vector.transpose %[[BCAST]], [0, 2, 1] : vector<8x2x[4]xf32> to vector<8x[4]x2xf32>
 // CHECK:           return %[[TRANSPOSE]] : vector<8x[4]x2xf32>
-func.func @permutation_with_mask_xfer_read_scalable(
+func.func @xfer_read_minor_identity_tranposed_with_mask_scalable(
     %mem: memref<?x?xf32>,
     %dim_1: index,
-    %dim_2: index) -> (vector<8x[4]x2xf32>) {
+    %dim_2: index,
+    %idx: index) -> (vector<8x[4]x2xf32>) {
 
-  %c0 = arith.constant 0 : index
-  %cst_0 = arith.constant 0.000000e+00 : f32
+  %pad = arith.constant 0.000000e+00 : f32
 
   %mask = vector.create_mask %dim_2, %dim_1 : vector<2x[4]xi1>
-  %res = vector.transfer_read %mem[%c0, %c0], %cst_0, %mask {
+  %res = vector.transfer_read %mem[%idx, %idx], %pad, %mask {
     in_bounds = [true, true, true],
     permutation_map = affine_map<(d0, d1) -> (0,...
[truncated]

``````````

</details>


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


More information about the Mlir-commits mailing list