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