[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