[Mlir-commits] [mlir] [MLIR][EmitC] Add support for pointer-array types in the TypeConverter and related MemRef-to-EmitC operations, and update the C emitter. (PR #160159)

[Matthias Gehre]

================
@@ -51,9 +57,15 @@ module @globals {
   // CHECK-LABEL: use_global
   func.func @use_global() {
     // CHECK-NEXT: emitc.get_global @public_global : !emitc.array<3x7xf32>
+    // CHECK-NEXT: "emitc.constant"() <{value = 0 : index}> : () -> index
----------------
mgehre-amd wrote:

Hm, this is working fine for us. Maybe we are using `memref.alloca` to make local variables of array type instead of using `alloc` to make dynamic allocations.
Will our use case continue to work? It looks like this PR maps memrefs to an different C type, so our calls to external C functions would break?

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