[llvm] [AMDGPU] Support waterfall loop for SOffset in VBUFFER (PR #95249)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 12 06:51:00 PDT 2024


================
@@ -6718,6 +6718,13 @@ SIInstrInfo::legalizeOperands(MachineInstr &MI,
     if (SRsrc && !RI.isSGPRClass(MRI.getRegClass(SRsrc->getReg())))
       CreatedBB = loadMBUFScalarOperandsFromVGPR(*this, MI, {SRsrc}, MDT);
 
+    if (isMUBUF(MI) || isMTBUF(MI)) {
----------------
arsenm wrote:

We shouldn't really be restricting the opcodes for this. This should just be the behavior for any scalar required operand, including inline asm. Why wouldn't this cover images? The calling convention check above is also nonsense 

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


More information about the llvm-commits mailing list