remove write-only array in rev_post_order_and_mark_dfs_back_seme

Message ID nycvar.YFH.7.76.2007201600070.9963@zhemvz.fhfr.qr
State New
Headers show
Series
  • remove write-only array in rev_post_order_and_mark_dfs_back_seme
Related show

Commit Message

Richard Biener July 20, 2020, 2 p.m.
This removes a write-only array in
rev_post_order_and_mark_dfs_back_seme.

Bootstrapped / tested on x86_64-unknown-linux-gnu.

2020-07-20  Richard Biener  <rguenther@suse.de>

	* cfganal.c (rev_post_order_and_mark_dfs_back_seme): Remove
	write-only post array.
---
 gcc/cfganal.c | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

-- 
2.26.2

Patch

diff --git a/gcc/cfganal.c b/gcc/cfganal.c
index b057c90368b..1c91c1e6bf5 100644
--- a/gcc/cfganal.c
+++ b/gcc/cfganal.c
@@ -1181,13 +1181,12 @@  rev_post_order_and_mark_dfs_back_seme (struct function *fn, edge entry,
      a need to re-allocate.  */
   auto_vec<edge, 20> stack (2 * n_basic_blocks_for_fn (fn));
 
-  int *pre = XNEWVEC (int, 2 * last_basic_block_for_fn (fn));
-  int *post = pre + last_basic_block_for_fn (fn);
+  int *pre = XNEWVEC (int, last_basic_block_for_fn (fn));
 
   /* BB flag to track nodes that have been visited.  */
   auto_bb_flag visited (fn);
-  /* BB flag to track which nodes have post[] assigned to avoid
-     zeroing post.  */
+  /* BB flag to track which nodes have postorder visting completed.  Used
+     for backedge marking.  */
   auto_bb_flag post_assigned (fn);
 
   /* Push the first edge on to the stack.  */
@@ -1235,7 +1234,6 @@  rev_post_order_and_mark_dfs_back_seme (struct function *fn, edge entry,
 	    {
 	      /* There are no successors for the DEST node so assign
 		 its reverse completion number.  */
-	      post[dest->index] = rev_post_order_num;
 	      dest->flags |= post_assigned;
 	      rev_post_order[rev_post_order_num] = dest->index;
 	      rev_post_order_num++;
@@ -1244,16 +1242,15 @@  rev_post_order_and_mark_dfs_back_seme (struct function *fn, edge entry,
       else
 	{
 	  if (dest->flags & visited
+	      && !(dest->flags & post_assigned)
 	      && src != entry->src
-	      && pre[src->index] >= pre[dest->index]
-	      && !(dest->flags & post_assigned))
+	      && pre[src->index] >= pre[dest->index])
 	    e->flags |= EDGE_DFS_BACK;
 
 	  if (idx != 0 && stack[idx - 1]->src != src)
 	    {
 	      /* There are no more successors for the SRC node
 		 so assign its reverse completion number.  */
-	      post[src->index] = rev_post_order_num;
 	      src->flags |= post_assigned;
 	      rev_post_order[rev_post_order_num] = src->index;
 	      rev_post_order_num++;