[PATCH] D99875: [Statepoint] Factor-out utility function to get non-foldable area of STATEPOINT like instructions. NFC

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 5 08:46:24 PDT 2021


reames added a comment.

For unclear reasons, phab dropped a long inline comment from my previous review.  Rewritten here.

The comment and naming was confusing to me.  I'd suggest the following changes.

1. Restrict the API and naming to patchpoint, stackmap, and statepoint explicitly.
2. Be clear about the fact that operands are "potentially foldable", not "guaranteed foldable".

A suggested name and comment: getPatchpointUnfoldableRange.

For a patchpoint, stackmap, or statepoint intrinsic, return the range of operands which can't be folded into stack references.  These instructions are unique in that stack references for some operands have the same execution cost (e.g. none) as the unfolded register forms.  The ranged return is guaranteed to include all operands which can't be folded at zero cost.


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

https://reviews.llvm.org/D99875



More information about the llvm-commits mailing list