This is an archive of the discontinued LLVM Phabricator instance.

[Doc] Modernize programmers manual
ClosedPublic

Authored by Prazek on Feb 24 2017, 7:42 AM.

Details

Summary

Fixed bunch of for loops to range based for loop
and bunch of rendundat types with auto.

Event Timeline

Prazek created this revision.Feb 24 2017, 7:42 AM
Prazek updated this revision to Diff 89673.Feb 24 2017, 7:44 AM

moved * to the right

This revision is now accepted and ready to land.Feb 24 2017, 9:15 AM
kparzysz added inline comments.
docs/ProgrammersManual.rst
2263–2264

BB.size()

Prazek updated this revision to Diff 89685.Feb 24 2017, 9:37 AM
Prazek marked an inline comment as done.

Small fixes.

Prazek added inline comments.Feb 24 2017, 9:37 AM
docs/ProgrammersManual.rst
2263–2264

thanks, missed that

Prazek updated this revision to Diff 89686.Feb 24 2017, 9:40 AM

other changes to comments nearby changed code

Prazek added inline comments.Feb 24 2017, 9:42 AM
docs/ProgrammersManual.rst
2267–2271

Mehdi, is it fine if I will delete this? The example doesn't contain the iterator now, and I would expect someone who would use the iterator to know how iterators works.

mehdi_amini added inline comments.Feb 24 2017, 10:27 AM
docs/ProgrammersManual.rst
2267–2271

Fine with me.

echristo accepted this revision.Feb 24 2017, 8:47 PM
This revision was automatically updated to reflect the committed changes.