From 1562d8e3aa9cb04eb19254c88245448ce787dba4 Mon Sep 17 00:00:00 2001 From: "haitao.feng@intel.com" Date: Thu, 12 Jun 2014 09:43:51 +0000 Subject: [PATCH] Rename kIs64BitArch with kRequiresCodeRange. R=verwaest@chromium.org Review URL: https://codereview.chromium.org/335473003 git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@21800 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- src/api.cc | 7 ++++--- src/globals.h | 6 ++++-- src/spaces.cc | 10 ++++++---- 3 files changed, 14 insertions(+), 9 deletions(-) diff --git a/src/api.cc b/src/api.cc index cc9cd3a..6e76d9a 100644 --- a/src/api.cc +++ b/src/api.cc @@ -465,11 +465,12 @@ void ResourceConstraints::ConfigureDefaults(uint64_t physical_memory, set_max_available_threads(i::Max(i::Min(number_of_processors, 4u), 1u)); - if (virtual_memory_limit > 0 && i::kIs64BitArch) { + if (virtual_memory_limit > 0 && i::kRequiresCodeRange) { // Reserve no more than 1/8 of the memory for the code range, but at most - // 512 MB. + // kMaximalCodeRangeSize. set_code_range_size( - i::Min(512, static_cast((virtual_memory_limit >> 3) / i::MB))); + i::Min(i::kMaximalCodeRangeSize / i::MB, + static_cast((virtual_memory_limit >> 3) / i::MB))); } } diff --git a/src/globals.h b/src/globals.h index 0c24c6e..f3f0788 100644 --- a/src/globals.h +++ b/src/globals.h @@ -153,12 +153,14 @@ const int kDoubleSizeLog2 = 3; const int kPointerSizeLog2 = 3; const intptr_t kIntptrSignBit = V8_INT64_C(0x8000000000000000); const uintptr_t kUintptrAllBitsSet = V8_UINT64_C(0xFFFFFFFFFFFFFFFF); -const bool kIs64BitArch = true; +const bool kRequiresCodeRange = true; +const int kMaximalCodeRangeSize = 512 * MB; #else const int kPointerSizeLog2 = 2; const intptr_t kIntptrSignBit = 0x80000000; const uintptr_t kUintptrAllBitsSet = 0xFFFFFFFFu; -const bool kIs64BitArch = false; +const bool kRequiresCodeRange = false; +const int kMaximalCodeRangeSize = 0 * MB; #endif const int kBitsPerByte = 8; diff --git a/src/spaces.cc b/src/spaces.cc index 7966a04..9d21529 100644 --- a/src/spaces.cc +++ b/src/spaces.cc @@ -115,15 +115,17 @@ bool CodeRange::SetUp(size_t requested) { ASSERT(code_range_ == NULL); if (requested == 0) { - // On 64-bit platform(s), we put all code objects in a 512 MB range of - // virtual address space, so that they can call each other with near calls. - if (kIs64BitArch) { - requested = 512 * MB; + // When a target requires the code range feature, we put all code objects + // in a kMaximalCodeRangeSize range of virtual address space, so that + // they can call each other with near calls. + if (kRequiresCodeRange) { + requested = kMaximalCodeRangeSize; } else { return true; } } + ASSERT(requested <= kMaximalCodeRangeSize); code_range_ = new VirtualMemory(requested); CHECK(code_range_ != NULL); if (!code_range_->IsReserved()) { -- 2.7.4