Author: Naveen Seth Hanig Date: 2025-12-15T17:34:37+01:00 New Revision: 57fcbd46bfb5833eed51074cd5d8e45f9afe7712
URL: https://github.com/llvm/llvm-project/commit/57fcbd46bfb5833eed51074cd5d8e45f9afe7712 DIFF: https://github.com/llvm/llvm-project/commit/57fcbd46bfb5833eed51074cd5d8e45f9afe7712.diff LOG: [clang][DependencyScanning] Unify initVFS helpers to return OverlayFileSystem (NFC) (#172166) This patch is part of a series of splitting the work from #169964 into smaller changes (see review linked below). As part of this work, the regular and by-name dependency scanning APIs need to be extended to allow the VFS to be initialized via the initVFS helpers before invoking the scanning API, with the resulting overlay filesystem passed into the worker instead. To make the newly added API changes consistent, this patch updates both helpers to return an OverlayFileSystem. https://github.com/llvm/llvm-project/pull/169964#pullrequestreview-3545879529 Added: Modified: clang/include/clang/DependencyScanning/DependencyScannerImpl.h clang/lib/DependencyScanning/DependencyScannerImpl.cpp Removed: ################################################################################ diff --git a/clang/include/clang/DependencyScanning/DependencyScannerImpl.h b/clang/include/clang/DependencyScanning/DependencyScannerImpl.h index 9e23c0f87f273..f43c7f70183fd 100644 --- a/clang/include/clang/DependencyScanning/DependencyScannerImpl.h +++ b/clang/include/clang/DependencyScanning/DependencyScannerImpl.h @@ -98,7 +98,8 @@ std::unique_ptr<CompilerInvocation> createCompilerInvocation(ArrayRef<std::string> CommandLine, DiagnosticsEngine &Diags); -std::pair<IntrusiveRefCntPtr<llvm::vfs::FileSystem>, std::vector<std::string>> +std::pair<IntrusiveRefCntPtr<llvm::vfs::OverlayFileSystem>, + std::vector<std::string>> initVFSForTUBufferScanning(IntrusiveRefCntPtr<llvm::vfs::FileSystem> BaseFS, ArrayRef<std::string> CommandLine, StringRef WorkingDirectory, diff --git a/clang/lib/DependencyScanning/DependencyScannerImpl.cpp b/clang/lib/DependencyScanning/DependencyScannerImpl.cpp index 191ffabeea4f2..201230d7d6a8e 100644 --- a/clang/lib/DependencyScanning/DependencyScannerImpl.cpp +++ b/clang/lib/DependencyScanning/DependencyScannerImpl.cpp @@ -430,7 +430,8 @@ dependencies::createCompilerInvocation(ArrayRef<std::string> CommandLine, return Invocation; } -std::pair<IntrusiveRefCntPtr<llvm::vfs::FileSystem>, std::vector<std::string>> +std::pair<IntrusiveRefCntPtr<llvm::vfs::OverlayFileSystem>, + std::vector<std::string>> dependencies::initVFSForTUBufferScanning( IntrusiveRefCntPtr<llvm::vfs::FileSystem> BaseFS, ArrayRef<std::string> CommandLine, StringRef WorkingDirectory, @@ -438,7 +439,6 @@ dependencies::initVFSForTUBufferScanning( // Reset what might have been modified in the previous worker invocation. BaseFS->setCurrentWorkingDirectory(WorkingDirectory); - IntrusiveRefCntPtr<llvm::vfs::FileSystem> ModifiedFS; auto OverlayFS = llvm::makeIntrusiveRefCnt<llvm::vfs::OverlayFileSystem>(BaseFS); auto InMemoryFS = llvm::makeIntrusiveRefCnt<llvm::vfs::InMemoryFileSystem>(); @@ -449,11 +449,10 @@ dependencies::initVFSForTUBufferScanning( IntrusiveRefCntPtr<llvm::vfs::FileSystem> InMemoryOverlay = InMemoryFS; OverlayFS->pushOverlay(InMemoryOverlay); - ModifiedFS = OverlayFS; std::vector<std::string> ModifiedCommandLine(CommandLine); ModifiedCommandLine.emplace_back(InputPath); - return std::make_pair(ModifiedFS, ModifiedCommandLine); + return std::make_pair(OverlayFS, ModifiedCommandLine); } std::pair<IntrusiveRefCntPtr<llvm::vfs::OverlayFileSystem>, _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
