[PATCH] D138088: [clang][docs] Use `option` directive in User's Manual

2022-11-21 Thread KAWASHIMA Takahiro via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG799b6b9f3193: [clang][docs] Use `option` directive in User's Manual (authored by kawashima-fj). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138088/new/ ht

[PATCH] D138088: [clang][docs] Use `option` directive in User's Manual

2022-11-21 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. In D138088#3938687 , @kawashima-fj wrote: > In D138088#3937680 , @aaron.ballman > wrote: > >>

[PATCH] D138088: [clang][docs] Use `option` directive in User's Manual

2022-11-18 Thread KAWASHIMA Takahiro via Phabricator via cfe-commits
kawashima-fj added a comment. In D138088#3937680 , @aaron.ballman wrote: > Thank you for this cleanup! In general, I thin this looks correct. However, I > know we've had to fix a bunch of options that cause the sphinx build to fail > (IIRC, oftentimes

[PATCH] D138088: [clang][docs] Use `option` directive in User's Manual

2022-11-18 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. Thank you for this cleanup! In general, I thin this looks correct. However, I know we've had to fix a bunch of options that cause the sphinx build to fail (IIRC, oftentimes due to duplicate options) and our precommit CI doesn't test the documentation build. Did yo

[PATCH] D138088: [clang][docs] Use `option` directive in User's Manual

2022-11-15 Thread KAWASHIMA Takahiro via Phabricator via cfe-commits
kawashima-fj created this revision. kawashima-fj added a project: clang. Herald added a project: All. kawashima-fj requested review of this revision. Herald added a subscriber: cfe-commits. Sphinx has the `option` directive. Most option descriptions in `clang/docs/UsersManual.rst` used it but som