[llvm-commits] [patch] Remove the TargetRegisterClass from CalleeSavedInfo
espindola at google.com
Wed Jun 2 12:37:58 PDT 2010
> This kind of thing is usually not what you want. Please do as you did for ARM: Mips::CPURegsRegisterClass->contains(Reg)
> Same issue with the PPC and SystemZ changes. As a general rule it is a mistake to test register class identity. You almost always want "is register in", or "is a subclass of" comparisons.
Good point. An updated patch is attached.
Rafael Ávila de Espíndola
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 18035 bytes
Desc: not available
More information about the llvm-commits