diff options
| author | Fangrui Song <maskray@google.com> | 2018-11-07 21:14:54 +0000 |
|---|---|---|
| committer | Fangrui Song <maskray@google.com> | 2018-11-07 21:14:54 +0000 |
| commit | db22af0f45c4782d8488e191f406b7aa65e4b62e (patch) | |
| tree | cd4070ffcd241d22a2922a7c487280c8ce75ba4b /llvm/lib/CodeGen/MachineFunction.cpp | |
| parent | 7d7d41debc9abdda7b3710cdabe2bc64e03d5578 (diff) | |
| download | llvm-db22af0f45c4782d8488e191f406b7aa65e4b62e.zip llvm-db22af0f45c4782d8488e191f406b7aa65e4b62e.tar.gz llvm-db22af0f45c4782d8488e191f406b7aa65e4b62e.tar.bz2 | |
[PPC64] Use INT32_MIN instead of std::numeric_limits<int32_t>::min()
Summary:
D53821 fixed the bogus MSVC (at least 2017) C4146 warning (unary minus applied on unsigned type)
by using std::numeric_limits<int32_t>::min().
The warning was because -2147483648 is incorrectly treated as unsigned long instead of long long)
Let's use INT32_MIN which is arguably more readable.
Note, on GCC or clang, -0x80000000 works fine (ILP64: long, LP64: long long).
Reviewers: ruiu, jhenderson, sfertile, espindola
Reviewed By: sfertile
Subscribers: emaste, nemanjai, arichardson, kbarton, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D54200
llvm-svn: 346356
Diffstat (limited to 'llvm/lib/CodeGen/MachineFunction.cpp')
0 files changed, 0 insertions, 0 deletions