This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] for PR 32773


> +   for (ei = ei_start (EXIT_BLOCK_PTR->preds); (e = ei_safe_edge (ei)); )
> +     {
> +       if (e->src == forwarder
> + 	  || !(e->flags & EDGE_FALLTHRU))
> + 	{
> + 	  ei_next (&ei);
> + 	  continue;
> + 	}

Could you rewrite this loop to something more standard?  Nobody understands 
what it does except you. :-)  Either do not use 'for' or use the natural form 
'for (init; cond; update)' with a 'break'.

-- 
Eric Botcazou


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]