[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