maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Forgot to remove some explicit locking when it became implicit in the ValueMap. 

llvm-svn: 73778
This commit is contained in:
Owen Anderson 2009-06-19 18:34:09 +00:00
parent 383f7b1f08
commit 59ba814f39
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
llvm/lib/VMCore/Constants.cpp
View File

@ -1473,6 +1473,7 @@ Constant *ConstantArray::get(const ArrayType *Ty,
/// destroyConstant - Remove the constant from the constant table...
///
void ConstantArray::destroyConstant() {
// Implicitly locked.
ArrayConstants->remove(this);
destroyConstantImpl();
}
@ -1601,6 +1602,7 @@ Constant *ConstantStruct::get(const std::vector<Constant*> &V, bool packed) {
// destroyConstant - Remove the constant from the constant table...
//
void ConstantStruct::destroyConstant() {
// Implicitly locked.
StructConstants->remove(this);
destroyConstantImpl();
}
@ -1668,7 +1670,7 @@ Constant *ConstantVector::get(const std::vector<Constant*> &V) {
// destroyConstant - Remove the constant from the constant table...
//
void ConstantVector::destroyConstant() {
sys::SmartScopedWriter<true> Write(&*ConstantsLock);
// Implicitly locked.
VectorConstants->remove(this);
destroyConstantImpl();
}
@ -1740,7 +1742,7 @@ ConstantPointerNull *ConstantPointerNull::get(const PointerType *Ty) {
// destroyConstant - Remove the constant from the constant table...
//
void ConstantPointerNull::destroyConstant() {
sys::SmartScopedWriter<true> Writer(&*ConstantsLock);
// Implicitly locked.
NullPtrConstants->remove(this);
destroyConstantImpl();
}
@ -2721,7 +2723,7 @@ Constant *ConstantExpr::getZeroValueForNegationExpr(const Type *Ty) {
// destroyConstant - Remove the constant from the constant table...
//
void ConstantExpr::destroyConstant() {
sys::SmartScopedWriter<true> Writer(&*ConstantsLock);
// Implicitly locked.
ExprConstants->remove(this);
destroyConstantImpl();
}