diff --git a/include/llvm/ADT/STLExtras.h b/include/llvm/ADT/STLExtras.h index 8dbf79031c3..32cf4590e99 100644 --- a/include/llvm/ADT/STLExtras.h +++ b/include/llvm/ADT/STLExtras.h @@ -279,28 +279,6 @@ static inline void array_pod_sort(IteratorTy Start, IteratorTy End, qsort(&*Start, End-Start, sizeof(*Start), Compare); } -//===----------------------------------------------------------------------===// -// Extra additions to -//===----------------------------------------------------------------------===// - -/// For a container of pointers, deletes the pointers and then clears the -/// container. -template -void DeleteContainerPointers(Container &C) { - for (typename Container::iterator I = C.begin(), E = C.end(); I != E; ++I) - delete *I; - C.clear(); -} - -/// In a container of pairs (usually a map) whose second element is a pointer, -/// deletes the second elements and then clears the container. -template -void DeleteContainerSeconds(Container &C) { - for (typename Container::iterator I = C.begin(), E = C.end(); I != E; ++I) - delete I->second; - C.clear(); -} - } // End llvm namespace #endif diff --git a/lib/CodeGen/AsmPrinter/DwarfDebug.cpp b/lib/CodeGen/AsmPrinter/DwarfDebug.cpp index d83cbf21d24..5ad1e5ea050 100644 --- a/lib/CodeGen/AsmPrinter/DwarfDebug.cpp +++ b/lib/CodeGen/AsmPrinter/DwarfDebug.cpp @@ -23,7 +23,6 @@ #include "llvm/Target/TargetFrameInfo.h" #include "llvm/Target/TargetLoweringObjectFile.h" #include "llvm/Target/TargetRegisterInfo.h" -#include "llvm/ADT/STLExtras.h" #include "llvm/ADT/StringExtras.h" #include "llvm/Support/Debug.h" #include "llvm/Support/ErrorHandling.h" @@ -175,10 +174,8 @@ class DbgScope { unsigned EndLabelID; // Label ID of the end of scope. const MachineInstr *LastInsn; // Last instruction of this scope. const MachineInstr *FirstInsn; // First instruction of this scope. - // Scopes defined in scope. Contents not owned. - SmallVector Scopes; - // Variables declared in scope. Contents owned. - SmallVector Variables; + SmallVector Scopes; // Scopes defined in scope. + SmallVector Variables;// Variables declared in scope. // Private state for dump() mutable unsigned IndentLevel; @@ -199,8 +196,8 @@ public: MDNode *getScopeNode() const { return Desc.getNode(); } unsigned getStartLabelID() const { return StartLabelID; } unsigned getEndLabelID() const { return EndLabelID; } - const SmallVector &getScopes() { return Scopes; } - const SmallVector &getVariables() { return Variables; } + SmallVector &getScopes() { return Scopes; } + SmallVector &getVariables() { return Variables; } void setStartLabelID(unsigned S) { StartLabelID = S; } void setEndLabelID(unsigned E) { EndLabelID = E; } void setLastInsn(const MachineInstr *MI) { LastInsn = MI; } @@ -223,14 +220,14 @@ public: assert (getFirstInsn() && "First instruction is missing!"); // Use the end of last child scope as end of this scope. - const SmallVector &Scopes = getScopes(); + SmallVector &Scopes = getScopes(); const MachineInstr *LastInsn = getFirstInsn(); unsigned LIndex = 0; if (Scopes.empty()) { assert (getLastInsn() && "Inner most scope does not have last insn!"); return; } - for (SmallVector::const_iterator SI = Scopes.begin(), + for (SmallVector::iterator SI = Scopes.begin(), SE = Scopes.end(); SI != SE; ++SI) { DbgScope *DS = *SI; DS->fixInstructionMarkers(MIIndexMap); @@ -282,6 +279,8 @@ void DbgScope::dump() const { #endif DbgScope::~DbgScope() { + for (unsigned i = 0, N = Scopes.size(); i < N; ++i) + delete Scopes[i]; for (unsigned j = 0, M = Variables.size(); j < M; ++j) delete Variables[j]; } @@ -1586,7 +1585,7 @@ DIE *DwarfDebug::constructScopeDIE(DbgScope *Scope) { } // Add variables to scope. - const SmallVector &Variables = Scope->getVariables(); + SmallVector &Variables = Scope->getVariables(); for (unsigned i = 0, N = Variables.size(); i < N; ++i) { DIE *VariableDIE = constructVariableDIE(Variables[i], Scope); if (VariableDIE) @@ -1594,7 +1593,7 @@ DIE *DwarfDebug::constructScopeDIE(DbgScope *Scope) { } // Add nested scopes. - const SmallVector &Scopes = Scope->getScopes(); + SmallVector &Scopes = Scope->getScopes(); for (unsigned j = 0, M = Scopes.size(); j < M; ++j) { // Define the Scope debug information entry. DIE *NestedDIE = constructScopeDIE(Scopes[j]); @@ -1697,6 +1696,7 @@ CompileUnit *DwarfDebug::constructCompileUnit(MDNode *N) { } CompileUnitMap[DIUnit.getNode()] = Unit; + CompileUnits.push_back(Unit); return Unit; } @@ -1802,7 +1802,7 @@ void DwarfDebug::beginModule(Module *M, MachineModuleInfo *mmi) { E = DbgFinder.compile_unit_end(); I != E; ++I) constructCompileUnit(*I); - if (CompileUnitMap.empty()) { + if (CompileUnits.empty()) { if (TimePassesIsEnabled) DebugTimer->stopTimer(); @@ -1812,7 +1812,7 @@ void DwarfDebug::beginModule(Module *M, MachineModuleInfo *mmi) { // If main compile unit for this module is not seen than randomly // select first compile unit. if (!ModuleCU) - ModuleCU = CompileUnitMap.begin()->second; + ModuleCU = CompileUnits[0]; // Create DIEs for each subprogram. for (DebugInfoFinder::iterator I = DbgFinder.subprogram_begin(), @@ -1944,10 +1944,6 @@ void DwarfDebug::endModule() { // Emit inline info. emitDebugInlineInfo(); - // Clear debug info in preparation for the next Module. - ModuleCU = NULL; - DeleteContainerSeconds(CompileUnitMap); - if (TimePassesIsEnabled) DebugTimer->stopTimer(); } @@ -2118,9 +2114,9 @@ bool DwarfDebug::extractScopeInformation() { while (!WorkList.empty()) { DbgScope *S = WorkList.back(); WorkList.pop_back(); - const SmallVector &Children = S->getScopes(); + SmallVector &Children = S->getScopes(); if (!Children.empty()) - for (SmallVector::const_iterator SI = Children.begin(), + for (SmallVector::iterator SI = Children.begin(), SE = Children.end(); SI != SE; ++SI) WorkList.push_back(*SI); @@ -2225,11 +2221,10 @@ void DwarfDebug::endFunction(const MachineFunction *MF) { // Clear debug info CurrentFnDbgScope = NULL; - DeleteContainerSeconds(DbgScopeMap); + DbgScopeMap.clear(); DbgScopeBeginMap.clear(); DbgScopeEndMap.clear(); ConcreteScopes.clear(); - DeleteContainerSeconds(AbstractScopes); AbstractScopesList.clear(); Lines.clear(); diff --git a/lib/CodeGen/AsmPrinter/DwarfDebug.h b/lib/CodeGen/AsmPrinter/DwarfDebug.h index 8d75b1ff937..55baa921006 100644 --- a/lib/CodeGen/AsmPrinter/DwarfDebug.h +++ b/lib/CodeGen/AsmPrinter/DwarfDebug.h @@ -66,6 +66,10 @@ class DwarfDebug : public DwarfPrinter { /// compile units. DenseMap CompileUnitMap; + /// CompileUnits - All the compile units in this module. + /// + SmallVector CompileUnits; + /// ModuleCU - All DIEs are inserted in ModuleCU. CompileUnit *ModuleCU; @@ -130,8 +134,7 @@ class DwarfDebug : public DwarfPrinter { // DbgScope *CurrentFnDbgScope; - /// DbgScopeMap - Tracks the scopes in the current function. Owns the - /// contained DbgScope*s. + /// DbgScopeMap - Tracks the scopes in the current function. /// DenseMap DbgScopeMap; @@ -140,7 +143,7 @@ class DwarfDebug : public DwarfPrinter { DenseMap ConcreteScopes; /// AbstractScopes - Tracks the abstract scopes a module. These scopes are - /// not included DbgScopeMap. AbstractScopes owns its DbgScope*s. + /// not included DbgScopeMap. DenseMap AbstractScopes; SmallVectorAbstractScopesList;