From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <pheeck@sourceware.org>
Received: by sourceware.org (Postfix, from userid 7879)
	id 8263038582AE; Tue,  6 Sep 2022 09:42:20 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 8263038582AE
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1662457340;
	bh=qKAqdZc4c/UZx9OPtFRqjU0l0JbUYrTOyQZmoeO+s4k=;
	h=From:To:Subject:Date:From;
	b=TFVclr+fy5m2EnewEwVaQLNAxGtMYM0oXELfJGIg+iESDwGzKbLO98q7DHUQwoeWL
	 oFnzdOVv02pMiVr1+xdq1++RZTkdMij/WsWJKSnltQNcooL7fIWjWUajgmPhbiSl16
	 AHD7mnoHF1FHG73GBqgs/MstkUzTTLJnl+/wsSBI=
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
From: Filip Kastl <pheeck@gcc.gnu.org>
To: gcc-cvs@gcc.gnu.org
Subject: [gcc(refs/users/pheeck/heads/sccp)] added freeing vector memory
X-Act-Checkin: gcc
X-Git-Author: Filip Kastl <filip.kastl@gmail.com>
X-Git-Refname: refs/users/pheeck/heads/sccp
X-Git-Oldrev: 768444c46e52c181076428902bd6a3f1ad0a6997
X-Git-Newrev: bbbbeaac9644776f06f5b03c366567c21b7b7288
Message-Id: <20220906094220.8263038582AE@sourceware.org>
Date: Tue,  6 Sep 2022 09:42:20 +0000 (GMT)
List-Id: <gcc-cvs.sourceware.org>

https://gcc.gnu.org/g:bbbbeaac9644776f06f5b03c366567c21b7b7288

commit bbbbeaac9644776f06f5b03c366567c21b7b7288
Author: Filip Kastl <filip.kastl@gmail.com>
Date:   Tue Sep 6 11:42:08 2022 +0200

    added freeing vector memory

Diff:
---
 gcc/sccp.cc | 39 +++++++++++++++++++++------------------
 1 file changed, 21 insertions(+), 18 deletions(-)

diff --git a/gcc/sccp.cc b/gcc/sccp.cc
index 8574d424b66..c202c9f76e5 100644
--- a/gcc/sccp.cc
+++ b/gcc/sccp.cc
@@ -145,10 +145,10 @@ debug_phis (void) // DEBUG
 
 /* Return vector of all PHI functions from all basic blocks.  */
 
-static vec<gphi *>
+static auto_vec<gphi *>
 get_all_phis (void)
 {
-  vec<gphi *> result = vNULL;
+  auto_vec<gphi *> result;
 
   basic_block bb;
   FOR_EACH_BB_FN (bb, cfun)
@@ -164,10 +164,10 @@ get_all_phis (void)
   return result;
 }
 
-static vec<vertex>
-tarjan_phis_to_vertices (vec<gphi *> phis)
+static auto_vec<vertex>
+tarjan_phis_to_vertices (auto_vec<gphi *> &phis)
 {
-  vec<vertex> result = vNULL;
+  auto_vec<vertex> result;
   for (gphi *phi : phis)
     {
       vertex v;
@@ -181,15 +181,15 @@ tarjan_phis_to_vertices (vec<gphi *> phis)
   return result;
 }
 
-static vec<vec<gphi *>>
-tarjan_compute_sccs (vec<gphi *> phis)
+static auto_vec<vec<gphi *>>
+tarjan_compute_sccs (auto_vec<gphi *> &phis)
 {
-  vec<vec<gphi *>> sccs = vNULL;
-  vec<unsigned> worklist = vNULL;
-  vec<unsigned> stack = vNULL;
+  auto_vec<vec<gphi *>> sccs;
+  auto_vec<unsigned> worklist;
+  auto_vec<unsigned> stack;
   unsigned curr_index = 0;
 
-  vec<vertex> vs = tarjan_phis_to_vertices (phis);
+  auto_vec<vertex> vs = tarjan_phis_to_vertices (phis);
 
   /* Assign phinums and set PHI functions' 'using' flag.  */
   unsigned i;
@@ -297,7 +297,7 @@ tarjan_compute_sccs (vec<gphi *> phis)
 	  /* If this is the root of an scc, pop it from stack.  */
 	  if (vs[i].lowlink == vs[i].index)
 	    {
-	      vec<gphi *> scc = vNULL;
+	      auto_vec<gphi *> scc;
 
 	      unsigned j;
 	      do
@@ -330,7 +330,7 @@ tarjan_compute_sccs (vec<gphi *> phis)
 }
 
 static void
-replace_scc_by_value (vec<gphi *> scc, tree v)
+replace_scc_by_value (vec<gphi *> &scc, tree v)
 {
   for (gphi *phi : scc)
     {
@@ -381,16 +381,16 @@ remove_zero_uses_phis ()
 }
 
 static void
-remove_redundant_phis (vec<gphi *> phis)
+remove_redundant_phis (auto_vec<gphi *> &phis)
 {
-  vec<vec<gphi *>> worklist = vNULL;
+  auto_vec<vec<gphi *>> worklist;
   worklist = tarjan_compute_sccs (phis);
 
   while (!worklist.is_empty ())
     {
       vec<gphi *> scc = worklist.pop ();
 
-      vec<gphi *> inner = vNULL;
+      auto_vec<gphi *> inner;
       hash_set<tree> outer_ops;
 
       for (gphi *phi : scc)
@@ -448,12 +448,14 @@ remove_redundant_phis (vec<gphi *> phis)
       else if (outer_ops.elements () > 1)
 	{
 	  /* Add inner sccs to worklist.  */
-	  vec<vec<gphi *>> inner_sccs = tarjan_compute_sccs (inner);
+	  auto_vec<vec<gphi *>> inner_sccs = tarjan_compute_sccs (inner);
 	  for (vec<gphi *> inner_scc : inner_sccs)
 	    {
 	      worklist.safe_push (inner_scc);
 	    }
 	}
+
+      scc.release();
     }
 
   remove_zero_uses_phis ();
@@ -494,7 +496,8 @@ pass_sccp::execute (function *)
   //debug_phis (); // DEBUG
 
   init_sccp ();
-  remove_redundant_phis (get_all_phis ());
+  auto_vec<gphi *> phis = get_all_phis ();
+  remove_redundant_phis (phis);
   finalize_sccp ();
 
   /*