This is an archive of the discontinued LLVM Phabricator instance.

[docs] Fix typo in the example code in ProgrammersManual.
ClosedPublic

Authored by HsiangKai on Jan 19 2022, 5:38 AM.

Diff Detail

Event Timeline

HsiangKai requested review of this revision.Jan 19 2022, 5:38 AM
HsiangKai created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJan 19 2022, 5:38 AM
dexonsmith accepted this revision.Jan 19 2022, 1:33 PM

Thanks for finding this! I think processBuffer() pairs better with processBufferAtPath() though so I suggest a different fix...

LGTM if you go with my alternate suggestion.

llvm/docs/ProgrammersManual.rst
579

I think this should be left alone...

588

... and this should be changed to processBuffer().

This revision is now accepted and ready to land.Jan 19 2022, 1:33 PM
HsiangKai updated this revision to Diff 401451.Jan 19 2022, 5:22 PM

Address @dexonsmith's comments.

This revision was landed with ongoing or failed builds.Jan 19 2022, 5:26 PM
This revision was automatically updated to reflect the committed changes.