[PATCH] D80674: Default to generating statepoints with deopt and gc-transition bundles if needed
    Denis Antrushin via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu May 28 05:57:25 PDT 2020
    
    
  
dantrushin accepted this revision.
dantrushin added a comment.
This revision is now accepted and ready to land.
LGTM with two notes:
- Does it makes sense to make GCArgs `Optional<>` too? Implementation might change later, but you won't need to change interface again.
- Now that Deopt and Transition args are `Optional<>`, do we need such many `CreageGCStatepointCall` variants?
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80674/new/
https://reviews.llvm.org/D80674
    
    
More information about the llvm-commits
mailing list