[PATCH] D115043: [clang][deps] Use MemoryBuffer in minimizing FS

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 8 06:54:17 PST 2021


jansvoboda11 added inline comments.


================
Comment at: clang/include/clang/Tooling/DependencyScanning/DependencyScanningFilesystem.h:49
-  /// running out of file descriptors.
+  /// mismatching size of the file.
   static CachedFileSystemEntry createFileEntry(StringRef Filename,
                                                llvm::vfs::FileSystem &FS,
----------------
In the implementation of this function we get `std::unique_ptr<llvm::vfs::File>` from the underlying filesystem. When that goes out of scope, the file descriptor will be closed immediately. How did copying contents into memory prevent running out of file descriptors? @arphaman


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D115043/new/

https://reviews.llvm.org/D115043



More information about the cfe-commits mailing list