[PATCH] D111880: [Support] [Windows] Convert paths to the preferred form

[Aaron Ballman via Phabricator via llvm-commits]

aaron.ballman added inline comments.


================
Comment at: llvm/lib/Support/Windows/Path.inc:79-80
+  // back to backslashes here.
+  const char *const LongPathPrefixSlash = "//?/";
+  if (Path8Str.startswith(LongPathPrefixSlash))
+    llvm::sys::path::native(Path8Str, path::Style::windows_backslash);
----------------
I'm assuming we purposefully don't care about complete nonsense like `\/?\` not getting normalized into `\\?\`?

We should add tests for both `//?/` and the nonsense form.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111880