[flang-commits] [flang] [llvm] [flang][OpenMP] Overhaul implementation of ATOMIC construct (PR #137852)

[via flang-commits]

================
@@ -777,5 +777,153 @@ inline bool checkForSymbolMatch(const Fortran::semantics::SomeExpr *lhs,
   }
   return false;
 }
+
+namespace operation {
+
+enum class Operator {
----------------
NimishMishra wrote:

The standard also has PREVIOUS and NEXT as possible intrinsic procedures. Do we want to add a TODO for the same here? 

"intrinsic-procedure-name is one of MAX, MIN, IAND, IOR, IEOR, PREVIOUS, or NEXT" (sec 6.3.3)

Currently, this patch exits with "A call to this function is not a valid ATOMIC UPDATE operation". gfortran also does not accept this, but I think we should since we are working with v6.0

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