Remove special casing of pointers and treat them generically as integers of

the appopriate size.  This gives us the ability to eliminate int -> ptr -> int

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@15063 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp
index 376222ffe2786b455539f8933e922e45f246c2cf..4634b6d9c420cd1c5723de5aa5f31d790053af2e 100644 (file)
--- a/lib/Transforms/Scalar/InstructionCombining.cpp
+++ b/lib/Transforms/Scalar/InstructionCombining.cpp
@@ -1974,14 +1974,11 @@ static inline bool isEliminableCastOfCast(const Type *SrcTy, const Type *MidTy,
   if (SrcTy == DstTy && SrcTy->isLosslesslyConvertibleTo(MidTy))
     return true;
 
-  // If the source and destination types are pointer types, and the intermediate
-  // is an integer type bigger than a pointer, eliminate the casts.
-  if (isa<PointerType>(SrcTy) && isa<PointerType>(DstTy)) {
-    if (isa<PointerType>(MidTy)) return true;
-
-    if (MidTy->isInteger() && MidTy->getPrimitiveSize() >= TD->getPointerSize())
-      return true;
-  }
+  // If we are casting between pointer and integer types, treat pointers as
+  // integers of the appropriate size for the code below.
+  if (isa<PointerType>(SrcTy)) SrcTy = TD->getIntPtrType();
+  if (isa<PointerType>(MidTy)) MidTy = TD->getIntPtrType();
+  if (isa<PointerType>(DstTy)) DstTy = TD->getIntPtrType();
 
   // Allow free casting and conversion of sizes as long as the sign doesn't
   // change...