From: Heejin Ahn Date: Fri, 2 Nov 2018 19:25:09 +0000 (+0000) Subject: [WebAssembly] Change indices types to unsined int (NFC) X-Git-Tag: llvmorg-8.0.0-rc1~5100 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4808914a5829ebdb19d0eb5ab93281169db8ad2d;p=platform%2Fupstream%2Fllvm.git [WebAssembly] Change indices types to unsined int (NFC) Summary: This changes int types to unsigned int in a few places: function indices and `wasm::Valtype` (which is unsigend int enum). Currently these values cannot have negative values anyway, so this should not be a functional change for now. Reviewers: sbc100 Subscribers: dschuff, jgravelle-google, sunfish, llvm-commits Differential Revision: https://reviews.llvm.org/D54044 llvm-svn: 346031 --- diff --git a/llvm/lib/MC/WasmObjectWriter.cpp b/llvm/lib/MC/WasmObjectWriter.cpp index f9318ad..c1e0b7a 100644 --- a/llvm/lib/MC/WasmObjectWriter.cpp +++ b/llvm/lib/MC/WasmObjectWriter.cpp @@ -92,9 +92,9 @@ struct WasmFunctionTypeDenseMapInfo { static unsigned getHashValue(const WasmFunctionType &FuncTy) { uintptr_t Value = FuncTy.State; for (wasm::ValType Ret : FuncTy.Returns) - Value += DenseMapInfo::getHashValue(int32_t(Ret)); + Value += DenseMapInfo::getHashValue(uint32_t(Ret)); for (wasm::ValType Param : FuncTy.Params) - Value += DenseMapInfo::getHashValue(int32_t(Param)); + Value += DenseMapInfo::getHashValue(uint32_t(Param)); return Value; } static bool isEqual(const WasmFunctionType &LHS, @@ -118,7 +118,7 @@ struct WasmDataSegment { // A wasm function to be written into the function section. struct WasmFunction { - int32_t Type; + uint32_t Type; const MCSymbolWasm *Sym; }; @@ -231,7 +231,7 @@ class WasmObjectWriter : public MCObjectWriter { // Map from section to defining function symbol. DenseMap SectionFunctions; - DenseMap + DenseMap FunctionTypeIndices; SmallVector FunctionTypes; SmallVector Globals;