[PATCH] D118089: [AMDGPU] Validate dst and src2 non-overlapping restriction in asm

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 24 16:46:47 PST 2022


arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:3643
+
+  if (RUCount <= 8) // 4 VGPRs or less
+    return true;
----------------
rampitec wrote:
> arsenm wrote:
> > Can't you get the width directly from the operand register class without making an assumption about the number of register units?
> I cannot get RC itself.
It's in the MCInstrDesc which you have


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

https://reviews.llvm.org/D118089



More information about the llvm-commits mailing list