[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