[PATCH] D54138: [WebAssembly] Read prefixed opcodes as ULEB128s

[Thomas Lively via Phabricator via llvm-commits]

tlively added inline comments.


================
Comment at: llvm/trunk/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp:69
 
-static int nextByte(ArrayRef<uint8_t> Bytes, uint64_t &Size) {
+static uint8_t nextByte(ArrayRef<uint8_t> Bytes, uint64_t &Size) {
   if (Size >= Bytes.size())
----------------
MTC wrote:
> @tlively  Just to remind, change `int` to `uint8_t` will trigger the following warning, see line 123.
> 
> ```
> warning: comparison is always false due to limited range of data type [-Wtype-limits]
> if (Opc < 0)
>            ^
> ```
Thanks for the heads up! I'll fix this soon.


Repository:
  rL LLVM

https://reviews.llvm.org/D54138