[PATCH] D111879: [Support] Add a new path style for Windows with forward slashes

[Aaron Ballman via Phabricator via llvm-commits]

aaron.ballman added a comment.

In D111879#3101310 <https://reviews.llvm.org/D111879#3101310>, @rnk wrote:

> I am a bit busy at the moment.
>
> @aaron.ballman , can I use you as a representative of the MSVC ecosystem developer community? Are you on board with this change?

Sure can! I need to think about this for a bit though -- I'm not super comfortable initially, but I can become comfortable. Windows silently accepts forward slashes as path separators and has for a long time, but there are circumstances where the forward slashes are NOT supported too. Those cases should hopefully be rare, but it'd be good to know how we treat them. For example, Win32 file namespace path names do not undergo separator normalization (e.g., paths that start with `\\?\` so that they work with path components larger than max path, etc, such as `\\?\C:\Users\aballman\Desktop\test.c`). I think I'd prefer *not* supporting mixed path separators with that style of path because the system does not support it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111879