[llvm] [MachineLateInstrsCleanup] Reuse redundant spill-slot reloads (PR #220180)

[Jonas Paulsson via llvm-commits]

================
@@ -168,15 +177,61 @@ void MachineLateInstrsCleanup::removeRedundantDef(MachineInstr *MI) {
   ++NumRemoved;
 }
 
+// Return true if MI accesses a register-allocation spill slot through a single
----------------
JonPsson1 wrote:

It seems this function is really checking for a spill reloads, right? I think it would be more clear to rename it to something like isSpillSlotReload(), and put the mayLoad() check inside it.

The comment about FI doesn't quite match the behavior... It is setting FI to the FrameIndex if present, and returning true if it goes to a spillslot.



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