diff --git a/lib/VMCore/Constants.cpp b/lib/VMCore/Constants.cpp index 7bbbdb32023..ab4bdb390b9 100644 --- a/lib/VMCore/Constants.cpp +++ b/lib/VMCore/Constants.cpp @@ -463,6 +463,26 @@ void ConstantExpr::replaceUsesOfWithOnConstant(Value *From, Value *ToV) { //===----------------------------------------------------------------------===// // Factory Function Implementation +// ReplaceUsesOfWith - This is exactly the same as Value::replaceAllUsesWith, +// except that it doesn't have all of the asserts. The asserts fail because we +// are half-way done resolving types, which causes some types to exist as two +// different Type*'s at the same time. This is a sledgehammer to work around +// this problem. +// +static void ReplaceUsesOfWith(Value *Old, Value *New) { + while (!Old->use_empty()) { + User *Use = Old->use_back(); + // Must handle Constants specially, we cannot call replaceUsesOfWith on a + // constant! + if (Constant *C = dyn_cast(Use)) { + C->replaceUsesOfWithOnConstant(Old, New); + } else { + Use->replaceUsesOfWith(Old, New); + } + } +} + + // ConstantCreator - A class that is used to create constants by // ValueMap*. This class should be partially specialized if there is // something strange that needs to be done to interface to the ctor for the @@ -573,7 +593,7 @@ void ConstantArray::refineAbstractType(const DerivedType *OldTy, C.push_back(cast(getOperand(i))); Constant *New = ConstantArray::get(cast(NewTy), C); if (New != this) { - replaceAllUsesWith(New); + ReplaceUsesOfWith(this, New); destroyConstant(); // This constant is now dead, destroy it. } } @@ -642,7 +662,7 @@ void ConstantStruct::refineAbstractType(const DerivedType *OldTy, C.push_back(cast(getOperand(i))); Constant *New = ConstantStruct::get(cast(NewTy), C); if (New != this) { - replaceAllUsesWith(New); + ReplaceUsesOfWith(this, New); destroyConstant(); // This constant is now dead, destroy it. } } @@ -683,7 +703,7 @@ void ConstantPointerNull::refineAbstractType(const DerivedType *OldTy, // Make everyone now use a constant of the new type... Constant *New = ConstantPointerNull::get(cast(NewTy)); if (New != this) { - replaceAllUsesWith(New); + ReplaceUsesOfWith(this, New); // This constant is now dead, destroy it. destroyConstant(); @@ -836,7 +856,7 @@ void ConstantExpr::refineAbstractType(const DerivedType *OldTy, New = ConstantExpr::getGetElementPtr(getOperand(0), C); } if (New != this) { - replaceAllUsesWith(New); + ReplaceUsesOfWith(this, New); destroyConstant(); // This constant is now dead, destroy it. } }