[PATCH] D86564: [Support][Windows] Fix incorrect GetFinalPathNameByHandleW() return value check in realPathFromHandle()

[Adrian McCarthy via Phabricator via llvm-commits]

amccarth requested changes to this revision.
amccarth added a comment.
This revision now requires changes to proceed.

I know it's not part of this patch, but Phabricator is showing a clang-tidy lint error on the inclusion of `<io.h>`.  While Windows does have one of these, it's only a compatibility hack, and it doesn't seem to be needed.  Can you yank that `#include <io.h>` line before submitting?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86564