RPCS3/llvm
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm.git synced 2025-02-12 13:48:45 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

LTO: don't fatal when value for cache key already exists

Browse Source 
Summary:
LTO/Caching.cpp uses file rename to atomically set the value for a
cache key. On Windows, this fails when the destination file already
exists. Previously, LLVM would report_fatal_error in such
cases. However, because the old and the new value for the cache key
are supposed to be equivalent, it actually doesn't matter which one we
keep. This change makes it so that failing the rename when an openable
file with the desired name already exists causes us to report success
instead of fataling.

Reviewers: pcc, hans

Subscribers: mehdi_amini, llvm-commits, hiraditya

Differential Revision: https://reviews.llvm.org/D39874

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@317899 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Bob Haarman 2017-11-10 17:08:21 +00:00
parent dc7bbb86a4
commit d4f1cfbb84
1 changed files with 15 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
lib/LTO/Caching.cpp
View File

@ -14,9 +14,9 @@
#include "llvm/LTO/Caching.h"
#include "llvm/ADT/StringExtras.h"
#include "llvm/Support/Errc.h"
#include "llvm/Support/FileSystem.h"
#include "llvm/Support/MemoryBuffer.h"
#include "llvm/Support/Path.h"
#include "llvm/Support/Process.h"
#include "llvm/Support/raw_ostream.h"
using namespace llvm;
@ -72,10 +72,23 @@ Expected<NativeObjectCache> lto::localCache(StringRef CacheDirectoryPath,
MBOrErr.getError().message() + "\n");
// This is atomic on POSIX systems.
if (auto EC = sys::fs::rename(TempFilename, EntryPath))
// On Windows, it can fail with permission denied if the destination
// file already exists. Since the existing file should be semantically
// equivalent to the one we are trying to write, we give AddBuffer
// a copy of the bytes we wrote in that case. We do this instead of
// just using the existing file, because the pruner might delete the
// file before we get a chance to use it.
auto EC = sys::fs::rename(TempFilename, EntryPath);
if (EC == errc::permission_denied) {
auto MBCopy = MemoryBuffer::getMemBufferCopy(
(*MBOrErr)->getBuffer(), EntryPath);
MBOrErr = std::move(MBCopy);
sys::fs::remove(TempFilename);
} else if (EC) {
report_fatal_error(Twine("Failed to rename temporary file ") +
TempFilename + " to " + EntryPath + ": " +
EC.message() + "\n");
}
AddBuffer(Task, std::move(*MBOrErr), EntryPath);
}