[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