[PATCH] D43921: [WebAssembly] Use uint8_t for single byte values to match the spec

Derek Schuff via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 1 09:56:32 PST 2018


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

I'm fine with doing this change since it does reflect the spec. Although IIRC the relevant types don't yet have any values over 128 and we've kept it that way so we can always upgrade to varint when we need to. So in principle we could keep the code using varint and then it won't have to change in the future.


Repository:
  rL LLVM

https://reviews.llvm.org/D43921





More information about the llvm-commits mailing list