[llvm] [NVPTX] Add syncscope support for cmpxchg (PR #140812)

via llvm-commits llvm-commits at lists.llvm.org
Fri May 30 09:33:37 PDT 2025


================
@@ -131,7 +131,7 @@ define void @test(ptr %dp0, ptr addrspace(1) %dp1, ptr addrspace(3) %dp3, bfloat
 ; CHECKPTX71-NEXT:    shl.b32 %r51, %r50, %r17;
 ; CHECKPTX71-NEXT:    and.b32 %r52, %r57, %r18;
 ; CHECKPTX71-NEXT:    or.b32 %r53, %r52, %r51;
-; CHECKPTX71-NEXT:    atom.relaxed.shared.cas.b32 %r21, [%r16], %r57, %r53;
+; CHECKPTX71-NEXT:    atom.relaxed.sys.shared.cas.b32 %r21, [%r16], %r57, %r53;
----------------
gonzalobg wrote:

This test has 4 different atomic operations back to back.
I think it'd be good to split it into 4 different tests, otherwise it is a bit hard to review (e.g. I see a few non-atomic ld.global and ld in between, which I'm not sure where they are coming from).
Could that be done in a separate PR that's merged before this one? (and then this one rebased on top?)
@Artem-B thoughts?

https://github.com/llvm/llvm-project/pull/140812


More information about the llvm-commits mailing list