[Intepreter] Addition of BytecodeArrayBuilder and accumulator based bytecodes.

src/objects.cc

Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index 3df88e300c3ad8f53028e452dc8f0a5a24976b73..3496e72fb2233e3c1987156999f4288785e9afbd 100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -11615,7 +11615,6 @@ void BytecodeArray::Disassemble(std::ostream& os) {
   for (int i = 0; i < this->length(); i += bytes) {
     interpreter::Bytecode bytecode = interpreter::Bytecodes::FromByte(get(i));
     bytes = interpreter::Bytecodes::Size(bytecode);
-
     SNPrintF(buf, "%p : ", GetFirstBytecodeAddress() + i);
     os << buf.start();
     for (int j = 0; j < bytes; j++) {
@@ -11625,7 +11624,26 @@ void BytecodeArray::Disassemble(std::ostream& os) {
     for (int j = bytes; j < interpreter::Bytecodes::MaximumSize(); j++) {
       os << "   ";
     }
-    os << bytecode << "\n";
+    os << bytecode << " ";
+    for (int j = 1; j < bytes; j++) {
+      interpreter::OperandType op_type =
+          interpreter::Bytecodes::GetOperandType(bytecode, j - 1);
+      uint8_t operand = get(i + j);
+      switch (op_type) {
+        case interpreter::OperandType::kNone:

[rmcilroy, 2015/07/31 09:56:19]

We should never hit this case, right? If so, maybe just add UNREACHABLE() here
(and move to the bottom)

[oth, 2015/07/31 11:20:59]

Done.

+          break;
+        case interpreter::OperandType::kImm8:
+          os << "#" << static_cast<int>(operand);
+          break;
+        case interpreter::OperandType::kReg:
+          os << "r" << static_cast<int>(operand);
+          break;
+      }
+      if (j + 1 < bytes) {
+        os << ", ";
+      }
+    }
+    os << "\n";
   }
 }