<div dir="ltr">LGTM.<div><br></div><div>-Eli</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 26, 2013 at 8:05 PM, Matt Arsenault <span dir="ltr"><<a href="mailto:Matthew.Arsenault@amd.com" target="_blank">Matthew.Arsenault@amd.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">  Rewrite test<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1464" target="_blank">http://llvm-reviews.chandlerc.com/D1464</a><br>
<br>
CHANGE SINCE LAST DIFF<br>
  <a href="http://llvm-reviews.chandlerc.com/D1464?vs=3638&id=3784#toc" target="_blank">http://llvm-reviews.chandlerc.com/D1464?vs=3638&id=3784#toc</a><br>
<div class="im"><br>
Files:<br>
  lib/Analysis/TargetTransformInfo.cpp<br>
  test/Transforms/Inline/ptr-diff.ll<br>
<br>
Index: lib/Analysis/TargetTransformInfo.cpp<br>
===================================================================<br>
--- lib/Analysis/TargetTransformInfo.cpp<br>
+++ lib/Analysis/TargetTransformInfo.cpp<br>
</div>@@ -269,26 +269,34 @@<br>
<div><div class="h5">       // Otherwise, the default basic cost is used.<br>
       return TCC_Basic;<br>
<br>
-    case Instruction::IntToPtr:<br>
+    case Instruction::IntToPtr: {<br>
+      if (!DL)<br>
+        return TCC_Basic;<br>
+<br>
       // An inttoptr cast is free so long as the input is a legal integer type<br>
       // which doesn't contain values outside the range of a pointer.<br>
-      if (DL && DL->isLegalInteger(OpTy->getScalarSizeInBits()) &&<br>
-          OpTy->getScalarSizeInBits() <= DL->getPointerSizeInBits())<br>
+      unsigned OpSize = OpTy->getScalarSizeInBits();<br>
+      if (DL->isLegalInteger(OpSize) &&<br>
+          OpSize <= DL->getPointerTypeSizeInBits(Ty))<br>
         return TCC_Free;<br>
<br>
       // Otherwise it's not a no-op.<br>
       return TCC_Basic;<br>
+    }<br>
+    case Instruction::PtrToInt: {<br>
+      if (!DL)<br>
+        return TCC_Basic;<br>
<br>
-    case Instruction::PtrToInt:<br>
       // A ptrtoint cast is free so long as the result is large enough to store<br>
       // the pointer, and a legal integer type.<br>
-      if (DL && DL->isLegalInteger(Ty->getScalarSizeInBits()) &&<br>
-          Ty->getScalarSizeInBits() >= DL->getPointerSizeInBits())<br>
+      unsigned DestSize = Ty->getScalarSizeInBits();<br>
+      if (DL->isLegalInteger(DestSize) &&<br>
+          DestSize >= DL->getPointerTypeSizeInBits(OpTy))<br>
         return TCC_Free;<br>
<br>
       // Otherwise it's not a no-op.<br>
       return TCC_Basic;<br>
-<br>
+    }<br>
     case Instruction::Trunc:<br>
       // trunc to a native type is free (assuming the target has compare and<br>
       // shift-right of the same width).<br>
Index: test/Transforms/Inline/ptr-diff.ll<br>
===================================================================<br>
--- test/Transforms/Inline/ptr-diff.ll<br>
+++ test/Transforms/Inline/ptr-diff.ll<br>
</div></div>@@ -1,6 +1,7 @@<br>
 ; RUN: opt -inline < %s -S -o - -inline-threshold=10 | FileCheck %s<br>
<br>
-target datalayout = "p:32:32-p1:64:64-n32"<br>
+target datalayout = "p:32:32-p1:64:64-p2:16:16-n16:32:64"<br>
+<br>
<br>
 define i32 @outer1() {<br>
 ; CHECK-LABEL: @outer1(<br>
@@ -56,6 +57,48 @@<br>
   ret i32 %t<br>
 }<br>
<br>
+; The inttoptrs are free since it is a smaller integer to a larger<br>
+; pointer size<br>
+define i32 @inttoptr_free_cost(i32 %a, i32 %b, i32 %c) {<br>
+  %p1 = inttoptr i32 %a to i32 addrspace(1)*<br>
+  %p2 = inttoptr i32 %b to i32 addrspace(1)*<br>
+  %p3 = inttoptr i32 %c to i32 addrspace(1)*<br>
+  %t1 = load i32 addrspace(1)* %p1<br>
+  %t2 = load i32 addrspace(1)* %p2<br>
+  %t3 = load i32 addrspace(1)* %p3<br>
+  %s = add i32 %t1, %t2<br>
+  %s1 = add i32 %s, %t3<br>
+  ret i32 %s1<br>
+}<br>
+<br>
+define i32 @inttoptr_free_cost_user(i32 %begin, i32 %end) {<br>
+; CHECK-LABEL: @inttoptr_free_cost_user(<br>
+; CHECK-NOT: call<br>
+  %x = call i32 @inttoptr_free_cost(i32 %begin, i32 %end, i32 9)<br>
+  ret i32 %x<br>
+}<br>
+<br>
+; The inttoptrs have a cost since it is a larger integer to a smaller<br>
+; pointer size<br>
+define i32 @inttoptr_cost_smaller_ptr(i32 %a, i32 %b, i32 %c) {<br>
+  %p1 = inttoptr i32 %a to i32 addrspace(2)*<br>
+  %p2 = inttoptr i32 %b to i32 addrspace(2)*<br>
+  %p3 = inttoptr i32 %c to i32 addrspace(2)*<br>
+  %t1 = load i32 addrspace(2)* %p1<br>
+  %t2 = load i32 addrspace(2)* %p2<br>
+  %t3 = load i32 addrspace(2)* %p3<br>
+  %s = add i32 %t1, %t2<br>
+  %s1 = add i32 %s, %t3<br>
+  ret i32 %s1<br>
+}<br>
+<br>
+define i32 @inttoptr_cost_smaller_ptr_user(i32 %begin, i32 %end) {<br>
+; CHECK-LABEL: @inttoptr_cost_smaller_ptr_user(<br>
+; CHECK: call<br>
+  %x = call i32 @inttoptr_cost_smaller_ptr(i32 %begin, i32 %end, i32 9)<br>
+  ret i32 %x<br>
<div class="im">+}<br>
+<br>
 ; Make sure that assertion isn't hit when ptrtoint is used with an<br>
 ; integer size different from the pointer size<br>
 define i32 @outer1_ptrtoint_larger() {<br>
</div>@@ -113,3 +156,4 @@<br>
   %t = load i32* %begin<br>
   ret i32 %t<br>
 }<br>
+<br>
</blockquote></div><br></div>