arsenm wrote:

> @b-sumner has useful context on this. I'll try not to speak for him but it 
> sounds like the block deleted here has the right semantics for opencl, where 
> "seqcst" has some special meaning and generally the semantics don't totally 
> make sense to me. Suggest we amend this to "if opencl, do this thing, 
> otherwise leave it alone"

I'd rather not infect this with poorly understood language based logic. The 
one-as should be more aggressive and safe to ignore.

> (can't see how to remove the green tick from the review)

"Request changes"

https://github.com/llvm/llvm-project/pull/120095
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to