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

Thu Jun 24 05:09:41 PDT 2021

critson marked an inline comment as done.
critson 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
----------------
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.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103800