[PATCH] D135849: [llvm][vfs] For virtual directories, use the virtual path as real path

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 6 06:26:39 PDT 2023


jansvoboda11 added a comment.

I think this patch should now stand on its own. Dealing with a VFS definition that uses the wrong case for virtual paths is not necessary for correctness. If Clang asks the FS for a file using the correct case, the query either fails and we never get far enough to need module map path canonicalization, or the query succeeds thanks to case-insensitivity of the VFS and canonicalizing to the VFS spelling means all future queries using it will succeed as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135849



More information about the cfe-commits mailing list