diff options
author | Michael Spencer <bigcheesegs@gmail.com> | 2024-03-08 23:30:33 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-03-08 23:30:33 -0800 |
commit | ba13fa2a5d57581bff1a7e9322234af30f4882f6 (patch) | |
tree | 9664647feae1b24a0519bf6ff0143c71b2badd2f /llvm/lib/Support/LockFileManager.cpp | |
parent | abbf1f18825440338f6e08c94c54d8c8b4fe57d4 (diff) | |
download | llvm-ba13fa2a5d57581bff1a7e9322234af30f4882f6.zip llvm-ba13fa2a5d57581bff1a7e9322234af30f4882f6.tar.gz llvm-ba13fa2a5d57581bff1a7e9322234af30f4882f6.tar.bz2 |
[llvm][Support] Add and use errnoAsErrorCode (#84423)
LLVM is inconsistent about how it converts `errno` to `std::error_code`.
This can cause problems because values outside of `std::errc` compare
differently if one is system and one is generic on POSIX systems.
This is even more of a problem on Windows where use of the system
category is just wrong, as that is for Windows errors, which have a
completely different mapping than POSIX/generic errors. This patch fixes
one instance of this mistake in `JSONTransport.cpp`.
This patch adds `errnoAsErrorCode()` which makes it so people do not
need to think about this issue in the future. It also cleans up a lot of
usage of `errno` in LLVM and Clang.
Diffstat (limited to 'llvm/lib/Support/LockFileManager.cpp')
-rw-r--r-- | llvm/lib/Support/LockFileManager.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Support/LockFileManager.cpp b/llvm/lib/Support/LockFileManager.cpp index facdc5a..083f8d7 100644 --- a/llvm/lib/Support/LockFileManager.cpp +++ b/llvm/lib/Support/LockFileManager.cpp @@ -87,7 +87,7 @@ static std::error_code getHostID(SmallVectorImpl<char> &HostID) { struct timespec wait = {1, 0}; // 1 second. uuid_t uuid; if (gethostuuid(uuid, &wait) != 0) - return std::error_code(errno, std::system_category()); + return errnoAsErrorCode(); uuid_string_t UUIDStr; uuid_unparse(uuid, UUIDStr); |