search for: otherptr

On Sep 13, 2008, at 1:07 PM, Matthijs Kooijman wrote: > Hi Dan, > >> Changing PromoteCastOfAllocation to not replace aggregate allocas >> with >> non-aggregate allocas if they have GEP users sounds reasonable to me. > This sounds reasonable indeed, but still a bit arbitrary. Haven't > figured out > anything better yet, though. > >> Finding the

Hi Dan, > Changing PromoteCastOfAllocation to not replace aggregate allocas with > non-aggregate allocas if they have GEP users sounds reasonable to me. This sounds reasonable indeed, but still a bit arbitrary. Haven't figured out anything better yet, though. > Finding the maximum alignment is sometimes still useful though, so > it would be nice to update the alignment field of

...all indices are zero. + if (hasVector && !IsAllZeroIndices) return MarkUnsafe(Info); - - IsAllZeroIndices &= IdxVal->isZero(); } // If there are any non-simple uses of this getelementptr, make sure to reject @@ -661,6 +669,11 @@ // It is likely that OtherPtr is a bitcast, if so, remove it. if (BitCastInst *BC = dyn_cast<BitCastInst>(OtherPtr)) OtherPtr = BC->getOperand(0); + // All zero GEPs are effectively casts + if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(OtherPtr)) + if (GEP->hasAllZer...