Re: [PATCH] C++-ify and simplify loop iterators

2013-11-24 Thread H.J. Lu
On Sun, Nov 24, 2013 at 5:29 AM, H.J. Lu wrote: > On Tue, Nov 19, 2013 at 7:19 PM, H.J. Lu wrote: >> On Tue, Nov 19, 2013 at 7:01 AM, Richard Biener wrote: >>> >>> $subject - the following turns >>> >>> loop_iterator li; >>> FOR_EACH_LOOP (li, loop, LI_ONLY_INNERMOST) >>>{ >>> ... >>>

Re: [PATCH] C++-ify and simplify loop iterators

2013-11-24 Thread H.J. Lu
On Tue, Nov 19, 2013 at 7:19 PM, H.J. Lu wrote: > On Tue, Nov 19, 2013 at 7:01 AM, Richard Biener wrote: >> >> $subject - the following turns >> >> loop_iterator li; >> FOR_EACH_LOOP (li, loop, LI_ONLY_INNERMOST) >>{ >> ... >> if () >>FOR_EACH_LOOP_BREAK; >>} >> >> int

Re: [PATCH] C++-ify and simplify loop iterators

2013-11-19 Thread H.J. Lu
On Tue, Nov 19, 2013 at 7:01 AM, Richard Biener wrote: > > $subject - the following turns > > loop_iterator li; > FOR_EACH_LOOP (li, loop, LI_ONLY_INNERMOST) >{ > ... > if () >FOR_EACH_LOOP_BREAK; >} > > into > > FOR_EACH_LOOP (loop, LI_ONLY_INNERMOST) >{ > ..

Re: [PATCH] C++-ify and simplify loop iterators

2013-11-19 Thread Diego Novillo
On Tue, Nov 19, 2013 at 10:09 AM, Michael Matz wrote: > OMG, finally a c++ification that _shrinks_ client code instead of > bloating it. Not quite. The patch that converted VEC macros removed a non-trivial amount of client code. $ git log -p -n1 f1f41a6cdc47d5123dd30ab110cc35c90f8189cb -- $(fin

Re: [PATCH] C++-ify and simplify loop iterators

2013-11-19 Thread Peter Bergner
On Tue, 2013-11-19 at 16:01 +0100, Richard Biener wrote: > $subject - the following turns > > loop_iterator li; > FOR_EACH_LOOP (li, loop, LI_ONLY_INNERMOST) >{ > ... > if () >FOR_EACH_LOOP_BREAK; >} > > into > > FOR_EACH_LOOP (loop, LI_ONLY_INNERMOST) >{ >

Re: [PATCH] C++-ify and simplify loop iterators

2013-11-19 Thread Michael Matz
Hi, On Tue, 19 Nov 2013, Richard Biener wrote: > $subject - the following turns > > loop_iterator li; > FOR_EACH_LOOP (li, loop, LI_ONLY_INNERMOST) >{ > ... > if () >FOR_EACH_LOOP_BREAK; >} > > into > > FOR_EACH_LOOP (loop, LI_ONLY_INNERMOST) >{ > ... >