This is an archive of the discontinued LLVM Phabricator instance.

[docs] Fix LibTooling documentation for OptionsParser change
AcceptedPublic

Authored by motemen on Jun 12 2023, 9:14 PM.

Details

Summary

Additional documentation fix per https://reviews.llvm.org/D94420

Diff Detail

Event Timeline

motemen created this revision.Jun 12 2023, 9:14 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2023, 9:14 PM
motemen requested review of this revision.Jun 12 2023, 9:14 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2023, 9:14 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript

I'm fine with the change. It would be great if this could literalinclude these two sample codes (as in https://www.sphinx-doc.org/en/master/usage/restructuredtext/directives.html#directive-literalinclude) and have them being compiled as part of the validation process. That way we would avoid such doc-bit-rotting (?) in the future. This require some effort so I'm still okay if you don't give it a try, in which case I will do. Please tell me!

This revision is now accepted and ready to land.Jun 12 2023, 10:37 PM

Thank you for your review. Though I'm interested in it, I'm not familiar with the build system. So I won't go further in this patch.
Thanks!