[Interpreter] Local flow control in the bytecode graph builder.

src/interpreter/bytecode-array-iterator.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_INTERPRETER_BYTECODE_ARRAY_ITERATOR_H_
 #define V8_INTERPRETER_BYTECODE_ARRAY_ITERATOR_H_
 
 #include "src/handles.h"
 #include "src/interpreter/bytecodes.h"
 #include "src/objects.h"
@@ skipping 17 matching lines @@
   int8_t GetImmediateOperand(int operand_index) const;
   int GetIndexOperand(int operand_index) const;
   int GetCountOperand(int operand_index) const;
   Register GetRegisterOperand(int operand_index) const;
   Handle<Object> GetConstantForIndexOperand(int operand_index) const;
 
   // Get the raw byte for the given operand. Note: you should prefer using the
   // typed versions above which cast the return to an appropriate type.
   uint32_t GetRawOperand(int operand_index, OperandType operand_type) const;
 
+  // Returns the absolute offset of the branch target if the current bytecode
+  // is jump or conditional jump, kMinInt otherwise.

[rmcilroy, 2015/12/15 17:35:44]

It never actually returns kMinInt since there is an UNREACHABLE there. I'd just
remove this part of the comment and mention this should only be called fi the
current bytecode is a jump.

[oth, 2015/12/15 22:42:17]

Done. Yes, doh!

+  int GetJumpTargetOffset() const;
+
  private:
   Handle<BytecodeArray> bytecode_array_;
   int bytecode_offset_;
 
   DISALLOW_COPY_AND_ASSIGN(BytecodeArrayIterator);
 };
 
 }  // namespace interpreter
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_INTERPRETER_BYTECODE_GRAPH_ITERATOR_H_