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

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 28 13:26:50 PDT 2015


MatzeB added a comment.

The fact that LiveIntervals requires LiveVariables appears to be hack. From LiveIntervalAnalaysis.cpp:

  // LiveVariables isn't really required by this analysis, it is only required
  // here to make sure it is live during TwoAddressInstructionPass and
  // PHIElimination. This is temporary.

This is probably not the right place for an in-depth discussion but kill flags is another feature that we hope to remove long term... Yes the current state of TwoAddressInstruction and PhiElimination prohibits those changes and I don't see this getting fixed in the near future.


http://reviews.llvm.org/D13197





More information about the llvm-commits mailing list