[Mlir-commits] [mlir] [mlir][EmitC] Expand the MemRefToEmitC pass - Lowering `CopyOp` (PR #151206)

[Gil Rapaport]

================
@@ -57,31 +77,30 @@ struct ConvertMemRefToEmitCPass
     mlir::ModuleOp module = getOperation();
     module.walk([&](mlir::emitc::CallOpaqueOp callOp) {
----------------
aniragil wrote:

Not sure this works as expected: the same module may contain both `malloc` and `memcpy` calls. The code seems to settle for a single call that needs an `include` directive, and IINM not necessarily the right one (as `isExpectedIncludeOp` doesn't compare `includeOp` against the specific `callOp.getCallee()`). Am I missing something? (In any case please add such a test).

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