[wasm] Fix OpcodeLength() calculation.

test/unittests/wasm/wasm-macro-gen-unittest.cc

Index: test/unittests/wasm/wasm-macro-gen-unittest.cc
diff --git a/test/unittests/wasm/wasm-macro-gen-unittest.cc b/test/unittests/wasm/wasm-macro-gen-unittest.cc
index 45e7b3983b3ceaaa3364387e5eb3e7f544499067..ed2fee65d01b5324f9020c95b2b4bb861b02d818 100644
--- a/test/unittests/wasm/wasm-macro-gen-unittest.cc
+++ b/test/unittests/wasm/wasm-macro-gen-unittest.cc
@@ -27,12 +27,16 @@ TEST_F(WasmMacroGenTest, Constants) {
   EXPECT_SIZE(2, WASM_I8(254));
 
   EXPECT_SIZE(2, WASM_I32V_1(1));
+  EXPECT_SIZE(3, WASM_I32V_2(200));
   EXPECT_SIZE(4, WASM_I32V_3(10000));
   EXPECT_SIZE(5, WASM_I32V_4(-9828934));
+  EXPECT_SIZE(6, WASM_I32V_5(-1119828934));

[Mircea Trofin, 2016/03/11 17:15:41]

Do these numbers mean anything? Should they be a const indicating their
semantics?

[titzer, 2016/03/11 17:17:35]

They're just random values with no semantic meaning. Factoring them out just
creates noise.

 
   EXPECT_SIZE(2, WASM_I64V_1(1));
+  EXPECT_SIZE(3, WASM_I64V_2(300));
   EXPECT_SIZE(4, WASM_I64V_3(10000));
   EXPECT_SIZE(5, WASM_I64V_4(-9828934));
+  EXPECT_SIZE(6, WASM_I64V_5(-1119828934));
   EXPECT_SIZE(10, WASM_I64V_9(0x123456789abcdef0ULL));
 
   EXPECT_SIZE(5, WASM_F32(1.0f));