[ignition] Add a property call bytecode

src/interpreter/bytecode-array-builder.cc

Index: src/interpreter/bytecode-array-builder.cc
diff --git a/src/interpreter/bytecode-array-builder.cc b/src/interpreter/bytecode-array-builder.cc
index 7f22ccbb4b7eb00efcecdecc0b6e0f38c59bf9fc..3c56ec9068dd0d36e37711ec1422774b7f97394e 100644
--- a/src/interpreter/bytecode-array-builder.cc
+++ b/src/interpreter/bytecode-array-builder.cc
@@ -824,9 +824,15 @@ BytecodeArrayBuilder& BytecodeArrayBuilder::MarkTryEnd(int handler_id) {
 BytecodeArrayBuilder& BytecodeArrayBuilder::Call(Register callable,
                                                  RegisterList args,
                                                  int feedback_slot,
-                                                 TailCallMode tail_call_mode) {
+                                                 TailCallMode tail_call_mode,
+                                                 Call::CallType call_type) {
   if (tail_call_mode == TailCallMode::kDisallow) {
-    OutputCall(callable, args, args.register_count(), feedback_slot);
+    if (call_type == Call::NAMED_PROPERTY_CALL ||
+        call_type == Call::KEYED_PROPERTY_CALL) {
+      OutputCallProperty(callable, args, args.register_count(), feedback_slot);
+    } else {
+      OutputCall(callable, args, args.register_count(), feedback_slot);
+    }
   } else {
     DCHECK(tail_call_mode == TailCallMode::kAllow);

[rmcilroy, 2016/10/26 16:20:16]

We should we do this for tail-calls too? Michi - any idea what is the status of
shipping TCO is?

[Michael Starzinger, 2016/10/26 18:03:01]

It is currently staged as an experimental feature. Shipping plans are unclear. I
don't have strong opinions about whether to add a "TailCallProperty" bytecode or
not, either way is fine with me.

[Leszek Swirski, 2016/10/27 09:08:24]

I left it as-is to not have to introduce another bytecode -- I'd be surprised if
we often had tail-calls for property calls, but I may be proven wrong by, e.g.,
callback style programming

     OutputTailCall(callable, args, args.register_count(), feedback_slot);