Remove lazy sweeping of new space and corresponding complicated pointer updating logic.

Remove lazy sweeping of new space and corresponding complicated pointer updating logic.

We can do that now since we have the invariant that the store buffer always has valid slots after marking.

BUG=

Committed: https://crrev.com/02ce445394413cd65a8888078bb86fc90d90e3ff
Cr-Commit-Position: refs/heads/master@{#27105}

[Hannes Payer (out of office), 2015-03-09 13:33:16]

hpayer@chromium.org changed reviewers:
+ jarin@chromium.org

[Hannes Payer (out of office), 2015-03-09 20:40:00]

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

https://codereview.chromium.org/991853002/diff/20001/src/heap/mark-compact.cc...
src/heap/mark-compact.cc:3013: // Ignore these pointers here.
Note that we could already filter out these pointers in IsSlotInBlackObject by
calling object->GetHeap()->lo_space()->FindObject for the given slot. I am
worried a bit about performance here, this is why I implemented the filtering
lazily. However, cleaning it up immediately would be cleaner.

[Hannes Payer (out of office), 2015-03-10 10:33:25]

hpayer@chromium.org changed reviewers:
+ ulan@chromium.org

[ulan, 2015-03-10 10:37:33]

lgtm, nice!

https://codereview.chromium.org/991853002/diff/20001/src/heap/store-buffer.cc
File src/heap/store-buffer.cc (left):

https://codereview.chromium.org/991853002/diff/20001/src/heap/store-buffer.cc...
src/heap/store-buffer.cc:395: if (clear_maps) ClearDeadObject(heap_object);
We can remove the ClearDeadObject function now.

[Hannes Payer (out of office), 2015-03-10 13:07:01]

https://codereview.chromium.org/991853002/diff/20001/src/heap/store-buffer.cc
File src/heap/store-buffer.cc (left):

https://codereview.chromium.org/991853002/diff/20001/src/heap/store-buffer.cc...
src/heap/store-buffer.cc:395: if (clear_maps) ClearDeadObject(heap_object);
On 2015/03/10 10:37:32, ulan wrote:
> We can remove the ClearDeadObject function now.

Done.

[Hannes Payer (out of office), 2015-03-10 13:07:05]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2015-03-10 13:07:06]

The patchset sent to the CQ was uploaded after l-g-t-m from ulan@chromium.org
Link to the patchset: https://codereview.chromium.org/991853002/#ps40001 (title:
" ")

[commit-bot: I haz the power, 2015-03-10 13:07:22]

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

[commit-bot: I haz the power, 2015-03-10 13:42:19]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-10 13:42:32]

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