[PATCH] D24176: Remove temoprary files.
Reid Kleckner via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 2 08:29:33 PDT 2016
rnk added inline comments.
================
Comment at: COFF/DriverUtils.cpp:585
@@ -570,3 +584,3 @@
E.run();
- return check(MemoryBuffer::getFile(Path), "could not open " + Path);
+ return check(MemoryBuffer::getFile(File.Path), "could not open " + File.Path);
}
----------------
I think the explanation is that MemoryBuffer only uses MapViewOfFile when shouldUseMmap returns true (if the file is small or its size is a multiple of 4K and RequiresNullTerminator is set). I suspect if you arrange for it to return true, then this removal will fail.
https://reviews.llvm.org/D24176
More information about the llvm-commits
mailing list