diff mbox

Don't free dominators after sincos

Message ID 87eggcwqr2.fsf@e105548-lin.cambridge.arm.com
State New
Headers show

Commit Message

Richard Sandiford Oct. 30, 2015, 11:16 a.m. UTC 
sincos has always freed dominators at the end, but AFAICT they should
still be up-to-date.  (In particular, gimple_purge_dead_eh_edges
updates the information.)

Tested on x86_64-linux-gnu, arm-linux-gnueabi and aarch64-linux-gnu.
OK to install?

Thanks,
Richard


gcc/
	* tree-ssa-math-opts.c (pass_cse_sincos::execute): Don't free
	CDI_DOMINATORS.

Comments

Richard Biener Oct. 30, 2015, 11:31 a.m. UTC | #1 
On Fri, Oct 30, 2015 at 12:16 PM, Richard Sandiford
<richard.sandiford@arm.com> wrote:
> sincos has always freed dominators at the end, but AFAICT they should
> still be up-to-date.  (In particular, gimple_purge_dead_eh_edges
> updates the information.)
>
> Tested on x86_64-linux-gnu, arm-linux-gnueabi and aarch64-linux-gnu.
> OK to install?

Ok.

Thanks,
Richard.

> Thanks,
> Richard
>
>
> gcc/
>         * tree-ssa-math-opts.c (pass_cse_sincos::execute): Don't free
>         CDI_DOMINATORS.
>
> diff --git a/gcc/tree-ssa-math-opts.c b/gcc/tree-ssa-math-opts.c
> index 2080328..1802754 100644
> --- a/gcc/tree-ssa-math-opts.c
> +++ b/gcc/tree-ssa-math-opts.c
> @@ -1857,7 +1857,6 @@ pass_cse_sincos::execute (function *fun)
>    statistics_counter_event (fun, "sincos statements inserted",
>                             sincos_stats.inserted);
>
> -  free_dominance_info (CDI_DOMINATORS);
>    return cfg_changed ? TODO_cleanup_cfg : 0;
>  }
>
>
diff mbox

Patch

diff --git a/gcc/tree-ssa-math-opts.c b/gcc/tree-ssa-math-opts.c
index 2080328..1802754 100644
--- a/gcc/tree-ssa-math-opts.c
+++ b/gcc/tree-ssa-math-opts.c
@@ -1857,7 +1857,6 @@  pass_cse_sincos::execute (function *fun)
   statistics_counter_event (fun, "sincos statements inserted",
 			    sincos_stats.inserted);
 
-  free_dominance_info (CDI_DOMINATORS);
   return cfg_changed ? TODO_cleanup_cfg : 0;
 }