[PATCH] D28566: Disable Callee Saved Registers
Matthias Braun via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 30 11:10:05 PST 2017
MatzeB added a comment.
This is going to make the code confusing. We keep TRI::getCalleeSavedRegs() just that now it doesn't always give you the right answer and you have to check that other thing as well. We really should find a way so that getCalleeSavedRegs() (or some other new function that replaces the former) gives you a single correct answer.
Comment at: include/llvm/CodeGen/RegisterClassInfo.h:56
+ // runOnFunction() call. Used only to determine if an update was made.
+ SmallVector<uint8_t, 16> CalleeSavedRegs;
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.
More information about the llvm-commits