[llvm] [ReachingDefAnalysis] Extend the analysis to stack objects. (PR #118097)

Michael Maitland via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 29 07:20:10 PST 2024


================
@@ -529,14 +612,21 @@ ReachingDefAnalysis::getLocalLiveOutMIDef(MachineBasicBlock *MBB,
                                           MCRegister PhysReg) const {
   LiveRegUnits LiveRegs(*TRI);
   LiveRegs.addLiveOuts(*MBB);
-  if (LiveRegs.available(PhysReg))
+  if (Register::isPhysicalRegister(PhysReg) && LiveRegs.available(PhysReg))
----------------
michaelmaitland wrote:

If `PhysReg` isn't `isPhysicalRegister`, then is there something wrong with the naming here? Or is this statement always true and can be dropped?

https://github.com/llvm/llvm-project/pull/118097


More information about the llvm-commits mailing list