[interpreter] Support for ES6 class literals.

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 cb3bc2fadaeb9a17a3a36afa9fd1b1358b9553d7..ce4af2448cc1c985f6208e0d2f3fd9fcb01b5c67 100644
--- a/src/compiler/bytecode-graph-builder.cc
+++ b/src/compiler/bytecode-graph-builder.cc
@@ -978,6 +978,13 @@ void BytecodeGraphBuilder::VisitKeyedStoreICStrictWide() {
   BuildKeyedStore();
 }
 
+void BytecodeGraphBuilder::VisitLdaInitialMap() {
+  FrameStateBeforeAndAfter states(this);

[Michael Starzinger, 2016/02/04 13:40:20]

nit: Shouldn't need a frame state. Can we drop this line?

[oth, 2016/02/04 14:00:59]

Done.

+  Node* js_function = environment()->LookupAccumulator();
+  environment()->BindAccumulator(BuildLoadObjectField(

[Michael Starzinger, 2016/02/04 13:40:20]

nit: This is a dangerous pattern, because C++ does not specify whether
"environment()" or "BuildLoadObjectField()" is evaluated first. So if the latter
modifies the current environment, then we are relying on implementation-specific
behavior here. Can we instead pull out the result into a local variable?

[oth, 2016/02/04 14:00:59]

Thanks, aware of the issue. Hadn't thought this instance through.

+      js_function, JSFunction::kPrototypeOrInitialMapOffset));
+}
+
 void BytecodeGraphBuilder::VisitPushContext() {
   Node* new_context = environment()->LookupAccumulator();
   environment()->BindRegister(bytecode_iterator().GetRegisterOperand(0),