[turbofan] use ZonePool in most places in the compiler pipeline a temp zone is used.

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 <sstream>
 
 #include "src/base/platform/elapsed-timer.h"
 #include "src/compiler/ast-graph-builder.h"
@@ skipping 160 matching lines @@
 void Pipeline::PrintAllocator(const char* phase,
                               const RegisterAllocator* allocator) {
   std::ofstream turbo_cfg_stream;
   OpenTurboCfgFile(&turbo_cfg_stream);
   turbo_cfg_stream << AsC1VAllocator(phase, allocator);
 }
 
 
 class AstGraphBuilderWithPositions : public AstGraphBuilder {
  public:
-  explicit AstGraphBuilderWithPositions(CompilationInfo* info, JSGraph* jsgraph,
+  explicit AstGraphBuilderWithPositions(Zone* local_zone, CompilationInfo* info,
+                                        JSGraph* jsgraph,
                                         SourcePositionTable* source_positions)
-      : AstGraphBuilder(info, jsgraph), source_positions_(source_positions) {}
+      : AstGraphBuilder(local_zone, info, jsgraph),
+        source_positions_(source_positions) {}
 
   bool CreateGraph() {
     SourcePositionTable::Scope pos(source_positions_,
                                    SourcePosition::Unknown());
     return AstGraphBuilder::CreateGraph();
   }
 
 #define DEF_VISIT(type)                                               \
   virtual void Visit##type(type* node) OVERRIDE {                     \
     SourcePositionTable::Scope pos(source_positions_,                 \
@@ skipping 52 matching lines @@
   // typer could sweep over later.
   Typer typer(&graph, info()->context());
   MachineOperatorBuilder machine;
   CommonOperatorBuilder common(zone());
   JSOperatorBuilder javascript(zone());
   JSGraph jsgraph(&graph, &common, &javascript, &machine);
   Node* context_node;
   {
     PhaseStats graph_builder_stats(info(), &zone_pool, PhaseStats::CREATE_GRAPH,
                                    "graph builder");
-    AstGraphBuilderWithPositions graph_builder(info(), &jsgraph,
-                                               &source_positions);
+    ZonePool::Scope zone_scope(&zone_pool);
+    AstGraphBuilderWithPositions graph_builder(zone_scope.zone(), info(),
+                                               &jsgraph, &source_positions);
     graph_builder.CreateGraph();
     context_node = graph_builder.GetFunctionContext();
   }
   {
     PhaseStats phi_reducer_stats(info(), &zone_pool, PhaseStats::CREATE_GRAPH,
                                  "phi reduction");
     PhiReducer phi_reducer;
     GraphReducer graph_reducer(&graph);
     graph_reducer.AddReducer(&phi_reducer);
     graph_reducer.ReduceGraph();
     // TODO(mstarzinger): Running reducer once ought to be enough for everyone.
     graph_reducer.ReduceGraph();
     graph_reducer.ReduceGraph();
   }
 
   VerifyAndPrintGraph(&graph, "Initial untyped", true);
 
   if (info()->is_context_specializing()) {
     SourcePositionTable::Scope pos(&source_positions,
                                    SourcePosition::Unknown());
     // Specialize the code to the context as aggressively as possible.
     JSContextSpecializer spec(info(), &jsgraph, context_node);
     spec.SpecializeToContext();
     VerifyAndPrintGraph(&graph, "Context specialized", true);
   }
 
   if (info()->is_inlining_enabled()) {
     SourcePositionTable::Scope pos(&source_positions,
                                    SourcePosition::Unknown());
-    JSInliner inliner(info(), &jsgraph);
+    ZonePool::Scope zone_scope(&zone_pool);
+    JSInliner inliner(zone_scope.zone(), info(), &jsgraph);
     inliner.Inline();
     VerifyAndPrintGraph(&graph, "Inlined", true);
   }
 
   // Print a replay of the initial graph.
   if (FLAG_print_turbo_replay) {
     GraphReplayPrinter::PrintReplay(&graph);
   }
 
   // Bailout here in case target architecture is not supported.
@@ skipping 62 matching lines @@
 
       // TODO(jarin, rossberg): Remove UNTYPED once machine typing works.
       VerifyAndPrintGraph(&graph, "Lowered changes", true);
     }
 
     {
       SourcePositionTable::Scope pos(&source_positions,
                                      SourcePosition::Unknown());
       PhaseStats control_reducer_stats(
           info(), &zone_pool, PhaseStats::CREATE_GRAPH, "control reduction");
-      ControlReducer::ReduceGraph(&jsgraph, &common);
+      ZonePool::Scope zone_scope(&zone_pool);
+      ControlReducer::ReduceGraph(zone_scope.zone(), &jsgraph, &common);
 
       VerifyAndPrintGraph(&graph, "Control reduced");
     }
   }
 
   {
     // Lower any remaining generic JSOperators.
     PhaseStats lowering_stats(info(), &zone_pool, PhaseStats::CREATE_GRAPH,
                               "generic lowering");
     SourcePositionTable::Scope pos(&source_positions,
@@ skipping 10 matching lines @@
   source_positions.RemoveDecorator();
 
   Handle<Code> code = Handle<Code>::null();
   {
     // Compute a schedule.
     Schedule* schedule = ComputeSchedule(&zone_pool, &graph);
     // Generate optimized code.
     PhaseStats codegen_stats(info(), &zone_pool, PhaseStats::CODEGEN,
                              "codegen");
     Linkage linkage(info());
-    code = GenerateCode(&linkage, &graph, schedule, &source_positions);
+    code =
+        GenerateCode(&zone_pool, &linkage, &graph, schedule, &source_positions);
     info()->SetCode(code);
   }
 
   // Print optimized code.
   v8::internal::CodeGenerator::PrintCode(code, info());
 
   if (FLAG_trace_turbo) {
     OFStream os(stdout);
     os << "--------------------------------------------------\n"
        << "Finished compiling method "
@@ skipping 11 matching lines @@
   Schedule* schedule = Scheduler::ComputeSchedule(zone_pool, graph);
   TraceSchedule(schedule);
   if (VerifyGraphs()) ScheduleVerifier::Run(schedule);
   return schedule;
 }
 
 
 Handle<Code> Pipeline::GenerateCodeForMachineGraph(Linkage* linkage,
                                                    Graph* graph,
                                                    Schedule* schedule) {
+  ZonePool zone_pool(isolate());
   CHECK(SupportedBackend());
   if (schedule == NULL) {
     // TODO(rossberg): Should this really be untyped?
     VerifyAndPrintGraph(graph, "Machine", true);
-    ZonePool zone_pool(isolate());
     schedule = ComputeSchedule(&zone_pool, graph);
   }
   TraceSchedule(schedule);
 
   SourcePositionTable source_positions(graph);
-  Handle<Code> code = GenerateCode(linkage, graph, schedule, &source_positions);
+  Handle<Code> code =
+      GenerateCode(&zone_pool, linkage, graph, schedule, &source_positions);
 #if ENABLE_DISASSEMBLER
   if (!code.is_null() && FLAG_print_opt_code) {
     CodeTracer::Scope tracing_scope(isolate()->GetCodeTracer());
     OFStream os(tracing_scope.file());
     code->Disassemble("test code", os);
   }
 #endif
   return code;
 }
 
 
-Handle<Code> Pipeline::GenerateCode(Linkage* linkage, Graph* graph,
-                                    Schedule* schedule,
+Handle<Code> Pipeline::GenerateCode(ZonePool* zone_pool, Linkage* linkage,
+                                    Graph* graph, Schedule* schedule,
                                     SourcePositionTable* source_positions) {
   DCHECK_NOT_NULL(graph);
   DCHECK_NOT_NULL(linkage);
   DCHECK_NOT_NULL(schedule);
   CHECK(SupportedBackend());
 
   BasicBlockProfiler::Data* profiler_data = NULL;
   if (FLAG_turbo_profiling) {
     profiler_data = BasicBlockInstrumentor::Instrument(info_, graph, schedule);
   }
 
   Zone* instruction_zone = schedule->zone();
   InstructionSequence sequence(instruction_zone, graph, schedule);
 
   // Select and schedule instructions covering the scheduled graph.
   {
-    InstructionSelector selector(linkage, &sequence, schedule,
-                                 source_positions);
+    ZonePool::Scope zone_scope(zone_pool);
+    InstructionSelector selector(zone_scope.zone(), linkage, &sequence,
+                                 schedule, source_positions);
     selector.SelectInstructions();
   }
 
   if (FLAG_trace_turbo) {
     OFStream os(stdout);
     os << "----- Instruction sequence before register allocation -----\n"
        << sequence;
     PrintScheduleAndInstructions("CodeGen", schedule, source_positions,
                                  &sequence);
   }
 
   // Allocate registers.
   Frame frame;
   {
     int node_count = graph->NodeCount();
     if (node_count > UnallocatedOperand::kMaxVirtualRegisters) {
       linkage->info()->AbortOptimization(kNotEnoughVirtualRegistersForValues);
       return Handle<Code>::null();
     }
-    RegisterAllocator allocator(&frame, linkage->info(), &sequence);
-    if (!allocator.Allocate()) {
+    ZonePool::Scope zone_scope(zone_pool);
+    RegisterAllocator allocator(zone_scope.zone(), &frame, linkage->info(),
+                                &sequence);
+    if (!allocator.Allocate(zone_pool)) {
       linkage->info()->AbortOptimization(kNotEnoughVirtualRegistersRegalloc);
       return Handle<Code>::null();
     }
     if (FLAG_trace_turbo) {
       PrintAllocator("CodeGen", &allocator);
     }
   }
 
   if (FLAG_trace_turbo) {
     OFStream os(stdout);
@@ skipping 20 matching lines @@
 }
 
 
 void Pipeline::TearDown() {
   InstructionOperand::TearDownCaches();
 }
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8