Reviewers: Hannes Payer,
Description:
PPC: fix allocation issues for PPC64.
[email protected]
Please review this at https://codereview.chromium.org/1009433002/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+3, -1 lines):
M src/snapshot-common.cc
M test/cctest/test-strings.cc
Index: src/snapshot-common.cc
diff --git a/src/snapshot-common.cc b/src/snapshot-common.cc
index
6bcabffb8111121b2360fd388c377034acab5099..a3e80185702622a48dabb1b8a2ef9130757bf2c5
100644
--- a/src/snapshot-common.cc
+++ b/src/snapshot-common.cc
@@ -135,6 +135,8 @@ void CalculateFirstPageSizes(bool is_default_snapshot,
required = (startup_reservations[startup_index].chunk_size() +
2 * context_reservations[context_index].chunk_size()) +
Page::kObjectStartOffset;
+ // Add a small allowance to the code space for small scripts.
+ if (space == CODE_SPACE) required += 32 * KB;
} else {
// We expect the vanilla snapshot to only require on page per space.
DCHECK(!is_default_snapshot);
Index: test/cctest/test-strings.cc
diff --git a/test/cctest/test-strings.cc b/test/cctest/test-strings.cc
index
9a4e96ffd5cc1c9c06f4bbb30387fa66c7522906..8d5129cd05e50b42338f4079f585c677f9f73bd4
100644
--- a/test/cctest/test-strings.cc
+++ b/test/cctest/test-strings.cc
@@ -1209,7 +1209,7 @@ UNINITIALIZED_TEST(OneByteArrayJoin) {
v8::Isolate::CreateParams create_params;
// Set heap limits.
create_params.constraints.set_max_semi_space_size(1);
- create_params.constraints.set_max_old_space_size(5);
+ create_params.constraints.set_max_old_space_size(6);
v8::Isolate* isolate = v8::Isolate::New(create_params);
isolate->Enter();
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
