This is an archive of the discontinued LLVM Phabricator instance.

[docs] NFC: Fix typos in documents
ClosedPublic

Authored by kiszk on Jan 2 2020, 9:27 AM.

Details

Summary

"the the" -> "the"
"an" -> "a"

Event Timeline

kiszk created this revision.Jan 2 2020, 9:27 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 2 2020, 9:27 AM
hans added a comment.Jan 7 2020, 1:33 AM

Thanks! I just added a few comments.
Do you have commit access, or will you need someone to commit this for you?

llvm/docs/LangRef.rst
10304

I think it should be "to the" here.

llvm/docs/ORCv2.rst
43

I think it should be "at runtime".

kiszk updated this revision to Diff 236576.Jan 7 2020, 6:41 AM
  • address review comments
  • rebase with master
kiszk added a comment.Jan 7 2020, 6:43 AM

Thank you for your comments. I have just addressed them.

Since I do not have commit rights to the repository yet, could someone commit this change if it is OK?
Kazuaki Ishizaki <ishizaki@jp.ibm.com>

hans accepted this revision.Jan 7 2020, 7:06 AM

Looks good to me. I'll commit it for you.

This revision is now accepted and ready to land.Jan 7 2020, 7:06 AM
This revision was automatically updated to reflect the committed changes.