[Interpreter] Add CreateClosure to BytecodeGraphBuilder.

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 5bb208ce94f7e5d8a2550d24cb9d0a1f4268c29b..ef97af294840e423a696f2477bc135479edb1e7a 100644
--- a/src/compiler/bytecode-graph-builder.cc
+++ b/src/compiler/bytecode-graph-builder.cc
@@ -643,7 +643,15 @@ void BytecodeGraphBuilder::VisitPopContext(
 
 void BytecodeGraphBuilder::VisitCreateClosure(
     const interpreter::BytecodeArrayIterator& iterator) {
-  UNIMPLEMENTED();
+  Node* accumulator = environment()->LookupAccumulator();
+  Handle<SharedFunctionInfo> shared_info(
+      OpParameter<Handle<SharedFunctionInfo>>(accumulator));

[Michael Starzinger, 2015/11/20 17:11:43]

Not sure what this OpParameter call is trying to achieve, but I am not aware of
any nodes whose parameter is Handle<SharedFunctionInfo>. This won't work.

If the assumption is that the "accumulator" node is a HeapConstant, then please
use a HeapObjectMatcher to access it.

[oth, 2015/11/21 14:19:47]

A JSCreateClosure node taking a Handle<SharedFunctionInfo> argument is
constructed two lines below.

HeapObjectMatcher is a wrapper around OpParameter. One of these is widely used.

+  PretenureFlag pretenure =
+      iterator.GetImmediateOperand(0) ? TENURED : NOT_TENURED;
+  const Operator* op = javascript()->CreateClosure(shared_info, pretenure);
+  Node* closure = NewNode(op);
+  AddEmptyFrameStateInputs(closure);
+  environment()->BindAccumulator(closure);
 }