[llvm-branch-commits] [llvm] 60692a6 - [Support] Silence warning with Clang ToT.
Tobias Hieta via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Tue Feb 28 00:40:59 PST 2023
Author: Alexandre Ganea
Date: 2023-02-28T09:40:33+01:00
New Revision: 60692a66ced6dfa6c3ecbf57f518a59cf3845553
URL: https://github.com/llvm/llvm-project/commit/60692a66ced6dfa6c3ecbf57f518a59cf3845553
DIFF: https://github.com/llvm/llvm-project/commit/60692a66ced6dfa6c3ecbf57f518a59cf3845553.diff
LOG: [Support] Silence warning with Clang ToT.
This fixes the following warning on Windows with latest Clang:
```
[160/3057] Building CXX object lib/Support/CMakeFiles/LLVMSupport.dir/Signals.cpp.obj
In file included from C:/git/llvm-project/llvm/lib/Support/Signals.cpp:260:
C:/git/llvm-project/llvm/lib/Support/Windows/Signals.inc(834,15): warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
if (RetCode == (0xE0000000 | EX_IOERR))
~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~
1 warning generated.```
(cherry picked from commit 1d0a5f11c04e6ac4dab578b81d02eabb83b31428)
Added:
Modified:
llvm/lib/Support/Windows/Signals.inc
Removed:
################################################################################
diff --git a/llvm/lib/Support/Windows/Signals.inc b/llvm/lib/Support/Windows/Signals.inc
index 4bf699f2bccf0..cb82f55fc38be 100644
--- a/llvm/lib/Support/Windows/Signals.inc
+++ b/llvm/lib/Support/Windows/Signals.inc
@@ -830,7 +830,7 @@ void sys::CleanupOnSignal(uintptr_t Context) {
//
// 0xE0000000 is combined with the return code in the exception raised in
// CrashRecoveryContext::HandleExit().
- int RetCode = (int)EP->ExceptionRecord->ExceptionCode;
+ unsigned RetCode = EP->ExceptionRecord->ExceptionCode;
if (RetCode == (0xE0000000 | EX_IOERR))
return;
LLVMUnhandledExceptionFilter(EP);
More information about the llvm-branch-commits
mailing list