Changeset 57682 in vbox

Timestamp:

Sep 10, 2015 9:47:45 AM (10 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

102585

Message:

ASMBitClearRange()/ASMBitSetRange(): use 1U for bit shifts, otherwise the gcc sanitizer complains about negating of INT_MIN which is not possible

File:

• trunk/include/iprt/asm.h (modified) (6 diffs)

trunk/include/iprt/asm.h

@@ -4322 +4322 @@
         int iEnd   = iBitEnd & ~31;
         if (iStart == iEnd)
-            *pu32 &= ((1 << (iBitStart & 31)) - 1) | ~((1 << (iBitEnd & 31)) - 1);
+            *pu32 &= ((1U << (iBitStart & 31)) - 1) | ~((1U << (iBitEnd & 31)) - 1);
         else
         {
@@ -4328 +4328 @@
             if (iBitStart & 31)
             {
-                *pu32 &= (1 << (iBitStart & 31)) - 1;
+                *pu32 &= (1U << (iBitStart & 31)) - 1;
                 pu32++;
                 iBitStart = iStart + 32;
@@ -4341 +4341 @@
             {
                 pu32 = (volatile uint32_t *)pvBitmap + (iBitEnd >> 5);
-                *pu32 &= ~((1 << (iBitEnd & 31)) - 1);
+                *pu32 &= ~((1U << (iBitEnd & 31)) - 1);
             }
         }
@@ -4363 +4363 @@
         int iEnd   = iBitEnd & ~31;
         if (iStart == iEnd)
-            *pu32 |= ((1 << (iBitEnd - iBitStart)) - 1) << (iBitStart & 31);
+            *pu32 |= ((1U << (iBitEnd - iBitStart)) - 1) << (iBitStart & 31);
         else
         {
@@ -4369 +4369 @@
             if (iBitStart & 31)
             {
-                *pu32 |= ~((1 << (iBitStart & 31)) - 1);
+                *pu32 |= ~((1U << (iBitStart & 31)) - 1);
                 pu32++;
                 iBitStart = iStart + 32;
@@ -4382 +4382 @@
             {
                 pu32 = (volatile uint32_t *)pvBitmap + (iBitEnd >> 5);
-                *pu32 |= (1 << (iBitEnd & 31)) - 1;
+                *pu32 |= (1U << (iBitEnd & 31)) - 1;
             }
         }