[Mlir-commits] [mlir] [mlir][OpenMP] add attribute for privatization barrier (PR #140089)

[llvmlistbot at llvm.org]

================
@@ -1102,7 +1102,10 @@ class OpenMP_PrivateClauseSkip<
 
   let arguments = (ins
     Variadic<AnyType>:$private_vars,
-    OptionalAttr<SymbolRefArrayAttr>:$private_syms
+    OptionalAttr<SymbolRefArrayAttr>:$private_syms,
+    // Set this attribute if a barrier is needed after initialization and
+    // copying of lastprivate variables.
+    UnitAttr:$private_needs_barrier
----------------
NimishMishra wrote:

Small nit: This attribute is already within private clause. Is `private_needs_barrier` a better name? In OpenMPDialect.cpp too, `needsBarrier` is used, which is self-sufficient I think.

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