replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Bug fix in TargetData::getIndexedOffset: We were using the pointer 

type instead of the element type for the element size for arrays.

llvm-svn: 3302
This commit is contained in:
Vikram S. Adve 2002-08-13 18:17:56 +00:00
parent 4f729affe6
commit a3efa260ea
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
llvm/lib/Target/TargetData.cpp
View File

@ -159,15 +159,15 @@ uint64_t TargetData::getIndexedOffset(const Type *ptrTy,
for (unsigned CurIDX = 0; CurIDX < Idx.size(); ++CurIDX) { for (unsigned CurIDX = 0; CurIDX < Idx.size(); ++CurIDX) {
if (Idx[CurIDX]->getType() == Type::UIntTy) { if (Idx[CurIDX]->getType() == Type::UIntTy) {
// Update Ty to refer to current element
Ty = cast<SequentialType>(Ty)->getElementType();
// Get the array index and the size of each array element. // Get the array index and the size of each array element.
// Both must be known constants, or this will fail. // Both must be known constants, or this will fail.
unsigned arrayIdx = cast<ConstantUInt>(Idx[CurIDX])->getValue(); unsigned arrayIdx = cast<ConstantUInt>(Idx[CurIDX])->getValue();
uint64_t elementSize = this->getTypeSize(Ty); uint64_t elementSize = this->getTypeSize(Ty);
Result += arrayIdx * elementSize; Result += arrayIdx * elementSize;
// Update Ty to refer to current element
Ty = cast<SequentialType>(Ty)->getElementType();
} else if (const StructType *STy = dyn_cast<const StructType>(Ty)) { } else if (const StructType *STy = dyn_cast<const StructType>(Ty)) {
assert(Idx[CurIDX]->getType() == Type::UByteTy && "Illegal struct idx"); assert(Idx[CurIDX]->getType() == Type::UByteTy && "Illegal struct idx");
unsigned FieldNo = cast<ConstantUInt>(Idx[CurIDX])->getValue(); unsigned FieldNo = cast<ConstantUInt>(Idx[CurIDX])->getValue();