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

[Sam Clegg via Phabricator via llvm-commits]

sbc100 created this revision.
Herald added subscribers: llvm-commits, sunfish, aheejin, jgravelle-google, dschuff, jfb.
sbc100 edited the summary of this revision.

The original BinaryEncoding.md document used to specify that
these values were `varint7`, but the official spec lists them
explicitly as single byte values and not LEB.

A similar change for wabt is in flight:
 https://github.com/WebAssembly/wabt/pull/782

And also lld: 
 https://reviews.llvm.org/D43922


Repository:
  rL LLVM

https://reviews.llvm.org/D43921

Files:
  include/llvm/BinaryFormat/Wasm.h
  include/llvm/ObjectYAML/WasmYAML.h
  lib/MC/WasmObjectWriter.cpp
  lib/Object/WasmObjectFile.cpp
  lib/Target/WebAssembly/MCTargetDesc/WebAssemblyTargetStreamer.cpp
  lib/Target/WebAssembly/WebAssemblyMCInstLower.cpp
  tools/yaml2obj/yaml2wasm.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43921.136438.patch
Type: text/x-patch
Size: 18212 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180301/13a4fb58/attachment.bin>