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

Unified Diff: src/heap/mark-compact.h

Issue 1863983002: 🏄 [heap] Add page evacuation mode for new->old (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Cleanup Created 4 years, 8 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
Index: src/heap/mark-compact.h
diff --git a/src/heap/mark-compact.h b/src/heap/mark-compact.h
index cd207bcda27ba792ec2291abead68006c86932be..aa7685829fd82c9e0aeae882ca984b224e38ff3f 100644
--- a/src/heap/mark-compact.h
+++ b/src/heap/mark-compact.h
@@ -498,6 +498,9 @@ class MarkCompactCollector {
int SweepInParallel(PagedSpace* space, int required_freed_bytes,
int max_pages = 0);
+ int SweepInParallel(std::vector<Page*>& pages, PagedSpace* space,
+ int required_freed_bytes, int max_pages = 0);
+
// Sweeps a given page concurrently to the sweeper threads. It returns the
// size of the maximum continuous freed memory chunk.
int SweepInParallel(Page* page, PagedSpace* space);
@@ -578,6 +581,7 @@ class MarkCompactCollector {
}
private:
+ class EvacuateNewSpacePageVisitor;
class EvacuateNewSpaceVisitor;
class EvacuateOldSpaceVisitor;
class EvacuateVisitorBase;
@@ -827,6 +831,7 @@ class MarkCompactCollector {
SweepingList sweeping_list_old_space_;
SweepingList sweeping_list_code_space_;
SweepingList sweeping_list_map_space_;
+ SweepingList* sweeping_list_shared_;
Michael Lippautz 2016/04/08 07:09:58 Not too happy about cluttering up all the sweeper
Hannes Payer (out of office) 2016/04/08 10:42:40 Yeah, that does not really scale. You can do that
Michael Lippautz 2016/04/08 11:30:00 It's sweeping_list_promoted_pages_ for now.
// True if we are collecting slots to perform evacuation from evacuation
// candidates.

Powered by Google App Engine
This is Rietveld 408576698