[PATCH] D29436: RegisterCoalescer: Fix joinReservedPhysReg()

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 6 16:54:24 PST 2017


MatzeB added a comment.

Good catch Wei! Looks like the only sensible way to fix this today is to require the liverange to be whithin a single basic block.

I put this simple fix into a different review: https://reviews.llvm.org/D29436 that we can hopefully pick into the llvm 4.0 branch.
I will update this review as a followup patch to https://reviews.llvm.org/D29436 as it has grown some interesting features like proper support for earlyclobber defs and multiple uses of the vreg and some code cleanups (they are however not critical enough that they must go into the 4.0 branch).


Repository:
  rL LLVM

https://reviews.llvm.org/D29436





More information about the llvm-commits mailing list