[llvm] [LiveDebugVariables] Use VirtRegMap::hasPhys. NFC (PR #106186)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 26 22:33:31 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-debuginfo
Author: Craig Topper (topperc)
<details>
<summary>Changes</summary>
Use hasPhys instead of MCRegister::isPhysical.
---
Full diff: https://github.com/llvm/llvm-project/pull/106186.diff
1 Files Affected:
- (modified) llvm/lib/CodeGen/LiveDebugVariables.cpp (+2-4)
``````````diff
diff --git a/llvm/lib/CodeGen/LiveDebugVariables.cpp b/llvm/lib/CodeGen/LiveDebugVariables.cpp
index d1341f116a5479..48bcc0a61e30c9 100644
--- a/llvm/lib/CodeGen/LiveDebugVariables.cpp
+++ b/llvm/lib/CodeGen/LiveDebugVariables.cpp
@@ -1529,8 +1529,7 @@ void UserValue::rewriteLocations(VirtRegMap &VRM, const MachineFunction &MF,
// Only virtual registers are rewritten.
if (Loc.isReg() && Loc.getReg() && Loc.getReg().isVirtual()) {
Register VirtReg = Loc.getReg();
- if (VRM.isAssignedReg(VirtReg) &&
- Register::isPhysicalRegister(VRM.getPhys(VirtReg))) {
+ if (VRM.isAssignedReg(VirtReg) && VRM.hasPhys(VirtReg)) {
// This can create a %noreg operand in rare cases when the sub-register
// index is no longer available. That means the user value is in a
// non-existent sub-register, and %noreg is exactly what we want.
@@ -1839,8 +1838,7 @@ void LDVImpl::emitDebugValues(VirtRegMap *VRM) {
unsigned SubReg = It.second.SubReg;
MachineBasicBlock *OrigMBB = Slots->getMBBFromIndex(Slot);
- if (VRM->isAssignedReg(Reg) &&
- Register::isPhysicalRegister(VRM->getPhys(Reg))) {
+ if (VRM->isAssignedReg(Reg) && VRM->hasPhys(Reg)) {
unsigned PhysReg = VRM->getPhys(Reg);
if (SubReg != 0)
PhysReg = TRI->getSubReg(PhysReg, SubReg);
``````````
</details>
https://github.com/llvm/llvm-project/pull/106186
More information about the llvm-commits
mailing list