[Mlir-commits] [mlir] [OpenMP][MLIR] Add `private` clause to `omp.target` (PR #91202)

[Sergio Afonso]

================
@@ -1262,7 +1308,8 @@ void TargetOp::build(OpBuilder &builder, OperationState &state,
       builder, state, clauses.ifVar, clauses.deviceVar, clauses.threadLimitVar,
       makeArrayAttr(ctx, clauses.dependTypeAttrs), clauses.dependVars,
       clauses.nowaitAttr, clauses.isDevicePtrVars, clauses.hasDeviceAddrVars,
-      clauses.mapVars);
+      clauses.mapVars, clauses.privateVars,
+      ArrayAttr::get(builder.getContext(), clauses.privatizers));
----------------
skatrak wrote:

Nit: It's best to use this helper whenever possible, which was created just to reduce verbosity in these constructors.
```suggestion
      makeArrayAttr(ctx, clauses.privatizers);
```

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