This is an archive of the discontinued LLVM Phabricator instance.

RegisterPressure: Make liveness tracking subregister aware
ClosedPublic

Authored by MatzeB on Nov 24 2015, 2:09 PM.

Details

Summary

Some comments:

  • I did not see any compiletime impact of this change if subregister/lanemask tracking is disabled
  • Compiletime when subregister/lanemask tracking is affected, this is because the only way to see which lanes are actually dead or use an undefined value is to perform a liveness query. Unfortunately we have to perform this query for each register operand.

Diff Detail

Repository
rL LLVM

Event Timeline

MatzeB updated this revision to Diff 41084.Nov 24 2015, 2:09 PM
MatzeB retitled this revision from to RegisterPressure: Make liveness tracking subregister aware.
MatzeB updated this object.
MatzeB added a reviewer: atrick.
MatzeB set the repository for this revision to rL LLVM.
MatzeB added a subscriber: llvm-commits.
atrick accepted this revision.Nov 30 2015, 1:14 AM
atrick edited edge metadata.

Nice. I can't claim to know if you've covered everything that needed to be changed, but the new code looks good.

This revision is now accepted and ready to land.Nov 30 2015, 1:14 AM
MatzeB updated this revision to Diff 42622.Dec 11 2015, 7:39 PM
MatzeB edited edge metadata.

Rebase patch onto RegisterOperand changes

This revision was automatically updated to reflect the committed changes.