Force sign-extension for uint array indexes from 32-bit to 64-bits
authorVikram S. Adve <vadve@cs.uiuc.edu>
Fri, 23 Aug 2002 02:55:51 +0000 (02:55 +0000)
committerVikram S. Adve <vadve@cs.uiuc.edu>
Fri, 23 Aug 2002 02:55:51 +0000 (02:55 +0000)
since uint is not normally sign-extended when casting to uint64_t.

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

lib/Target/TargetData.cpp

index 54acd08966e1bcd3dda175d911922c3fed3a5f6e..48fca1ebdf40107147c9e66be5edfcadc6d0b538 100644 (file)
@@ -164,8 +164,13 @@ uint64_t TargetData::getIndexedOffset(const Type *ptrTy,
 
       // Get the array index and the size of each array element.
       // Both must be known constants, or this will fail.
-      unsigned arrayIdx = cast<ConstantUInt>(Idx[CurIDX])->getValue();
+      // Also, the arrayIdx needs to be sign-extended from uint
+      // to the machine register size if the reg. size > sizeof(uint).
       uint64_t elementSize = this->getTypeSize(Ty);
+      uint64_t arrayIdx = cast<ConstantUInt>(Idx[CurIDX])->getValue();
+      if (getIntegerRegize() > sizeof(uint)) {
+        arrayIdx = (uint64_t) (int) arrayIdx; // sign-extend from 32 to 64 bits
+      }
       Result += arrayIdx * elementSize;
 
     } else if (const StructType *STy = dyn_cast<const StructType>(Ty)) {