This is an archive of the discontinued LLVM Phabricator instance.

Reformulate OrcJIT tutorial doc to make it more clear.
ClosedPublic

Authored by sonson on Jul 13 2021, 8:43 AM.

Details

Summary

Fixed a minor writing error. The text was hard to understand, so I reviewed it.

Diff Detail

Event Timeline

sonson requested review of this revision.Jul 13 2021, 8:43 AM
sonson created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJul 13 2021, 8:43 AM
mehdi_amini accepted this revision.Jul 13 2021, 9:21 PM

LGTM, can you retitle to something descriptive, for example Reformulate OrcJIT tutorial doc to make it more clear or something like that?

Also do you have commit access or do you need someone to land this for you?

This revision is now accepted and ready to land.Jul 13 2021, 9:21 PM
sonson retitled this revision from Fixed a minor writing error. The text was hard to understand, so I reviewed it. to Reformulate OrcJIT tutorial doc to make it more clear..Jul 13 2021, 10:25 PM
sonson edited the summary of this revision. (Show Details)

I changed the title and I will keep it in mind about titles.
I do not have commit access, I need someone to land it.

Thank you for your review.

This revision was landed with ongoing or failed builds.Jul 13 2021, 10:48 PM
This revision was automatically updated to reflect the committed changes.