[Interpreter] Implement ForIn in 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 f4c5055ec043a9b788e0e4b61fb546bd5b5deb2d..0a38b8934cebf7aee0ecbeea300a465f35c82d1a 100644
--- a/src/compiler/bytecode-graph-builder.cc
+++ b/src/compiler/bytecode-graph-builder.cc
@@ -1013,7 +1013,6 @@ void BytecodeGraphBuilder::BuildBinaryOp(
   Node* left = environment()->LookupRegister(iterator.GetRegisterOperand(0));
   Node* right = environment()->LookupAccumulator();
   Node* node = NewNode(js_op, left, right);
-
   AddEmptyFrameStateInputs(node);
   environment()->BindAccumulator(node);
 }
@@ -1369,19 +1368,53 @@ void BytecodeGraphBuilder::VisitReturn(
 
 void BytecodeGraphBuilder::VisitForInPrepare(
     const interpreter::BytecodeArrayIterator& iterator) {
-  UNIMPLEMENTED();
+  Node* receiver =
+      environment()->LookupRegister(iterator.GetRegisterOperand(0));
+  Node* property_names = environment()->LookupAccumulator();
+  const Operator* call_runtime =
+      javascript()->CallRuntime(Runtime::kInterpreterForInPrepare, 2);

[Benedikt Meurer, 2015/12/16 10:53:25]

What about JSForInPrepare? Calling interpreter runtime functions from TurboFan
doesn't look right to me.

+  Node** args = info()->zone()->NewArray<Node*>(2);
+  args[0] = receiver;
+  args[1] = property_names;
+  Node* value = MakeNode(call_runtime, 2, args, false);
+  AddEmptyFrameStateInputs(value);
+  environment()->BindAccumulator(value);
+}
+
+
+void BytecodeGraphBuilder::VisitForInDone(
+    const interpreter::BytecodeArrayIterator& iterator) {
+  Node* index = environment()->LookupRegister(iterator.GetRegisterOperand(0));
+  Node* cache_length =
+      environment()->LookupRegister(iterator.GetRegisterOperand(1));
+  Node* exit_cond = NewNode(javascript()->ForInDone(), index, cache_length);
+  AddEmptyFrameStateInputs(exit_cond);
+  environment()->BindAccumulator(exit_cond);
 }
 
 
 void BytecodeGraphBuilder::VisitForInNext(
     const interpreter::BytecodeArrayIterator& iterator) {
-  UNIMPLEMENTED();
+  Node* receiver =
+      environment()->LookupRegister(iterator.GetRegisterOperand(0));
+  Node* index = environment()->LookupRegister(iterator.GetRegisterOperand(1));
+  Node* cache_type =
+      environment()->LookupRegister(iterator.GetRegisterOperand(2));
+  Node* cache_array =
+      environment()->LookupRegister(iterator.GetRegisterOperand(3));
+  Node* value = NewNode(javascript()->ForInNext(), receiver, cache_array,
+                        cache_type, index);
+  AddEmptyFrameStateInputs(value);
+  environment()->BindAccumulator(value);
 }
 
 
-void BytecodeGraphBuilder::VisitForInDone(
+void BytecodeGraphBuilder::VisitForInStep(
     const interpreter::BytecodeArrayIterator& iterator) {
-  UNIMPLEMENTED();
+  Node* index = environment()->LookupRegister(iterator.GetRegisterOperand(0));
+  index = NewNode(javascript()->ForInStep(), index);
+  AddEmptyFrameStateInputs(index);
+  environment()->BindRegister(iterator.GetRegisterOperand(0), index);
 }