[Mlir-commits] [flang] [mlir] [mlir][OpenMP] Add iterator support to map/motion clause (PR #197047)

[Sergio Afonso]

================
@@ -4371,7 +4371,8 @@ genOpenMPDeclareMapperImpl(lower::AbstractConverter &converter,
   List<Clause> clauses = makeClauses(construct.v.Clauses(), semaCtx);
   ClauseProcessor cp(converter, semaCtx, clauses);
   cp.processMap(loc, stmtCtx, clauseOps);
-  mlir::omp::DeclareMapperInfoOp::create(firOpBuilder, loc, clauseOps.mapVars);
+  mlir::omp::DeclareMapperInfoOp::create(firOpBuilder, loc, clauseOps.mapVars,
----------------
skatrak wrote:

Nit: Here too, we should be able to just pass `clauseOps`. I think what's missing is a custom builder to `DeclareMapperOp`. This is a common pattern in the OpenMP dialect, there's plenty of examples you can use as reference:
```
let builders = [
  OpBuilder<(ins CArg<"const DeclareMapperInfoOperands &">:$clauses)>
];
```

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