Introduce {FAST,SLOW}_STRING_WRAPPER_ELEMENTS

src/interpreter/bytecode-array-iterator.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/bytecode-array-iterator.h"
 
 #include "src/objects-inl.h"
 
 namespace v8 {
 namespace internal {
@@ skipping 89 matching lines @@
       return Register::FromWideOperand(static_cast<uint16_t>(operand));
     case OperandSize::kNone:
       UNREACHABLE();
   }
   return Register();
 }
 
 
 Handle<Object> BytecodeArrayIterator::GetConstantForIndexOperand(
     int operand_index) const {
   Handle<FixedArray> constants = handle(bytecode_array()->constant_pool());

[rmcilroy, 2016/01/26 15:53:27]

Just skip the handle creation here and call get on
bytecode_array()->constant_pool() directly ?

[Jakob Kummerow, 2016/01/29 14:31:26]

Most excellent point, dunno why I didn't see that before. Done.

-  return FixedArray::get(constants, GetIndexOperand(operand_index));
+  return FixedArray::get(*constants, GetIndexOperand(operand_index),
+                         constants->GetIsolate());
 }
 
 
 int BytecodeArrayIterator::GetJumpTargetOffset() const {
   Bytecode bytecode = current_bytecode();
   if (interpreter::Bytecodes::IsJumpImmediate(bytecode)) {
     int relative_offset = GetImmediateOperand(0);
     return current_offset() + relative_offset;
   } else if (interpreter::Bytecodes::IsJumpConstant(bytecode) ||
              interpreter::Bytecodes::IsJumpConstantWide(bytecode)) {
     Smi* smi = Smi::cast(*GetConstantForIndexOperand(0));
     return current_offset() + smi->value();
   } else {
     UNREACHABLE();
     return kMinInt;
   }
 }
 
 }  // namespace interpreter
 }  // namespace internal
 }  // namespace v8