[Mlir-commits] [mlir] [OpenMP][mlir] Add Groupprivate op in omp dialect. (PR #162704)

[Sergio Afonso]

================

----------------
skatrak wrote:

We should add the `SymbolUserOpInterface` to the operation and then implement `LogicalResult GroupprivateOp::verifySymbolUses(SymbolTableCollection &symbolTable)`. That's where the verification logic to check the symbol exists and is not a function should go.
```suggestion
def GroupprivateOp : OpenMP_Op<"groupprivate", [Pure, DeclareOpInterfaceMethods<SymbolUserOpInterface>]> {
```
That should also remove the need for a custom verifier, since that's the only thing that is currently checked.

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