[llvm] r205468 - Add comments and test case for [X86TTI] Make constant base pointers for GetElementPtr opaque (r204739).

[Juergen Ributzka]

Author: ributzka
Date: Wed Apr  2 16:45:36 2014
New Revision: 205468

URL: http://llvm.org/viewvc/llvm-project?rev=205468&view=rev
Log:
Add comments and test case for [X86TTI] Make constant base pointers for GetElementPtr opaque (r204739).

Added:
    llvm/trunk/test/Transforms/ConstantHoisting/X86/const-base-addr.ll
Modified:
    llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp

Modified: llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp?rev=205468&r1=205467&r2=205468&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp Wed Apr  2 16:45:36 2014
@@ -915,6 +915,9 @@ unsigned X86TTI::getIntImmCost(unsigned
   switch (Opcode) {
   default: return TCC_Free;
   case Instruction::GetElementPtr:
+    // Always hoist the base address of a GetElementPtr. This prevents the
+    // creation of new constants for every base constant that gets constant
+    // folded with the offset.
     if (Idx == 0)
       return 2 * TCC_Basic;
     return TCC_Free;

Added: llvm/trunk/test/Transforms/ConstantHoisting/X86/const-base-addr.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/ConstantHoisting/X86/const-base-addr.ll?rev=205468&view=auto
==============================================================================
--- llvm/trunk/test/Transforms/ConstantHoisting/X86/const-base-addr.ll (added)
+++ llvm/trunk/test/Transforms/ConstantHoisting/X86/const-base-addr.ll Wed Apr  2 16:45:36 2014
@@ -0,0 +1,24 @@
+; RUN: opt -S -consthoist < %s | FileCheck %s
+
+target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-apple-macosx10.9.0"
+
+%T = type { i32, i32, i32, i32 }
+
+; Test if even cheap base addreses are hoisted.
+define i32 @test1() nounwind {
+; CHECK-LABEL:  @test1
+; CHECK:        %const = bitcast i32 12345678 to i32
+; CHECK:        %1 = inttoptr i32 %const to %T*
+; CHECK:        %addr1 = getelementptr %T* %1, i32 0, i32 1
+  %addr1 = getelementptr %T* inttoptr (i32 12345678 to %T*), i32 0, i32 1
+  %tmp1 = load i32* %addr1
+  %addr2 = getelementptr %T* inttoptr (i32 12345678 to %T*), i32 0, i32 2
+  %tmp2 = load i32* %addr2
+  %addr3 = getelementptr %T* inttoptr (i32 12345678 to %T*), i32 0, i32 3
+  %tmp3 = load i32* %addr3
+  %tmp4 = add i32 %tmp1, %tmp2
+  %tmp5 = add i32 %tmp3, %tmp4
+  ret i32 %tmp5
+}
+