6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-02-11 11:34:02 +00:00
Code Issues Projects Releases Wiki Activity

Use SmartMutex to simplify.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@73712 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Owen Anderson 2009-06-18 18:36:34 +00:00
parent f0eeb9b7f5
commit 0e4d606684
1 changed files with 12 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
lib/VMCore/Type.cpp
View File

@ -47,7 +47,9 @@ AbstractTypeUser::~AbstractTypeUser() {}
static ManagedStatic<sys::RWMutex> TypeMapLock; static ManagedStatic<sys::RWMutex> TypeMapLock;
// Recursive lock used for guarding access to AbstractTypeUsers. // Recursive lock used for guarding access to AbstractTypeUsers.
static ManagedStatic<sys::Mutex> AbstractTypeUsersLock; // NOTE: The false template parameter means this will no-op when we're not in
// multithreaded mode.
static ManagedStatic<sys::SmartMutex<true> > AbstractTypeUsersLock;
// Concrete/Abstract TypeDescriptions - We lazily calculate type descriptions // Concrete/Abstract TypeDescriptions - We lazily calculate type descriptions
// for types as they are needed. Because resolution of types must invalidate // for types as they are needed. Because resolution of types must invalidate
@ -1469,13 +1471,9 @@ bool PointerType::isValidElementType(const Type *ElemTy) {
// it. This function is called primarily by the PATypeHandle class. // it. This function is called primarily by the PATypeHandle class.
void Type::addAbstractTypeUser(AbstractTypeUser *U) const { void Type::addAbstractTypeUser(AbstractTypeUser *U) const {
assert(isAbstract() && "addAbstractTypeUser: Current type not abstract!"); assert(isAbstract() && "addAbstractTypeUser: Current type not abstract!");
if (llvm_is_multithreaded()) { AbstractTypeUsersLock->acquire();
AbstractTypeUsersLock->acquire(); AbstractTypeUsers.push_back(U);
AbstractTypeUsers.push_back(U); AbstractTypeUsersLock->release();
AbstractTypeUsersLock->release();
} else {
AbstractTypeUsers.push_back(U);
}
} }
@ -1485,7 +1483,7 @@ void Type::addAbstractTypeUser(AbstractTypeUser *U) const {
// is annihilated, because there is no way to get a reference to it ever again. // is annihilated, because there is no way to get a reference to it ever again.
// //
void Type::removeAbstractTypeUser(AbstractTypeUser *U) const { void Type::removeAbstractTypeUser(AbstractTypeUser *U) const {
if (llvm_is_multithreaded()) AbstractTypeUsersLock->acquire(); AbstractTypeUsersLock->acquire();
// Search from back to front because we will notify users from back to // Search from back to front because we will notify users from back to
// front. Also, it is likely that there will be a stack like behavior to // front. Also, it is likely that there will be a stack like behavior to
@ -1514,7 +1512,7 @@ void Type::removeAbstractTypeUser(AbstractTypeUser *U) const {
this->destroy(); this->destroy();
} }
if (llvm_is_multithreaded()) AbstractTypeUsersLock->release(); AbstractTypeUsersLock->release();
} }
// unlockedRefineAbstractTypeTo - This function is used when it is discovered // unlockedRefineAbstractTypeTo - This function is used when it is discovered
@ -1565,7 +1563,7 @@ void DerivedType::unlockedRefineAbstractTypeTo(const Type *NewType) {
// will not cause users to drop off of the use list. If we resolve to ourself // will not cause users to drop off of the use list. If we resolve to ourself
// we succeed! // we succeed!
// //
if (llvm_is_multithreaded()) AbstractTypeUsersLock->acquire(); AbstractTypeUsersLock->acquire();
while (!AbstractTypeUsers.empty() && NewTy != this) { while (!AbstractTypeUsers.empty() && NewTy != this) {
AbstractTypeUser *User = AbstractTypeUsers.back(); AbstractTypeUser *User = AbstractTypeUsers.back();
@ -1581,7 +1579,7 @@ void DerivedType::unlockedRefineAbstractTypeTo(const Type *NewType) {
assert(AbstractTypeUsers.size() != OldSize && assert(AbstractTypeUsers.size() != OldSize &&
"AbsTyUser did not remove self from user list!"); "AbsTyUser did not remove self from user list!");
} }
if (llvm_is_multithreaded()) AbstractTypeUsersLock->release(); AbstractTypeUsersLock->release();
// If we were successful removing all users from the type, 'this' will be // If we were successful removing all users from the type, 'this' will be
// deleted when the last PATypeHolder is destroyed or updated from this type. // deleted when the last PATypeHolder is destroyed or updated from this type.
@ -1612,7 +1610,7 @@ void DerivedType::notifyUsesThatTypeBecameConcrete() {
DOUT << "typeIsREFINED type: " << (void*)this << " " << *this << "\n"; DOUT << "typeIsREFINED type: " << (void*)this << " " << *this << "\n";
#endif #endif
if (llvm_is_multithreaded()) AbstractTypeUsersLock->acquire(); AbstractTypeUsersLock->acquire();
unsigned OldSize = AbstractTypeUsers.size(); OldSize=OldSize; unsigned OldSize = AbstractTypeUsers.size(); OldSize=OldSize;
while (!AbstractTypeUsers.empty()) { while (!AbstractTypeUsers.empty()) {
AbstractTypeUser *ATU = AbstractTypeUsers.back(); AbstractTypeUser *ATU = AbstractTypeUsers.back();
@ -1621,7 +1619,7 @@ void DerivedType::notifyUsesThatTypeBecameConcrete() {
assert(AbstractTypeUsers.size() < OldSize-- && assert(AbstractTypeUsers.size() < OldSize-- &&
"AbstractTypeUser did not remove itself from the use list!"); "AbstractTypeUser did not remove itself from the use list!");
} }
if (llvm_is_multithreaded()) AbstractTypeUsersLock->release(); AbstractTypeUsersLock->release();
} }
// refineAbstractType - Called when a contained type is found to be more // refineAbstractType - Called when a contained type is found to be more