[coverage] Block coverage with support for IfStatements

test/unittests/interpreter/bytecode-array-builder-unittest.cc

 // Copyright 2014 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/v8.h"
 
 #include "src/ast/scopes.h"
 #include "src/interpreter/bytecode-array-builder.h"
 #include "src/interpreter/bytecode-array-iterator.h"
 #include "src/interpreter/bytecode-jump-table.h"
@@ skipping 353 matching lines @@
   builder.CallRuntime(Runtime::kInlineIsArray, reg_list);
 
   // Emit debugger bytecode.
   builder.Debugger();
 
   // Insert dummy ops to force longer jumps.
   for (int i = 0; i < 256; i++) {
     builder.Debugger();
   }
 
+  // Emit block counter increments.
+  {
+    SaveFlags save_flags;
+    FLAG_block_coverage = true;  // Satisfy DCHECKs.

[Michael Starzinger, 2017/06/06 11:49:32]

suggestion: We could also just remove the DCHECKs in the BytecodeArrayBuilder. I
think those we could live without.

[rmcilroy, 2017/06/06 13:02:32]

+1

[jgruber, 2017/06/06 14:52:46]

Done.

+    builder.IncBlockCounter(0);
+  }
+
   // Bind labels for long jumps at the very end.
   for (size_t i = 0; i < arraysize(end); i++) {
     builder.Bind(&end[i]);
   }
 
   // Return must be the last instruction.
   builder.Return();
 
   // Generate BytecodeArray.
   scope.SetScriptScopeInfo(factory->NewScopeInfo(1));
@@ skipping 444 matching lines @@
     iterator.Advance();
   }
   CHECK_EQ(iterator.current_bytecode(), Bytecode::kReturn);
   iterator.Advance();
   CHECK(iterator.done());
 }
 
 }  // namespace interpreter
 }  // namespace internal
 }  // namespace v8