[PATCH] D78559: [MLIR] Verify there are no side-effecting ops in GenericAtomicRMWOp body.

[Stephan Herhut via Phabricator via llvm-commits]

herhut accepted this revision.
herhut added inline comments.
This revision is now accepted and ready to land.


================
Comment at: mlir/lib/Dialect/StandardOps/IR/Ops.cpp:520
+  if (hasSideEffects)
+    return op.emitError("body of 'generic_atomic_rmw' should contain "
+                        "only operations with no side effects");
----------------
It would be extra nice if we could get a note where the side-effecting operation is. But that is icing on the cake.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78559/new/

https://reviews.llvm.org/D78559