From: law <law@138bc75d-0d04-0410-961f-82ee72b054a4>
Date: Fri, 9 Oct 2015 21:16:16 +0000 (+0000)
Subject: Re: [PATCH] Update SSA_NAME manager to use two lists
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=388e5184a32c6358c975bad5e63be6d6d3fe6225;p=thirdparty%2Fgcc.git

Re: [PATCH] Update SSA_NAME manager to use two lists

	* tree-ssanames.c (flush_ssaname_freelist): Use splice and truncate
	rather than moving each name to the freelist individually.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@228668 138bc75d-0d04-0410-961f-82ee72b054a4
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 69743c318fb8..4b81033bc601 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2015-10-09  Jeff Law  <law@redhat.com>
+
+	* tree-ssanames.c (flush_ssaname_freelist): Use splice and truncate
+	rather than moving each name to the freelist individually.
+
 2015-10-09  Steve Ellcey  <sellcey@imgtec.com>
 
 	* config.gcc (mips*-*-*): Add frame-header-opt.o to extra_objs.
diff --git a/gcc/tree-ssanames.c b/gcc/tree-ssanames.c
index 91f4ed8d7f16..82fd4a15f859 100644
--- a/gcc/tree-ssanames.c
+++ b/gcc/tree-ssanames.c
@@ -127,11 +127,8 @@ ssanames_print_statistics (void)
 void
 flush_ssaname_freelist (void)
 {
-  while (!vec_safe_is_empty (FREE_SSANAMES_QUEUE (cfun)))
-    {
-      tree t = FREE_SSANAMES_QUEUE (cfun)->pop ();
-      vec_safe_push (FREE_SSANAMES (cfun), t);
-    }
+  vec_safe_splice (FREE_SSANAMES (cfun), FREE_SSANAMES_QUEUE (cfun));
+  vec_safe_truncate (FREE_SSANAMES_QUEUE (cfun), 0);
 }
 
 /* Return an SSA_NAME node for variable VAR defined in statement STMT