[flang-commits] [flang] [mlir] [Flang] [OpenMP] Allow any type as argument to the FlushOp (PR #143844)

[via flang-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

Author: Thirumalai Shaktivel (Thirumalai-Shaktivel)

<details>
<summary>Changes</summary>

Fixes: #<!-- -->143842 

---
Full diff: https://github.com/llvm/llvm-project/pull/143844.diff


2 Files Affected:

- (modified) flang/test/Fir/fir-ops.fir (+8) 
- (modified) mlir/include/mlir/Dialect/OpenMP/OpenMPOps.td (+1-1) 


``````````diff
diff --git a/flang/test/Fir/fir-ops.fir b/flang/test/Fir/fir-ops.fir
index 9c444d2f4e0bc..56c5220487f5e 100644
--- a/flang/test/Fir/fir-ops.fir
+++ b/flang/test/Fir/fir-ops.fir
@@ -1015,3 +1015,11 @@ func.func @test_box_total_elements(%arg0: !fir.class<!fir.type<sometype{i:i32}>>
   %6 = arith.addi %2, %5 : index
   return %6 : index
 }
+
+// omp.flush operation
+// CHECK-LABEL: func.func @omp_flush
+func.func @omp_flush(%arg0: !fir.class<!fir.type<_QMmymodTmyt1{t:i32}>>) {
+  // CHECK: omp.flush(%{{.*}} : !fir.class<!fir.type<_QMmymodTmyt1{t:i32}>>)
+  omp.flush(%arg0 : !fir.class<!fir.type<_QMmymodTmyt1{t:i32}>>)
+  return
+}
diff --git a/mlir/include/mlir/Dialect/OpenMP/OpenMPOps.td b/mlir/include/mlir/Dialect/OpenMP/OpenMPOps.td
index 036c6a6e350a8..ac80926053a2d 100644
--- a/mlir/include/mlir/Dialect/OpenMP/OpenMPOps.td
+++ b/mlir/include/mlir/Dialect/OpenMP/OpenMPOps.td
@@ -889,7 +889,7 @@ def FlushOp : OpenMP_Op<"flush", clauses = [
     specified or implied.
   }] # clausesDescription;
 
-  let arguments = !con((ins Variadic<OpenMP_PointerLikeType>:$varList),
+  let arguments = !con((ins Variadic<AnyType>:$varList),
                        clausesArgs);
 
   // Override inherited assembly format to include `varList`.

``````````

</details>


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