[llvm] [llvm-exegesis] Begin replacing unsigned with MCRegister. NFC (PR #123109)
Min-Yih Hsu via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 15 13:49:02 PST 2025
================
@@ -46,7 +46,7 @@ struct RegisterAliasingTracker {
// Construct a tracker from an MCPhysReg.
RegisterAliasingTracker(const MCRegisterInfo &RegInfo,
- const MCPhysReg Register);
+ const MCRegister Register);
----------------
mshockwave wrote:
> I want to add an assertion for that, but NVPTX and WebAssembly don't compile to traditional assembly and have their own virtual register space
I have been wondering why virtual registers are allowed while the comments in MCRegister's header file say it's for physical registers. Thanks for the clarifications.
https://github.com/llvm/llvm-project/pull/123109
More information about the llvm-commits
mailing list