jayfoad wrote: Makes sense to me, but then I suggested it, so hopefully someone else will take a look too. It might also make sense to refactor `getNumWavesPerEUWithNumVGPRs` or others to use the new `getAllocatedNumVGPRBlocks`. https://github.com/llvm/llvm-project/pull/84161