[interpreter] Add checks for source position to test-bytecode-generator.

test/cctest/interpreter/test-bytecode-generator.cc

 // Copyright 2015 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 <fstream>
 
 #include "src/v8.h"
 
 #include "src/compiler.h"
 #include "src/interpreter/bytecode-array-iterator.h"
@@ skipping 90 matching lines @@
 std::string BuildActual(const BytecodeExpectationsPrinter& printer,
                         const char* (&snippet_list)[N],
                         const char* prologue = nullptr,
                         const char* epilogue = nullptr) {
   std::ostringstream actual_stream;
   for (const char* snippet : snippet_list) {
     std::string source_code;
     if (prologue) source_code += prologue;
     source_code += snippet;
     if (epilogue) source_code += epilogue;
+    if (source_code.size() > 0 && source_code.back() != '\n') {
+      source_code += '\n';

[rmcilroy, 2016/05/09 15:36:52]

Does this get hit in the current code? Is there a corresponding implicit newline
in the generate_bytecode_expectations?

+    }
     printer.PrintExpectation(actual_stream, source_code);
   }
   return actual_stream.str();
 }
 
 using ConstantPoolType = BytecodeExpectationsPrinter::ConstantPoolType;
 
 TEST(PrimitiveReturnStatements) {
   InitializedIgnitionHandleScope scope;
   BytecodeExpectationsPrinter printer(CcTest::isolate(),
@@ skipping 2047 matching lines @@
   };
 
   CHECK_EQ(BuildActual(printer, snippets), LoadGolden("Generators.golden"));
 
   FLAG_ignition_generators = old_flag;
 }
 
 }  // namespace interpreter
 }  // namespace internal
 }  // namespace v8