[PATCH] D32629: [StackMaps] Increase the size of the "location size" field

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 27 18:42:36 PDT 2017


reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: docs/StackMaps.rst:342
     uint16 : NumLocations
     Location[NumLocations] {
       uint8  : Register | Direct | Indirect | Constant | ConstantIndex
----------------
It's really unfortunate that we had to grow the size so much here, but given this is intended to be immediately parsed and thrown away, I guess it's not too much of a problem.


https://reviews.llvm.org/D32629





More information about the llvm-commits mailing list