[PATCH] D136918: [AMDGPU] Scheduler: fix RP calculation for a MBB with one successor

Wed Nov 2 10:12:32 PDT 2022

arsenm added a comment.

In D136918#3902686 <https://reviews.llvm.org/D136918#3902686>, @arsenm wrote:

> In D136918#3902682 <https://reviews.llvm.org/D136918#3902682>, @vpykhtin wrote:
>
>>> (Liveness is really "due to" uses, not defs.)
>>
>> This sounds like if we need to track RP upward from uses to defs.
>
> Yes, this is how things are supposed to work. There's a lot of infrastructure that needs to be inverted

In particular RegScavenging is used in the forward direction and relies on accurate kill flags. stepBackwards doesn't, but most everything needs to move to use that instead

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136918/new/

https://reviews.llvm.org/D136918