[wasm] Fix turbolizer output

The header / start of the json file was missing before.

R=ahaas@chromium.org, titzer@chromium.org

Review-Url: https://codereview.chromium.org/1928503002
Cr-Commit-Position: refs/heads/master@{#35901}

src/compiler/pipeline.cc

@@ -572,6 +572,8 @@ PipelineCompilationJob::Status PipelineCompilationJob::GenerateCodeImpl() {
   return SUCCEEDED;
 }
 
+}  // namespace
+
 class PipelineWasmCompilationJob final : public CompilationJob {
  public:
   explicit PipelineWasmCompilationJob(CompilationInfo* info, Graph* graph,
@@ -602,6 +604,18 @@ PipelineWasmCompilationJob::CreateGraphImpl() {
 
 PipelineWasmCompilationJob::Status
 PipelineWasmCompilationJob::OptimizeGraphImpl() {
+  if (FLAG_trace_turbo) {
+    FILE* json_file = OpenVisualizerLogFile(info(), nullptr, "json", "w+");
+    if (json_file != nullptr) {
+      OFStream json_of(json_file);
+      json_of << "{\"function\":\"" << info()->GetDebugName().get()
+              << "\", \"source\":\"\",\n\"phases\":[";
+      fclose(json_file);
+    }
+  }
+
+  pipeline_.RunPrintAndVerify("Machine", true);
+
   if (!pipeline_.ScheduleAndSelectInstructions(&linkage_)) return FAILED;
   return SUCCEEDED;
 }
@@ -612,8 +626,6 @@ PipelineWasmCompilationJob::GenerateCodeImpl() {
   return SUCCEEDED;
 }
 
-}  // namespace
-
 
 template <typename Phase>
 void Pipeline::Run() {
@@ -1452,10 +1464,18 @@ Handle<Code> Pipeline::GenerateCodeForTesting(CompilationInfo* info,
   }
 
   Pipeline pipeline(&data);
-  if (data.schedule() == nullptr) {
+
+  if (FLAG_trace_turbo) {
+    FILE* json_file = OpenVisualizerLogFile(info, nullptr, "json", "w+");
+    if (json_file != nullptr) {
+      OFStream json_of(json_file);
+      json_of << "{\"function\":\"" << info->GetDebugName().get()
+              << "\", \"source\":\"\",\n\"phases\":[";
+      fclose(json_file);
+    }
+  }
   // TODO(rossberg): Should this really be untyped?
   pipeline.RunPrintAndVerify("Machine", true);
-  }
 
   return pipeline.ScheduleAndGenerateCode(call_descriptor);
 }

src/compiler/pipeline.h

@@ -80,10 +80,11 @@ class Pipeline {
       CompilationInfo* info, Graph* graph, CallDescriptor* descriptor,
       SourcePositionTable* source_positions);
 
-  // TODO(mstarzinger, bmeurer): This shouldn't be public!
-  bool ScheduleAndSelectInstructions(Linkage* linkage);
-
  private:
+  // The wasm compilation job calls ScheduleAndSelectInstructions and
+  // RunPrintAndVerify, so we make it a member class.
+  friend class PipelineWasmCompilationJob;
+
   // Helpers for executing pipeline phases.
   template <typename Phase>
   void Run();
@@ -94,6 +95,7 @@ class Pipeline {
 
   void BeginPhaseKind(const char* phase_kind);
   void EndPhaseKind();
+  bool ScheduleAndSelectInstructions(Linkage* linkage);
   void RunPrintAndVerify(const char* phase, bool untyped = false);
   Handle<Code> ScheduleAndGenerateCode(CallDescriptor* call_descriptor);
   void AllocateRegisters(const RegisterConfiguration* config,