[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
parent
b71b22e4
Please register or sign in to comment