[PATCH] D13197: AMDGPU: Move SIFixSGPRLiveRanges to be a regalloc pass

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 28 13:17:43 PDT 2015


arsenm added a comment.

In http://reviews.llvm.org/D13197#254947, @MatzeB wrote:

> Note that long-term we plan to remove the LiveVariables analysis and just keep LiveIntervals around; though I don't see this happening soon as  I don't see an incentive to actually do the work of fixing TwoAddressInstructions and PhiElimination to use LiveIntervals...


I'm kind of confused what the relationship between LiveVariables and LiveIntervals is, but I've seen the comments about removing LiveVariables. LiveIntervals seems to require LiveVariables, but then the calculation of LiveIntervals seems to clear kill flags?


http://reviews.llvm.org/D13197





More information about the llvm-commits mailing list