[PATCH] D99212: [Support] Fix Windows 7 path handling

[Ronald Wampler via Phabricator via llvm-commits]

rdwampler added inline comments.


================
Comment at: llvm/lib/Support/Windows/Path.inc:408
+  // When Delete is false skip the check, since on Windows 7 the function
+  // realPathFromHandle() below would fail. In that case, network files would
+  // also get the 'false' flag, but that is fine.
----------------
IIUC, realPathFromHandle() fails because FILE_DISPOSITION_INFO is already set to true, if so can you update the comment here to clarify. I think this would still fail if setDeleteDisposition() is called twice with Delete being true.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99212