[PATCH] D122549: [VFS] RedirectingFileSystem only replace path if not already mapped

Mon Mar 28 11:24:34 PDT 2022

bnbarham added a comment.

In D122549#3412064 <https://reviews.llvm.org/D122549#3412064>, @dexonsmith wrote:

> In D122549#3412021 <https://reviews.llvm.org/D122549#3412021>, @bnbarham wrote:
>
>> `clang-apply-replacements/relative-paths.cpp` is failing, I haven't looked into it but my guess would be that it's from the `Status.getName() == Filename` -> `!Status.IsVFSMapped` change. That seems very odd to me.
>
> Is it just failing on Windows? I wonder (rather speculatively...) whether https://reviews.llvm.org/D121733 would help.

No, also debian. Not sure why it isn't saying debian failed in the review.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122549