GC. Delay/avoid entering high promotion mode

GC. Delay/avoid entering high promotion mode

R=hpayer@chromium.org
BUG=

Committed: https://crrev.com/a5221d0779d0839055c764b88f412a0793d959b3
Cr-Commit-Position: refs/heads/master@{#29429}

[Erik Corry Chromium.org, 2015-07-01 08:46:25]

The CQ bit was checked by erikcorry@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-07-01 08:46:35]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1217813012/1

[commit-bot: I haz the power, 2015-07-01 09:19:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-07-01 09:19:48]

Dry run: This issue passed the CQ dry run.

[Hannes Payer (out of office), 2015-07-01 09:53:20]

https://codereview.chromium.org/1217813012/diff/1/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1217813012/diff/1/src/heap/heap.cc#newcode1257
src/heap/heap.cc:1257: if (!deopted) {
There is already logic to avoid shrinking of young generation when we deopt
because of pretenuring. We can assume that the deopt will happen soon. It should
be good enough to wait for 2 GCs then, WDYT?

[Erik Corry Chromium.org, 2015-07-01 14:46:40]

PTAL - simpler approach that doesn't involve the compiler

[Erik Corry Chromium.org, 2015-07-01 15:19:28]

The CQ bit was checked by erikcorry@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-07-01 15:19:36]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1217813012/20001

[Hannes Payer (out of office), 2015-07-01 15:31:31]

looking good, just naming nits

https://codereview.chromium.org/1217813012/diff/20001/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1217813012/diff/20001/src/heap/heap.cc#newcod...
src/heap/heap.cc:2641: bool hope_high_promotion_problem_will_fix_itself =
funny name :) but can we please have a shorter version.
still_gathering_lifetime_data? or something?

https://codereview.chromium.org/1217813012/diff/20001/src/heap/heap.h
File src/heap/heap.h (right):

https://codereview.chromium.org/1217813012/diff/20001/src/heap/heap.h#newcode...
src/heap/heap.h:1378:
recently_deopted_code_that_will_be_reopted_with_pretenuring_ = 2;
it is not clear if the code will be tenured. you could call it:
gather_lifetime_feedback

[Erik Corry Chromium.org, 2015-07-01 15:37:54]

The CQ bit was checked by erikcorry@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-07-01 15:38:05]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1217813012/40001

[Erik Corry Chromium.org, 2015-07-01 15:38:40]

https://codereview.chromium.org/1217813012/diff/20001/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1217813012/diff/20001/src/heap/heap.cc#newcod...
src/heap/heap.cc:2641: bool hope_high_promotion_problem_will_fix_itself =
On 2015/07/01 15:31:31, Hannes Payer wrote:
> funny name :) but can we please have a shorter version.
> still_gathering_lifetime_data? or something?

Done.

https://codereview.chromium.org/1217813012/diff/20001/src/heap/heap.h
File src/heap/heap.h (right):

https://codereview.chromium.org/1217813012/diff/20001/src/heap/heap.h#newcode...
src/heap/heap.h:1378:
recently_deopted_code_that_will_be_reopted_with_pretenuring_ = 2;
On 2015/07/01 15:31:31, Hannes Payer wrote:
> it is not clear if the code will be tenured. you could call it:
> gather_lifetime_feedback

Renamed to gathering_lifetime_feedback_

[commit-bot: I haz the power, 2015-07-01 16:05:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-07-01 16:05:51]

Dry run: This issue passed the CQ dry run.

[Hannes Payer (out of office), 2015-07-01 17:40:30]

LGTM

[Hannes Payer (out of office), 2015-07-01 17:40:36]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2015-07-01 17:40:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1217813012/40001

[commit-bot: I haz the power, 2015-07-01 17:42:22]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-07-01 17:42:36]

Patchset 3 (id:??) landed as
https://crrev.com/a5221d0779d0839055c764b88f412a0793d959b3
Cr-Commit-Position: refs/heads/master@{#29429}