shraiysh added a comment. In D111992#3080308 <https://reviews.llvm.org/D111992#3080308>, @kiranchandramohan wrote:
> LGTM. > > My preference is the following one. If you agree then please make the change, > otherwise, you can keep it as is. Also, wait a couple of days to see whether > others have comments. > > %8 = omp.atomic.read %addr hint(speculative, contended) > memory_order(seq_cst) : memref<i32> -> i32 Sure, will wait for other's comments. Meanwhile, I will update it with this syntax. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111992/new/ https://reviews.llvm.org/D111992 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
