Pass debug name as Vector instead of const char*

src/wasm/wasm-opcodes.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_OPCODES_H_
 #define V8_WASM_OPCODES_H_
 
 #include "src/machine-type.h"
 #include "src/signature.h"
 
@@ skipping 34 matching lines @@
 const LocalType kAstF64 = MachineRepresentation::kFloat64;
 // We use kTagged here because kNone is already used by kAstStmt.
 const LocalType kAstEnd = MachineRepresentation::kTagged;
 
 typedef Signature<LocalType> FunctionSig;
 std::ostream& operator<<(std::ostream& os, const FunctionSig& function);
 
 struct WasmName {
   const char* name;
   uint32_t length;
+  // TODO(clemensh): Remove whole WasmName, replace by Vector<const char>

[titzer, 2016/04/22 11:46:30]

This should be inline, to prevent a proliferation of itty bitty helper
functions.

[Clemens Hammacher, 2016/04/22 13:47:54]

I will remove the whole WasmName in https://codereview.chromium.org/1910213004.

+  Vector<const char> toVec() const { return Vector<const char>(name, length); }
 };
 
 // TODO(titzer): Renumber all the opcodes to fill in holes.
 
 // Control expressions and blocks.
 #define FOREACH_CONTROL_OPCODE(V) \
   V(Nop, 0x00, _)                 \
   V(Block, 0x01, _)               \
   V(Loop, 0x02, _)                \
   V(If, 0x03, _)                  \
@@ skipping 469 matching lines @@
       default:
         return "<unknown>";
     }
   }
 };
 }  // namespace wasm
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_WASM_OPCODES_H_