153465 was incorrect. In this code we wanted to check that the pointer operand is...
authorNadav Rotem <nadav.rotem@intel.com>
Mon, 26 Mar 2012 21:00:53 +0000 (21:00 +0000)
committerNadav Rotem <nadav.rotem@intel.com>
Mon, 26 Mar 2012 21:00:53 +0000 (21:00 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@153468 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/InstCombine/InstructionCombining.cpp

index 8802ba448e777d778c4f01d4086a5d9845822ae4..066b2ec89c3e8fd6158bbf3da40096a8b3195603 100644 (file)
@@ -915,13 +915,12 @@ Instruction *InstCombiner::visitGetElementPtrInst(GetElementPtrInst &GEP) {
 
   // Handle gep(bitcast x) and gep(gep x, 0, 0, 0).
   Value *StrippedPtr = PtrOp->stripPointerCasts();
+  PointerType *StrippedPtrTy = dyn_cast<PointerType>(StrippedPtr->getType());
 
-  // We do not handle pointer-vector geps here
-  if (!StrippedPtr)
+  // We do not handle pointer-vector geps here.
+  if (!StrippedPtrTy)
     return 0;
 
-  PointerType *StrippedPtrTy = dyn_cast<PointerType>(StrippedPtr->getType());
-
   if (StrippedPtr != PtrOp &&
     StrippedPtrTy->getAddressSpace() == GEP.getPointerAddressSpace()) {