<p dir="ltr">I see no value in eagerly converting all the existing uses and a lot of churn. Please don't.</p>
<p dir="ltr">I only converted the use_begin stuff because the iterator there was changing anyways forcing the churn.</p>
<div class="gmail_quote">On Mar 26, 2014 11:03 AM, "David Blaikie" <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, Mar 26, 2014 at 10:36 AM, Eli Bendersky <<a href="mailto:eliben@google.com">eliben@google.com</a>> wrote:<br>
> Hello,<br>
><br>
> This patch is in similar vein to what done earlier to<br>
> Module::globals/aliases etc. I would really like to iterate over function<br>
> arguments like this in new code:<br>
><br>
>   for (Argument Arg : F.args()) {<br>
>     ...<br>
>   }<br>
><br>
> The question is: would it be OK to change existing code from explicit<br>
> iteration with the .args_begin/args_end idiom to this new idiom?<br>
<br>
There seems to be lots of precedent for such changes already in both<br>
Clang and LLVM - is there anything that makes this particular example<br>
worthy of more review (see Owen Anderson, Chandler Carruth, and Aaron<br>
Ballman's patches to LLVM and Clang doing widescale changes/migrations<br>
to range-for).<br>
<br>
If there's nothing particularly noteworthy about this range over the<br>
others that have been migrated, you can probably just skip the<br>
precommit review here. (or take my words here as sufficient precommit<br>
review - essentially "if there's nothing special about this range, I<br>
(David Blaikie) would feel comfortable committing this, so I'm<br>
comfortable signing off on someone else committing it")<br>
<br>
- David<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>