[PATCH] D42200: Fix RegScavenger::unprocess

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 17 15:05:05 PST 2018


MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.

LGTM.

That said the medium to short term plan is to get rid of kill flags, which means RegisterScavenger::forward() and RegisterScavenger::unprocess() will not any more (or at least give you very conservative results which are probably not too useful). I'd recommend to switch your code to LiveRegUnits/LivePhysRegs (in case you don't need emergency spilling) or RegisterScavenger::backward() instead!


Repository:
  rL LLVM

https://reviews.llvm.org/D42200





More information about the llvm-commits mailing list