diff options
| author | Sean Stangl <sean.stangl@gmail.com> | 2019-07-10 14:35:11 -0600 |
|---|---|---|
| committer | Alon Zakai <azakai@google.com> | 2019-07-10 13:35:11 -0700 |
| commit | 9838b396d72e96741348b6c11c6fd02500b54171 (patch) | |
| tree | 433272b87ff817a174faa03d75762e6407a1244b /src/passes/OptimizeInstructions.cpp | |
| parent | 90449a5699567283586460aa127c311f72f81704 (diff) | |
| download | binaryen-9838b396d72e96741348b6c11c6fd02500b54171.tar.gz binaryen-9838b396d72e96741348b6c11c6fd02500b54171.tar.bz2 binaryen-9838b396d72e96741348b6c11c6fd02500b54171.zip | |
Fix comparison signedness errors in optimizeMemoryAccess() (#2211)
Diffstat (limited to 'src/passes/OptimizeInstructions.cpp')
| -rw-r--r-- | src/passes/OptimizeInstructions.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/passes/OptimizeInstructions.cpp b/src/passes/OptimizeInstructions.cpp index f8fc21913..3b053a5ca 100644 --- a/src/passes/OptimizeInstructions.cpp +++ b/src/passes/OptimizeInstructions.cpp @@ -1219,9 +1219,9 @@ private: // don't do this if it would wrap the pointer uint64_t value64 = last->value.geti32(); uint64_t offset64 = offset; - if (value64 <= std::numeric_limits<int32_t>::max() && - offset64 <= std::numeric_limits<int32_t>::max() && - value64 + offset64 <= std::numeric_limits<int32_t>::max()) { + if (value64 <= uint64_t(std::numeric_limits<int32_t>::max()) && + offset64 <= uint64_t(std::numeric_limits<int32_t>::max()) && + value64 + offset64 <= uint64_t(std::numeric_limits<int32_t>::max())) { last->value = Literal(int32_t(value64 + offset64)); offset = 0; } |