Clean up the ownership model a bit so that nodes actually get deleted more

frequently. This still leaks edges quite a bit, but it leaks no nodes
(I think).


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@2190 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner 2002-04-09 05:15:33 +00:00
parent fb005fe5c2
commit f3dd05ca03
2 changed files with 16 additions and 24 deletions

View File

@ -168,6 +168,9 @@ SchedGraphNode::SchedGraphNode(unsigned int _nodeId,
/*dtor*/
SchedGraphNode::~SchedGraphNode()
{
// for each node, delete its out-edges
std::for_each(beginOutEdges(), endOutEdges(),
deleter<SchedGraphEdge>);
}
void SchedGraphNode::dump(int indent=0) const {
@ -233,16 +236,9 @@ SchedGraph::SchedGraph(const BasicBlock* bb,
SchedGraph::~SchedGraph()
{
for (const_iterator I = begin(); I != end(); ++I)
{
SchedGraphNode *node = I->second;
// for each node, delete its out-edges
std::for_each(node->beginOutEdges(), node->endOutEdges(),
deleter<SchedGraphEdge>);
// then delete the node itself.
delete node;
}
delete I->second;
delete graphRoot;
delete graphLeaf;
}
@ -967,8 +963,8 @@ SchedGraphSet::SchedGraphSet(const Function* _function,
SchedGraphSet::~SchedGraphSet()
{
// delete all the graphs
for (const_iterator I = begin(); I != end(); ++I)
delete *I;
for(iterator I = begin(), E = end(); I != E; ++I)
delete *I; // destructor is a friend
}

View File

@ -168,6 +168,9 @@ SchedGraphNode::SchedGraphNode(unsigned int _nodeId,
/*dtor*/
SchedGraphNode::~SchedGraphNode()
{
// for each node, delete its out-edges
std::for_each(beginOutEdges(), endOutEdges(),
deleter<SchedGraphEdge>);
}
void SchedGraphNode::dump(int indent=0) const {
@ -233,16 +236,9 @@ SchedGraph::SchedGraph(const BasicBlock* bb,
SchedGraph::~SchedGraph()
{
for (const_iterator I = begin(); I != end(); ++I)
{
SchedGraphNode *node = I->second;
// for each node, delete its out-edges
std::for_each(node->beginOutEdges(), node->endOutEdges(),
deleter<SchedGraphEdge>);
// then delete the node itself.
delete node;
}
delete I->second;
delete graphRoot;
delete graphLeaf;
}
@ -967,8 +963,8 @@ SchedGraphSet::SchedGraphSet(const Function* _function,
SchedGraphSet::~SchedGraphSet()
{
// delete all the graphs
for (const_iterator I = begin(); I != end(); ++I)
delete *I;
for(iterator I = begin(), E = end(); I != E; ++I)
delete *I; // destructor is a friend
}