This is an archive of the discontinued LLVM Phabricator instance.

[docs] Typos and whitespace fixed in LTO docs.
ClosedPublic

Authored by JDevlieghere on Nov 29 2016, 2:58 AM.

Details

Summary

While reading the LTO docs I fixed few small typos and whitespace issues.

Diff Detail

Repository
rL LLVM

Event Timeline

JDevlieghere retitled this revision from to [docs] Typos and whitespace fixed in LTO docs..
JDevlieghere updated this object.
JDevlieghere added a reviewer: pcc.
JDevlieghere set the repository for this revision to rL LLVM.
mehdi_amini added inline comments.Nov 29 2016, 9:49 AM
docs/LinkTimeOptimization.rst
297 ↗(On Diff #79527)

This is still part of the same sentence, I wouldn't use a capital here.

  • Fixed capitalization of "which"
JDevlieghere marked an inline comment as done.Nov 29 2016, 10:01 AM
JDevlieghere added inline comments.
docs/LinkTimeOptimization.rst
297 ↗(On Diff #79527)

You're totally right! I changed it for consistency with a previous sentence, which I now made lowercase in the updated diff. (I guess I should've done that in the first place...)

mehdi_amini accepted this revision.Nov 29 2016, 10:06 AM
mehdi_amini added a reviewer: mehdi_amini.

LGTM!

This revision is now accepted and ready to land.Nov 29 2016, 10:06 AM
JDevlieghere marked an inline comment as done.Nov 29 2016, 10:07 AM

Would you mind landing this for me please? I don't have commit access yet.

This revision was automatically updated to reflect the committed changes.