[PATCH] D103800: [AMDGPU] Add VReg_192/VReg_224 support for MIMG instructions

Thu Jun 24 06:17:31 PDT 2021

foad added inline comments.

================
Comment at: llvm/lib/Target/AMDGPU/MIMGInstructions.td:765-766
                   !listconcat(lhs.List, [MIMGAddrSize<dw, !empty(lhs.List)>]),
-                  !if(!or(!eq(dw, 3), !eq(dw, 5)), dw, !add(dw, 1))>,
-                  // we still want _V4/_V8 for codegen with 3/5 dwords
+                  !if(!or(!eq(dw, 3), !eq(dw, 5)), dw,
+                      !if(!or(!eq(dw, 6), !eq(dw, 7)), lhs.Min, !add(dw, 1)))>,
+                  // we still want _V4 for 3 dwords codegen
----------------
critson wrote:
> foad wrote:
> > I have never understood this bit and I understand it even less now that 6 and 7 are treated differently from 3 and 5.
> I have rewritten this to be more explicit and hence hopefully easier to understand.
> Please let me know your opinion.
Yes I think that is better thanks. I will ponder further simplifications...

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103800/new/

https://reviews.llvm.org/D103800