Patchwork don't allocate/free unused pointer set in verify_cgraph_node

login
register
mail settings
Submitter Steven Bosscher
Date Dec. 13, 2012, 10:42 p.m.
Message ID <CABu31nPB-yJU2mAje=h6WSvXijWbUD9b-3pHXSwC6qwHhO6ksA@mail.gmail.com>
Download mbox | patch
Permalink /patch/206264/
State New
Headers show

Comments

Steven Bosscher - Dec. 13, 2012, 10:42 p.m.
Hello,

Looks like a remnant from pre GIMPLE tuples days.
Will commit as obvious.

Ciao!
Steven


        * cgraph.c (verify_cgraph_node): Don't allocate/free visited_nodes set.

Patch

Index: cgraph.c
===================================================================
--- cgraph.c	(revision 194491)
+++ cgraph.c	(working copy)
@@ -2498,9 +2498,6 @@  verify_cgraph_node (struct cgraph_node *
     {
       if (this_cfun->cfg)
 	{
-	  /* The nodes we're interested in are never shared, so walk
-	     the tree ignoring duplicates.  */
-	  struct pointer_set_t *visited_nodes = pointer_set_create ();
 	  /* Reach the trees by walking over the CFG, and note the
 	     enclosing basic-blocks in the call edges.  */
 	  FOR_EACH_BB_FN (this_block, this_cfun)
@@ -2550,7 +2547,6 @@  verify_cgraph_node (struct cgraph_node *
 		      }
 		  }
 	      }
-	  pointer_set_destroy (visited_nodes);
 	}
       else
 	/* No CFG available?!  */