[PATCH] D53603: [CodeGen] Take SPAdj into account for STATEPOINT liveness args

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 25 19:10:00 PST 2018


reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM.  I'm actually a bit uncertain of the semantics of this change, but I doubt we're going to find a better reviewer, so I did my best.  :)

p.s. Don't be surprised if you find other issues with statepoints and variable stack sizes.  In practice, we use fixed size frames, so the stack adjustment paths may be under tested.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D53603/new/

https://reviews.llvm.org/D53603





More information about the llvm-commits mailing list