[llvm] [AMDGPU] fix asyncmark soft waitcnt bug (PR #184851)

via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 6 08:40:18 PST 2026


vporpo wrote:

I think you are right, we should keep this precondition. Having any other type of instruction in the sequence other than waitcnts would probably make the transformation illegal. 

One change that would help enforce this rule in the future is to make a small change in the structure of the code like this:
```
} else if (Opcode == AMDGPU::S_WAITCNT_VSCNT) {
...
} else {
   llvm_unreachable("Expected a sequence of waitcnts!");
}
```
and a similar change in the other overload of applyPreexistingWaitcnts().


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


More information about the llvm-commits mailing list