[PATCH] D109128: [VFS] Use original path when falling back to external FS

Keith Smiley via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 3 12:49:48 PDT 2021


keith added a comment.

I'm realizing more now that these changes break the original intent of rG0be9ca7c0f9a733f846bb6bc4e8e36d46b518162 <https://reviews.llvm.org/rG0be9ca7c0f9a733f846bb6bc4e8e36d46b518162> as well, given that it seems like the core canonicalization of the path before passing it down was the intent, but that's also the part that appears to be causing the issue. @JDevlieghere I would definitely appreciate some advice here, I'm still trying to see if I can satisfy the new test case without breaking the tests you changed, but I think they might fundamentally be at odds


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109128



More information about the cfe-commits mailing list