pravinjagtap accepted this revision as: pravinjagtap.
pravinjagtap added a comment.

LGTM, In the context of atomic optimizer, when we extend the support for 
`atomicInc` and `atomicDec`, I think, the visitor of AtomicRMWInst will find 
the potential candidate for optimization now instead of visitor of 
IntrinsicInst because of this change. Right ?


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

https://reviews.llvm.org/D137524

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

Reply via email to