Re: [PATCH][2/2] Sanity check removed loops

2014-09-08 Thread Richard Biener
On Fri, 5 Sep 2014, Andreas Schwab wrote: > Richard Biener writes: > > > 2014-09-05 Richard Biener > > > > * cfgloop.c (mark_loop_for_removal): Record former header > > when ENABLE_CHECKING. > > * cfgloop.h (strut loop): Add former_header member when > > ENABLE_CHECKING. > >

Re: [PATCH][2/2] Sanity check removed loops

2014-09-05 Thread H.J. Lu
On Fri, Sep 5, 2014 at 3:00 AM, Richard Biener wrote: > > The following patch adds sanity checking for loops removed > via mark_loop_for_removal or fix_loop_structure to check > whether they reappear in the same place. > > The simplest case is where the loop header stays the same > and we definite

Re: [PATCH][2/2] Sanity check removed loops

2014-09-05 Thread Jeff Law
On 09/05/14 04:00, Richard Biener wrote: The following patch adds sanity checking for loops removed via mark_loop_for_removal or fix_loop_structure to check whether they reappear in the same place. The simplest case is where the loop header stays the same and we definitely should avoid all thes

Re: [PATCH][2/2] Sanity check removed loops

2014-09-05 Thread Andreas Schwab
Richard Biener writes: > 2014-09-05 Richard Biener > > * cfgloop.c (mark_loop_for_removal): Record former header > when ENABLE_CHECKING. > * cfgloop.h (strut loop): Add former_header member when > ENABLE_CHECKING. > * loop-init.c (fix_loop_structure): Sanity check