Concurrent/parallel precise sweeping.

Concurrent/parallel precise sweeping.

BUG=
R=jarin@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=22471

[Jarin, 2014-07-18 08:31:30]

https://codereview.chromium.org/398333002/diff/60001/src/mark-compact.cc
File src/mark-compact.cc (right):

https://codereview.chromium.org/398333002/diff/60001/src/mark-compact.cc#newc...
src/mark-compact.cc:3237: return space->Free(start, size);
Could we ASSERT(free_list == NULL) here?

https://codereview.chromium.org/398333002/diff/60001/src/mark-compact.cc#newc...
src/mark-compact.cc:4130: space->set_swept_precisely(sweeper == PRECISE);
I think the flag should be also set if sweeper is CONCURRENT_PRECISE or
PARALLEL_PRECISE.

https://codereview.chromium.org/398333002/diff/60001/src/mark-compact.cc#newc...
src/mark-compact.cc:4289: if (sweep_precisely_) how_to_sweep = PRECISE;
I am wondering whether we want sweep in parallel here (and also in the spaces
below).

[Hannes Payer (out of office), 2014-07-18 10:47:20]

https://codereview.chromium.org/398333002/diff/60001/src/mark-compact.cc
File src/mark-compact.cc (right):

https://codereview.chromium.org/398333002/diff/60001/src/mark-compact.cc#newc...
src/mark-compact.cc:3237: return space->Free(start, size);
On 2014/07/18 08:31:29, jarin wrote:
> Could we ASSERT(free_list == NULL) here?

Done.

https://codereview.chromium.org/398333002/diff/60001/src/mark-compact.cc#newc...
src/mark-compact.cc:4130: space->set_swept_precisely(sweeper == PRECISE);
On 2014/07/18 08:31:29, jarin wrote:
> I think the flag should be also set if sweeper is CONCURRENT_PRECISE or
> PARALLEL_PRECISE.

Good catch! Done.

https://codereview.chromium.org/398333002/diff/60001/src/mark-compact.cc#newc...
src/mark-compact.cc:4289: if (sweep_precisely_) how_to_sweep = PRECISE;
On 2014/07/18 08:31:29, jarin wrote:
> I am wondering whether we want sweep in parallel here (and also in the spaces
> below).

I will address this comment in a separate cl.

[Jarin, 2014-07-18 10:54:11]

lgtm

[Hannes Payer (out of office), 2014-07-18 11:04:31]

Committed patchset #5 manually as r22471 (presubmit successful).