[PATCH] D109498: [clang][deps] Stop using `ClangTool` for virtual files

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 9 03:42:50 PDT 2021


jansvoboda11 created this revision.
jansvoboda11 added reviewers: Bigcheese, dexonsmith, ahatanak.
jansvoboda11 requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

This patch changes how the dependency scanner creates the fake input file when scanning dependencies of a single module (introduced in D109485 <https://reviews.llvm.org/D109485>). The scanner now has its own `InMemoryFilesystem` which sits under the minimizing FS (when that's requested). This makes it possible to drop the duplicate work in `DependencyScanningActions::runInvocation` that sets up the main file ID. Besides that, this patch makes it possible to land D108979 <https://reviews.llvm.org/D108979>, where we drop `ClangTool` entirely.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D109498

Files:
  clang/include/clang/Tooling/DependencyScanning/DependencyScanningWorker.h
  clang/lib/Tooling/DependencyScanning/DependencyScanningWorker.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D109498.371540.patch
Type: text/x-patch
Size: 5416 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210909/4f068200/attachment.bin>


More information about the cfe-commits mailing list