[coverage] Block coverage with support for IfStatements

src/interpreter/bytecodes.h

Index: src/interpreter/bytecodes.h
diff --git a/src/interpreter/bytecodes.h b/src/interpreter/bytecodes.h
index a6c6d0be34761e9153789e2fb672ff4b322e66cd..b4dac2188454a5a3ce7993d1430eee358d7b1da4 100644
--- a/src/interpreter/bytecodes.h
+++ b/src/interpreter/bytecodes.h
@@ -338,6 +338,9 @@ namespace interpreter {
   V(DebugBreakWide, AccumulatorUse::kRead)                                     \
   V(DebugBreakExtraWide, AccumulatorUse::kRead)                                \
                                                                                \
+  /* Block Coverage */                                                         \
+  V(IncBlockCounter, AccumulatorUse::kNone, OperandType::kIdx)                 \

[rmcilroy, 2017/06/01 14:01:50]

Any reason we couldn't just emit the runtime call rather than being a bytecode
given that Ignition and TurboFan just do a runtime call here anyway?

[jgruber, 2017/06/02 06:39:27]

The plan is to remove the runtime call soon with a more efficient implementation
that just loads the CoverageInfo and increments the correct slot. I just haven't
gotten around to it yet.

If you agree, I'd like to land the bytecode in this CL to make the upcoming
transition from the runtime call easier.

[rmcilroy, 2017/06/02 09:35:59]

Sounds good.

+                                                                               \
   /* Illegal bytecode (terminates execution) */                                \
   V(Illegal, AccumulatorUse::kNone)                                            \
                                                                                \