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

Ben Barham via Phabricator via cfe-commits cfe-commits at lists.llvm.org
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



More information about the cfe-commits mailing list