- Reduce the number of Isolate::Current() calls.

runtime/vm/locations.cc

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/locations.h"
 
 #include "vm/assembler.h"
 #include "vm/il_printer.h"
 #include "vm/intermediate_language.h"
 #include "vm/flow_graph_compiler.h"
 #include "vm/stack_frame.h"
 
 namespace dart {
 
 intptr_t RegisterSet::RegisterCount(intptr_t registers) {
   // Brian Kernighan's algorithm for counting the bits set.
   intptr_t count = 0;
   while (registers != 0) {
     ++count;
     registers &= (registers - 1);  // Clear the least significant bit set.
   }
   return count;
 }
 
 
-LocationSummary::LocationSummary(intptr_t input_count,
+LocationSummary::LocationSummary(Isolate* isolate,
+                                 intptr_t input_count,
                                  intptr_t temp_count,
                                  LocationSummary::ContainsCall contains_call)
-    : input_locations_(input_count),
-      temp_locations_(temp_count),
-      output_locations_(1),
+    : input_locations_(isolate, input_count),
+      temp_locations_(isolate, temp_count),
+      output_locations_(isolate, 1),
       stack_bitmap_(NULL),
       contains_call_(contains_call),
       live_registers_() {
   for (intptr_t i = 0; i < input_count; i++) {
     input_locations_.Add(Location());
   }
   for (intptr_t i = 0; i < temp_count; i++) {
     temp_locations_.Add(Location());
   }
   output_locations_.Add(Location());
   ASSERT(output_locations_.length() == 1);
-  if (contains_call_ != kNoCall) {
-    stack_bitmap_ = new BitmapBuilder();
-  }
 }
 
 
-LocationSummary::LocationSummary(intptr_t input_count,
-                                intptr_t temp_count,
-                                intptr_t output_count,
-                                LocationSummary::ContainsCall contains_call)
-    : input_locations_(input_count),
-      temp_locations_(temp_count),
-      output_locations_(output_count),
+LocationSummary::LocationSummary(Isolate* isolate,
+                                 intptr_t input_count,
+                                 intptr_t temp_count,
+                                 intptr_t output_count,
+                                 LocationSummary::ContainsCall contains_call)
+    : input_locations_(isolate, input_count),
+      temp_locations_(isolate, temp_count),
+      output_locations_(isolate, output_count),
       stack_bitmap_(NULL),
       contains_call_(contains_call),
       live_registers_() {
   for (intptr_t i = 0; i < input_count; i++) {
     input_locations_.Add(Location());
   }
   for (intptr_t i = 0; i < temp_count; i++) {
     temp_locations_.Add(Location());
   }
   // TODO(johnmccutchan): Remove this assertion once support for multiple
   // outputs is complete.
   ASSERT(output_count == 1);
   for (intptr_t i = 0; i < output_count; i++) {
     output_locations_.Add(Location());
   }
-  if (contains_call_ != kNoCall) {
-    stack_bitmap_ = new BitmapBuilder();
-  }
 }
 
 
 LocationSummary* LocationSummary::Make(
     intptr_t input_count,
     Location out,
     LocationSummary::ContainsCall contains_call) {
-  LocationSummary* summary = new LocationSummary(input_count, 0, contains_call);
+  LocationSummary* summary = new LocationSummary(
+      Isolate::Current(),input_count, 0, contains_call);
   for (intptr_t i = 0; i < input_count; i++) {
     summary->set_in(i, Location::RequiresRegister());
   }
   summary->set_out(0, out);
   return summary;
 }
 
 
 Location Location::Pair(Location first, Location second) {
   PairLocation* pair_location = new PairLocation();
@@ skipping 181 matching lines @@
 
   if (!out(0).IsInvalid()) {
     f->Print(" => ");
     out(0).PrintTo(f);
   }
 
   if (always_calls()) f->Print(" C");
 }
 
 }  // namespace dart