[PATCH] D85596: [Docs] Link to --print-supported-cpus option
Travis Finkenauer via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Aug 23 18:57:33 PDT 2020
tmfink added inline comments.
================
Comment at: clang/docs/CommandGuide/clang.rst:346
- Aliases of --print-supported-cpus
+ Aliases of :option:`--print-supported-cpus`
----------------
nickdesaulniers wrote:
> Would you mind adding a `.` for punctuation while you're here?
This is not a complete sentence (no predicate/verb), so from a grammar point of view, a period does not make sense.
It seems like the other options use complete sentences (with a period).
I'll reword this description to match the style of the other options (so a period will make sense).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85596/new/
https://reviews.llvm.org/D85596
More information about the cfe-commits
mailing list