[PATCH] D28585: [RegisterCoalescing] Remove partial redundent copy

[Quentin Colombet via Phabricator via llvm-commits]

qcolombet added a comment.

Hi Wei,

Could you write a .mir test instead of a .ll one?
You should be able to get your input with `llc -stop-before=regalloc` or something like that.

In the assumption I would add that because of:

> we require that BB1 has only one successor so we always move a copy to a colder place.

and

> No B is referenced from the start of BB2 to B = A.

We know that B is not defined at the end of BB1, which is otherwise a requirement.

I'll have a look at the patch later, unless Matthias beats me at it.

Cheers,
-Quentin


Repository:
  rL LLVM

https://reviews.llvm.org/D28585