[llvm] [AMDGPU] Generate waterfall for calls with SGPR(inreg) argument (PR #146997)

Juan Manuel Martinez CaamaƱo via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 22 01:12:49 PDT 2025


https://github.com/jmmartinez commented:

The patch looks ok to me, but I'd like some other reviewer to take a look too. @arsenm @shiltian 

Just some naming concerns:

* `createWaterfall` doesn't create the waterfall code for any instruction, only for calls. So maybe we can name it `loadCallScalarOperandsFromVGPR` ?
* `loadMBUFScalarOperandsFromVGPR` creates the waterfall code for more than MBUF instructions. So maybe we can rename it `loadScalarOperandsFromVGPR`

Or switch the names to something related to `waterfall`; but be consistent across the two functions since they are closely related.

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


More information about the llvm-commits mailing list