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

Ben Barham via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 4 17:26:01 PDT 2022


bnbarham added a comment.

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`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123103

STAMPS
actor(@bnbarham) application(Differential) author(@bnbarham) herald(H224) herald(H225) herald(H423) herald(H576) herald(H615) herald(H688) herald(H864) monogram(D123103) object-type(DREV) phid(PHID-DREV-tt3oqfaxmuyojrbgy6z7) reviewer(@dexonsmith) revision-repository(rG) revision-status(needs-review) subscriber(@cfe-commits) subscriber(@hiraditya) subscriber(@llvm-commits) tag(#all) tag(#clang) tag(#llvm) via(web)



More information about the cfe-commits mailing list