[llvm] [AMDGPU] Change the immediate operand of s_waitcnt_depctr / s_wait_alu (PR #169378)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 25 00:10:22 PST 2025


jayfoad wrote:

> Is there any way you can keep the undefined/reserved bits in the mask as 1?

Yes, I could abandon this patch or break compatibility with SP3. But the whole point of this patch is that I thought on the whole it was better to change the bits to 0 and keep compatibility with SP3.

> While it's probably fine if they are 0 they have always been set, so we need to double check there aren't any unintended consequences of not setting them.

I can run some testing on real hardware to smoke out any unintended consequences.

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


More information about the llvm-commits mailing list