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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to