From: Vikram S. Adve Date: Tue, 13 Aug 2002 18:17:56 +0000 (+0000) Subject: Bug fix in TargetData::getIndexedOffset: We were using the pointer X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=ca710e9cc5991369ef1ed5dc9fa5063d8feb24a2;p=oota-llvm.git Bug fix in TargetData::getIndexedOffset: We were using the pointer type instead of the element type for the element size for arrays. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@3302 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Target/TargetData.cpp b/lib/Target/TargetData.cpp index 08a6536b78a..54acd08966e 100644 --- a/lib/Target/TargetData.cpp +++ b/lib/Target/TargetData.cpp @@ -159,15 +159,15 @@ uint64_t TargetData::getIndexedOffset(const Type *ptrTy, for (unsigned CurIDX = 0; CurIDX < Idx.size(); ++CurIDX) { if (Idx[CurIDX]->getType() == Type::UIntTy) { + // Update Ty to refer to current element + Ty = cast(Ty)->getElementType(); + // Get the array index and the size of each array element. // Both must be known constants, or this will fail. unsigned arrayIdx = cast(Idx[CurIDX])->getValue(); uint64_t elementSize = this->getTypeSize(Ty); Result += arrayIdx * elementSize; - // Update Ty to refer to current element - Ty = cast(Ty)->getElementType(); - } else if (const StructType *STy = dyn_cast(Ty)) { assert(Idx[CurIDX]->getType() == Type::UByteTy && "Illegal struct idx"); unsigned FieldNo = cast(Idx[CurIDX])->getValue();