diff options
author | Ivan Krasin <krasin@chromium.org> | 2011-09-15 23:13:00 +0000 |
---|---|---|
committer | Ivan Krasin <krasin@chromium.org> | 2011-09-15 23:13:00 +0000 |
commit | 71280b55a3406c7dd4215449bf4a3ab216e78ffd (patch) | |
tree | 81cd494c9225445a8ad0b41ec0793bf30a76460c | |
parent | 6b5b79c7e8aec3138727e102e85aaf1c3a01765e (diff) | |
download | external_llvm-71280b55a3406c7dd4215449bf4a3ab216e78ffd.zip external_llvm-71280b55a3406c7dd4215449bf4a3ab216e78ffd.tar.gz external_llvm-71280b55a3406c7dd4215449bf4a3ab216e78ffd.tar.bz2 |
use 64-bit types instead of off_t/size_t to avoid the issue when
gold plugin is built with Large File Support (sizeof(off_t) == 64 on i686)
and the rest of LLVM is built w/o Large File Support
(sizeof(off_t) == 32 on i686) which corrupts the stack.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@139873 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | include/llvm/Support/MemoryBuffer.h | 6 | ||||
-rw-r--r-- | lib/Support/MemoryBuffer.cpp | 8 | ||||
-rw-r--r-- | tools/gold/gold-plugin.cpp | 2 |
3 files changed, 8 insertions, 8 deletions
diff --git a/include/llvm/Support/MemoryBuffer.h b/include/llvm/Support/MemoryBuffer.h index 5e55bd9..06816de 100644 --- a/include/llvm/Support/MemoryBuffer.h +++ b/include/llvm/Support/MemoryBuffer.h @@ -75,9 +75,9 @@ public: /// return a MemoryBuffer. static error_code getOpenFile(int FD, const char *Filename, OwningPtr<MemoryBuffer> &result, - size_t FileSize = -1, - size_t MapSize = -1, - off_t Offset = 0, + uint64_t FileSize = -1, + uint64_t MapSize = -1, + int64_t Offset = 0, bool RequiresNullTerminator = true); /// getMemBuffer - Open the specified memory range as a MemoryBuffer. Note diff --git a/lib/Support/MemoryBuffer.cpp b/lib/Support/MemoryBuffer.cpp index d264be9..0771af5 100644 --- a/lib/Support/MemoryBuffer.cpp +++ b/lib/Support/MemoryBuffer.cpp @@ -275,16 +275,16 @@ static bool shouldUseMmap(int FD, error_code MemoryBuffer::getOpenFile(int FD, const char *Filename, OwningPtr<MemoryBuffer> &result, - size_t FileSize, size_t MapSize, - off_t Offset, + uint64_t FileSize, uint64_t MapSize, + int64_t Offset, bool RequiresNullTerminator) { static int PageSize = sys::Process::GetPageSize(); // Default is to map the full file. - if (MapSize == size_t(-1)) { + if (MapSize == uint64_t(-1)) { // If we don't know the file size, use fstat to find out. fstat on an open // file descriptor is cheaper than stat on a random path. - if (FileSize == size_t(-1)) { + if (FileSize == uint64_t(-1)) { struct stat FileInfo; // TODO: This should use fstat64 when available. if (fstat(FD, &FileInfo) == -1) { diff --git a/tools/gold/gold-plugin.cpp b/tools/gold/gold-plugin.cpp index 13d7ec2..6f547b3 100644 --- a/tools/gold/gold-plugin.cpp +++ b/tools/gold/gold-plugin.cpp @@ -246,7 +246,7 @@ static ld_plugin_status claim_file_hook(const ld_plugin_input_file *file, return LDPS_ERR; } } else { - off_t offset = 0; + int64_t offset = 0; // Gold has found what might be IR part-way inside of a file, such as // an .a archive. if (file->offset) { |