[PATCH] D78119: [Statepoint] Add getters to StatepointOpers. NFC.

Denis Antrushin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 14 09:06:22 PDT 2020


dantrushin created this revision.
dantrushin added reviewers: reames, skatkov.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.

To simplify future work on statepoint representation, hide
direct access ti statepoint field indices and provide getters
for them. Add getters for couple more statepoint fields.

This also fixes two bugs in MachineVerifier for statepoint:
First, the `break` statement was falling out of `if` statement
scope, thus disabling following checks.
Second, it was incorrectly accessing some fields like CallingConv -
StatepointOpers gives index to their value directly, not to
preceeding field type encoding.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D78119

Files:
  llvm/include/llvm/CodeGen/StackMaps.h
  llvm/lib/CodeGen/FixupStatepointCallerSaved.cpp
  llvm/lib/CodeGen/MachineVerifier.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78119.257364.patch
Type: text/x-patch
Size: 5347 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200414/0de74018/attachment.bin>


More information about the llvm-commits mailing list