From 17701ab5bd9e4137cbab32ea9a1ccd1e8dc2aa71 Mon Sep 17 00:00:00 2001 From: Hans Wennborg Date: Wed, 11 Oct 2017 22:04:14 +0000 Subject: Support: Work around missing SetFileInformationByHandle on Wine In r315079, fs::rename was reimplemented in terms of CreateFile and SetFileInformationByHandle. Unfortunately, the latter isn't supported by Wine. This adds a fallback to MoveFileEx for that case. Differential Revision: https://reviews.llvm.org/D38817 llvm-svn: 315520 --- llvm/lib/Support/Windows/Path.inc | 11 +++++++++++ 1 file changed, 11 insertions(+) (limited to 'llvm/lib/Support/Windows') diff --git a/llvm/lib/Support/Windows/Path.inc b/llvm/lib/Support/Windows/Path.inc index 94cb2e422c5..5866f9a403a 100644 --- a/llvm/lib/Support/Windows/Path.inc +++ b/llvm/lib/Support/Windows/Path.inc @@ -413,6 +413,17 @@ std::error_code rename(const Twine &From, const Twine &To) { // a true error, so stop trying. for (unsigned Retry = 0; Retry != 200; ++Retry) { auto EC = rename_internal(FromHandle, To, true); + + if (EC == + std::error_code(ERROR_CALL_NOT_IMPLEMENTED, std::system_category())) { + // Wine doesn't support SetFileInformationByHandle in rename_internal. + // Fall back to MoveFileEx. + if (::MoveFileExW(WideFrom.begin(), WideTo.begin(), + MOVEFILE_REPLACE_EXISTING)) + return std::error_code(); + return mapWindowsError(GetLastError()); + } + if (!EC || EC != errc::permission_denied) return EC; -- cgit v1.2.3