[PATCH] D121914: [AMDGPU] Stop using getMinimalPhysRegClass in LowerFormalArguments

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 17 07:54:09 PDT 2022


foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:2551
     Register Reg = VA.getLocReg();
-    const TargetRegisterClass *RC = TRI->getMinimalPhysRegClass(Reg, VT);
     EVT ValVT = VA.getValVT();
----------------
As an alternative, I did try implementing a getMaximalAllocatablePhysRegClass and using it here. That seemed to cause other problems, but I can pursue it further if you think it's the right way to go.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121914



More information about the llvm-commits mailing list