Re: DCE speedup

[Jan Hubicka <jh at suse dot cz>]

> >+ /* What to do when we cleaned up some statements.  */
> >+ #define DCE_TODOs (TODO_update_ssa | TODO_cleanup_cfg | TODO_ggc_collect 
> >\
> >+                  | TODO_remove_unused_locals)
> 
> I don't like this - you only have one user here:
> 
> >--- 831,863 ----
> >    tree_dce_done (aggressive);
> >
> >    free_edge_list (el);
> >+   return something_changed ? DCE_TODOs : 0;
> >  }
> 
> Just write it as
>  if (something_changed)
>    return ...;
> 
>  return 0;

Thanks, the define got in since I was expecting there to be separate
execute_* for the different DCE variants, but then I moved it to the
common code.

Honza