[Interpreter] Handle negative ints in generate-bytecode-expectations.

src/interpreter/bytecodes.cc

 // 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.
 
 #include "src/interpreter/bytecodes.h"
 
 #include "src/frames.h"
 #include "src/interpreter/bytecode-traits.h"
 
 namespace v8 {
@@ skipping 309 matching lines @@
 #define CASE(Name, _)        \
   case OperandType::k##Name: \
     break;
     NON_REGISTER_OPERAND_TYPE_LIST(CASE)
     REGISTER_INPUT_OPERAND_TYPE_LIST(CASE)
 #undef CASE
   }
   return false;
 }
 
+// static
+bool Bytecodes::IsImmediateOperandType(OperandType operand_type) {
+  return operand_type == OperandType::kImm8;
+}
+
+// static
+bool Bytecodes::IsIndexOperandType(OperandType operand_type) {
+  return operand_type == OperandType::kIdx8 ||
+         operand_type == OperandType::kIdx16;
+}

[rmcilroy, 2016/02/10 15:26:12]

nit - move above IsMaybeRegisterOperandType (and move IsMaybeRegisterOperandType
below IsRegisterCountOperandType)

[Stefano Sanfilippo, 2016/02/10 15:52:00]

Done.

+
 namespace {
 static Register DecodeRegister(const uint8_t* operand_start,
                                OperandType operand_type) {
   switch (Bytecodes::SizeOfOperand(operand_type)) {
     case OperandSize::kByte:
       return Register::FromOperand(*operand_start);
     case OperandSize::kShort:
       return Register::FromWideOperand(ReadUnalignedUInt16(operand_start));
     case OperandSize::kNone: {
       UNREACHABLE();
@@ skipping 239 matching lines @@
   } else {
     std::ostringstream s;
     s << "r" << index();
     return s.str();
   }
 }
 
 }  // namespace interpreter
 }  // namespace internal
 }  // namespace v8