[clang][deps] Stop sharing FileManager across module builds in scanner
authorBen Langmuir <blangmuir@apple.com>
Mon, 8 Aug 2022 16:52:35 +0000 (09:52 -0700)
committerBen Langmuir <blangmuir@apple.com>
Mon, 8 Aug 2022 19:13:54 +0000 (12:13 -0700)
commit33af4b22f8ea5dc8340002833802be95ae9f83a1
tree837e32da758c61a4d62bcd92ea224fdf47e3f020
parentb71b22e48ddcf98ada78dc1e98c2fbdb562e0bd2
[clang][deps] Stop sharing FileManager across module builds in scanner

Sharing the FileManager across implicit module builds currently leaks
paths looked up in an importer into the built module itself. This can
cause non-deterministic results across scans. It is especially bad for
modules since the path can be saved into the pcm file itself, leading to
stateful behaviour if the cache is shared.

This should not impact the number of real filesystem accesses in the
scanner, since it is already caching in the
DependencyScanningWorkerFilesystem.

Note: this change does not affect whether or not the FileManager is
shared across TUs in the scanner, which is a separate issue.

Differential Revision: https://reviews.llvm.org/D131412
clang/lib/Tooling/DependencyScanning/DependencyScanningWorker.cpp
clang/test/ClangScanDeps/modules-file-path-isolation.c [new file with mode: 0644]