[PATCH] D82638: [MachineCopyPropagation] BackwardPropagatableCopy: add check for hasOverlappingMultipleDef

[Simon Wallis via Phabricator via llvm-commits]

simonwallis2 added inline comments.


================
Comment at: llvm/test/CodeGen/ARM/mcp-dest-regs-no-dup.mir:1
+# RUN: llc -mtriple=arm-eabi -O1 -run-pass=machine-cp %s -o - 2>&1 | FileCheck %s
+--- |
----------------
lkail wrote:
> lkail wrote:
> > Prefer adding `-verify-machineinstrs`, `-simplify-mir` and keep the line under 80 columns.
> Any reason to redirect stderr?
Reason is that I saw it in the test I based this on.
I have removed the redirection.



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82638/new/

https://reviews.llvm.org/D82638