Revision: 18547
Author:   [email protected]
Date:     Fri Jan 10 20:31:03 2014 UTC
Log:      Turn on allocation site pretenuring.

BUG=
[email protected]

Review URL: https://codereview.chromium.org/133993006
http://code.google.com/p/v8/source/detail?r=18547

Modified:
 /branches/bleeding_edge/src/flag-definitions.h
 /branches/bleeding_edge/test/cctest/test-heap.cc

=======================================
--- /branches/bleeding_edge/src/flag-definitions.h Fri Jan 10 15:35:19 2014 UTC +++ /branches/bleeding_edge/src/flag-definitions.h Fri Jan 10 20:31:03 2014 UTC
@@ -220,7 +220,7 @@
 // TODO(hpayer): We will remove this flag as soon as we have pretenuring
 // support for specific allocation sites.
 DEFINE_bool(pretenuring_call_new, false, "pretenure call new")
-DEFINE_bool(allocation_site_pretenuring, false,
+DEFINE_bool(allocation_site_pretenuring, true,
             "pretenure with allocation sites")
 DEFINE_bool(trace_pretenuring, false,
             "trace pretenuring decisions of HAllocate instructions")
=======================================
--- /branches/bleeding_edge/test/cctest/test-heap.cc Fri Jan 10 15:35:19 2014 UTC +++ /branches/bleeding_edge/test/cctest/test-heap.cc Fri Jan 10 20:31:03 2014 UTC
@@ -2206,10 +2206,10 @@
       "  this.e = 1.3;"
       "  this.f = [{}];"
       "}"
-      "var number_elements = 20000;"
+      "var number_elements = 30000;"
       "var elements = new Array();"
       "function f() {"
-      "  for (var i = 0; i < 20000-1; i++) {"
+      "  for (var i = 0; i < number_elements-1; i++) {"
       "    elements[i] = new DataObject();"
       "  }"
       "  return new DataObject()"
@@ -2240,7 +2240,7 @@
   CcTest::heap()->SetNewSpaceHighPromotionModeActive(true);

   v8::Local<v8::Value> res = CompileRun(
-      "var number_elements = 3000;"
+      "var number_elements = 20000;"
       "var elements = new Array(number_elements);"
       "function DataObject() {"
       "  this.a = [{}];"
@@ -2345,7 +2345,7 @@
   v8::HandleScope scope(CcTest::isolate());

   v8::Local<v8::Value> res = CompileRun(
-      "var number_elements = 20000;"
+      "var number_elements = 30000;"
       "var elements = new Array(number_elements);"
       "function f() {"
       "  for (var i = 0; i < number_elements; i++) {"

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to