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

Jonas Devlieghere via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 14 12:34:25 PDT 2021


JDevlieghere added inline comments.


================
Comment at: llvm/lib/Support/VirtualFileSystem.cpp:1179-1180
 
+  if (ExternalFS)
+    ExternalFS->setCurrentWorkingDirectory(Path);
+
----------------
I'm pretty sure there was a reason we stopped doing this. There should be some discussion about that in my original patch. 


================
Comment at: llvm/lib/Support/VirtualFileSystem.cpp:2012
+      if (auto Result = ExternalFS->status(CanonicalPath)) {
+        return Result.get().copyWithNewName(Result.get(), OriginalPath);
+      } else {
----------------
Can we abstract this in a function similar to `getRedirectedFileStatus`, something like `getOrginialFileStatus` or `getNonCanonicalizedFileStatus` and have a comment explaining what it does and why? 


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