[PATCH] D11880: AMDGPU/SI: Only look at live out SGPR defs

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 10 07:46:02 PDT 2015


tstellarAMD added a comment.

> When trying to fix SGPR live ranges, skip defs that are

>  killed in the same block as the def. I don't think

>  we need to worry about these cases as long as the

>  live ranges of the SGPRs in dominating blocks are

>  correct.


I think this will break this case:

def vreg0 <- assigned sgpr0

if:
def vreg1 <- assigned sgpr0
kill vreg1  <- assigned sgpr0
br endif

else:
use vreg0 <- assigned sgpr0
br endif

endif:
 ...


http://reviews.llvm.org/D11880





More information about the llvm-commits mailing list