[PATCH] D117796: AMDGPU: Fix LiveVariables error after lowering SI_END_CF
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 26 18:45:24 PST 2022
arsenm added a comment.
In D117796#3274452 <https://reviews.llvm.org/D117796#3274452>, @ruiling wrote:
> I would hope we can move the block split logic into a separate pass that can be scheduled before we constructing liveness information, like during addPreRegAlloc(). It is expensive to update either LiveVariables or LiveIntervals. Sounds good?
The whole reason this pass is here is because it needs to be done after phi elimination, so you can't really move it anywhere else. LiveVariables we should also just be pushing to eliminate entirely
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117796/new/
https://reviews.llvm.org/D117796
More information about the llvm-commits
mailing list