[llvm] r203364 - [C++11] Add range based accessors for the Use-Def chain of a Value.
Chris Lattner
clattner at apple.com
Sat Mar 8 19:56:33 PST 2014
On Mar 8, 2014, at 7:16 PM, Chandler Carruth <chandlerc at gmail.com> wrote:
> Author: chandlerc
> Date: Sat Mar 8 21:16:01 2014
> New Revision: 203364
>
> URL: http://llvm.org/viewvc/llvm-project?rev=203364&view=rev
> Log:
> [C++11] Add range based accessors for the Use-Def chain of a Value.
Cool, thank you for working on this.
> Because #6 requires churning essentially everything that walked the
> Use-Def chains, I went ahead and added all of the range adaptors and
> switched them to range-based loops where appropriate. Also because the
> renaming requires at least churning every line of code, it didn't make
> any sense to split these up into multiple commits -- all of which would
> touch all of the same lies of code.
How did you check that it was safe to convert these cases to foreach loops? Did you only convert loops that explicitly precalculated the end() point?
-Chris
More information about the llvm-commits
mailing list