[PATCH] D49519: [RegisterCoalescer] Delay live interval update work until the rematerialization for all the uses from the same def is done

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 18 16:43:24 PDT 2018


chandlerc added a comment.

Nice approach!

I'd love to know if the old pattern has any real advantages. It would seem much simpler to just always defer the interval update until all the copies have been coalesced. Maybe its worth collecting perf data there too? If we can get away with the simpler (non-heuristic) model, it seems better.


Repository:
  rL LLVM

https://reviews.llvm.org/D49519





More information about the llvm-commits mailing list