Chromium Code Reviews
Help | Chromium Project | Sign in
(207)

Issue 11410080: Forward transition objects to grey if they are referenced by a large object (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 5 months ago by payer
Modified:
1 year, 5 months ago
Reviewers:
Michael Starzinger
CC:
v8-dev_googlegroups.com
Visibility:
Public.

Description

Forward transition objects to grey if they are referenced by a large object


BUG=

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

Patch Set 3 : #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -5 lines) Lint Patch
M src/incremental-marking-inl.h View 1 2 1 chunk +11 lines, -5 lines 4 comments 1 errors Download
Trybot results:
Commit:

Messages

Total messages: 5
payer
1 year, 5 months ago #1
Michael Starzinger
https://codereview.chromium.org/11410080/diff/1/src/incremental-marking-inl.h File src/incremental-marking-inl.h (right): https://codereview.chromium.org/11410080/diff/1/src/incremental-marking-inl.h#newcode55 src/incremental-marking-inl.h:55: return false; This will prevent the slot from being ...
1 year, 5 months ago #2
payer
https://codereview.chromium.org/11410080/diff/1/src/incremental-marking-inl.h File src/incremental-marking-inl.h (right): https://codereview.chromium.org/11410080/diff/1/src/incremental-marking-inl.h#newcode55 src/incremental-marking-inl.h:55: return false; On 2012/11/14 08:42:24, Michael Starzinger wrote: > ...
1 year, 5 months ago #3
Michael Starzinger
https://codereview.chromium.org/11410080/diff/6002/src/incremental-marking-inl.h File src/incremental-marking-inl.h (right): https://codereview.chromium.org/11410080/diff/6002/src/incremental-marking-inl.h#newcode44 src/incremental-marking-inl.h:44: // TODO(hpayer): make decision based on if object has ...
1 year, 5 months ago #4
payer
1 year, 5 months ago #5
https://codereview.chromium.org/11410080/diff/6002/src/incremental-marking-inl.h
File src/incremental-marking-inl.h (right):

https://codereview.chromium.org/11410080/diff/6002/src/incremental-marking-in...
src/incremental-marking-inl.h:44: // TODO(hpayer): make decision based on if
object has a progress bar
On 2012/11/15 18:04:26, Michael Starzinger wrote:
> As discussed offline, can we rebase that to the brand new progress bar and
check
> for the HAS_PROGRESS_BAR page flag here?

Done.

I will add the optimization in the next CL.

https://codereview.chromium.org/11410080/diff/6002/src/incremental-marking-in...
src/incremental-marking-inl.h:49: } else if (Marking::IsBlack(obj_bit)) {
On 2012/11/15 18:04:26, Michael Starzinger wrote:
> Can we rewrite the condition so that it does the Marking::IsBlack only once,
> unfortunately GCC isn't smart enough to optimize it away.

Done.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1275:d14800f88434