This is to avoid confusion with MachineRegisterInfo.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
I don't think this is an improvement. People got used to MRI (which is consistent with MII, MAI, etc.) and will continue to use it, which will introduce more inconsistencies.
This is to avoid confusion with MachineRegisterInfo.
IMHO there is no confusion. MachineRegisterInfo is only used in CodeGen, while MCRegisterInfo is primarily used in MC layer.
Just note that almost all affected files are in MCTargetDesc/AsmParser/Disassembler directories.
Comment Actions
I don't feel too strongly about it, but there are a bunch of files that use both, like lib/Target/*/*FrameLowering.cpp.
Comment Actions
I believe they could be refactored to use TargetRegisterInfo instead. This is at least true for getDwarfRegNum.