[clang] [llvm] [clang][deps] Overload `Filesystem::exists` in `DependencyScanningFilesystem` to have it use cached `status` (PR #88152)
Jan Svoboda via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 10 20:24:36 PDT 2024
================
@@ -270,6 +270,12 @@ DependencyScanningWorkerFilesystem::status(const Twine &Path) {
return Result->getStatus();
}
+bool
+DependencyScanningWorkerFilesystem::exists(const Twine &Path) {
+ llvm::ErrorOr<llvm::vfs::Status> Status = status(Path);
----------------
jansvoboda11 wrote:
Would be good to explain why it's necessary & okay to kill the `exists()` optimization here.
https://github.com/llvm/llvm-project/pull/88152
More information about the cfe-commits
mailing list