[debugger,interpreter] add source position to stack checks.

src/interpreter/bytecode-array-builder.cc

Index: src/interpreter/bytecode-array-builder.cc
diff --git a/src/interpreter/bytecode-array-builder.cc b/src/interpreter/bytecode-array-builder.cc
index 72421437f9b7e1cfced285166bca2e4767313467..f25bc9e8a4f7591cb459f45b7bfe5705a604fbed 100644
--- a/src/interpreter/bytecode-array-builder.cc
+++ b/src/interpreter/bytecode-array-builder.cc
@@ -861,7 +861,12 @@ BytecodeArrayBuilder& BytecodeArrayBuilder::JumpIfUndefined(
   return OutputJump(Bytecode::kJumpIfUndefined, label);
 }
 
-BytecodeArrayBuilder& BytecodeArrayBuilder::StackCheck() {
+BytecodeArrayBuilder& BytecodeArrayBuilder::StackCheck(int position) {
+  DCHECK_NE(RelocInfo::kNoPosition, position);
+  // We need to attach a non-breakable source position to a stack check, so we
+  // simply add it as expression position.
+  source_position_table_builder_.AddExpressionPosition(bytecodes_.size(),

[Michael Starzinger, 2016/04/28 12:06:52]

suggestion: Do you think it would make sense to add a separate
BytecodeArrayBuilder::SetStatementAsExpressionPosition instead of passing in the
position as a plain int? That seems to be more in sync with the rest of the
interface. WDYT?

+                                                       position);
   Output(Bytecode::kStackCheck);
   return *this;
 }