[Interpreter] Adds support for throw to bytecode graph builder.

src/compiler/bytecode-graph-builder.cc

Index: src/compiler/bytecode-graph-builder.cc
diff --git a/src/compiler/bytecode-graph-builder.cc b/src/compiler/bytecode-graph-builder.cc
index 47a22df239854b4ef2568736d4c8bcaa87a2a32a..2a80992f8e8b3e4e324a047d831c783cebcda70e 100644
--- a/src/compiler/bytecode-graph-builder.cc
+++ b/src/compiler/bytecode-graph-builder.cc
@@ -816,7 +816,11 @@ void BytecodeGraphBuilder::VisitNew(
 
 void BytecodeGraphBuilder::VisitThrow(
     const interpreter::BytecodeArrayIterator& iterator) {
-  UNIMPLEMENTED();
+  Node* value = environment()->LookupAccumulator();
+  NewNode(javascript()->CallRuntime(Runtime::kReThrow, 1), value);

[Michael Starzinger, 2015/11/26 17:27:45]

This should be Runtime::kThrow, not Runtime::kReThrow, otherwise the exception
message will not be created and the throw location won't be setup.

[Michael Starzinger, 2015/11/27 09:28:46]

I realized later that creating the exception message will require deoptimization
information, which we don't yet have for these graphs. So I would be fine with
keeping kReThrow here for now (until we have deopt info) but at least leave a
big TODO here.

[mythria, 2015/11/27 14:54:47]

Thanks for pointing this. We should use Runtime::kReThrow because interpreter
does not support deopt yet right? I just wanted to be explicit in the TODO
comment.

[Michael Starzinger, 2015/11/27 15:09:18]

Acknowledged. Yep, correct, TODO sounds good to me.

+  Node* control = NewNode(common()->Throw(), value);
+  UpdateControlDependencyToLeaveFunction(control);
+  environment()->BindAccumulator(value);
 }