[Mlir-commits] [mlir] [mlir][acc] Add acc.on_device op (PR #208096)

[Ivan R. Ivanov]

================
@@ -3713,6 +3713,33 @@ def OpenACC_WaitOp : OpenACC_Op<"wait", [AttrSizedOperandSegments]> {
   let hasVerifier = 1;
 }
 
+//===----------------------------------------------------------------------===//
+// acc_on_device runtime call
+//===----------------------------------------------------------------------===//
+
+def OpenACC_OnDeviceOp : OpenACC_Op<"on_device", [NoMemoryEffect]> {
----------------
ivanradanov wrote:

Yes this is a _very_ good point, thank you. I was only thinking about when we are already in an IsolatedFromAbove region, but that's not the case at the start of the pipeline - cse will gladly merge these. I think I will just remove the memory interface.

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