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

[Sergio Afonso]

================
@@ -256,7 +256,8 @@ mlir::FlatSymbolRefAttr getOrGenImplicitDefaultDeclareMapper(
       /*partialMap=*/true);
 
   clauseMapVars.emplace_back(mapOp);
-  mlir::omp::DeclareMapperInfoOp::create(firOpBuilder, loc, clauseMapVars);
+  mlir::omp::DeclareMapperInfoOp::create(firOpBuilder, loc, clauseMapVars,
----------------
skatrak wrote:

Nit: It might be a good idea to pass an instance of the auto-generated `DeclareMapperInfoOperands` struct here, in place of using the bare `clauseMapVars`, so we don't need to keep updating this whenever we add operands to that operation that need to remain empty.

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