[llvm] [CodeGen] Use MCRegister instead of MCPhysReg in RegisterMaskPair. NFC (PR #123688)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 20 22:03:14 PST 2025


topperc wrote:

> Most of these are really MCPhysReg. There is a slight difference since MCPhysReg is 16-bit and MCRegister is 32

My understanding was that MCPhysReg was 16-bit primarily to reduce static data size in *GenRegisterInfo.inc. I'm probably the one who made it 16-bit. It doesn't provide any size savings in `RegisterMaskPair` because `LaneMask` has 32-bit alignment.

The `MachineBasicBlock::addLiveIn` function that creates `RegisterMaskPair` uses MCRegister in its interface. Are you suggesting to change that interface.

https://github.com/llvm/llvm-project/pull/123688


More information about the llvm-commits mailing list