Atomic ops that do arithmetic use signed arithmetic.
authorOwen Anderson <resistor@mac.com>
Tue, 23 Jun 2009 18:30:27 +0000 (18:30 +0000)
committerOwen Anderson <resistor@mac.com>
Tue, 23 Jun 2009 18:30:27 +0000 (18:30 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@73980 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/System/Atomic.h
include/llvm/Type.h
lib/System/Atomic.cpp
lib/VMCore/Mangler.cpp

index a94ad2d92e5510cc085c595974070c34054971e5..c0612f9976c4cbf3914111bc0ccd3b77e0427d4e 100644 (file)
@@ -23,11 +23,11 @@ namespace llvm {
     uint32_t CompareAndSwap32(volatile uint32_t* ptr,
                             uint32_t new_value,
                             uint32_t old_value);
-    uint32_t AtomicIncrement32(volatile uint32_t* ptr);
-    uint32_t AtomicDecrement32(volatile uint32_t* ptr);
-    uint32_t AtomicAdd32(volatile uint32_t* ptr, uint32_t val);
+    int32_t AtomicIncrement32(volatile int32_t* ptr);
+    int32_t AtomicDecrement32(volatile int32_t* ptr);
+    int32_t AtomicAdd32(volatile int32_t* ptr, int32_t val);
     
-    uint64_t AtomicAdd64(volatile uint64_t* ptr, uint64_t val);
+    int64_t AtomicAdd64(volatile int64_t* ptr, int64_t val);
   }
 }
 
index 8c07b3e2a06982d9686bd06fa7f4bc1b8a361a43..97d5043dc416e6b20ddebae5bd0abc57d56c7705 100644 (file)
@@ -103,7 +103,7 @@ private:
   /// has no AbstractTypeUsers, the type is deleted.  This is only sensical for
   /// derived types.
   ///
-  mutable uint32_t RefCount;
+  mutable int32_t RefCount;
 
   const Type *getForwardedTypeInternal() const;
 
@@ -347,7 +347,7 @@ public:
 
     // If this is the last PATypeHolder using this object, and there are no
     // PATypeHandles using it, the type is dead, delete it now.
-    uint32_t Count = sys::AtomicDecrement32(&RefCount);
+    int32_t Count = sys::AtomicDecrement32(&RefCount);
     if (Count == 0 && AbstractTypeUsers.empty())
       this->destroy();
   }
index 65e14697ede469dd8787528b8fd2c62f44eff2b1..fda27088da136cd9775237561e9d97a6a781aa38 100644 (file)
@@ -52,7 +52,7 @@ uint32_t sys::CompareAndSwap32(volatile uint32_t* ptr,
 #endif
 }
 
-uint32_t sys::AtomicIncrement32(volatile uint32_t* ptr) {
+int32_t sys::AtomicIncrement32(volatile int32_t* ptr) {
 #if LLVM_MULTITHREADED==0
   ++(*ptr);
   return *ptr;
@@ -65,7 +65,7 @@ uint32_t sys::AtomicIncrement32(volatile uint32_t* ptr) {
 #endif
 }
 
-uint32_t sys::AtomicDecrement32(volatile uint32_t* ptr) {
+int32_t sys::AtomicDecrement32(volatile int32_t* ptr) {
 #if LLVM_MULTITHREADED==0
   --(*ptr);
   return *ptr;
@@ -78,7 +78,7 @@ uint32_t sys::AtomicDecrement32(volatile uint32_t* ptr) {
 #endif
 }
 
-uint32_t sys::AtomicAdd32(volatile uint32_t* ptr, uint32_t val) {
+int32_t sys::AtomicAdd32(volatile int32_t* ptr, int32_t val) {
 #if LLVM_MULTITHREADED==0
   *ptr += val;
   return *ptr;
@@ -91,7 +91,7 @@ uint32_t sys::AtomicAdd32(volatile uint32_t* ptr, uint32_t val) {
 #endif
 }
 
-uint64_t sys::AtomicAdd64(volatile uint64_t* ptr, uint64_t val) {
+int64_t sys::AtomicAdd64(volatile int64_t* ptr, int64_t val) {
 #if LLVM_MULTITHREADED==0
   *ptr += val;
   return *ptr;
index 0f6f216ceb805b15225422f43ee9c06761a8304e..6be06d22168a1e13c451629c8302fec54a565ef7 100644 (file)
@@ -165,9 +165,9 @@ std::string Mangler::getValueName(const GlobalValue *GV, const char * Suffix) {
   } else if (!GV->hasName()) {
     // Must mangle the global into a unique ID.
     unsigned TypeUniqueID = getTypeID(GV->getType());
-    static uint32_t GlobalID = 0;
+    static int32_t GlobalID = 0;
     
-    unsigned OldID = GlobalID;
+    int32_t OldID = GlobalID;
     sys::AtomicIncrement32(&GlobalID);
     
     Name = "__unnamed_" + utostr(TypeUniqueID) + "_" + utostr(OldID);