Revision: 21326
Author:   [email protected]
Date:     Thu May 15 11:09:22 2014 UTC
Log: Only allow allocation site lifetime transitions from undicided to tenure or don't tenure.

BUG=
[email protected]

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

Modified:
 /branches/bleeding_edge/src/objects-inl.h

=======================================
--- /branches/bleeding_edge/src/objects-inl.h   Mon May 12 15:30:00 2014 UTC
+++ /branches/bleeding_edge/src/objects-inl.h   Thu May 15 11:09:22 2014 UTC
@@ -1587,7 +1587,10 @@
           static_cast<double>(found_count) / create_count : 0.0;
   PretenureFlag current_mode = GetPretenureMode();

-  if (minimum_mementos_created) {
+  // TODO(hpayer): Add an intermediate state MAYBE_TENURE which collects
+  // more lifetime feedback for tenuring candidates. In the meantime, we
+  // just allow transitions from undecided to tenured or not tenured.
+  if (minimum_mementos_created && pretenure_decision() == kUndecided) {
     PretenureDecision result = ratio >= kPretenureRatio
         ? kTenure
         : kDontTenure;

--
--
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/d/optout.

Reply via email to