[llvm] [RISCV][MCP] Remove redundant move from tail duplication (PR #89865)
Piyou Chen via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 23 23:05:08 PDT 2024
BeMg wrote:
> Removing `CopyOperands.Source->isRenamable()` from `isBackwardPropagatableCopy` is another way to fix this without changing `copyPhysReg`. `propagateDefs` checks the renamable flag on the def it is propagating into. So the check in `isBackwardPropagatableCopy` might not be functionally needed.
How about inherit the `isRenamable` during the `TargetInstrInfo::lowerCopy`?
https://github.com/llvm/llvm-project/pull/89865
More information about the llvm-commits
mailing list