Commit fac33fb6 authored by ager@chromium.org's avatar ager@chromium.org

Another fix after the executable memory limit introduction.

The max executable size must be less than or equal to the max old
space size. It is an extra limit in addition to the old space size.

Review URL: http://codereview.chromium.org/4730002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@5802 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent c20cf6d3
...@@ -4349,6 +4349,12 @@ bool Heap::ConfigureHeap(int max_semispace_size, ...@@ -4349,6 +4349,12 @@ bool Heap::ConfigureHeap(int max_semispace_size,
max_executable_size_ = RoundUp(max_executable_size, Page::kPageSize); max_executable_size_ = RoundUp(max_executable_size, Page::kPageSize);
} }
// The max executable size must be less than or equal to the max old
// generation size.
if (max_executable_size_ > max_old_generation_size_) {
max_executable_size_ = max_old_generation_size_;
}
// The new space size must be a power of two to support single-bit testing // The new space size must be a power of two to support single-bit testing
// for containment. // for containment.
max_semispace_size_ = RoundUpToPowerOf2(max_semispace_size_); max_semispace_size_ = RoundUpToPowerOf2(max_semispace_size_);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment