[llvm-commits] [patch] Remove the TargetRegisterClass from CalleeSavedInfo

Jakob Stoklund Olesen stoklund at 2pi.dk
Wed Jun 2 12:48:12 PDT 2010


On Jun 2, 2010, at 12:37 PM, Rafael Espindola wrote:

>> 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.

Looks good.





More information about the llvm-commits mailing list