[llvm] [MCP] Enhance MCP copy Instruction removal for special case (PR #70778)

Kai Luo via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 9 00:01:41 PST 2023


bzEq wrote:

> Shouldn't we forget about L1 when we get to L2?

I agree with we should forget about L1 when we get to L2. The semantic of `Copies[Unit].DefRegs` should be Registers defined by Unit in COPY instructions, we want to keep this invariant during tracking. I think it's appropriate to keep this property during `trackCopy`. After L2, `Copies[r9].DefRegs` still containing `r0` is violating this property.

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


More information about the llvm-commits mailing list