[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