public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] Remove unused modified_noreturn_calls
@ 2015-12-17  9:59 Richard Biener
  2015-12-17 13:05 ` Bernd Schmidt
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Biener @ 2015-12-17  9:59 UTC (permalink / raw)
  To: gcc-patches


Bootstrapped on x86_64-unknown-linux-gnu, applied.

Richard.

2015-12-16  Richard Biener  <rguenther@suse.de>

	* gimple-ssa.h (struct gimple_df): Remove modified_noreturn_calls
	field.
	* tree-ssa.c (delete_tree_ssa): Do not zero it.

Index: gcc/gimple-ssa.h
===================================================================
--- gcc/gimple-ssa.h	(revision 231696)
+++ gcc/gimple-ssa.h	(working copy)
@@ -44,6 +44,9 @@ struct tm_restart_hasher : ggc_ptr_hash<
   }
 };
 
+extern void gt_ggc_mx (gimple *&);
+extern void gt_pch_nx (gimple *&);
+
 struct ssa_name_hasher : ggc_ptr_hash<tree_node>
 {
   /* Hash a tree in a uid_decl_map.  */
@@ -67,13 +70,6 @@ struct ssa_name_hasher : ggc_ptr_hash<tr
    gimple_ accessor defined, all publicly modifiable fields should have
    gimple_set accessor.  */
 struct GTY(()) gimple_df {
-  /* A vector of all the noreturn calls passed to modify_stmt.
-     cleanup_control_flow uses it to detect cases where a mid-block
-     indirect call has been turned into a noreturn call.  When this
-     happens, all the instructions after the call are no longer
-     reachable and must be deleted as dead.  */
-  vec<gimple *, va_gc> *modified_noreturn_calls;
-
   /* Array of all SSA_NAMEs used in the function.  */
   vec<tree, va_gc> *ssa_names;
 
Index: gcc/tree-ssa.c
===================================================================
--- gcc/tree-ssa.c	(revision 231696)
+++ gcc/tree-ssa.c	(working copy)
@@ -1124,7 +1124,6 @@ delete_tree_ssa (struct function *fn)
   if (fn->gimple_df->decls_to_pointers != NULL)
     delete fn->gimple_df->decls_to_pointers;
   fn->gimple_df->decls_to_pointers = NULL;
-  fn->gimple_df->modified_noreturn_calls = NULL;
   fn->gimple_df = NULL;
 
   /* We no longer need the edge variable maps.  */

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Remove unused modified_noreturn_calls
  2015-12-17  9:59 [PATCH] Remove unused modified_noreturn_calls Richard Biener
@ 2015-12-17 13:05 ` Bernd Schmidt
  2015-12-17 13:22   ` Richard Biener
  0 siblings, 1 reply; 3+ messages in thread
From: Bernd Schmidt @ 2015-12-17 13:05 UTC (permalink / raw)
  To: Richard Biener, gcc-patches

On 12/17/2015 10:59 AM, Richard Biener wrote:
>
> +extern void gt_ggc_mx (gimple *&);
> +extern void gt_pch_nx (gimple *&);
> +

This doesn't occur in the ChangeLog - unrelated change?


Bernd

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Remove unused modified_noreturn_calls
  2015-12-17 13:05 ` Bernd Schmidt
@ 2015-12-17 13:22   ` Richard Biener
  0 siblings, 0 replies; 3+ messages in thread
From: Richard Biener @ 2015-12-17 13:22 UTC (permalink / raw)
  To: Bernd Schmidt; +Cc: gcc-patches

On Thu, 17 Dec 2015, Bernd Schmidt wrote:

> On 12/17/2015 10:59 AM, Richard Biener wrote:
> > 
> > +extern void gt_ggc_mx (gimple *&);
> > +extern void gt_pch_nx (gimple *&);
> > +
> 
> This doesn't occur in the ChangeLog - unrelated change?

Not unrelated, it's required to make gtype-desc.c compile.  See
other occurances of these forward-decls.  They are needed from
hash_map/table.

Took me quite a while to figure out ;)

Richard.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-12-17 13:22 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-17  9:59 [PATCH] Remove unused modified_noreturn_calls Richard Biener
2015-12-17 13:05 ` Bernd Schmidt
2015-12-17 13:22   ` Richard Biener

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).