Remove conservative sweeping.

Remove conservative sweeping.

BUG=
R=jarin@chromium.org

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

[Jarin, 2014-08-21 08:29:12]

Very nice. I have a couple of questions.

https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.cc
File src/heap/mark-compact.cc (right):

https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.cc...
src/heap/mark-compact.cc:3303: static int SweepPrecisely(PagedSpace* space,
FreeList* free_list, Page* p,
How about renaming to just "Sweep"?

https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.h
File src/heap/mark-compact.h (left):

https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.h#...
src/heap/mark-compact.h:698: bool sweep_precisely_;
Is this flag still needed?

[Hannes Payer (out of office), 2014-08-21 09:03:17]

https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.cc
File src/heap/mark-compact.cc (right):

https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.cc...
src/heap/mark-compact.cc:3303: static int SweepPrecisely(PagedSpace* space,
FreeList* free_list, Page* p,
On 2014/08/21 08:29:12, jarin wrote:
> How about renaming to just "Sweep"?

Done.

https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.h
File src/heap/mark-compact.h (left):

https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.h#...
src/heap/mark-compact.h:698: bool sweep_precisely_;
On 2014/08/21 08:29:12, jarin wrote:
> Is this flag still needed?

Done.

[Jarin, 2014-08-21 10:59:25]

On 2014/08/21 09:03:17, Hannes Payer wrote:
> https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.cc
> File src/heap/mark-compact.cc (right):
> 
>
https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.cc...
> src/heap/mark-compact.cc:3303: static int SweepPrecisely(PagedSpace* space,
> FreeList* free_list, Page* p,
> On 2014/08/21 08:29:12, jarin wrote:
> > How about renaming to just "Sweep"?
> 
> Done.
> 
> https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.h
> File src/heap/mark-compact.h (left):
> 
>
https://codereview.chromium.org/479113004/diff/20001/src/heap/mark-compact.h#...
> src/heap/mark-compact.h:698: bool sweep_precisely_;
> On 2014/08/21 08:29:12, jarin wrote:
> > Is this flag still needed?
> 
> Done.

lgtm. Very nice cleanup.

I am wondering whether we should also rename CONCURRENT_PRECISE ->
CONCURRENT_SWEEPING, PARALLEL_PRECISE -> PARALLEL_SWEEPING, PRECISE ->
SEQUENTIAL_SWEEPING.

[Hannes Payer (out of office), 2014-08-21 14:48:47]

Yes, makes perfect sense. Thanks!

[Hannes Payer (out of office), 2014-08-21 14:50:29]

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