[Mlir-commits] [mlir] [mlir][acc] Introduce acc data bounds accessors (PR #156545)

[Razvan Lupusoru]

================
@@ -1180,30 +1258,48 @@ def OpenACC_PrivateRecipeOp
       1. The initializer region specifies how to allocate and initialize a new
          private value. For example in Fortran, a derived-type might have a
          default initialization. The region has an argument that contains the
-         value that need to be privatized. This is useful if the type is not
-         known at compile time and the private value is needed to create its
-         copy.
+         original value that needs to be privatized, followed by bounds arguments
+         (if any) in order from innermost to outermost dimension. The region
+         must yield the privatized copy.
       2. The destroy region specifies how to destruct the value when it reaches
-         its end of life. It takes the privatized value as argument.
+         its end of life. It takes the original value, the privatized value, and
----------------
razvanlupusoru wrote:

> At one point I think the 'verifier' should start doing some level of failures for this signature though, which would help catch any issues I have.

I agree. I have not added verifier yet because Flang does not yet comply with this.

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