[Interpreter] Add support for global loads / stores / calls to BytecodeGraphBuilder.

src/compiler/bytecode-graph-builder.h

Index: src/compiler/bytecode-graph-builder.h
diff --git a/src/compiler/bytecode-graph-builder.h b/src/compiler/bytecode-graph-builder.h
index d0ea1ff15d64a9f633a27a7af39f4d67ea7656e2..af0e47c1559a8ab81cdaf8e6d5c169f8f2300a9f 100644
--- a/src/compiler/bytecode-graph-builder.h
+++ b/src/compiler/bytecode-graph-builder.h
@@ -86,15 +86,16 @@ class BytecodeGraphBuilder {
 
   void UpdateControlDependencyToLeaveFunction(Node* exit);
 
-  void BuildBinaryOp(const Operator* op,
-                     const interpreter::BytecodeArrayIterator& iterator);
-
-  void BuildNamedLoad(const interpreter::BytecodeArrayIterator& iterator);
-
   Node* ProcessCallArguments(const Operator* call_op,
                              interpreter::Register callee,
                              interpreter::Register receiver, size_t arity);
 
+  void BuildLoadGlobal(const interpreter::BytecodeArrayIterator& iterator,
+                       TypeofMode typeof_mode);
+  void BuildStoreGlobal(const interpreter::BytecodeArrayIterator& iterator);
+  void BuildBinaryOp(const Operator* op,

[mythria, 2015/11/17 10:52:33]

May be we can move BuildBinaryOp above BuildLoadGlobal or move BuildNamedLoad
before BuildBinaryOp. Just to group together all loads at one place.

[oth, 2015/11/17 11:12:50]

Good idea. BuildBinaryOp now sits below BuildCall and the order of the
BuildCalls matches their definition order in bytecode-graph-builder.cc.

+                     const interpreter::BytecodeArrayIterator& iterator);
+  void BuildNamedLoad(const interpreter::BytecodeArrayIterator& iterator);
   void BuildCall(const interpreter::BytecodeArrayIterator& iterator);
 
   // Growth increment for the temporary buffer used to construct input lists to