[PATCH] D137361: IR: Add atomicrmw inc and dec

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

I'm very much in favor. One question, should it be `uinc` and `udec`, just for consistency? I guess we can simply add nvptx support afterwards too (@tra).



================
Comment at: llvm/docs/LangRef.rst:10524
+-  inc: ``*ptr = (old >= val) ? 0 : (*ptr + 1)``
+-  dec: ``*ptr = (*ptr == 0) || (*ptr > v)) ? val : (val - 1)``
 
----------------
Mark the comparisons as unsigned.


================
Comment at: llvm/include/llvm/IR/Instructions.h:770
+    /// Decrement one until a minimum value or zero.
+    /// *p = (old == 0) || (old > v)) ? v : (v - 1)
+    Dec,
----------------
Mark the comparisons as unsigned.


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

https://reviews.llvm.org/D137361