On Tue, Jan 29, 2013 at 4:49 PM, Marek Polacek <pola...@redhat.com> wrote:
> I don't see any reason why we'd want to mark BBs in this case,
> changed_bbs are thrown away right after fix_loop_structure
> marks them.
>
> Regtested/bootstrapped on x86_64-linux, ok for trunk?
Ok.
Thanks,
Richard.
> 2013-01-29 Marek Polacek <pola...@redhat.com>
>
> * cfgcleanup.c (cleanup_cfg): Don't mark affected BBs.
>
> --- gcc/cfgcleanup.c.mp 2013-01-29 13:53:21.740473075 +0100
> +++ gcc/cfgcleanup.c 2013-01-29 13:53:27.942491341 +0100
> @@ -3017,14 +3017,11 @@ cleanup_cfg (int mode)
> && (changed
> || (mode & CLEANUP_CFG_CHANGED)))
> {
> - bitmap changed_bbs;
> timevar_push (TV_REPAIR_LOOPS);
> /* The above doesn't preserve dominance info if available. */
> gcc_assert (!dom_info_available_p (CDI_DOMINATORS));
> calculate_dominance_info (CDI_DOMINATORS);
> - changed_bbs = BITMAP_ALLOC (NULL);
> - fix_loop_structure (changed_bbs);
> - BITMAP_FREE (changed_bbs);
> + fix_loop_structure (NULL);
> free_dominance_info (CDI_DOMINATORS);
> timevar_pop (TV_REPAIR_LOOPS);
> }
>
> Marek
