Skip to content

Commit cd499f9

Browse files
committed
fixup! [mlir][memref] Revert llvm#140730
Add a TODO to re-use check lines, remove LIT variables (for consistency with existing examples)
1 parent c6cedb2 commit cd499f9

File tree

1 file changed

+15
-12
lines changed

1 file changed

+15
-12
lines changed

mlir/test/Conversion/MemRefToLLVM/memref-to-llvm.mlir

Lines changed: 15 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@
66
// because the conversion target is set up differently.
77
// RUN: mlir-opt --convert-to-llvm="filter-dialects=memref" --split-input-file %s | FileCheck --check-prefixes=ALL,CHECK-INTERFACE %s
88

9+
// TODO: In some (all?) cases, CHECK and CHECK-INTERFACE outputs are identical.
10+
// Use a common prefix instead (e.g. ALL).
11+
912
// CHECK-LABEL: func @view(
1013
// CHECK: %[[ARG0F:.*]]: index, %[[ARG1F:.*]]: index, %[[ARG2F:.*]]: index
1114
// CHECK-INTERFACE-LABEL: func @view(
@@ -133,20 +136,20 @@ func.func @view_empty_memref(%offset: index, %mem: memref<0xi8>) {
133136
// -----
134137

135138
// ALL-LABEL: func.func @view_memref_as_rank0(
136-
// ALL-SAME: %[[ARG0:.*]]: index,
137-
// ALL-SAME: %[[ARG1:.*]]: memref<2xi8>) {
139+
// ALL-SAME: %[[OFFSET:.*]]: index,
140+
// ALL-SAME: %[[MEM:.*]]: memref<2xi8>) {
138141
func.func @view_memref_as_rank0(%offset: index, %mem: memref<2xi8>) {
139142

140-
// ALL: %[[VAL_0:.*]] = builtin.unrealized_conversion_cast %[[ARG0]] : index to i64
141-
// ALL: %[[VAL_1:.*]] = builtin.unrealized_conversion_cast %[[ARG1]] : memref<2xi8> to !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
142-
// ALL: %[[VAL_2:.*]] = llvm.mlir.poison : !llvm.struct<(ptr, ptr, i64)>
143-
// ALL: %[[VAL_3:.*]] = llvm.extractvalue %[[VAL_1]][0] : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
144-
// ALL: %[[VAL_4:.*]] = llvm.insertvalue %[[VAL_3]], %[[VAL_2]][0] : !llvm.struct<(ptr, ptr, i64)>
145-
// ALL: %[[VAL_5:.*]] = llvm.extractvalue %[[VAL_1]][1] : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
146-
// ALL: %[[VAL_6:.*]] = llvm.getelementptr %[[VAL_5]]{{\[}}%[[VAL_0]]] : (!llvm.ptr, i64) -> !llvm.ptr, i8
147-
// ALL: %[[VAL_7:.*]] = llvm.insertvalue %[[VAL_6]], %[[VAL_4]][1] : !llvm.struct<(ptr, ptr, i64)>
148-
// ALL: %[[VAL_8:.*]] = llvm.mlir.constant(0 : index) : i64
149-
// ALL: %[[VAL_9:.*]] = llvm.insertvalue %[[VAL_8]], %[[VAL_7]][2] : !llvm.struct<(ptr, ptr, i64)>
143+
// ALL: builtin.unrealized_conversion_cast %[[OFFSET]] : index to i64
144+
// ALL: builtin.unrealized_conversion_cast %[[MEM]] : memref<2xi8> to !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
145+
// ALL: llvm.mlir.poison : !llvm.struct<(ptr, ptr, i64)>
146+
// ALL: llvm.extractvalue %{{.*}}[0] : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
147+
// ALL: llvm.insertvalue %{{.*}}, %{{.*}}[0] : !llvm.struct<(ptr, ptr, i64)>
148+
// ALL: llvm.extractvalue %{{.*}}[1] : !llvm.struct<(ptr, ptr, i64, array<1 x i64>, array<1 x i64>)>
149+
// ALL: llvm.getelementptr %{{.*}}[%{{.*}}] : (!llvm.ptr, i64) -> !llvm.ptr, i8
150+
// ALL: llvm.insertvalue %{{.*}}, %{{.*}}[1] : !llvm.struct<(ptr, ptr, i64)>
151+
// ALL: llvm.mlir.constant(0 : index) : i64
152+
// ALL: llvm.insertvalue %{{.*}}, %{{.*}}[2] : !llvm.struct<(ptr, ptr, i64)>
150153
%memref_view_bf16 = memref.view %mem[%offset][] : memref<2xi8> to memref<bf16>
151154

152155
return

0 commit comments

Comments
 (0)