[llvm] 37fdde6 - [CodeGen] Remove implict conversions from Register to unsigned from MachineOperand. NFC
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Sat Jan 25 23:49:53 PST 2025
Author: Craig Topper
Date: 2025-01-25T23:12:14-08:00
New Revision: 37fdde6025c8ead27a7608643b63e0d4498211e2
URL: https://github.com/llvm/llvm-project/commit/37fdde6025c8ead27a7608643b63e0d4498211e2
DIFF: https://github.com/llvm/llvm-project/commit/37fdde6025c8ead27a7608643b63e0d4498211e2.diff
LOG: [CodeGen] Remove implict conversions from Register to unsigned from MachineOperand. NFC
Added:
Modified:
llvm/include/llvm/CodeGen/MachineOperand.h
llvm/lib/CodeGen/MachineOperand.cpp
Removed:
################################################################################
diff --git a/llvm/include/llvm/CodeGen/MachineOperand.h b/llvm/include/llvm/CodeGen/MachineOperand.h
index be1b4fb7d54fb2..3ec46afa781ab6 100644
--- a/llvm/include/llvm/CodeGen/MachineOperand.h
+++ b/llvm/include/llvm/CodeGen/MachineOperand.h
@@ -854,7 +854,7 @@ class MachineOperand {
Op.IsEarlyClobber = isEarlyClobber;
Op.TiedTo = 0;
Op.IsDebug = isDebug;
- Op.SmallContents.RegNo = Reg;
+ Op.SmallContents.RegNo = Reg.id();
Op.Contents.Reg.Prev = nullptr;
Op.Contents.Reg.Next = nullptr;
Op.setSubReg(SubReg);
diff --git a/llvm/lib/CodeGen/MachineOperand.cpp b/llvm/lib/CodeGen/MachineOperand.cpp
index d11ac614ace356..f498491164e14f 100644
--- a/llvm/lib/CodeGen/MachineOperand.cpp
+++ b/llvm/lib/CodeGen/MachineOperand.cpp
@@ -71,13 +71,13 @@ void MachineOperand::setReg(Register Reg) {
if (MachineFunction *MF = getMFIfAvailable(*this)) {
MachineRegisterInfo &MRI = MF->getRegInfo();
MRI.removeRegOperandFromUseList(this);
- SmallContents.RegNo = Reg;
+ SmallContents.RegNo = Reg.id();
MRI.addRegOperandToUseList(this);
return;
}
// Otherwise, just change the register, no problem. :)
- SmallContents.RegNo = Reg;
+ SmallContents.RegNo = Reg.id();
}
void MachineOperand::substVirtReg(Register Reg, unsigned SubIdx,
@@ -291,7 +291,7 @@ void MachineOperand::ChangeToRegister(Register Reg, bool isDef, bool isImp,
assert(!(isDead && !isDef) && "Dead flag on non-def");
assert(!(isKill && isDef) && "Kill flag on def");
OpKind = MO_Register;
- SmallContents.RegNo = Reg;
+ SmallContents.RegNo = Reg.id();
SubReg_TargetFlags = 0;
IsDef = isDef;
IsImp = isImp;
@@ -390,7 +390,8 @@ hash_code llvm::hash_value(const MachineOperand &MO) {
switch (MO.getType()) {
case MachineOperand::MO_Register:
// Register operands don't have target flags.
- return hash_combine(MO.getType(), (unsigned)MO.getReg(), MO.getSubReg(), MO.isDef());
+ return hash_combine(MO.getType(), MO.getReg().id(), MO.getSubReg(),
+ MO.isDef());
case MachineOperand::MO_Immediate:
return hash_combine(MO.getType(), MO.getTargetFlags(), MO.getImm());
case MachineOperand::MO_CImmediate:
More information about the llvm-commits
mailing list