[llvm-branch-commits] [llvm] [mcp-frameinst: 2/4]: [MCP][NFC] Cleanup and prepare to preserve frame-setup/destroy (PR #186240)

[Scott Linder via llvm-branch-commits]

================
@@ -941,7 +965,9 @@ void MachineCopyPropagation::ForwardCopyPropagateBlock(MachineBasicBlock &MBB) {
       if (!TRI->regsOverlap(RegDef, RegSrc)) {
         // Copy is now a candidate for deletion.
         MCRegister Def = RegDef.asMCReg();
-        if (!MRI->isReserved(Def))
+        // FIXME: what about src? is it target dependant? am I misunderstanding
----------------
slinder1 wrote:

If nobody knows the answer, then I think it could be (or a cleaned up version of it). I still haven't worked out if this is just a difference in heuristic or a fundamental limitation.

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