[Interpreter] Add BytecodeArray class and add to SharedFunctionInfo.

src/objects-printer.cc

 // Copyright 2012 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/v8.h"
 
 #include "src/disasm.h"
 #include "src/disassembler.h"
 #include "src/heap/objects-visiting.h"
+#include "src/interpreter/bytecodes.h"
 #include "src/jsregexp.h"
 #include "src/ostreams.h"
 
 namespace v8 {
 namespace internal {
 
 #ifdef OBJECT_PRINT
 
 void Object::Print() {
   OFStream os(stdout);
@@ skipping 45 matching lines @@
       break;
     case FIXED_DOUBLE_ARRAY_TYPE:
       FixedDoubleArray::cast(this)->FixedDoubleArrayPrint(os);
       break;
     case FIXED_ARRAY_TYPE:
       FixedArray::cast(this)->FixedArrayPrint(os);
       break;
     case BYTE_ARRAY_TYPE:
       ByteArray::cast(this)->ByteArrayPrint(os);
       break;
+    case BYTECODE_ARRAY_TYPE:
+      BytecodeArray::cast(this)->BytecodeArrayPrint(os);
+      break;
     case FREE_SPACE_TYPE:
       FreeSpace::cast(this)->FreeSpacePrint(os);
       break;
 
 #define PRINT_EXTERNAL_ARRAY(Type, type, TYPE, ctype, size)            \
   case EXTERNAL_##TYPE##_ARRAY_TYPE:                                   \
     External##Type##Array::cast(this)->External##Type##ArrayPrint(os); \
     break;
 
      TYPED_ARRAYS(PRINT_EXTERNAL_ARRAY)
@@ skipping 105 matching lines @@
 
     default:
       os << "UNKNOWN TYPE " << map()->instance_type();
       UNREACHABLE();
       break;
   }
 }
 
 
 void ByteArray::ByteArrayPrint(std::ostream& os) {  // NOLINT
-  os << "byte array, data starts at " << GetDataStartAddress();
+  ScopedVector<char> buf(100);
+  SNPrintF(buf, "byte array, data starts at %p", GetDataStartAddress());
+  os << buf.start();

[rmcilroy, 2015/07/20 11:39:46]

nit - do you need to change this function?

[oth, 2015/07/20 13:47:30]

Done.

 }
 
 
+void BytecodeArray::BytecodeArrayPrint(std::ostream& os) {  // NOLINT
+  ScopedVector<char> buf(100);
+  SNPrintF(buf, "bytecode array, starts at %p (%d bytes)\n",
+           GetFirstBytecodeAddress(), length());
+  os << buf.start();

[rmcilroy, 2015/07/20 11:39:46]

I don't think these lines is necessary - it should be clear from Dissasemble
which prints the addresses of the bytecodes as it prints them.

[oth, 2015/07/20 13:47:30]

Done.

+  Disassemble(os);
+}
+
+
 void FreeSpace::FreeSpacePrint(std::ostream& os) {  // NOLINT
   os << "free space, size " << Size();
 }
 
 
 #define EXTERNAL_ARRAY_PRINTER(Type, type, TYPE, ctype, size)                \
   void External##Type##Array::External##Type##ArrayPrint(std::ostream& os) { \
     os << "external " #type " array";                                        \
   }
 
@@ skipping 543 matching lines @@
   // script()->Print(os);
   os << "\n - function token position = " << function_token_position();
   os << "\n - start position = " << start_position();
   os << "\n - end position = " << end_position();
   os << "\n - is expression = " << is_expression();
   os << "\n - debug info = " << Brief(debug_info());
   os << "\n - length = " << length();
   os << "\n - optimized_code_map = " << Brief(optimized_code_map());
   os << "\n - feedback_vector = ";
   feedback_vector()->FixedArrayPrint(os);
+  os << "\n - bytecode_array = " << bytecode_array();
   os << "\n";
 }
 
 
 void JSGlobalProxy::JSGlobalProxyPrint(std::ostream& os) {  // NOLINT
   os << "global_proxy ";
   JSObjectPrint(os);
   os << "native context : " << Brief(native_context());
   os << "\n";
 }
@@ skipping 394 matching lines @@
   }
 }
 
 
 void JSObject::PrintTransitions(std::ostream& os) {  // NOLINT
   TransitionArray::PrintTransitions(os, map()->raw_transitions());
 }
 #endif  // defined(DEBUG) || defined(OBJECT_PRINT)
 }  // namespace internal
 }  // namespace v8