Fix a regression in the last patch. When constructing a BitMask, be careful
authorReid Spencer <rspencer@reidspencer.com>
Thu, 18 Jan 2007 18:01:32 +0000 (18:01 +0000)
committerReid Spencer <rspencer@reidspencer.com>
Thu, 18 Jan 2007 18:01:32 +0000 (18:01 +0000)
not to overflow 64-bits and end up with a 0 mask. This caused i64 values to
always be stored as 0 with lots of consequential damage to nightly test.

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

lib/ExecutionEngine/ExecutionEngine.cpp

index 2cb4a8ef9640d7e9e5198128602da49776795d79..771ce0b1aa369cbc1689ad290b1ed8d087c04ec0 100644 (file)
@@ -457,6 +457,8 @@ void ExecutionEngine::StoreValueToMemory(GenericValue Val, GenericValue *Ptr,
     case Type::IntegerTyID: {
       unsigned BitWidth = cast<IntegerType>(Ty)->getBitWidth();
       uint64_t BitMask = (1ull << BitWidth) - 1;
+      if (BitWidth >= 64)
+        BitMask = (uint64_t)-1;
       GenericValue TmpVal = Val;
       if (BitWidth <= 8)
         Ptr->Untyped[0] = Val.Int8Val & BitMask;
@@ -513,6 +515,8 @@ Store4BytesLittleEndian:
     case Type::IntegerTyID: {
       unsigned BitWidth = cast<IntegerType>(Ty)->getBitWidth();
       uint64_t BitMask = (1ull << BitWidth) - 1;
+      if (BitWidth >= 64)
+        BitMask = (uint64_t)-1;
       GenericValue TmpVal = Val;
       if (BitWidth <= 8)
         Ptr->Untyped[0] = Val.Int8Val & BitMask;