[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