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!

Reply via email to