On Fri, Oct 30, 2015 at 12:16 PM, Richard Sandiford <richard.sandif...@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; > } > >
