[wasm] Fix turbolizer output

src/compiler/pipeline.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/compiler/pipeline.h"
 
 #include <fstream>  // NOLINT(readability/streams)
 #include <sstream>
 
 #include "src/base/adapters.h"
@@ skipping 584 matching lines @@
   Linkage linkage_;
 };
 
 PipelineWasmCompilationJob::Status
 PipelineWasmCompilationJob::CreateGraphImpl() {
   return SUCCEEDED;
 }
 
 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;
 }
 
 PipelineWasmCompilationJob::Status
 PipelineWasmCompilationJob::GenerateCodeImpl() {
   pipeline_.GenerateCode(&linkage_);
   return SUCCEEDED;
 }
 
@@ skipping 831 matching lines @@
   ZonePool zone_pool(info->isolate()->allocator());
   PipelineData data(&zone_pool, info, graph, schedule);
   base::SmartPointer<PipelineStatistics> pipeline_statistics;
   if (FLAG_turbo_stats) {
     pipeline_statistics.Reset(new PipelineStatistics(info, &zone_pool));
     pipeline_statistics->BeginPhaseKind("test codegen");
   }
 
   Pipeline pipeline(&data);
   if (data.schedule() == nullptr) {
+    if (FLAG_trace_turbo) {

[ahaas, 2016/04/27 14:45:22]

Why do you write the header only if there is no schedule?

[Clemens Hammacher, 2016/04/27 14:58:55]

Because also the RunPrintAndVerity is only executed if there is no schedule. I
assumed there is some rationale behind that.
But since I do not find it either, I will remove the whole check for
schedule==nullptr if no one objects.

For wasm, it does not matter anyway, since we never pass a schedule. Since there
are no tests for such cases, let's hope that it is correct in the general case.

+      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);
 }
 
 // static
 OptimizedCompileJob* Pipeline::NewCompilationJob(CompilationInfo* info) {
   return new PipelineCompilationJob(info);
@@ skipping 232 matching lines @@
   data->DeleteRegisterAllocationZone();
 }
 
 CompilationInfo* Pipeline::info() const { return data_->info(); }
 
 Isolate* Pipeline::isolate() const { return info()->isolate(); }
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8