[PATCH] D124816: [Tooling] use different FileManager for each CWD

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 5 09:31:30 PDT 2022


dexonsmith added a comment.

In D124816#3494243 <https://reviews.llvm.org/D124816#3494243>, @dexonsmith wrote:

> - AFAICT, every call to `FileManager::setVirtualFileSystem()` is fundamentally unsound unless the new FS is equivalent to the old one or the filemanager hasn't been used yet.

More precisely, I think calling `FileManager::setVirtualFileSystem()` is fundamentally unsound unless the new VFS behaves identically for every path the FileManager has accessed so far.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124816/new/

https://reviews.llvm.org/D124816



More information about the cfe-commits mailing list