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

Unified Diff: src/heap.cc

Issue 6905127: Make newspace semispaces be single page-sized memory chunks. (Closed) Base URL: https://v8.googlecode.com/svn/branches/experimental/gc
Patch Set: Created 9 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
« no previous file with comments | « no previous file | src/spaces.h » ('j') | src/spaces.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/heap.cc
diff --git a/src/heap.cc b/src/heap.cc
index e0c04d9633fa0d1e38776a6f26011623f07e19fb..c291365b37c81cb840271a69d8236d94dd8e2f57 100644
--- a/src/heap.cc
+++ b/src/heap.cc
@@ -84,7 +84,7 @@ Heap::Heap()
#else
reserved_semispace_size_(4*MB),
max_semispace_size_(4*MB),
- initial_semispace_size_(512*KB),
+ initial_semispace_size_(1*MB),
max_old_generation_size_(512*MB),
max_executable_size_(128*MB),
code_range_size_(0),
@@ -4644,7 +4644,12 @@ bool Heap::ConfigureHeap(intptr_t max_semispace_size,
intptr_t max_executable_size) {
if (HasBeenSetup()) return false;
- if (max_semispace_size > 0) max_semispace_size_ = max_semispace_size;
+ if (max_semispace_size > 0) {
+ if (max_semispace_size < Page::kPageSize) {
+ max_semispace_size = Page::kPageSize;
+ }
+ max_semispace_size_ = max_semispace_size;
+ }
if (Snapshot::IsEnabled()) {
// If we are using a snapshot we always reserve the default amount
« no previous file with comments | « no previous file | src/spaces.h » ('j') | src/spaces.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698