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

[Kareem Ergawy]

================
@@ -6030,7 +6030,7 @@ static bool isTargetDeviceOp(Operation *op) {
   // by taking it in as an operand, so we must always lower these in
   // some manner or result in an ICE (whether they end up in a no-op
   // or otherwise).
-  if (mlir::isa<omp::ThreadprivateOp>(op))
+  if (mlir::isa<omp::ThreadprivateOp, omp::GroupprivateOp>(op))
----------------
ergawy wrote:

Should we `return false` when the `DeclareTargetDeviceTypeAttr` is `host`?

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