Remove an unnecessary use of pointee types introduced in r194220
authorDavid Blaikie <dblaikie@gmail.com>
Fri, 21 Aug 2015 17:37:41 +0000 (17:37 +0000)
committerDavid Blaikie <dblaikie@gmail.com>
Fri, 21 Aug 2015 17:37:41 +0000 (17:37 +0000)
David Majnemer (the original author) believes this to be an impossible
condition to reach anyway, and no test cases cover this so we'll go with
that.

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

lib/IR/ConstantFold.cpp

index 5dd075a..131c5c5 100644 (file)
@@ -1999,9 +1999,8 @@ static bool isInBoundsIndices(ArrayRef<IndexTy> Idxs) {
 /// \brief Test whether a given ConstantInt is in-range for a SequentialType.
 static bool isIndexInRangeOfSequentialType(SequentialType *STy,
                                            const ConstantInt *CI) {
-  if (auto *PTy = dyn_cast<PointerType>(STy))
-    // Only handle pointers to sized types, not pointers to functions.
-    return PTy->getElementType()->isSized();
+  if (isa<PointerType>(STy))
+    return true;
 
   uint64_t NumElements = 0;
   // Determine the number of elements in our sequential type.