On Wed, Sep 27, 2017 at 6:48 AM, Richard Biener <rguent...@suse.de> wrote: > > Turns out loop_nest recorded in scop-info isn't really necessary as > we can simply process parameters in loop bounds during the gather_bbs > walk where we encounter each loop (identified by its header) once. > > This avoids the linear search in record_loop_in_sese. > > Bootstrap / regtest running on x86_64-unknown-linux-gnu, will apply. > > Richard. > > 2017-09-27 Richard Biener <rguent...@suse.de> > > * graphite-scop-detection.c (find_scop_parameters): Move > loop bound handling ... > (gather_bbs::before_dom_children): ... here, avoiding the need > to build scop_info->loop_nest. > (record_loop_in_sese): Remove. > * sese.h (sese_info_t::loop_nest): Remove. > * sese.c (new_sese_info): Do not allocate loop_nest. > (free_sese_info): Do not free loop_nest.
Looks good. Thanks!