dexonsmith added a subscriber: rnk. dexonsmith added inline comments.
================ Comment at: llvm/unittests/Support/VirtualFileSystemTest.cpp:1878 + // with native path seperator, regardless of the actual path seperator + // used in YAMLFilePath field. +#ifndef _WIN32 ---------------- bnbarham wrote: > `separator` -> `separator` > > As per my comment above, this is only true because we're either using posix > or windows_backslash. I'm not sure what we really want here, should we always > convert to native? Any thoughts @dexonsmith? > > FWIW the non-windows case is identical to the above block. It'd be > interesting to test `/` and `\` on both filesystems. I think we might need a Windows person to help answer; @rnk, can you suggest someone to help answer @bnbarham's question? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137473/new/ https://reviews.llvm.org/D137473 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
