saiislam added a comment.

In D80804#2063451 <https://reviews.llvm.org/D80804#2063451>, @sameerds wrote:

> Actually, the question really is about why inc/dec are needed as separate 
> operations either as IR intrinsics or Clang builtins. Why not just expose a 
> __builtin_amdgcn_atomicrmw that takes a scope, and map it to the LLVM 
> atomicrmw? That would be way cleaner. The language can provide convenience 
> functions for inc/dec that internally call the rmw builtin.


At the moment, atomic inc/dec exist along with atomicrmw. This patch only aims 
to devise a way to make it (inc/dec) accessible from the language level.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80804



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to