diff options
Diffstat (limited to 'clang/lib/Basic/FileSystemStatCache.cpp')
-rw-r--r-- | clang/lib/Basic/FileSystemStatCache.cpp | 44 |
1 files changed, 21 insertions, 23 deletions
diff --git a/clang/lib/Basic/FileSystemStatCache.cpp b/clang/lib/Basic/FileSystemStatCache.cpp index b225facbad9..7a01bffcd95 100644 --- a/clang/lib/Basic/FileSystemStatCache.cpp +++ b/clang/lib/Basic/FileSystemStatCache.cpp @@ -12,7 +12,7 @@ //===----------------------------------------------------------------------===// #include "clang/Basic/FileSystemStatCache.h" -#include "clang/Basic/VirtualFileSystem.h" +#include "llvm/Support/FileSystem.h" #include "llvm/Support/Path.h" // FIXME: This is terrible, we need this for ::close. @@ -30,13 +30,13 @@ using namespace clang; void FileSystemStatCache::anchor() { } -static void copyStatusToFileData(const vfs::Status &Status, +static void copyStatusToFileData(const llvm::sys::fs::file_status &Status, FileData &Data) { Data.Size = Status.getSize(); Data.ModTime = Status.getLastModificationTime().toEpochTime(); Data.UniqueID = Status.getUniqueID(); - Data.IsDirectory = Status.isDirectory(); - Data.IsNamedPipe = Status.getType() == llvm::sys::fs::file_type::fifo_file; + Data.IsDirectory = is_directory(Status); + Data.IsNamedPipe = Status.type() == llvm::sys::fs::file_type::fifo_file; Data.InPCH = false; } @@ -50,23 +50,22 @@ static void copyStatusToFileData(const vfs::Status &Status, /// implementation can optionally fill in FileDescriptor with a valid /// descriptor and the client guarantees that it will close it. bool FileSystemStatCache::get(const char *Path, FileData &Data, bool isFile, - vfs::File **F, FileSystemStatCache *Cache, - vfs::FileSystem &FS) { + int *FileDescriptor, FileSystemStatCache *Cache) { LookupResult R; bool isForDir = !isFile; // If we have a cache, use it to resolve the stat query. if (Cache) - R = Cache->getStat(Path, Data, isFile, F, FS); - else if (isForDir || !F) { + R = Cache->getStat(Path, Data, isFile, FileDescriptor); + else if (isForDir || !FileDescriptor) { // If this is a directory or a file descriptor is not needed and we have // no cache, just go to the file system. - llvm::ErrorOr<vfs::Status> Status = FS.status(Path); - if (!Status) { + llvm::sys::fs::file_status Status; + if (llvm::sys::fs::status(Path, Status)) { R = CacheMissing; } else { R = CacheExists; - copyStatusToFileData(*Status, Data); + copyStatusToFileData(Status, Data); } } else { // Otherwise, we have to go to the filesystem. We can always just use @@ -76,8 +75,7 @@ bool FileSystemStatCache::get(const char *Path, FileData &Data, bool isFile, // // Because of this, check to see if the file exists with 'open'. If the // open succeeds, use fstat to get the stat info. - llvm::OwningPtr<vfs::File> OwnedFile; - llvm::error_code EC = FS.openFileForRead(Path, OwnedFile); + llvm::error_code EC = llvm::sys::fs::openFileForRead(Path, *FileDescriptor); if (EC) { // If the open fails, our "stat" fails. @@ -86,16 +84,16 @@ bool FileSystemStatCache::get(const char *Path, FileData &Data, bool isFile, // Otherwise, the open succeeded. Do an fstat to get the information // about the file. We'll end up returning the open file descriptor to the // client to do what they please with it. - llvm::ErrorOr<vfs::Status> Status = OwnedFile->status(); - if (Status) { + llvm::sys::fs::file_status Status; + if (!llvm::sys::fs::status(*FileDescriptor, Status)) { R = CacheExists; - copyStatusToFileData(*Status, Data); - *F = OwnedFile.take(); + copyStatusToFileData(Status, Data); } else { // fstat rarely fails. If it does, claim the initial open didn't // succeed. R = CacheMissing; - *F = 0; + ::close(*FileDescriptor); + *FileDescriptor = -1; } } } @@ -107,9 +105,9 @@ bool FileSystemStatCache::get(const char *Path, FileData &Data, bool isFile, // demands. if (Data.IsDirectory != isForDir) { // If not, close the file if opened. - if (F && *F) { - (*F)->close(); - *F = 0; + if (FileDescriptor && *FileDescriptor != -1) { + ::close(*FileDescriptor); + *FileDescriptor = -1; } return true; @@ -120,8 +118,8 @@ bool FileSystemStatCache::get(const char *Path, FileData &Data, bool isFile, MemorizeStatCalls::LookupResult MemorizeStatCalls::getStat(const char *Path, FileData &Data, bool isFile, - vfs::File **F, vfs::FileSystem &FS) { - LookupResult Result = statChained(Path, Data, isFile, F, FS); + int *FileDescriptor) { + LookupResult Result = statChained(Path, Data, isFile, FileDescriptor); // Do not cache failed stats, it is easy to construct common inconsistent // situations if we do, and they are not important for PCH performance (which |