On Fri, Dec 7, 2012 at 1:05 AM, Steven Bosscher <stevenb....@gmail.com> wrote:
> Hello,
>
> ree.c's data structures are not GC-allocated, so its GTY markers and
> its TODO_ggc_collect are unnecessary.
>
> Will commit as obvious after the usual testing on my favourite platforms :-)
IMHO TODO_ggc_collect should go and we should collect after each pass run.
Richard.
> * ree.c (struct ext_cand): Remove GTY markers.
> (pass_ree): Remove TODO_ggc_collect.
>
> Index: ree.c
> ===================================================================
> --- ree.c (revision 194247)
> +++ ree.c (working copy)
> @@ -243,7 +243,7 @@
>
> /* This structure represents a candidate for elimination. */
>
> -typedef struct GTY(()) ext_cand
> +typedef struct ext_cand
> {
> /* The expression. */
> const_rtx expr;
> @@ -958,7 +958,6 @@
> 0, /* properties_provided */
> 0, /* properties_destroyed */
> 0, /* todo_flags_start */
> - TODO_ggc_collect |
> TODO_verify_rtl_sharing, /* todo_flags_finish */
> }
> };
