diff options
-rw-r--r-- | lib/VMCore/Attributes.cpp | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/lib/VMCore/Attributes.cpp b/lib/VMCore/Attributes.cpp index 6f5ecd278d..d854628603 100644 --- a/lib/VMCore/Attributes.cpp +++ b/lib/VMCore/Attributes.cpp @@ -106,7 +106,10 @@ Attributes Attribute::typeIncompatible(const Type *Ty) { // AttributeListImpl Definition //===----------------------------------------------------------------------===// + namespace llvm { +static ManagedStatic<sys::SmartMutex<true> > ALMutex; + class AttributeListImpl : public FoldingSetNode { sys::cas_flag RefCount; @@ -122,10 +125,15 @@ public: RefCount = 0; } - void AddRef() { sys::AtomicIncrement(&RefCount); } + void AddRef() { + sys::SmartScopedLock<true> Lock(*ALMutex); + ++RefCount; + } void DropRef() { - sys::cas_flag old = sys::AtomicDecrement(&RefCount); - if (old == 0) delete this; + sys::SmartScopedLock<true> Lock(*ALMutex); + sys::cas_flag old = RefCount++; + if (old == 0) + delete this; } void Profile(FoldingSetNodeID &ID) const { @@ -139,11 +147,10 @@ public: }; } -static ManagedStatic<sys::SmartMutex<true> > ALMutex; static ManagedStatic<FoldingSet<AttributeListImpl> > AttributesLists; AttributeListImpl::~AttributeListImpl() { - sys::SmartScopedLock<true> Lock(*ALMutex); + // NOTE: Lock must be acquired by caller. AttributesLists->RemoveNode(this); } |