[PATCH] D89018: [NFC][MC] MCRegister API typing.

[Gaurav Jain via Phabricator via llvm-commits]

gjain added inline comments.


================
Comment at: llvm/lib/CodeGen/RegisterCoalescer.cpp:2396
 public:
-  JoinVals(LiveRange &LR, unsigned Reg, unsigned SubIdx, LaneBitmask LaneMask,
-           SmallVectorImpl<VNInfo*> &newVNInfo, const CoalescerPair &cp,
+  JoinVals(LiveRange &LR, Register Reg, unsigned SubIdx, LaneBitmask LaneMask,
+           SmallVectorImpl<VNInfo *> &newVNInfo, const CoalescerPair &cp,
----------------
Should this be a reference?


================
Comment at: llvm/lib/CodeGen/RegisterCoalescer.h:33
     /// virtual or physical register.
-    unsigned DstReg = 0;
+    Register DstReg = 0;
 
----------------
Should we omit the assignment since we are already using the default constructor?


================
Comment at: llvm/lib/CodeGen/RegisterCoalescer.h:36
     /// The virtual register that will be coalesced into dstReg.
-    unsigned SrcReg = 0;
+    Register SrcReg = 0;
 
----------------
ditto


================
Comment at: llvm/lib/Target/SystemZ/SystemZRegisterInfo.cpp:112
           Register Reg = MO->getReg();
-          Register PhysReg =
-            Register::isPhysicalRegister(Reg) ? Reg : VRM->getPhys(Reg);
+          Register PhysReg = Register::isPhysicalRegister(Reg)
+                                 ? Reg
----------------
Should this be instead?

```
Register PhysReg(Register::isPhysicalRegister(MO->getReg()) ? MO->getReg() : VRM->getPhys(Reg))
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89018/new/

https://reviews.llvm.org/D89018