[Mlir-commits] [mlir] [flang][MLIR][OpenMP] Add support for `target update` directive. (PR #75047)

[Akash Banerjee]

================
@@ -1658,4 +1658,77 @@ func.func @omp_target_exit_data(%map1: memref<?xi32>) {
   return
 }
 
+// -----
+
+func.func @omp_target_update_data_if(%if_cond : i1) {
+  // expected-error @below {{`if` clause can appear at most once in the expansion of the oilist directive}}
+  omp.target_update_data if(%if_cond : i1) if(%if_cond : i1)
+  return
+}
----------------
TIFitis wrote:

The op builder wouldn't let you create an op with two nowait or if clause or whatever that isn't specified in the op definition.

By your argument why not test that it doesn't work for other random clauses as well?

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