summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp
diff options
context:
space:
mode:
authorFlorian Hahn <flo@fhahn.com>2018-12-15 01:32:58 +0000
committerFlorian Hahn <flo@fhahn.com>2018-12-15 01:32:58 +0000
commitabe32c9125ca29639f56afa57ad93cbb66c19df8 (patch)
treeb2fc79d153ca8f131cf3540c9719531ab62a682a /llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp
parent1fc257d97f99513427597502b9b46237e97b3f2a (diff)
downloadbcm5719-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.cpp4
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;
OpenPOWER on IntegriCloud