[PATCH] D38817: Support: Work around missing SetFileInformationByHandle on Wine

Hans Wennborg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 11 14:25:31 PDT 2017


hans added inline comments.


================
Comment at: lib/Support/Windows/Path.inc:436
     auto EC = rename_internal(FromHandle, To, true);
     if (!EC || EC != errc::permission_denied)
       return EC;
----------------
pcc wrote:
> Wine returns `ERROR_CALL_NOT_IMPLEMENTED` if it does not support the call to `SetFileInformationByHandle`. Would it be simpler to detect that here and try the `MoveFileEx` if it fails in that way?
Yes, that seems much nicer. Thanks!


https://reviews.llvm.org/D38817





More information about the llvm-commits mailing list