[PATCH] D112982: [MLIR][OpenMP] Added omp.atomic.update

[Shraiysh via Phabricator via llvm-commits]

shraiysh added a comment.

Thanks for the review @kiranchandramohan.

In D112982#3160888 <https://reviews.llvm.org/D112982#3160888>, @kiranchandramohan wrote:

> What will prevent the builder from building operations which has the errors that you are catching during parsing?

The BinOp error will be taken care of by the AtomicBinOpKindAttr. An integer value from the allowed ones will be forced because of that constraint.

The other error - about X being in RHS - that is taken care of by design as the in memory structure has X, expr, binOp and isXBinOpExpr. It is not possible to run into that error while building the operation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112982