diff options
author | Florian Hahn <flo@fhahn.com> | 2018-12-15 01:32:58 +0000 |
---|---|---|
committer | Florian Hahn <flo@fhahn.com> | 2018-12-15 01:32:58 +0000 |
commit | abe32c9125ca29639f56afa57ad93cbb66c19df8 (patch) | |
tree | b2fc79d153ca8f131cf3540c9719531ab62a682a /llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp | |
parent | 1fc257d97f99513427597502b9b46237e97b3f2a (diff) | |
download | bcm5719-llvm-abe32c9125ca29639f56afa57ad93cbb66c19df8.tar.gz bcm5719-llvm-abe32c9125ca29639f56afa57ad93cbb66c19df8.zip |
[SILoadStoreOptimizer] Use std::abs to avoid truncation.
Using regular abs() causes the following warning
error: absolute value function 'abs' given an argument of type 'int64_t' (aka 'long') but has parameter of type 'int' which may cause truncation of value [-Werror,-Wabsolute-value]
(uint32_t)abs(Dist) > MaxDist) {
^
lib/Target/AMDGPU/SILoadStoreOptimizer.cpp:1369:19: note: use function 'std::abs' instead
which causes a bot to fail:
http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/18284/steps/bootstrap%20clang/logs/stdio
llvm-svn: 349224
Diffstat (limited to 'llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp')
-rw-r--r-- | llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp b/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp index d755f7609cc..9f3983c7256 100644 --- a/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp +++ b/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp @@ -1366,8 +1366,8 @@ bool SILoadStoreOptimizer::promoteConstantOffsetToImm( AM.HasBaseReg = true; AM.BaseOffs = Dist; if (TLI->isLegalGlobalAddressingMode(AM) && - (uint32_t)abs(Dist) > MaxDist) { - MaxDist = abs(Dist); + (uint32_t)std::abs(Dist) > MaxDist) { + MaxDist = std::abs(Dist); AnchorAddr = MAddrNext; AnchorInst = &MINext; |