[PATCH] D123103: Revert "[VFS] RedirectingFileSystem only replace path if not already mapped"

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 4 18:34:10 PDT 2022


dexonsmith accepted this revision.
dexonsmith added a comment.
This revision is now accepted and ready to land.

In D123103#3428298 <https://reviews.llvm.org/D123103#3428298>, @bnbarham wrote:

> This broke crash reproducers in very specific circumstances, see https://reviews.llvm.org/D123104. I'll re-commit with adding `ExposesExternalVFSPath` instead of replacing `IsVFSMapped`.

Please expand the commit message to include some details (such as these) so git-log is more useful. With that, LGTM.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123103



More information about the cfe-commits mailing list