<div dir="ltr">I'm pretty sure the test passes without your change...<div><br></div><div>-Eli</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 21, 2013 at 1:01 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"><a href="http://llvm-reviews.chandlerc.com/D1464" target="_blank">http://llvm-reviews.chandlerc.com/D1464</a><br>
<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>
@@ -265,26 +265,34 @@<br>
       // 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>
@@ -56,6 +56,34 @@<br>
   ret i32 %t<br>
 }<br>
<br>
+define i32 @outer_inttoptr_as_cost(i32 addrspace(1)* %ptr) {<br>
+; CHECK-LABEL: @outer_inttoptr_as_cost(<br>
+; CHECK: call<br>
+; CHECK: ret i32<br>
+  %ptr1 = getelementptr inbounds i32 addrspace(1)* %ptr, i32 0<br>
+  %ptr2 = getelementptr inbounds i32 addrspace(1)* %ptr, i32 42<br>
+  %int1 = ptrtoint i32 addrspace(1)* %ptr1 to i32<br>
+  %int2 = ptrtoint i32 addrspace(1)* %ptr2 to i32<br>
+  %result = call i32 @inner_inttoptr_as_cost(i32 %int1, i32 %int2)<br>
+  ret i32 %result<br>
+}<br>
+<br>
+; Test that the operation cost for inttoptr is hit<br>
+define i32 @inner_inttoptr_as_cost(i32 %begin, i32 %end) {<br>
+  %distance = sub i32 %end, %begin<br>
+  %begin.p = inttoptr i32 %begin to i32 addrspace(1)*<br>
+  %end.p = inttoptr i32 %end to i32 addrspace(1)*<br>
+  %icmp = icmp sle i32 %distance, 42<br>
+  br i1 %icmp, label %then, label %else<br>
+<br>
+then:<br>
+  ret i32 3<br>
+<br>
+else:<br>
+  %t = load i32 addrspace(1)* %begin.p<br>
+  ret i32 %t<br>
+}<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>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>