[PATCH] D21668: [ARM] Fix cost computation for constant hoisting

[Weiming Zhao via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL274073: [ARM] Fix 28282: cost computation for constant hoisting (authored by weimingz).

Changed prior to commit:
  http://reviews.llvm.org/D21668?vs=61931&id=62143#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D21668

Files:
  llvm/trunk/lib/Target/ARM/ARMTargetTransformInfo.cpp
  llvm/trunk/test/Transforms/ConstantHoisting/ARM/bad-cases.ll

Index: llvm/trunk/lib/Target/ARM/ARMTargetTransformInfo.cpp
===================================================================
--- llvm/trunk/lib/Target/ARM/ARMTargetTransformInfo.cpp
+++ llvm/trunk/lib/Target/ARM/ARMTargetTransformInfo.cpp
@@ -19,7 +19,7 @@
   assert(Ty->isIntegerTy());
 
  unsigned Bits = Ty->getPrimitiveSizeInBits();
- if (Bits == 0 || Bits > 64)
+ if (Bits == 0 || Imm.getActiveBits() >= 64)
    return 4;
 
   int64_t SImmVal = Imm.getSExtValue();
Index: llvm/trunk/test/Transforms/ConstantHoisting/ARM/bad-cases.ll
===================================================================
--- llvm/trunk/test/Transforms/ConstantHoisting/ARM/bad-cases.ll
+++ llvm/trunk/test/Transforms/ConstantHoisting/ARM/bad-cases.ll
@@ -90,3 +90,20 @@
 end:
   ret void
 }
+
+;PR 28282: even when data type is larger than 64-bit, the bit width of the
+;constant operand could be smaller than 64-bit. In this case, there is no
+;benefit to hoist the constant.
+define i32 @struct_type_test(i96 %a0, i96 %a1) {
+;CHECK-LABEL: @struct_type_test
+entry:
+;CHECK-NOT: %const = bitcast i96 32 to i96
+;CHECK: lshr0 = lshr i96 %a0, 32
+  %lshr0 = lshr i96 %a0, 32
+  %cast0 = trunc i96 %lshr0 to i32
+;CHECK: lshr1 = lshr i96 %a1, 32
+  %lshr1 = lshr i96 %a1, 32
+  %cast1 = trunc i96 %lshr1 to i32
+  %ret = add i32 %cast0, %cast1
+  ret i32 %ret
+}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21668.62143.patch
Type: text/x-patch
Size: 1350 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160628/94a11de2/attachment.bin>