Issue 2428493003: [heap] Reland move slot filtering logic into sweeper.

Issue 2428493003: [heap] Reland move slot filtering logic into sweeper. (Closed)

Created:
4 years, 2 months ago by Hannes Payer (out of office)

Modified:
4 years, 2 months ago

Reviewers:
Michael Lippautz, ulan

CC:
v8-reviews_googlegroups.com, Hannes Payer (out of office), ulan

Target Ref:
refs/pending/heads/master

Project:
v8

Visibility:
Public.

[heap] Reland move slot filtering logic into sweeper. BUG=chromium:648568 Committed: https://crrev.com/60cb6013d40c4c202b9389dc4dc2b0be07b4b267 Cr-Commit-Position: refs/heads/master@{#40393}

Created: 4 years, 2 months ago

	Unified diffs	Side-by-side diffs	Stats (+90 lines, -67 lines)
M	src/base/atomic-utils.h	View	1 chunk	+7 lines, -0 lines	0 comments
M	src/heap/mark-compact.cc	View	8 chunks	+29 lines, -10 lines	0 comments
M	src/heap/remembered-set.h	View	1 chunk	+0 lines, -5 lines	0 comments
M	src/heap/remembered-set.cc	View	2 chunks	+0 lines, -19 lines	0 comments
M	src/heap/slot-set.h	View	4 chunks	+32 lines, -17 lines	0 comments
M	test/unittests/heap/slot-set-unittest.cc	View	2 chunks	+22 lines, -16 lines	0 comments

Total messages: 14 (8 generated)

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2428493003/20001

4 years, 2 months ago (2016-10-18 09:40:06 UTC) #11

commit-bot: I haz the power

4 years, 2 months ago (2016-10-18 10:07:17 UTC) #14

Message was sent while issue was closed.

Patchset 2 (id:??) landed as
https://crrev.com/60cb6013d40c4c202b9389dc4dc2b0be07b4b267
Cr-Commit-Position: refs/heads/master@{#40393}