Initialize zone lists in the register allocator with more reasonable initial ...

src/lithium-allocator.h

Index: src/lithium-allocator.h
===================================================================
--- src/lithium-allocator.h	(revision 7109)
+++ src/lithium-allocator.h	(working copy)
@@ -428,22 +428,7 @@
 
 class LAllocator BASE_EMBEDDED {
  public:
-  explicit LAllocator(int first_virtual_register, HGraph* graph)
-      : chunk_(NULL),
-        live_in_sets_(0),
-        live_ranges_(16),
-        fixed_live_ranges_(8),
-        fixed_double_live_ranges_(8),
-        unhandled_live_ranges_(8),
-        active_live_ranges_(8),
-        inactive_live_ranges_(8),
-        reusable_slots_(8),
-        next_virtual_register_(first_virtual_register),
-        first_artificial_register_(first_virtual_register),
-        mode_(NONE),
-        num_registers_(-1),
-        graph_(graph),
-        has_osr_entry_(false) {}
+  LAllocator(int first_virtual_register, HGraph* graph);
 
   static void Setup();
   static void TraceAlloc(const char* msg, ...);
@@ -468,10 +453,10 @@
   void Allocate(LChunk* chunk);
 
   const ZoneList<LiveRange*>* live_ranges() const { return &live_ranges_; }
-  const ZoneList<LiveRange*>* fixed_live_ranges() const {
+  const Vector<LiveRange*>* fixed_live_ranges() const {
     return &fixed_live_ranges_;
   }
-  const ZoneList<LiveRange*>* fixed_double_live_ranges() const {
+  const Vector<LiveRange*>* fixed_double_live_ranges() const {
     return &fixed_double_live_ranges_;
   }
 
@@ -616,8 +601,10 @@
   ZoneList<LiveRange*> live_ranges_;
 
   // Lists of live ranges
-  ZoneList<LiveRange*> fixed_live_ranges_;
-  ZoneList<LiveRange*> fixed_double_live_ranges_;
+  EmbeddedVector<LiveRange*,

[Kevin Millikin (Chromium), 2011/03/10 11:23:08]

No matter what you do here, it will be a nasty line break.  I like to break at
the lowest precedence operator that makes it fit on the fewest lines.  Here, the
lowest precedence ought to be the space between the type and identifier:

EmbeddedVector<LiveRange*, Register::kNumAllocatableRegisters>
    fixed_live_ranges_;

I like that a little better, but what do you think?

[fschneider, 2011/03/10 12:04:16]

I'm fine with both.

+      Register::kNumAllocatableRegisters> fixed_live_ranges_;
+  EmbeddedVector<LiveRange*,
+      DoubleRegister::kNumAllocatableRegisters> fixed_double_live_ranges_;
   ZoneList<LiveRange*> unhandled_live_ranges_;
   ZoneList<LiveRange*> active_live_ranges_;
   ZoneList<LiveRange*> inactive_live_ranges_;