summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
diff options
context:
space:
mode:
authorJames Molloy <james.molloy@arm.com>2016-09-08 12:58:04 +0000
committerJames Molloy <james.molloy@arm.com>2016-09-08 12:58:04 +0000
commit7c7255e40b9115802987690516fb10301e22a313 (patch)
tree623d933c0b418a3ea47c78fcaaaa0094926e7a6d /llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
parentcc7b4b511bb366dd2b404b4ec176638954e90a3d (diff)
downloadbcm5719-llvm-7c7255e40b9115802987690516fb10301e22a313.tar.gz
bcm5719-llvm-7c7255e40b9115802987690516fb10301e22a313.zip
[Thumb1] Fix cost calculation for complemented immediates
Materializing something like "-3" can be done as 2 instructions: MOV r0, #3 MVN r0, r0 This has a cost of 2, not 3. It looks like we were already trying to detect this pattern in TII::getIntImmCost(), but were taking the complement of the zero-extended value instead of the sign-extended value which is unlikely to ever produce a number < 256. There were no tests failing after changing this... :/ llvm-svn: 280928
Diffstat (limited to 'llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp')
-rw-r--r--llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp b/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
index 13c5dc61acd..2d44adc9951 100644
--- a/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
+++ b/llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
@@ -41,7 +41,7 @@ int ARMTTIImpl::getIntImmCost(const APInt &Imm, Type *Ty) {
// Thumb1.
if (SImmVal >= 0 && SImmVal < 256)
return 1;
- if ((~ZImmVal < 256) || ARM_AM::isThumbImmShiftedVal(ZImmVal))
+ if ((~SImmVal < 256) || ARM_AM::isThumbImmShiftedVal(ZImmVal))
return 2;
// Load from constantpool.
return 3;
OpenPOWER on IntegriCloud