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

Fri Oct 28 10:16:01 PDT 2022

foad added a comment.

In D136918#3892323 <https://reviews.llvm.org/D136918#3892323>, @vpykhtin wrote:

> In D136918#3891234 <https://reviews.llvm.org/D136918#3891234>, @foad wrote:
>
>> I don't understand this. Why would the liveouts of A and B be different from the liveins of C?
>
> You're right - live-outs of A //and// B are live-ins to C, but in this case we're trying to use live-outs of only A or only B as live-ins to C.

No, I mean that the live-outs of A should be identical to the live-outs of B. And both should be identical to the live-ins of C. That's how liveness works.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136918