Skip to content

Commit 58a7f32

Browse files
committed
Revert "Modify transform-op-hoist-pad/transform-op-hoist-pad-build-packing-loop-nest.mlir:"
This reverts commit 390a8e6.
1 parent e607c87 commit 58a7f32

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

mlir/test/Dialect/Linalg/transform-op-hoist-pad-build-packing-loop-nest.mlir

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ func.func @pad_and_hoist_rhs(
66
%arg0: tensor<24x12xf32>, %arg1: tensor<12x25xf32>, %arg2: tensor<24x25xf32>)
77
-> tensor<24x25xf32>
88
{
9-
// expected-note @below {{target op}}
109
%0 = linalg.matmul ins(%arg0, %arg1 : tensor<24x12xf32>, tensor<12x25xf32>) outs(%arg2 : tensor<24x25xf32>) -> tensor<24x25xf32>
1110
func.return %0 : tensor<24x25xf32>
1211
}
@@ -25,11 +24,10 @@ module attributes {transform.with_named_sequence} {
2524

2625
// In this case, the pad op is actually empty: we only tile the first dimension
2726
// and it does not have an impact on the RHS operand.
28-
// expected-error @below {{could not find a producer for operand number: 1}}
2927
%pad = transform.get_producer_of_operand %matmul_padded[1]
3028
: (!transform.any_op) -> !transform.any_op
3129

32-
// We do not even reach this transform op.
30+
// expected-error @below {{requires exactly 2 non-null handles}}
3331
transform.structured.hoist_pad.build_packing_loop_nest %pad above %loops_l1
3432
: (!transform.any_op, !transform.any_op) -> !transform.any_op
3533
transform.yield

mlir/test/Dialect/Linalg/transform-op-hoist-pad.mlir

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ func.func @pad_and_hoist_rhs(
44
%arg0: tensor<24x12xf32>, %arg1: tensor<12x25xf32>, %arg2: tensor<24x25xf32>)
55
-> tensor<24x25xf32>
66
{
7-
// expected-note @below {{target op}}
7+
// expected-note @below {{payload operation}}
88
%0 = linalg.matmul ins(%arg0, %arg1 : tensor<24x12xf32>, tensor<12x25xf32>) outs(%arg2 : tensor<24x25xf32>) -> tensor<24x25xf32>
99
func.return %0 : tensor<24x25xf32>
1010
}
@@ -25,7 +25,7 @@ module attributes {transform.with_named_sequence} {
2525

2626
// In this case, the pad op is actually empty: we only tile the first dimension
2727
// and it does not have an impact on the RHS operand.
28-
// expected-error @below {{could not find a producer for operand number: 1}}
28+
// expected-error @below {{incompatible payload operation name}}
2929
%pad = transform.get_producer_of_operand %matmul_padded[1]
3030
: (!transform.any_op) -> !transform.op<"tensor.pad">
3131

0 commit comments

Comments
 (0)