Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(344)

Issue 7784001: Add specialized sweeper for cell and map space.

Created:
9 years, 3 months ago by Michael Starzinger
Modified:
9 years, 3 months ago
CC:
v8-dev
Visibility:
Public.

Description

Add specialized sweeper for cell and map space. R=vegorov@chromium.org BUG=v8:1613

Patch Set 1 #

Patch Set 2 : Fix specialized sweepers for X64. #

Total comments: 2

Patch Set 3 : Remove obsolete printf used for debugging. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+57 lines, -3 lines) Patch
M src/mark-compact.h View 1 chunk +3 lines, -1 line 0 comments Download
M src/mark-compact.cc View 1 2 3 chunks +54 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Michael Starzinger
9 years, 3 months ago (2011-08-29 15:15:48 UTC) #1
Michael Starzinger
Added new patch set.
9 years, 3 months ago (2011-08-29 16:48:22 UTC) #2
Erik Corry
For data and pointer space we have a small number of buckets and discard the ...
9 years, 3 months ago (2011-08-29 20:09:14 UTC) #3
Michael Starzinger
9 years, 3 months ago (2011-08-30 07:33:07 UTC) #4
Added new patch set.

http://codereview.chromium.org/7784001/diff/2001/src/mark-compact.cc
File src/mark-compact.cc (right):

http://codereview.chromium.org/7784001/diff/2001/src/mark-compact.cc#newcode2695
src/mark-compact.cc:2695: printf("cell:0x%08x mask:0x%08x, idx=%d\n", mark_bits,
mark_mask, cell_index);
On 2011/08/29 20:09:14, Erik Corry wrote:
> You don't want this printf.

Done (ooops).

Powered by Google App Engine
This is Rietveld 408576698