From: Nadav Rotem <nadav.rotem@intel.com>
Date: Mon, 26 Mar 2012 21:00:53 +0000 (+0000)
Subject: 153465 was incorrect. In this code we wanted to check that the pointer operand is... 
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=02f0a49bbaa9a0f881ef18abd3b06168c2b35135;p=oota-llvm.git

153465 was incorrect. In this code we wanted to check that the pointer operand is of pointer type (and not vector type).

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

diff --git a/lib/Transforms/InstCombine/InstructionCombining.cpp b/lib/Transforms/InstCombine/InstructionCombining.cpp
index 8802ba448e7..066b2ec89c3 100644
--- a/lib/Transforms/InstCombine/InstructionCombining.cpp
+++ b/lib/Transforms/InstCombine/InstructionCombining.cpp
@@ -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()) {