[PATCH] D31048: AMDGPU: Add all atomicrmw fields to atomic.inc/dec

[Konstantin Zhuravlyov via Phabricator via llvm-commits]

kzhuravl added inline comments.


================
Comment at: include/llvm/IR/IntrinsicsAMDGPU.td:253-255
+  llvm_i32_ty, // ordering
+  llvm_i32_ty, // scope
+  llvm_i1_ty], // isVolatile
----------------
atomicrmw has these fields in the following order:
[volatile], [singlethread], <ordering>

should we mirror it in the same order?


https://reviews.llvm.org/D31048