DebugInfo: Remove CompileUnit::constructTypeDIEImpl now that it's just a simple wrapper again.

r195698 moved the type unit checking up into getOrCreateTypeDIE so
remove the redundant check and fold the functions back together again.

llvm-svn: 195700
This commit is contained in:
David Blaikie 2013-11-26 00:35:04 +00:00
parent 0decd7da5a
commit fbd29eb3b6
2 changed files with 2 additions and 15 deletions

View File

@ -923,7 +923,7 @@ DIE *CompileUnit::createTypeDIE(DICompositeType Ty) {
// Create new type.
TyDIE = createAndAddDIE(Ty.getTag(), *ContextDIE, Ty);
constructTypeDIEImpl(*TyDIE, Ty);
constructTypeDIE(*TyDIE, Ty);
updateAcceleratorTables(Context, Ty, TyDIE);
return TyDIE;
@ -994,7 +994,7 @@ DIE *CompileUnit::getOrCreateTypeDIE(const MDNode *TyNode) {
// Skip updating the accellerator tables since this is not the full type
return TyDIE;
}
constructTypeDIEImpl(*TyDIE, CTy);
constructTypeDIE(*TyDIE, CTy);
} else {
assert(Ty.isDerivedType() && "Unknown kind of DIType");
constructTypeDIE(*TyDIE, DIDerivedType(Ty));
@ -1169,15 +1169,6 @@ void CompileUnit::constructTypeDIE(DIE &Buffer, DIDerivedType DTy) {
/// constructTypeDIE - Construct type DIE from DICompositeType.
void CompileUnit::constructTypeDIE(DIE &Buffer, DICompositeType CTy) {
// If this is a type applicable to a type unit it then add it to the
// list of types we'll compute a hash for later.
if (shouldCreateTypeUnit(CTy, DD))
DD->addTypeUnitType(&Buffer, CTy);
else
constructTypeDIEImpl(Buffer, CTy);
}
void CompileUnit::constructTypeDIEImpl(DIE &Buffer, DICompositeType CTy) {
// Add name if not anonymous or intermediate type.
StringRef Name = CTy.getName();

View File

@ -327,10 +327,6 @@ public:
DIE *createAndAddDIE(unsigned Tag, DIE &Parent,
DIDescriptor N = DIDescriptor());
/// constructTypeDIEImpl - Construct type DIE that is not a type unit
/// reference from a DICompositeType.
void constructTypeDIEImpl(DIE &Buffer, DICompositeType CTy);
/// Compute the size of a header for this unit, not including the initial
/// length field.
unsigned getHeaderSize() const {