[wasm] All strings are length-prefixed and inline

src/wasm/ast-decoder.h

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_WASM_AST_DECODER_H_
 #define V8_WASM_AST_DECODER_H_
 
 #include "src/signature.h"
 #include "src/wasm/decoder.h"
 #include "src/wasm/wasm-opcodes.h"
@@ skipping 140 matching lines @@
     length = varint_length + (table_count + 1) * sizeof(uint32_t);
 
     uint32_t table_start = 1 + varint_length;
     if (decoder->check(pc, table_start, (table_count + 1) * sizeof(uint32_t),
                        "expected <table entries>")) {
       table = pc + table_start;
     } else {
       table = nullptr;
     }
   }
-  inline uint32_t read_entry(Decoder* decoder, int i) {
+  inline uint16_t read_entry(Decoder* decoder, int i) {

[titzer, 2016/03/09 08:18:17]

does this need a rebase?

[binji, 2016/03/09 18:34:17]

Done.

     DCHECK(i >= 0 && static_cast<uint32_t>(i) <= table_count);
     return table ? decoder->read_u32(table + i * sizeof(uint32_t)) : 0;
   }
 };
 
 struct MemoryAccessOperand {
   bool aligned;
   uint32_t offset;
   int length;
   inline MemoryAccessOperand(Decoder* decoder, const byte* pc) {
@@ skipping 57 matching lines @@
 int OpcodeLength(const byte* pc, const byte* end);
 
 // Computes the arity (number of sub-nodes) of the opcode at the given address.
 int OpcodeArity(ModuleEnv* module, FunctionSig* sig, const byte* pc,
                 const byte* end);
 }  // namespace wasm
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_WASM_AST_DECODER_H_