[PATCH] D97218: [AMDGPU] Set threshold for regbanks reassign pass

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 23 12:28:30 PST 2021


rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/GCNRegBankReassign.cpp:668
 
   for (MCRegister Reg : RC->getRegisters()) {
     // Check occupancy limit.
----------------
arsenm wrote:
> Can you redo this search process in terms of regunits instead? 
I will need to supply a physreg for VRM at the end. Plus all these isAllocatable() et all checks are not for reg units.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97218



More information about the llvm-commits mailing list