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.