diff options
| author | Peter Collingbourne <peter@pcc.me.uk> | 2017-09-05 19:51:38 +0000 |
|---|---|---|
| committer | Peter Collingbourne <peter@pcc.me.uk> | 2017-09-05 19:51:38 +0000 |
| commit | d0e9c167d8a6ab934bfa20ec0b774488176777a8 (patch) | |
| tree | 93e9c71a903eb8c81bf4712fa3a6a302af246c0f /llvm/lib/CodeGen/MachineFunction.cpp | |
| parent | 420c4863a932cef3e7cb74fd1bf6cabbf6e5f62b (diff) | |
| download | llvm-d0e9c167d8a6ab934bfa20ec0b774488176777a8.zip llvm-d0e9c167d8a6ab934bfa20ec0b774488176777a8.tar.gz llvm-d0e9c167d8a6ab934bfa20ec0b774488176777a8.tar.bz2 | |
LTO: Try to open cache files before renaming them.
It appears that a potential race between the cache client and the cache
pruner that I thought was unlikely actually happened in practice [1].
Try to avoid the race condition by opening the temporary file before
renaming it. Do this only on non-Windows platforms because we cannot
rename open files on Windows using the sys::fs::rename function.
[1] https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.memory%2FLinux_CFI%2F1610%2F%2B%2Frecipes%2Fsteps%2Fcompile%2F0%2Fstdout
Differential Revision: https://reviews.llvm.org/D37410
llvm-svn: 312567
Diffstat (limited to 'llvm/lib/CodeGen/MachineFunction.cpp')
0 files changed, 0 insertions, 0 deletions