diff options
Diffstat (limited to 'lib/Support')
-rw-r--r-- | lib/Support/Process.cpp | 8 | ||||
-rw-r--r-- | lib/Support/Unix/Process.inc | 11 | ||||
-rw-r--r-- | lib/Support/Windows/Process.inc | 14 |
3 files changed, 21 insertions, 12 deletions
diff --git a/lib/Support/Process.cpp b/lib/Support/Process.cpp index 1e21d64e60..9d87b7744b 100644 --- a/lib/Support/Process.cpp +++ b/lib/Support/Process.cpp @@ -55,6 +55,14 @@ self_process::~self_process() { #endif +//===----------------------------------------------------------------------===// +// Implementations of legacy functions in terms of the new self_process object. + +unsigned Process::GetPageSize() { + return process::get_self()->page_size(); +} + + // Include the platform-specific parts of this class. #ifdef LLVM_ON_UNIX #include "Unix/Process.inc" diff --git a/lib/Support/Unix/Process.inc b/lib/Support/Unix/Process.inc index e96d37ce56..9ad6272bab 100644 --- a/lib/Support/Unix/Process.inc +++ b/lib/Support/Unix/Process.inc @@ -49,10 +49,7 @@ process::id_type self_process::get_id() { return getpid(); } - -unsigned -Process::GetPageSize() -{ +static unsigned getPageSize() { #if defined(__CYGWIN__) // On Cygwin, getpagesize() returns 64k but the page size for the purposes of // memory protection and mmap() is 4k. @@ -68,6 +65,12 @@ Process::GetPageSize() return static_cast<unsigned>(page_size); } +// This constructor guaranteed to be run exactly once on a single thread, and +// sets up various process invariants that can be queried cheaply from then on. +self_process::self_process() : PageSize(getPageSize()) { +} + + size_t Process::GetMallocUsage() { #if defined(HAVE_MALLINFO) struct mallinfo mi; diff --git a/lib/Support/Windows/Process.inc b/lib/Support/Windows/Process.inc index d3a0c94e1c..9eb611be87 100644 --- a/lib/Support/Windows/Process.inc +++ b/lib/Support/Windows/Process.inc @@ -43,11 +43,9 @@ process::id_type self_process::get_id() { return GetCurrentProcess(); } - // This function retrieves the page size using GetSystemInfo and is present -// solely so it can be called once in Process::GetPageSize to initialize the -// static variable PageSize. -inline unsigned GetPageSizeOnce() { +// solely so it can be called once to initialize the self_process member below. +static unsigned getPageSize() { // NOTE: A 32-bit application running under WOW64 is supposed to use // GetNativeSystemInfo. However, this interface is not present prior // to Windows XP so to use it requires dynamic linking. It is not clear @@ -58,12 +56,12 @@ inline unsigned GetPageSizeOnce() { return static_cast<unsigned>(info.dwPageSize); } -unsigned -Process::GetPageSize() { - static const unsigned PageSize = GetPageSizeOnce(); - return PageSize; +// This constructor guaranteed to be run exactly once on a single thread, and +// sets up various process invariants that can be queried cheaply from then on. +self_process::self_process() : PageSize(getPageSize()) { } + size_t Process::GetMallocUsage() { |