This is an archive of the discontinued LLVM Phabricator instance.

Use more range-based for loops
ClosedPublic

Authored by dougk on Jun 25 2015, 10:15 AM.

Details

Summary

And add Arg::getValues method with const 'this' and const result.

As requested by Sean Silva in http://reviews.llvm.org/D10689 but split out for separate review.

Diff Detail

Event Timeline

dougk updated this revision to Diff 28474.Jun 25 2015, 10:15 AM
dougk retitled this revision from to Use more range-based for loops.
dougk updated this object.
dougk edited the test plan for this revision. (Show Details)
dougk added a reviewer: silvas.
dougk added subscribers: Unknown Object (MLST), Unknown Object (MLST).
dblaikie accepted this revision.Jun 25 2015, 11:36 AM
dblaikie added a reviewer: dblaikie.
dblaikie added a subscriber: dblaikie.

Looks good - feel free to commit this sort of cleanup in areas you're working directly/with post-commit review.

This revision is now accepted and ready to land.Jun 25 2015, 11:36 AM
This revision was automatically updated to reflect the committed changes.