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

Unified Diff: src/incremental-marking.cc

Issue 6639024: Get rid of distinction between below- and above-watermark in page allocation.... (Closed) Base URL: http://v8.googlecode.com/svn/branches/experimental/gc/
Patch Set: '' Created 9 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/heap-profiler.cc ('k') | src/liveedit.cc » ('j') | src/spaces-inl.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/incremental-marking.cc
===================================================================
--- src/incremental-marking.cc (revision 7216)
+++ src/incremental-marking.cc (working copy)
@@ -87,7 +87,7 @@
static void ClearMarkbits(PagedSpace* space) {
- PageIterator it(space, PageIterator::PAGES_IN_USE);
+ PageIterator it(space);
while (it.has_next()) {
Page* p = it.next();
@@ -97,16 +97,18 @@
static void ClearMarkbits() {
- // We are sweeping code and map spaces precisely so clearing is not required.
+ // TODO: Clear the mark bits in the sweeper.
ClearMarkbits(Heap::old_pointer_space());
ClearMarkbits(Heap::old_data_space());
ClearMarkbits(Heap::cell_space());
+ ClearMarkbits(Heap::map_space());
+ ClearMarkbits(Heap::code_space());
}
#ifdef DEBUG
static void VerifyMarkbitsAreClean(PagedSpace* space) {
- PageIterator it(space, PageIterator::PAGES_IN_USE);
+ PageIterator it(space);
while (it.has_next()) {
Page* p = it.next();
« no previous file with comments | « src/heap-profiler.cc ('k') | src/liveedit.cc » ('j') | src/spaces-inl.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698