[PATCH] D33105: [AMDGPU] Turn register pressure estimation into forward tracker

[Valery Pykhtin via Phabricator via llvm-commits]

vpykhtin added a comment.

In https://reviews.llvm.org/D33105#755142, @rampitec wrote:

> In https://reviews.llvm.org/D33105#754976, @vpykhtin wrote:
>
> > If I don't mistake downward tracker cannot work on arbitrary instruction order, so let's change it's interface so that it tracks current instruction and move it, likewise like llvm standard tracker, that is reset(MI), advance()
>
>
> In general it cannot work on an arbitrary order, but it can be used as a probe to schedule next arbitrary instruction and then reset to the previous state.
>  Then in https://reviews.llvm.org/D33117 I'm using advanceBefore to cross basic block boundary and I do not really want to slow down advance method by checking for the iterator end condition which is needed relatively seldom.


Why are you sure it would work on arbitrary out-of-original-LIS-constructed-order instruction?


Repository:
  rL LLVM

https://reviews.llvm.org/D33105