Remove dangerous constructor from CompilationInfoWithZone.

test/cctest/compiler/test-instruction.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 "test/cctest/cctest.h"
 
 #include "src/compiler/code-generator.h"
 #include "src/compiler/common-operator.h"
 #include "src/compiler/graph.h"
@@ skipping 13 matching lines @@
 typedef v8::internal::compiler::InstructionSequence TestInstrSeq;
 
 // A testing helper for the register code abstraction.
 class InstructionTester : public HandleAndZoneScope {
  public:  // We're all friends here.
   InstructionTester()
       : isolate(main_isolate()),
         graph(zone()),
         schedule(zone()),
         fake_stub(main_isolate()),
-        info(&fake_stub, main_isolate()),
+        info(&fake_stub, main_isolate(), zone()),
         common(zone()),
         machine(zone()),
         code(NULL) {}
 
   Isolate* isolate;
   Graph graph;
   Schedule schedule;
   FakeStubForTesting fake_stub;
-  CompilationInfoWithZone info;
+  CompilationInfo info;

[titzer, 2015/03/23 17:04:34]

Do we even need this field anymore?

[Michael Starzinger, 2015/03/23 17:10:54]

Done. Same for some other fields.

   CommonOperatorBuilder common;
   MachineOperatorBuilder machine;
   TestInstrSeq* code;
 
   Zone* zone() { return main_zone(); }
 
   void allocCode() {
     if (schedule.rpo_order()->size() == 0) {
       // Compute the RPO order.
       Scheduler::ComputeSpecialRPO(main_zone(), &schedule);
@@ skipping 280 matching lines @@
           CHECK(inputs[z].Equals(m->InputAt(z)));
         }
 
         for (size_t z = 0; z < k; z++) {
           CHECK(temps[z].Equals(m->TempAt(z)));
         }
       }
     }
   }
 }