From ca710e9cc5991369ef1ed5dc9fa5063d8feb24a2 Mon Sep 17 00:00:00 2001 From: "Vikram S. Adve" Date: Tue, 13 Aug 2002 18:17:56 +0000 Subject: [PATCH] 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 --- lib/Target/TargetData.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) 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(); -- 2.34.1