[PATCH] D28566: Disable Callee Saved Registers
Matthias Braun via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 1 11:00:50 PST 2017
MatzeB added inline comments.
Comment at: include/llvm/CodeGen/RegisterClassInfo.h:56
+ // runOnFunction() call. Used only to determine if an update was made.
+ SmallVector<uint8_t, 16> CalleeSavedRegs;
> MatzeB wrote:
> > There are several targets out there with more than 256 registers! AMDGPU already has > 1000, and I know of even crazier out-of-tree targets.
> The size here is just the initial size. The size is updated according to the number of registers.
> Notice that following code exmaple that actually changes the size (using push_back):
> SmallVector<uint8_t, 16> ActualCalleeSaved;
> for (unsigned I = 0; unsigned Reg = CSR[I]; ++I)
> CalleeSavedRegs = ActualCalleeSaved;
You hold a register number in an uint8_t with UINT8_MAX==256, don't you?
You should use MCPhysReg (or `unsigned` if you want to store phys+virtual regs)
More information about the llvm-commits