diff options
author | Thomas Lively <tlively@google.com> | 2018-11-15 18:56:49 +0000 |
---|---|---|
committer | Thomas Lively <tlively@google.com> | 2018-11-15 18:56:49 +0000 |
commit | fc3163b67a87e93e8beda976ccd16418ca879284 (patch) | |
tree | c079e3f39c9d70e1b462a59185e9156886292c19 /llvm/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp | |
parent | 919fbbbcca6a6c137b2319aecf70cfb0f48136a0 (diff) | |
download | bcm5719-llvm-fc3163b67a87e93e8beda976ccd16418ca879284.tar.gz bcm5719-llvm-fc3163b67a87e93e8beda976ccd16418ca879284.zip |
[WebAssembly] Fix return type of nextByte
Summary:
The old return type did not allow for correct error reporting and was
causing a compiler warning.
Reviewers: aheejin
Subscribers: dschuff, sbc100, jgravelle-google, sunfish, llvm-commits
Differential Revision: https://reviews.llvm.org/D54586
llvm-svn: 346979
Diffstat (limited to 'llvm/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp')
-rw-r--r-- | llvm/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp b/llvm/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp index 971a48c89bd..c068d6b43d4 100644 --- a/llvm/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp +++ b/llvm/lib/Target/WebAssembly/Disassembler/WebAssemblyDisassembler.cpp @@ -68,7 +68,7 @@ extern "C" void LLVMInitializeWebAssemblyDisassembler() { createWebAssemblyDisassembler); } -static uint8_t nextByte(ArrayRef<uint8_t> Bytes, uint64_t &Size) { +static int nextByte(ArrayRef<uint8_t> Bytes, uint64_t &Size) { if (Size >= Bytes.size()) return -1; auto V = Bytes[Size]; @@ -121,7 +121,7 @@ MCDisassembler::DecodeStatus WebAssemblyDisassembler::getInstruction( raw_ostream & /*OS*/, raw_ostream &CS) const { CommentStream = &CS; Size = 0; - auto Opc = nextByte(Bytes, Size); + int Opc = nextByte(Bytes, Size); if (Opc < 0) return MCDisassembler::Fail; const auto *WasmInst = &InstructionTable0[Opc]; |