Reviewers: Kevin Millikin,

Description:
Reuse backing store of two zone lists by using Rewind(0) instead of Clear().


Please review this at http://codereview.chromium.org/6647019/

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/hydrogen.h
  M     src/hydrogen.cc


Index: src/hydrogen.cc
===================================================================
--- src/hydrogen.cc     (revision 7102)
+++ src/hydrogen.cc     (working copy)
@@ -640,7 +640,7 @@
   HBasicBlock* start = blocks_[0];
   Postorder(start, &visited, &reverse_result, NULL);

-  blocks_.Clear();
+  blocks_.Rewind(0);
   int index = 0;
   for (int i = reverse_result.length() - 1; i >= 0; --i) {
     HBasicBlock* b = reverse_result[i];
Index: src/hydrogen.h
===================================================================
--- src/hydrogen.h      (revision 7102)
+++ src/hydrogen.h      (working copy)
@@ -379,7 +379,7 @@
   void ClearHistory() {
     pop_count_ = 0;
     push_count_ = 0;
-    assigned_variables_.Clear();
+    assigned_variables_.Rewind(0);
   }

   void SetValueAt(int index, HValue* value) {


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to