This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Patch for cfgloopmanip.c - misplaced semicolon
- From: Zdenek Dvorak <rakdver at atrey dot karlin dot mff dot cuni dot cz>
- To: fnf at intrinsity dot com
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 5 Feb 2003 20:57:51 +0100
- Subject: Re: Patch for cfgloopmanip.c - misplaced semicolon
- References: <200302051650.h15Gokin001792@fred.ninemoons.com>
Hello,
> 2003-02-05 Fred Fish <fnf@intrinsity.com>
>
> * cfgloopmanip.c (force_single_succ_latches): Remove semicolon
> that nullified intended for loop body.
no, it is correct this way -- this loop just looks up the edge from
latch.
Zdenek
> Index: cfgloopmanip.c
> ===================================================================
> RCS file: /cvs/gcc/gcc/gcc/cfgloopmanip.c,v
> retrieving revision 1.3
> diff -c -p -r1.3 cfgloopmanip.c
> *** cfgloopmanip.c 28 Jan 2003 11:05:18 -0000 1.3
> --- cfgloopmanip.c 5 Feb 2003 16:48:19 -0000
> *************** force_single_succ_latches (loops)
> *** 159,165 ****
> if (!loop->latch->succ->succ_next)
> continue;
>
> ! for (e = loop->header->pred; e->src != loop->latch; e = e->pred_next);
> loop_split_edge_with (e, NULL_RTX, loops);
> }
> loops->state |= LOOPS_HAVE_SIMPLE_LATCHES;
> --- 159,165 ----
> if (!loop->latch->succ->succ_next)
> continue;
>
> ! for (e = loop->header->pred; e->src != loop->latch; e = e->pred_next)
> loop_split_edge_with (e, NULL_RTX, loops);
> }
> loops->state |= LOOPS_HAVE_SIMPLE_LATCHES;